257123 2023-05-22 08:30:15 -0700 Update 'testharness.js' from upstream for WPT tests 2023-05-22 16:17:50 -0700 1 1 1 Unclassified WebKit Tools / Tests Safari Technology Preview Unspecified Unspecified RESOLVED DUPLICATE 250872 P2 Normal --- 1 ahmad.saleem792 webkit-unassigned annevk ap cdumez gsnedders ntim oldest_to_newest 1956955 0 ahmad.saleem792 2023-05-22 08:30:15 -0700 Hi Team, I noticed that we updated / upstreamed 'testharness.js' last time about two years ago. (Last Push) Commit: https://github.com/WebKit/WebKit/commit/3e631739e295a4f82140d130f26236c7721cbc8b Latest Version (with History from WPT GitHub): https://github.com/web-platform-tests/wpt/commits/74236ab210e6d6dc8bd735ce3a586541bf066bd4/resources/testharness.js I think it would be good to sync it after two years. Any thoughts or opinions? or if there is something super planned in WPT realm, so we should hold out? Thanks! 1957122 1 gsnedders 2023-05-22 15:14:39 -0700 If we do this (and we should do it at some point!) we should re-import all of resources/, not just testharness.js. 1957139 2 ahmad.saleem792 2023-05-22 16:12:52 -0700 (In reply to Sam Sneddon [:gsnedders] from comment #1) > If we do this (and we should do it at some point!) we should re-import all > of resources/, not just testharness.js. Thanks! I think it would be beyond my expertise, if it would've been just copy paste of 'testharness.js', I was happy to do PR. :-) Though still happy to learn something new on how to do this 'resource' stuff. :-) 1957142 3 cdumez 2023-05-22 16:17:34 -0700 What do you mean, I resync'd testharness.js from upstream in January of this year (as part of the whole resources/ folder): ``` commit 4a4574451a99cffd3197fec02b0df9437594bfd2 (fork/250872_resync_wpt_resources_jan23, cdumez/250872_resync_wpt_resources_jan23) Author: Chris Dumez <cdumez@apple.com> Date: Fri Jan 20 09:29:07 2023 -0800 Resync web-platform-tests/resources from upstream https://bugs.webkit.org/show_bug.cgi?id=250872 Reviewed by Tim Nguyen. Resync web-platform-tests/resources from upstream d20c3cdbf6e91f14dda92737e. * LayoutTests/imported/w3c/web-platform-tests/html/infrastructure/safe-passing-of-structured-data/structured-cloning-error-stack-optional.sub.window-expected.txt: * LayoutTests/imported/w3c/web-platfrom-tests/html/user-activation/*: Updated since they would break otherwise after the resources/ folder re-sync. * LayoutTests/imported/w3c/web-platform-tests/resources/*: Updated. Canonical link: https://commits.webkit.org/259143@main ``` Are you looking at the wrong testharness.js ? Note that there is an old version is LayoutTests/resources but it is not used by WPT tests, only by some WebKit-specific tests. 1957145 4 cdumez 2023-05-22 16:17:50 -0700 *** This bug has been marked as a duplicate of bug 250872 ***