Closed
Bug 1855295
Opened 1 year ago
Closed 1 year ago
convert .ini manifests to .toml: batch 8 dom/[a-l]**/mochitest.ini r=jmaher
Categories
(Testing :: General, task)
Tracking
(firefox120 fixed)
RESOLVED
FIXED
120 Branch
Tracking | Status | |
---|---|---|
firefox120 | --- | fixed |
People
(Reporter: tmarble, Assigned: tmarble)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
Convert batch 7 of *.ini files to *.toml (see list below)
Success criteria:
- INI files moved to TOML to preserve history
- References to INI files replaced with TOML equivalents (e.g. in moz.build)
Files:
1 ./browser/components/extensions/test/mochitest/mochitest.ini
2 ./browser/components/originattributes/test/mochitest/mochitest.ini
3 ./browser/components/resistfingerprinting/test/mochitest/mochitest.ini
4 ./browser/extensions/formautofill/test/mochitest/creditCard/mochitest.ini
5 ./browser/extensions/formautofill/test/mochitest/mochitest.ini
6 ./caps/tests/mochitest/mochitest.ini
7 ./docshell/test/iframesandbox/mochitest.ini
8 ./docshell/test/mochitest/mochitest.ini
9 ./docshell/test/navigation/mochitest.ini
10 ./dom/abort/tests/mochitest.ini
11 ./dom/animation/test/mochitest.ini
12 ./dom/base/test/fullscreen/mochitest.ini
13 ./dom/base/test/jsmodules/importmaps/mochitest.ini
14 ./dom/base/test/meta_viewport/mochitest.ini
15 ./dom/base/test/mochitest.ini
16 ./dom/base/test/useractivation/mochitest.ini
17 ./dom/battery/test/mochitest.ini
18 ./dom/bindings/test/mochitest.ini
19 ./dom/broadcastchannel/tests/mochitest.ini
20 ./dom/cache/test/mochitest/mochitest.ini
21 ./dom/canvas/test/crash/mochitest.ini
22 ./dom/canvas/test/crossorigin/mochitest.ini
23 ./dom/canvas/test/mochitest.ini
24 ./dom/canvas/test/webgl-mochitest/mochitest.ini
25 ./dom/console/tests/mochitest.ini
26 ./dom/credentialmanagement/identity/tests/mochitest/mochitest.ini
27 ./dom/credentialmanagement/tests/mochitest/mochitest.ini
28 ./dom/crypto/test/mochitest.ini
29 ./dom/encoding/test/mochitest.ini
30 ./dom/events/test/mochitest.ini
31 ./dom/events/test/pointerevents/mochitest.ini
32 ./dom/fetch/tests/mochitest.ini
33 ./dom/file/ipc/tests/mochitest.ini
34 ./dom/file/tests/mochitest.ini
35 ./dom/filesystem/compat/tests/mochitest.ini
36 ./dom/filesystem/tests/mochitest.ini
37 ./dom/fs/test/common/mochitest.ini
38 ./dom/fs/test/mochitest/mochitest.ini
39 ./dom/fs/test/mochitest/worker/mochitest.ini
40 ./dom/html/test/dialog/mochitest.ini
41 ./dom/html/test/forms/mochitest.ini
42 ./dom/html/test/forms/without_selectionchange/mochitest.ini
43 ./dom/html/test/mochitest.ini
44 ./dom/ipc/tests/mochitest.ini
45 ./dom/jsurl/test/mochitest.ini
46 ./dom/l10n/tests/mochitest/mochitest.ini
47 ./dom/localstorage/test/mochitest.ini
48 ./dom/locks/test/mochitest.ini
Assignee | ||
Comment 1•1 year ago
|
||
Should say batch 8 in the description
Assignee | ||
Comment 2•1 year ago
|
||
Updated•1 year ago
|
Assignee: nobody → tmarble
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Pushed by jmaher@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c366cc4c3190
convert .ini manifests to .toml: batch 8 dom/[a-l]**/mochitest.ini r=jmaher,jgilbert,extension-reviewers,credential-management-reviewers,robwu,sgalich
Comment 4•1 year ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
status-firefox120:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 120 Branch
Comment 5•1 year ago
|
||
It seems dom/indexedDB/test was skipped, is it being handled somewhere else ?
Updated•1 year ago
|
Flags: needinfo?(tmarble)
Comment 6•1 year ago
|
||
I noticed in mochitest.toml that the comments got put under the wrong test. They should be with test_bug1433073.html
.
You can see them in the original commit at https://hg.mozilla.org/mozilla-central/rev/7e93722dbf4d1f329a71daa10d379ad9a0a283ad
So it should be like this:
["test_bug1433073.html"]
skip-if = [
"os == 'android'", # See bug 1632196
"headless", # See bug 1632438
]
Assignee | ||
Comment 7•1 year ago
|
||
@janv unconventional manifest names will be handled in a subsequent batch
@Mathew I will fix the comments in a future changeset
Flags: needinfo?(tmarble)
You need to log in
before you can comment on or make changes to this bug.
Description
•