Address flaky unit tests

src/TriggerService.Tests/CromwellOnAzureEnvironmentTests.cs

@@ -19,7 +19,6 @@
 namespace TriggerService.Tests
 {
     [TestClass]
-    [Ignore]
     public class CromwellOnAzureEnvironmentTests
     {
         private const string azureName = "test";
@@ -160,8 +159,7 @@ private TriggerHostedService SetCromwellOnAzureEnvironment(string accountAuthori
                 ApplicationInsightsAccountName = "fakeappinsights"
             });
 
-            var azureCloudConfig = AzureCloudConfig.FromKnownCloudNameAsync().Result;
-            var environment = new TriggerHostedService(logger, triggerServiceOptions.Object, cromwellApiClient, repository, storageUtility.Object, azureCloudConfig);
+            var environment = new TriggerHostedService(logger, triggerServiceOptions.Object, cromwellApiClient, repository, storageUtility.Object, AzureCloudConfig.ForUnitTesting());
             return environment;
         }
 

src/TriggerService.Tests/ProcessAbortRequestTests.cs

@@ -19,7 +19,6 @@
 namespace TriggerService.Tests
 {
     [TestClass]
-    [Ignore]
     public class ProcessAbortRequestTests
     {
         public ProcessAbortRequestTests()
@@ -103,8 +102,7 @@ public async Task FailedAbortRequestFileGetsMovedToFailedSubdirectory()
                 .Setup(x => x.GetStorageAccountsUsingMsiAsync(It.IsAny<string>()))
                 .Returns(Task.FromResult((new List<IAzureStorage>(), azureStorage.Object)));
 
-            var azureCloudConfig = AzureCloudConfig.FromKnownCloudNameAsync().Result;
-            var cromwellOnAzureEnvironment = new TriggerHostedService(logger, triggerServiceOptions.Object, cromwellApiClient, repository.Object, storageUtility.Object, azureCloudConfig);
+            var cromwellOnAzureEnvironment = new TriggerHostedService(logger, triggerServiceOptions.Object, cromwellApiClient, repository.Object, storageUtility.Object, AzureCloudConfig.ForUnitTesting());
             await cromwellOnAzureEnvironment.ProcessAndAbortWorkflowsAsync();
             return (newTriggerName, newTriggerContent);
         }

src/TriggerService.Tests/ProcessNewWorkflowTests.cs

@@ -20,7 +20,6 @@
 namespace TriggerService.Tests
 {
     [TestClass]
-    [Ignore]
     public class ProcessNewWorkflowTests
     {
         public ProcessNewWorkflowTests()
@@ -121,14 +120,13 @@ public async Task NewWorkflowsThatFailToPostToCromwellAreMovedToFailedSubdirecto
             storageUtility
                 .Setup(x => x.GetStorageAccountsUsingMsiAsync(It.IsAny<string>()))
                 .Returns(Task.FromResult((new List<IAzureStorage>(), azureStorage.Object)));
-            var azureCloudConfig = AzureCloudConfig.FromKnownCloudNameAsync().Result;
             var cromwellOnAzureEnvironment = new TriggerHostedService(
                 logger,
                 optionsMock.Object,
                 cromwellApiClient,
                 tesTaskRepository,
                 storageUtility.Object,
-                azureCloudConfig);
+                AzureCloudConfig.ForUnitTesting());
 
             await cromwellOnAzureEnvironment.ProcessAndAbortWorkflowsAsync();
 
@@ -209,15 +207,13 @@ public async Task NewWorkflowsThatFailToParseAsJsonAreAnotatedAndMovedToFailedSu
                 .Setup(x => x.GetStorageAccountsUsingMsiAsync(It.IsAny<string>()))
                 .Returns(Task.FromResult((new List<IAzureStorage>(), azureStorage.Object)));
 
-            var azureCloudConfig = AzureCloudConfig.FromKnownCloudNameAsync().Result;
-
             var cromwellOnAzureEnvironment = new TriggerHostedService(
                 logger,
                 triggerServiceOptions.Object,
                 cromwellApiClient2,
                 tesTaskRepository,
                 storageUtility.Object,
-                azureCloudConfig);
+                AzureCloudConfig.ForUnitTesting());
 
             await cromwellOnAzureEnvironment.ProcessAndAbortWorkflowsAsync();
             Assert.IsTrue(newTriggerName.StartsWith("failed/"));

src/TriggerService.Tests/TriggerEngineTests.cs

@@ -101,8 +101,7 @@ public async Task TriggerEngineRunsAndOnlyLogsAvailabilityOncePerSystemUponAvail
                 .Setup(x => x.GetStorageAccountsUsingMsiAsync(It.IsAny<string>()))
                 .Returns(Task.FromResult((new List<IAzureStorage>(), azureStorage.Object)));
 
-            var azureCloudConfig = AzureCloudConfig.FromKnownCloudNameAsync().Result;
-            var triggerHostedService = new TriggerHostedService(logger, triggerServiceOptions.Object, cromwellApiClient, tesTaskRepository, storageUtility.Object, azureCloudConfig);
+            var triggerHostedService = new TriggerHostedService(logger, triggerServiceOptions.Object, cromwellApiClient, tesTaskRepository, storageUtility.Object, AzureCloudConfig.ForUnitTesting());
 
             //var engine = new TriggerHostedService(loggerFactory, environment.Object, TimeSpan.FromMilliseconds(25), TimeSpan.FromMilliseconds(25));
             _ = Task.Run(() => triggerHostedService.StartAsync(new System.Threading.CancellationToken()));

src/TriggerService.Tests/TriggerService.Tests.csproj

@@ -25,8 +25,6 @@
   </ItemGroup>
 
   <ItemGroup>
-    <ProjectReference Include="..\Common\Common.csproj" />
-    <ProjectReference Include="..\ga4gh-tes\src\Tes\Tes.csproj" />
     <ProjectReference Include="..\TriggerService\TriggerService.csproj" />
   </ItemGroup>
 </Project>

src/TriggerService.Tests/UpdateWorkflowStatusTests.cs

@@ -21,7 +21,6 @@
 namespace TriggerService.Tests
 {
     [TestClass]
-    [Ignore]
     public class UpdateWorkflowStatusTests
     {
         public UpdateWorkflowStatusTests()
@@ -431,8 +430,7 @@ static string ShardString(int shard) =>
                 .Setup(x => x.GetStorageAccountsUsingMsiAsync(It.IsAny<string>()))
                 .Returns(Task.FromResult((new List<IAzureStorage>(), azureStorage.Object)));
 
-            var azureCloudConfig = AzureCloudConfig.FromKnownCloudNameAsync().Result;
-            var cromwellOnAzureEnvironment = new TriggerHostedService(logger, triggerServiceOptions.Object, cromwellApiClient.Object, repository.Object, storageUtility.Object, azureCloudConfig);
+            var cromwellOnAzureEnvironment = new TriggerHostedService(logger, triggerServiceOptions.Object, cromwellApiClient.Object, repository.Object, storageUtility.Object, AzureCloudConfig.ForUnitTesting());
 
             await cromwellOnAzureEnvironment.UpdateWorkflowStatusesAsync();