Closed Bug 1581165 Opened 3 years ago Closed 1 year ago

[META] make mocha/sasslint tests run from `mach test`

Categories

(Firefox :: Messaging System, task, P3)

task

Tracking

()

RESOLVED WONTFIX
Tracking Status
firefox71 --- fix-optional

People

(Reporter: dmosedale, Unassigned)

References

(Blocks 2 open bugs)

Details

(Keywords: meta)

Attachments

(1 obsolete file)

Currently, to run the messaging system / new tab unit tests and sasslint in mozilla-central, one has to use a few npm commands, documented on firefox-source-docs. These should really just run automatically when using mach test on/in the appropriate directory. If we stick with SASS, we might well want to move that piece into mach lint, but just getting them running at all using one of the basic mach commands is the most important first step.

Depends on: 1581545
Priority: -- → P2
Assignee: nobody → dmose
Status: NEW → ASSIGNED

Comment on attachment 9093056 [details]
Bug 1581165 - Make New Tab docs reference bug to enable mach test, r?Mardak

Revision D46039 was moved to bug 1581545. Setting attachment 9093056 [details] to obsolete.

Attachment #9093056 - Attachment is obsolete: true
Assignee: dmose → nobody
Status: ASSIGNED → NEW

:ochameau mentioned that there was some work on a similar sort of issue in 1450655, and that we might find some inspiration there. Thanks, Alex!

See Also: → 1401189
See Also: → 1539441, 1575574

I believe dmose was looking to reference bug 1540655 not 145…

See Also: → 1540655
Summary: make karma/mocha/sasslint tests run from `mach test browser/components/newtab` → [META] make karma/mocha/sasslint tests run from `mach test browser/components/newtab`
No longer blocks: ujmc-test-automation
Blocks: bugzy-epic
No longer blocks: as-build-meta
Summary: [META] make karma/mocha/sasslint tests run from `mach test browser/components/newtab` → [META] make mocha/sasslint tests run from `mach test browser/components/newtab`
Summary: [META] make mocha/sasslint tests run from `mach test browser/components/newtab` → [META] make mocha/sasslint tests run from `mach test`
No longer blocks: node-toplevel-packages
No longer blocks: old-bugzy-epic
Priority: P2 → P3
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.