Commit Graph
547 Commits
Author SHA1 Message Date
Ted Campbell 909b390891 Bug 2011346 - Support xpcshelltestrunner http3server on macos. r=florian
The http3 server runs on host machine, so must set DYLD_LIBRARY_PATH if host
is macos.

Differential Revision: https://phabricator.services.mozilla.com/D279617
2026-06-04 02:49:57 +00:00
Florian Quèze 4f3fb8bf93 Bug 2042832 - Remove --total-chunks/--this-chunk and chunk_by_slice from xpcshell harness, r=ahal.
Same reasoning as the mochitest cleanup in the previous commit. Neither
xpcshell task in taskcluster/kinds/test/xpcshell.yml sets
test-manifest-loader: null, so taskgraph always pre-resolves their
manifests and passes them via MOZHARNESS_TEST_PATHS. desktop_unittest.py
never appends --total-chunks/--this-chunk to the xpcshell command,
self.totalChunks stays at its default of 1, and the chunk_by_slice call
in runxpcshelltests.py never fires.

Differential Revision: https://phabricator.services.mozilla.com/D302813
2026-05-28 07:56:14 +00:00
Sylvestre Ledru bdd074022b Bug 2015841 - ruff: fix 'String is split more times than necessary' r=linter-reviewers,perftest-reviewers,ahal,sergesanspaille,sparky
Differential Revision: https://phabricator.services.mozilla.com/D282635
2026-04-14 19:33:10 +00:00
Florian Quèze ea9c91f340 Bug 2028234 - Emit test_start before test setup so runtime data includes setup overhead, r=ahal.
Differential Revision: https://phabricator.services.mozilla.com/D291311
2026-04-03 05:57:40 +00:00
Adam Vandolder 80767fc4f2 Bug 1892551 - Part 14: Set sessionHistoryInParent test manifest variable to always be true. r=taskgraph-reviewers,ahal
Differential Revision: https://phabricator.services.mozilla.com/D284728
2026-03-18 17:26:43 +00:00
Adam Vandolder 33f5a412e7 Bug 1892551 - Part 3: Remove disableSessionHistoryInParent pref. r=jgraham,dom-core,farre
Differential Revision: https://phabricator.services.mozilla.com/D208062
2026-03-18 17:26:18 +00:00
Valentin Gosu f98ac74904 Bug 2001895 - Make NodeServers (https) work in mochitests r=ahal
Spawn moz-http2.js server for mochitests.

Differential Revision: https://phabricator.services.mozilla.com/D279128
2026-03-12 13:55:57 +00:00
Gabriele Svelto 2289e47fed Bug 1999263 - Prevent a race when attempting to generate a minidump in timed-out XPCShell tests r=ahal
Differential Revision: https://phabricator.services.mozilla.com/D274995
2026-02-27 08:37:32 +00:00
Geoff Lankow 9d6012d52e Bug 2018521 - Avoid multiple 'retry' keyword arguments. r=sergesanspaille
Differential Revision: https://phabricator.services.mozilla.com/D284418
2026-02-24 07:30:14 +00:00
Florian Quèze 2beaccdc7f Bug 2017139 - Make structured logging the default for all mozharness tests, r=ahal.
Differential Revision: https://phabricator.services.mozilla.com/D283569
2026-02-23 22:22:26 +00:00
Florian Quèze 857715d03e Bug 2018130 - Divide xpcshell parallel test times by thread count in error summary, r=ahal.
The error summary formatter sums raw test_start-to-test_end times for
xpcshell manifests, but this overestimates wall-clock time when tests run
in parallel across multiple threads. Pass the thread count from the
xpcshell harness via group_start extra data, and divide individual test
times by that count during the parallel phase.

Differential Revision: https://phabricator.services.mozilla.com/D284511
2026-02-23 22:20:58 +00:00
Cristina Horotan f807c9b27e Revert "Bug 2015841 - ruff: fix 'Lambda may be unnecessary; consider inlining inner function' r=linter-reviewers,perftest-reviewers,sessionstore-reviewers,ahal,sparky,sclements" for causing lint failure
This reverts commit 0382300c2d.

Revert "Bug 2015841 - ruff: fix 'String is split more times than necessary' r=linter-reviewers,ahal,sergesanspaille"

This reverts commit 1e0b4c3c7b.

Revert "Bug 2015841 - Reformat the code with ruff-format 0.15.0 r=linter-reviewers,perftest-reviewers,mozperftest-reviewers,releng-reviewers,sessionstore-reviewers,dom-storage-reviewers,dom-worker-reviewers,whimboo,chutten,ahal,sparky,sclements"

