bump newman dependency so we have a true OAV version to test publish

Author: scbedd

ChangeLog.md

@@ -1,5 +1,9 @@
 # Change Log - oav
 
+## 05/30/2024 3.3.6
+
+- Bump newman dependency `^5.3.2` -> `^6.0.0`
+
 ## 05/7/2024 3.3.5
 
 - Bump Node engine version requirement for security vulnerability.

eng/templates/stages/oav.yml

@@ -46,25 +46,29 @@ extends:
                   customCommand: test
 
               - task: PublishTestResults@2
+                condition: succeededOrFailed()
                 inputs:
                   testResultsFiles: "**/test-results.xml"
                   testRunTitle: "Test results for JavaScript"
 
               - task: PublishCodeCoverageResults@1
+                condition: succeededOrFailed()
                 inputs:
                   codeCoverageTool: Cobertura
                   summaryFileLocation: "$(System.DefaultWorkingDirectory)/**/*coverage.xml"
                   reportDirectory: "$(System.DefaultWorkingDirectory)/**/coverage"
 
               - task: Npm@1
                 displayName: "npm pack"
+                condition: succeededOrFailed()
                 inputs:
                   command: custom
                   verbose: false
                   customCommand: pack
 
               - task: CopyFiles@2
                 displayName: "Copy Files to: drop"
+                condition: succeededOrFailed()
                 inputs:
                   Contents: "*.tgz"
                   TargetFolder: $(Build.ArtifactStagingDirectory)
@@ -73,6 +77,7 @@ extends:
                 parameters:
                   ArtifactName: 'oav'
                   ArtifactPath: '$(Build.ArtifactStagingDirectory)'
+                  CustomCondition: succeededOrFailed()
 
       - template: /eng/templates/stages/examples-regression.yml