Rearrange docker copy and test results check commands

dotnet · Apr 17, 2024 · d757acf · d757acf
1 parent 3d4a494
commit d757acf
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/eng/common/templates/steps/test-images-linux-client.yml b/eng/common/templates/steps/test-images-linux-client.yml
@@ -74,12 +74,11 @@ steps:
     condition: and(always(), ${{ parameters.condition }})
     continueOnError: true
 - powershell: |
-    $hasTestResults = Test-Path $(Common.TestResultsDirectory)/$(testResultsDirectory)
+    $ErrorActionPreference = 'SilentlyContinue'
+    docker cp $(testRunner.container):/repo/$(testResultsDirectory) $(Common.TestResultsDirectory)/.
+    $hasTestResults = Test-Path $(Common.TestResultsDirectory)
     echo "##vso[task.setvariable variable=hasTestResults]$hasTestResults"
-    if ($hasTestResults) {
-      docker cp $(testRunner.container):/repo/$(testResultsDirectory) $(Common.TestResultsDirectory)/.
-    }
-    else {
+    if (!$hasTestResults) {
       echo "There were no test outputs in $(Common.TestResultsDirectory)/$(testResultsDirectory). Skipping test result publishing."
     }
   displayName: Copy Test Results