Skip to content

Commit

Permalink
chore(storetypesv2): Implement V2 Certificat Store Types that interpr…
Browse files Browse the repository at this point in the history
…et the Store Path as the Object ID

Signed-off-by: Hayden Roszell <[email protected]>
  • Loading branch information
m8rmclaren committed Oct 3, 2024
1 parent 2c03c6f commit ca30904
Show file tree
Hide file tree
Showing 45 changed files with 3,475 additions and 755 deletions.
20 changes: 10 additions & 10 deletions AzureEnterpriseApplicationOrchestrator.Tests/AzureApp.cs
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ public void AzureApp_Inventory_IntegrationTest_ReturnSuccess()
.WithTenantId(env.TenantId)
.WithApplicationId(env.ApplicationId)
.WithClientSecret(env.ClientSecret)
.WithTargetObjectId(env.TargetApplicationObjectId)
.WithTargetApplicationApplicationId(env.TargetApplicationApplicationId)
.Build();

// Set up the inventory job configuration
Expand All @@ -57,7 +57,7 @@ public void AzureApp_Inventory_IntegrationTest_ReturnSuccess()
CertificateStoreDetails = new CertificateStore
{
ClientMachine = env.TenantId,
StorePath = env.TargetApplicationObjectId,
StorePath = env.TargetApplicationApplicationId,
Properties = $"{{\"ServerUsername\":\"{env.ApplicationId}\",\"ServerPassword\":\"{env.ClientSecret}\",\"AzureCloud\":\"\"}}"
}
};
Expand Down Expand Up @@ -118,14 +118,14 @@ public void AzureApp_Inventory_ProcessJob_ValidClient_ReturnSuccess()

// Act
JobResult result = inventory.ProcessJob(config, (inventoryItems) =>
{
// Assert
Assert.Equal(1, inventoryItems.Count());
Assert.Equal("test", inventoryItems.First().Alias);
{
// Assert
Assert.Equal(1, inventoryItems.Count());

Check warning on line 123 in AzureEnterpriseApplicationOrchestrator.Tests/AzureApp.cs

View workflow job for this annotation

GitHub Actions / Build and Test dotnet project

Do not use Assert.Equal() to check for collection size. (https://xunit.github.io/xunit.analyzers/rules/xUnit2013)

Check warning on line 123 in AzureEnterpriseApplicationOrchestrator.Tests/AzureApp.cs

View workflow job for this annotation

GitHub Actions / Build and Test dotnet project

Do not use Assert.Equal() to check for collection size. (https://xunit.github.io/xunit.analyzers/rules/xUnit2013)

Check warning on line 123 in AzureEnterpriseApplicationOrchestrator.Tests/AzureApp.cs

View workflow job for this annotation

GitHub Actions / call-starter-workflow / call-generate-readme-workflow / Use private doctool action in public repository

Do not use Assert.Equal() to check for collection size. (https://xunit.github.io/xunit.analyzers/rules/xUnit2013)

Check warning on line 123 in AzureEnterpriseApplicationOrchestrator.Tests/AzureApp.cs

View workflow job for this annotation

GitHub Actions / call-starter-workflow / call-dotnet-build-and-release-workflow / dotnet-build-and-release

Do not use Assert.Equal() to check for collection size. (https://xunit.github.io/xunit.analyzers/rules/xUnit2013)

Check warning on line 123 in AzureEnterpriseApplicationOrchestrator.Tests/AzureApp.cs

View workflow job for this annotation

GitHub Actions / call-starter-workflow / call-dotnet-build-and-release-workflow / dotnet-build-and-release

Do not use Assert.Equal() to check for collection size. (https://xunit.github.io/xunit.analyzers/rules/xUnit2013)
Assert.Equal("test", inventoryItems.First().Alias);

_logger.LogInformation("AzureApp_Inventory_ProcessJob_ValidClient_ReturnSuccess - Success");
return true;
});
_logger.LogInformation("AzureApp_Inventory_ProcessJob_ValidClient_ReturnSuccess - Success");
return true;
});

// Assert
Assert.Equal(OrchestratorJobStatusJobResult.Success, result.Result);
Expand Down Expand Up @@ -481,7 +481,7 @@ public void AzureApp_Management_IntegrationTest_ReturnSuccess()
CertificateStoreDetails = new CertificateStore
{
ClientMachine = env.TenantId,
StorePath = env.TargetApplicationObjectId,
StorePath = env.TargetApplicationApplicationId,
Properties = $"{{\"ServerUsername\":\"{env.ApplicationId}\",\"ServerPassword\":\"{env.ClientSecret}\",\"AzureCloud\":\"\"}}"
},
JobCertificate = new ManagementJobCertificate
Expand Down
Loading

0 comments on commit ca30904

Please sign in to comment.