Closed Bug 1915333 Opened 1 year ago Closed 1 year ago

[wpt-sync] Sync PR 47831 - wpt: fix the path to wpt-prefs.json on WPT runner

Categories

(Testing :: web-platform-tests, task, P4)

task

Tracking

(firefox131 fixed)

RESOLVED FIXED
131 Branch
Tracking Status
firefox131 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 47831 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/47831
Details from upstream follow.

Mukilan Thiyagarajan <mukilan@igalia.com> wrote:

wpt: fix the path to wpt-prefs.json on WPT runner

The path added in #<!-- nolink -->33202 turned out to be incorrect. The logic had a fallback to pwd, but that is not the one that WPT runner uses. Instead, the path to the venv sub-directory is passed in from the caller.

The logs from WPT runner also show that the ./wpt run command is executed with the wpt repo as the CWD

2024-08-28 01:51:32,158 - tc-run - INFO - Executing command: python3 ./wpt run --channel=nightly --log-wptreport=../artifacts/wpt_report.json --log-wptscreenshot=../artifacts/wpt_screenshot.txt --no-fail-on-unexpected --this-chunk=4 --total-chunks=16 --test-type=testharness --log-mach-level=info --log-mach=- -y --no-pause --no-restart-on-unexpected --install-fonts --no-headless --verify-log-full --install-browser --processes=12 servo

Reviewed in servo/servo#33220

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d91b105f4126 [wpt PR 47831] - wpt: fix the path to wpt-prefs.json on WPT runner, a=testonly
Test result changes from PR not available.
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 131 Branch
You need to log in before you can comment on or make changes to this bug.