Peptest should be updated to reflect mozlog api changes introduced by bug 891488

RESOLVED FIXED

Status

Testing Graveyard
Peptest
RESOLVED FIXED
5 years ago
29 days ago

People

(Reporter: chmanchester, Unassigned)

Tracking

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
Bug 891488 removes the logfile parameter for mozlog's getLogger method. Peptest needs to be updated to reflect this change.
(Reporter)

Comment 1

5 years ago
Created attachment 777417 [details] [diff] [review]
Change peptest to work with changes to mozlog's getLogger.

Sadly requires some additional boilerplate over the current approach, but this preserves the existing behavior.
Attachment #777417 - Flags: review?(ahalberstadt)
Comment on attachment 777417 [details] [diff] [review]
Change peptest to work with changes to mozlog's getLogger.

Review of attachment 777417 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks!
Attachment #777417 - Flags: review?(ahalberstadt) → review+
(Reporter)

Comment 3

5 years ago
https://github.com/mozilla/peptest/commit/57b0b2a60f6ac19aa3483d71b57f79f58c5731be
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Updated

29 days ago
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.