Closed Bug 1425349 Opened 2 years ago Closed 2 years ago

Fix mozscreenshots logging where multiple log arguments were used

Categories

(Testing :: mozscreenshots, defect)

defect
Not set

Tracking

(firefox59 fixed)

RESOLVED FIXED
mozilla59
Tracking Status
firefox59 --- fixed

People

(Reporter: MattN, Assigned: MattN)

References

Details

Attachments

(1 file)

> log.info("Saving screenshots to:", screenshotPath);
was incorrectly converted to 
> this.mochitestScope.info("Saving screenshots to:", screenshotPath);

This is incorrect because `info` will only log the first argument.
Comment on attachment 8936955 [details]
Bug 1425349 - Fix mozscreenshots logging where multiple log arguments were used.

https://reviewboard.mozilla.org/r/207678/#review213656

Thanks!
Attachment #8936955 - Flags: review?(jhofmann) → review+
Pushed by mozilla@noorenberghe.ca:
https://hg.mozilla.org/integration/autoland/rev/a8c870edc43d
Fix mozscreenshots logging where multiple log arguments were used. r=johannh
https://hg.mozilla.org/mozilla-central/rev/a8c870edc43d
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
You need to log in before you can comment on or make changes to this bug.