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)

Default
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

Blocks: 1821199

Should say batch 8 in the description

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
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 120 Branch

It seems dom/indexedDB/test was skipped, is it being handled somewhere else ?

Flags: needinfo?(tmarble)

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
]

@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.

Attachment

General

Creator:
Created:
Updated:
Size: