Store the path to screenshots in reports for l10n tests

RESOLVED WONTFIX

Status

RESOLVED WONTFIX
7 years ago
3 years ago

People

(Reporter: Pike, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [lib])

(Reporter)

Description

7 years ago
The file.createUnique call in screenshot.js changes file.path and file.leafName to the file that it actually created, including the -1 -2 etc.

We can use that to map reports to the screenshot for them, and should.

That requires .create() to return that (*), and then the tests need to be refactored to expect.fail after the screenshot, because we need to pass the screenshot path in there. That might be the tricky part here.
Is is related to our shared modules. Mozmill 2 has its own implementation which is similar but not identical. We can check this later.
Component: Mozmill → Mozmill Shared Modules
Product: Testing → Mozilla QA
QA Contact: mozmill → mozmill-shared-modules
Component: Mozmill Shared Modules → Mozmill Tests
Whiteboard: [lib]
Mozmill tests are no longer in use. So closing this bug.
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.