This reverts commit 15a20a737e.

Revert "Bug 2015841 - Update to ruff 0.15.0 r=linter-reviewers,ahal"

This reverts commit 4b0785d39e.
2026-02-18 17:18:07 +00:00
Sylvestre Ledru 1e0b4c3c7b Bug 2015841 - ruff: fix 'String is split more times than necessary' r=linter-reviewers,ahal,sergesanspaille
Differential Revision: https://phabricator.services.mozilla.com/D282635
2026-02-18 16:47:12 +00:00
iulian moraru c616d6af1d Revert "Bug 2015841 - ruff: fix 'Lambda may be unnecessary; consider inlining inner function' r=linter-reviewers,perftest-reviewers,sessionstore-reviewers,ahal,sparky,sclements" for causing multiple android reftest failures.
This reverts commit 5eca473e96.

Revert "Bug 2015841 - ruff: fix 'String is split more times than necessary' r=linter-reviewers,ahal,sergesanspaille"

This reverts commit 654a09320c.

Revert "Bug 2015841 - Reformat the code with ruff-format 0.15.0 r=linter-reviewers,perftest-reviewers,mozperftest-reviewers,releng-reviewers,sessionstore-reviewers,dom-storage-reviewers,dom-worker-reviewers,whimboo,chutten,ahal,sparky,sclements"

This reverts commit 93b38f0d79.

Revert "Bug 2015841 - Update to ruff 0.15.0 r=linter-reviewers,ahal"

This reverts commit cc04e8271b.
2026-02-13 22:35:55 +00:00
Sylvestre Ledru 654a09320c Bug 2015841 - ruff: fix 'String is split more times than necessary' r=linter-reviewers,ahal,sergesanspaille
Differential Revision: https://phabricator.services.mozilla.com/D282635
2026-02-13 21:36:22 +00:00
Florian Quèze b82fc4432b Bug 2015029 - make xpcshell and mochitest output group_result status=SKIP lines in the error summary file for missing test manifests, r=ahal.
Differential Revision: https://phabricator.services.mozilla.com/D282170
2026-02-06 21:05:23 +00:00
Ted Campbell 33a56b3bc7 Bug 2009091 - Avoid clobbering host environment when running nodejs for xpcshellremote tests r=ahal,valentin
The remote XPCShell test configuration for Android builds up `XPCShellRemote.env` with environment
variables for the remote device. We also want some of these on the host for httpserver/NodeServer.sys.mjs,
but some of these may break host apps. In particular the 'HOME' variable is used on host for
'adb' to run so we want to avoid clobbering it with the value used for the Android device.

Differential Revision: https://phabricator.services.mozilla.com/D278226
2026-01-12 22:08:19 +00:00
Alex Hochheiden 02333707ab Bug 2006716 - Reformat entire repo with ruff-format (black replacement) r=ahal,sylvestre,linter-reviewers,perftest-reviewers,mozperftest-reviewers,geckoview-reviewers,extension-reviewers,media-playback-reviewers,webrtc-reviewers,releng-reviewers,sessionstore-reviewers,win-reviewers,dom-storage-reviewers,translations-reviewers,omc-reviewers,android-reviewers,home-newtab-reviewers,toolkit-telemetry-reviewers,firefox-style-system-reviewers,layout-reviewers,dom-worker-reviewers,geckodriver-reviewers,Sasha,emz,fchasen,sthompson,dholbert,sparky,padenot,emilio,Gijs,mconley,tcampbell,mjf,emcminn,willdurand,gstoll,jgraham,asuth
# ignore-this-changeset

Differential Revision: https://phabricator.services.mozilla.com/D276894
2026-01-08 21:15:53 +00:00
Atila Butkovits ce6aff66e2 Revert "Bug 2006716 - Remove unused default_bindir() from ruff.py r=ahal,linter-reviewers" for causing failures at tp5n-pageset.manifest.
This reverts commit a198379dd7.

Revert "Bug 2006716 - Add `ruff-format` linter r=ahal,linter-reviewers"

This reverts commit 8e2f1e924b.

Revert "Bug 2006716 - Remove `black` linter r=ahal,linter-reviewers"

This reverts commit 857218c417.

Revert "Bug 2006716 - Fix `ruff` linter `stderr` callback broken in D190696 r=ahal,linter-reviewers"

