Fixes for unit tests

Process runner test adds -y to overwrite existing files Add more debugging info to failing test result publishing
QutEcoacoustics · Apr 4, 2020 · bd65209 · bd65209
1 parent 45e6232
commit bd65209
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 6 deletions.
diff --git a/build/azure-pipelines-build.yml b/build/azure-pipelines-build.yml
@@ -141,13 +141,17 @@ jobs:
               RunConfiguration.DisableAppDomain=true
             displayName: Run Acoustics.Test
 
+          - pwsh: |
+              GetChild-Item -Recursive $(Agent.TempDirectory)/Acoustics.Test_Results
+              Get-Content $(Agent.TempDirectory)/Acoustics.Test_Results/*.trx
+
           - task: PublishTestResults@2
             inputs:
               testResultFormat: VSTest
               testResultsFiles: '**/*.trx'
               searchFolder: $(Agent.TempDirectory)/Acoustics.Test_Results
               testRunTitle: "Acoustics.Test for ${{ platform.rid }} ${{ configuration }}"
-              buildConfig: ${{ configuration }}
+              buildConfiguration: ${{ configuration }}
               buildPlatform: $(platformTag)
               publishRunAttachments: true
             condition: succeededOrFailed()

diff --git a/tests/Acoustics.Test/.runsettings b/tests/Acoustics.Test/.runsettings
@@ -7,7 +7,7 @@
     <Loggers>
       <Logger friendlyName="console">
         <Configuration>
-          <verbosity>detailed</verbosity>
+          <!-- <verbosity>detailed</verbosity> -->
         </Configuration>
       </Logger>
     </Loggers>

diff --git a/tests/Acoustics.Test/Shared/ProcessRunnerTests.cs b/tests/Acoustics.Test/Shared/ProcessRunnerTests.cs
@@ -34,15 +34,15 @@ public void ProcessRunnerSimple()
         [RetryTestMethod(2)]
         public void ProcessRunnerTimeOutDoesNotDeadlock()
         {
-            var result = Enumerable.Range(0, 100).AsParallel().Select(this.RunFfprobeIndefinite).ToArray();
+            var result = Enumerable.Range(0, 100).AsParallel().Select(this.RunFfmpegIndefinite).ToArray();
 
             Assert.IsTrue(result.All());
         }
 
         [RetryTestMethod(2)]
         public void ProcessRunnerTimeOutSimple()
         {
-            this.RunFfprobeIndefinite(0);
+            this.RunFfmpegIndefinite(0);
         }
 
         [TestMethod]
@@ -109,7 +109,7 @@ private bool RunFfprobe(int index)
             return result;
         }
 
-        private bool RunFfprobeIndefinite(int index)
+        private bool RunFfmpegIndefinite(int index)
         {
             var path = PathHelper.ResolveAssetPath(TestFile);
             var dest = PathHelper.GetTempFile(this.TestOutputDirectory, ".mp3");
@@ -121,7 +121,7 @@ private bool RunFfprobeIndefinite(int index)
 
                 Assert.ThrowsException<ProcessRunner.ProcessMaximumRetriesException>(() =>
                 {
-                    runner.Run($@"-i ""{path}"" -ar 8000 ""{dest}""", this.TestOutputDirectory.FullName);
+                    runner.Run($@"-y -i ""{path}"" -ar 8000 ""{dest}""", this.TestOutputDirectory.FullName);
 
                     Assert.Fail($"Process running finished without timing out - this should not happen. Exit code: {runner.ExitCode}.\nStdout:\n{runner.StandardOutput}\nStdErr\n{runner.ErrorOutput}");
                 });