Skip to content

Commit

Permalink
Correct capitalisation on Cosmos DB Role Assignment property access
Browse files Browse the repository at this point in the history
  • Loading branch information
rudiv committed May 27, 2024
1 parent f6e91e7 commit 6eb2d65
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/Achieve.Aspire.AzureProvisioning/CosmosDb.cs
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,7 @@ public CosmosDbAccountOptions WithRoleAssignment(BicepResource scope, BicepOutpu
}

public CosmosDbAccountOptions WithRoleAssignment(BicepResource scope, IResourceBuilder<AzureManagedIdentityResource> identity, CosmosDbSqlBuiltInRole role) =>
WithRoleAssignment(scope, identity.GetOutput("PrincipalId"), role);
WithRoleAssignment(scope, identity.GetOutput("principalId"), role);
}

public class CosmosDbDatabaseOptions(CosmosDbAccountOptions parent, CosmosDbSqlDatabaseResource sqlDatabase)

Check warning on line 147 in src/Achieve.Aspire.AzureProvisioning/CosmosDb.cs

View workflow job for this annotation

GitHub Actions / test

Parameter 'parent' is unread.

Check warning on line 147 in src/Achieve.Aspire.AzureProvisioning/CosmosDb.cs

View workflow job for this annotation

GitHub Actions / publish

Parameter 'parent' is unread.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,7 @@ public async Task CosmosAccountWithRbacGeneratesCorrectly()
"path": "cosmos.achieve.bicep",
"params": {
"principalId": "",
"testidPrincipal": "{testid.outputs.PrincipalId}"
"testidPrincipal": "{testid.outputs.principalId}"
}
}
""";
Expand Down

0 comments on commit 6eb2d65

Please sign in to comment.