Closed Bug 1815451 Opened 3 years ago Closed 3 years ago

[meta] Put all mochitest test manifest files under browser/ into alphabetical order.

Categories

(Testing :: Mochitest, task)

Default
task

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mconley, Unassigned)

References

Details

(Keywords: meta)

See bug 1815447 for details. Alphabetical order is what the testing harness is running these things in anyways, so this shouldn't change the behaviour of our test runs.

If someone wants to pick this up, we have a linter that'll help! You can run it using:

./mach lint -l test-manifest-alpha browser/ --warnings

To get a list of all of the manifest files that are out of order. Then the job is to put each of those files in alphabetical order so that the linter doesn't complain.

Hey Standard8, when we roll out a non-ESLint linting rule like this, do you know if there's a mechanism to enable error level reporting for individual directories? I'm basically trying to see if we can fix each directory at a time, and finish by putting that directory into the error level state for the linter.

Flags: needinfo?(standard8)

I think you could use the exclude list in test-manifest-alpha.yml

Flags: needinfo?(standard8)
Mentor: mconley
Keywords: good-first-bug
Summary: Put all mochitest test manifest files under browser/ into alphabetical order. → [meta] Put all mochitest test manifest files under browser/ into alphabetical order.
Depends on: 1816564
Depends on: 1817320
Depends on: 1829673
Depends on: 1830075
Depends on: 1830991

With bug 1830991 closed out, all of browser/ is now in alphabetical, and we can close this out!

Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.