This reverts commit da4a569cfb.

Revert "Bug 2006716 - Update `ruff` to version `0.14.9` and use consistent versions in `lint.txt` and `python-test.txt` r=ahal,linter-reviewers"

This reverts commit 0781186b2e.

Revert "Bug 2006716 - Reformat entire repo with `ruff-format` (`black` replacement) r=ahal,sylvestre,linter-reviewers,perftest-reviewers,mozperftest-reviewers,geckoview-reviewers,extension-reviewers,media-playback-reviewers,webrtc-reviewers,releng-reviewers,sessionstore-reviewers,win-reviewers,dom-storage-reviewers,translations-reviewers,omc-reviewers,android-reviewers,home-newtab-reviewers,toolkit-telemetry-reviewers,firefox-style-system-reviewers,layout-reviewers,dom-worker-reviewers,geckodriver-reviewers,Sasha,emz,fchasen,sthompson,dholbert,sparky,padenot,emilio,Gijs,mconley,tcampbell,mjf,emcminn,willdurand,gstoll,jgraham,asuth"

This reverts commit b2b8db2859.
2026-01-07 21:34:31 +00:00
Alex Hochheiden b2b8db2859 Bug 2006716 - Reformat entire repo with ruff-format (black replacement) r=ahal,sylvestre,linter-reviewers,perftest-reviewers,mozperftest-reviewers,geckoview-reviewers,extension-reviewers,media-playback-reviewers,webrtc-reviewers,releng-reviewers,sessionstore-reviewers,win-reviewers,dom-storage-reviewers,translations-reviewers,omc-reviewers,android-reviewers,home-newtab-reviewers,toolkit-telemetry-reviewers,firefox-style-system-reviewers,layout-reviewers,dom-worker-reviewers,geckodriver-reviewers,Sasha,emz,fchasen,sthompson,dholbert,sparky,padenot,emilio,Gijs,mconley,tcampbell,mjf,emcminn,willdurand,gstoll,jgraham,asuth
# ignore-this-changeset

Differential Revision: https://phabricator.services.mozilla.com/D276894
2026-01-07 18:08:27 +00:00
Alexandru Marc d296caccba Revert "Bug 2006716 - Remove unused default_bindir() from ruff.py r=ahal,linter-reviewers" for causing toolchain bustages
This reverts commit 76033c983e.

Revert "Bug 2006716 - Add `ruff-format` linter r=ahal,linter-reviewers"

This reverts commit 19fd8c23ca.

Revert "Bug 2006716 - Remove `black` linter r=ahal,linter-reviewers"

This reverts commit 0b01c783a6.

Revert "Bug 2006716 - Fix `ruff` linter `stderr` callback broken in D190696 r=ahal,linter-reviewers"

This reverts commit 72cab75517.

Revert "Bug 2006716 - Update `ruff` to version `0.14.9` and use consistent versions in `lint.txt` and `python-test.txt` r=ahal,linter-reviewers"

This reverts commit a434d066ce.

Revert "Bug 2006716 - Reformat entire repo with `ruff-format` (`black` replacement) r=ahal,sylvestre,linter-reviewers,perftest-reviewers,mozperftest-reviewers,geckoview-reviewers,extension-reviewers,media-playback-reviewers,webrtc-reviewers,releng-reviewers,sessionstore-reviewers,win-reviewers,dom-storage-reviewers,translations-reviewers,omc-reviewers,android-reviewers,home-newtab-reviewers,toolkit-telemetry-reviewers,firefox-style-system-reviewers,layout-reviewers,dom-worker-reviewers,geckodriver-reviewers,Sasha,emz,fchasen,sthompson,dholbert,sparky,padenot,emilio,Gijs,mconley,tcampbell,mjf,emcminn,willdurand,gstoll,jgraham,asuth"

This reverts commit 3ed06912eb.
2026-01-07 00:54:28 +00:00
Alex Hochheiden 3ed06912eb Bug 2006716 - Reformat entire repo with ruff-format (black replacement) r=ahal,sylvestre,linter-reviewers,perftest-reviewers,mozperftest-reviewers,geckoview-reviewers,extension-reviewers,media-playback-reviewers,webrtc-reviewers,releng-reviewers,sessionstore-reviewers,win-reviewers,dom-storage-reviewers,translations-reviewers,omc-reviewers,android-reviewers,home-newtab-reviewers,toolkit-telemetry-reviewers,firefox-style-system-reviewers,layout-reviewers,dom-worker-reviewers,geckodriver-reviewers,Sasha,emz,fchasen,sthompson,dholbert,sparky,padenot,emilio,Gijs,mconley,tcampbell,mjf,emcminn,willdurand,gstoll,jgraham,asuth
# ignore-this-changeset

Differential Revision: https://phabricator.services.mozilla.com/D276894
2026-01-07 00:32:15 +00:00
Alex Hochheiden 491e058e06 Bug 2003412 - Fix PLW0602 warnings: Remove unnecessary global declaration r=emilio,perftest-reviewers,geckoview-reviewers,tcampbell,sparky
https://docs.astral.sh/ruff/rules/global-variable-not-assigned/

Differential Revision: https://phabricator.services.mozilla.com/D274702
2025-12-10 06:06:27 +00:00
Alex Hochheiden c936a9217d Bug 2003412 - Fix PLW1508 warnings: Use string default for os.getenv r=emilio,android-reviewers,jdescottes,tcampbell
https://docs.astral.sh/ruff/rules/invalid-envvar-default/

Differential Revision: https://phabricator.services.mozilla.com/D274700
2025-12-10 06:06:22 +00:00
Alex Hochheiden 7054a1ab49 Bug 2003412 - PLR warnings auto-fixed by ./mach lint --fix . r=emilio,geckoview-reviewers,taskgraph-reviewers,releng-reviewers,toolkit-telemetry-reviewers,layout-reviewers,jdescottes,bhearsum,nalexander,perftest-reviewers,sparky
Differential Revision: https://phabricator.services.mozilla.com/D274687
2025-12-10 06:05:48 +00:00
Heitor Neiva 3c49559429 Bug 1996349 - Switch xpcshell tests to highcpu pool r=florian,releng-reviewers,taskgraph-reviewers,ahal
Differential Revision: https://phabricator.services.mozilla.com/D270050
2025-11-03 20:08:02 +00:00
Florian Quèze 2465f82a89 Bug 1995627 - show non-default timeout factors on test markers in resource usage profiles, r=ahal.
Differential Revision: https://phabricator.services.mozilla.com/D269626
2025-10-24 00:38:02 +00:00
Florian Quèze e0622dafb3 Bug 1995627 - optimize xpcshell test parallel execution by avoiding long-running tests at the end, r=jmaher.
Differential Revision: https://phabricator.services.mozilla.com/D269624
2025-10-22 19:26:48 +00:00
Florian Quèze 89fb98b5ec Bug 1995593 - don't upload crash dumps from self tests, r=jmaher.
Differential Revision: https://phabricator.services.mozilla.com/D269435
2025-10-21 20:37:43 +00:00
Florian Quèze 1d896cc495 Bug 1995593 - don't retry self tests that are expected to fail, r=jmaher.
Differential Revision: https://phabricator.services.mozilla.com/D269432
2025-10-21 20:37:41 +00:00
Florian Quèze 4ce73f1fab Bug 1995058 - set the default xpcshell test timeout to 30s, r=jmaher.
Differential Revision: https://phabricator.services.mozilla.com/D269185
2025-10-21 08:45:07 +00:00
Florian Quèze c221107ad8 Bug 1995058 - add support for setting the timeoutfactor per platform in xpcshell.yml, r=jmaher,taskgraph-reviewers.
Differential Revision: https://phabricator.services.mozilla.com/D269181
2025-10-21 08:44:59 +00:00
agoloman c5d27cd7ef Revert "Bug 1995058 - add support for setting the timeoutfactor per platform in xpcshell.yml, r=jmaher,taskgraph-reviewers." for causing gecko decions tasks failures.
This reverts commit 90107538e2.
2025-10-20 19:35:23 +00:00
Florian Quèze 90107538e2 Bug 1995058 - add support for setting the timeoutfactor per platform in xpcshell.yml, r=jmaher,taskgraph-reviewers.
Differential Revision: https://phabricator.services.mozilla.com/D269181
2025-10-20 19:24:10 +00:00
Florian Quèze 69a33a86ba Bug 1960759 - replace 'fatal error' with just 'error' when replaying the log of xpcshell tests failing in parallel to avoid filling the treeherder error summary with unimportant warnings, r=jmaher.
Differential Revision: https://phabricator.services.mozilla.com/D268442
2025-10-13 20:29:46 +00:00
Florian Quèze 9654e67b38 Bug 1993270 - when replaying the log of xpcshell tests failing in parallel, replace 'ERROR: AddressSanitizer' with 'ERROR (will retry): AddressSanitizer' to avoid failing the job, r=jmaher.
Differential Revision: https://phabricator.services.mozilla.com/D268441
2025-10-13 20:29:43 +00:00
Florian Quèze 5f9c20b03b Bug 1993483 - show a marker in resource usage profiles for time spent replaying full logs of failed xpcshell tests, r=jmaher.
Differential Revision: https://phabricator.services.mozilla.com/D268138
2025-10-09 15:43:09 +00:00
Florian Quèze 9719732528 Bug 1993255 - make the treeherder log not log PROCESS-CRASH for crashes that didn't fail the job, r=jmaher.
Differential Revision: https://phabricator.services.mozilla.com/D267997
2025-10-09 05:56:23 +00:00
Florian Quèze 3c4c1f8944 Bug 1993255 - xpcshell tests that crash while running in parallel should be retried, r=jmaher.
Differential Revision: https://phabricator.services.mozilla.com/D267967
2025-10-09 05:56:21 +00:00
Florian Quèze 7396723211 Bug 1992477 - replay the logs of failures that happened in parallel, without failing the treeherder jobs, r=jmaher.
Differential Revision: https://phabricator.services.mozilla.com/D267455
2025-10-07 19:34:11 +00:00
iulian moraru 472cfdd840 Revert "Bug 1992477, Bug 1992181 - replay the logs of failures that happened in parallel, without failing the treeherder jobs, r=jmaher." for causing xpcshell failures on test_ExperimentManager_prefs.
This reverts commit 71e9bf363f.

