Closed
Bug 1412012
Opened 7 years ago
Closed 7 years ago
[mozdebug] Add some basic unittests for mozdebug
Categories
(Testing :: Mozbase, enhancement)
Testing
Mozbase
Tracking
(firefox58 fixed)
RESOLVED
FIXED
mozilla58
Tracking | Status | |
---|---|---|
firefox58 | --- | fixed |
People
(Reporter: ahal, Assigned: ahal)
References
Details
Attachments
(1 file)
We recently broke the --debugger option on all test suites when fixing some linter problems. This would have been caught if mozdebug had even a trivial test. Let's add a test or two so this doesn't happen again.
Comment hidden (mozreview-request) |
Assignee | ||
Comment 2•7 years ago
|
||
Try: https://treeherder.mozilla.org/#/jobs?repo=try&author=ahalberstadt@mozilla.com&fromchange=dc2ee6d66f88686c3df7882fe7eb530faeb718d9&tochange=c5603c8ed9dd3a57730d9ed876ed2ec9364eb3e0
Assignee: nobody → ahalberstadt
Status: NEW → ASSIGNED
Comment 3•7 years ago
|
||
mozreview-review |
Comment on attachment 8922780 [details] Bug 1412012 - [mozdebug] Add a basic test for get_default_debugger_name, https://reviewboard.mozilla.org/r/193930/#review199036 r+ again
Attachment #8922780 -
Flags: review?(jmaher) → review+
Pushed by ahalberstadt@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/dab6d6b8d11e [mozdebug] Add a basic test for get_default_debugger_name, r=jmaher
Comment 5•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/dab6d6b8d11e
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox58:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in
before you can comment on or make changes to this bug.
Description
•