Open Bug 1713577 Opened 4 years ago Updated 1 year ago

Add missing license headers for tests

Categories

(Developer Infrastructure :: Lint and Formatting, task, P3)

Tracking

(Not tracked)

People

(Reporter: Sylvestre, Unassigned)

References

(Depends on 1 open bug)

Details

Started in bug 1562642 for the code itself.

The next step is to do the same but for tests with the public domain
https://searchfox.org/mozilla-central/rev/d3b116e554eb3546a95851e81f7ddc53d5b7e096/tools/lint/license/__init__.py#21-24

So, the goal would be:

  1. Uncomment https://searchfox.org/mozilla-central/rev/d3b116e554eb3546a95851e81f7ddc53d5b7e096/tools/lint/license/__init__.py#179-181
  2. Run it
  3. Fix as much as possible mach lint --linter license --fix <path>
  4. Check that tests can still run
  5. Add the (long list) of new items in the exclude list to https://searchfox.org/mozilla-central/rev/d3b116e554eb3546a95851e81f7ddc53d5b7e096/tools/lint/license.yml

I tried to run it and got:
✖ 101050 problems (101050 errors, 0 warnings, 0 fixed)
glad this linter has an autofix :)

For example:
./mach lint -l license --fix xpfe/appshell/test/test_windowlessBrowser.xhtml

Depends on: 1734807
Severity: -- → S3
Priority: -- → P3
Product: Firefox Build System → Developer Infrastructure
Depends on: 1898558
Component: Source Code Analysis → Lint and Formatting
Depends on: 1898890
You need to log in before you can comment on or make changes to this bug.