Revert "Bug 1992181 - show 'artifact' markers for files found in the upload dir when stopping the resource monitor, r=ahal."

This reverts commit aad0350f35.

Revert "Bug 1992181 - Include 'process_output' structured log messages in resource usage profiles, r=ahal."

This reverts commit 0009d2b54a.

Revert "Bug 1992181 - Resource usage profiles should show test_status and log messages, r=ahal."

This reverts commit d550c4e679.
2025-10-07 02:30:07 +00:00
Florian Quèze 71e9bf363f Bug 1992477 - replay the logs of failures that happened in parallel, without failing the treeherder jobs, r=jmaher.
Differential Revision: https://phabricator.services.mozilla.com/D267455
2025-10-06 23:46:03 +00:00
Florian Quèze c8451d8925 Bug 1991768 - use single file mode for xpcshell tests listed in multiple manifests but skipped in all but one, r=jmaher.
Differential Revision: https://phabricator.services.mozilla.com/D266903
2025-10-01 10:00:28 +00:00
Florian Quèze eb839e028d Bug 1991090 - support parsing run-sequentially annotations as manifestparser conditions, r=jmaher,application-update-reviewers,media-playback-reviewers,webrtc-reviewers,sync-reviewers,bytesized,ng,skhamis,alwu.
Differential Revision: https://phabricator.services.mozilla.com/D266378
2025-09-28 12:27:24 +00:00
Florian Quèze 8203a10161 Bug 1990180 - Stop limitting to 2 xpcshell tests running at once on msix, r=jmaher.
Differential Revision: https://phabricator.services.mozilla.com/D265760
2025-09-23 13:36:31 +00:00
Florian Quèze 1d3bfe7467 Bug 1989043 - Reduce the number of parallel xpcshell tests on Windows to twice the core count, r=jmaher.
Differential Revision: https://phabricator.services.mozilla.com/D265107
2025-09-17 16:27:07 +00:00
Florian Quèze afe82b2196 Bug 1649576 - run xpcshell self-tests in parallel, r=ahal.
Differential Revision: https://phabricator.services.mozilla.com/D263358
2025-09-15 13:24:55 +00:00
Florian Quèze 68a408e8dd Bug 1987939 - add 'selftests', 'parallel', 'sequential' and 'retry' markers in xpcshell resource usage profiles, r=ahal.
Differential Revision: https://phabricator.services.mozilla.com/D264411
2025-09-11 21:23:55 +00:00
Florian Quèze 035e0ec0e5 Bug 1987383 - fix the normalized xpcshell test ids, r=ahal.
Differential Revision: https://phabricator.services.mozilla.com/D264389
2025-09-10 16:27:04 +00:00
Valentin Gosu 5273750cc6 Bug 1943534 - Use correct npm path in win and mac xpcshell tests r=jmaher
Differential Revision: https://phabricator.services.mozilla.com/D263812
2025-09-07 19:59:59 +00:00