Skip to content

Commit a2d1acf

Browse files
authored
[Azure Pipelines] drop old workaround for safaridriver --version (#18454)
The `--version` argument is supported starting in Safari 12.1: web-platform-tests/results-collection#617
1 parent a8a1303 commit a2d1acf

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

.azure-pipelines.yml

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -334,10 +334,7 @@ jobs:
334334
channel: stable
335335
- template: tools/ci/azure/update_hosts.yml
336336
- template: tools/ci/azure/update_manifest.yml
337-
- script: |
338-
# TODO: drop this when `safaridriver --version` is supported.
339-
SAFARI_VERSION="$(/usr/libexec/PlistBuddy -c 'Print :CFBundleShortVersionString' /Applications/Safari.app/Contents/Info.plist) ($(/usr/libexec/PlistBuddy -c 'Print :CFBundleVersion' /Applications/Safari.app/Contents/Info.plist))"
340-
no_proxy='*' ./wpt run --no-manifest-update --no-restart-on-unexpected --no-fail-on-unexpected --this-chunk=$(System.JobPositionInPhase) --total-chunks=$(System.TotalJobsInPhase) --chunk-type hash --log-wptreport $(Build.ArtifactStagingDirectory)/wpt_report_$(System.JobPositionInPhase).json --log-wptscreenshot $(Build.ArtifactStagingDirectory)/wpt_screenshot_$(System.JobPositionInPhase).txt --log-tbpl - --log-tbpl-level info --browser-version "$SAFARI_VERSION" safari
337+
- script: no_proxy='*' ./wpt run --no-manifest-update --no-restart-on-unexpected --no-fail-on-unexpected --this-chunk=$(System.JobPositionInPhase) --total-chunks=$(System.TotalJobsInPhase) --chunk-type hash --log-wptreport $(Build.ArtifactStagingDirectory)/wpt_report_$(System.JobPositionInPhase).json --log-wptscreenshot $(Build.ArtifactStagingDirectory)/wpt_screenshot_$(System.JobPositionInPhase).txt --log-tbpl - --log-tbpl-level info safari
341338
displayName: 'Run tests'
342339
- task: PublishBuildArtifacts@1
343340
displayName: 'Publish results'

0 commit comments

Comments
 (0)