Closed Bug 745904 Opened 8 years ago Closed 7 years ago

GCLI needs extra unit tests for recently added code

Categories

(DevTools :: Console, defect, P2)

defect

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 15

People

(Reporter: jwalker, Assigned: jwalker)

References

Details

(Whiteboard: [fixed-in-fx-team])

Attachments

(1 file, 1 obsolete file)

Target Milestone: --- → Firefox 15
Attached patch Upload 1 (obsolete) — Splinter Review
a.k.a https://github.com/campd/gcli/pull/32 (with the 2 extra commits)
Assignee: nobody → jwalker
Status: NEW → ASSIGNED
Attachment #623119 - Flags: review?(dcamp)
Attached patch Upload 2Splinter Review
I'm not sure if we worry about file permissions - the previous patch had some files set to 755 rather than 644. This fixes that.
Attachment #623119 - Attachment is obsolete: true
Attachment #623119 - Flags: review?(dcamp)
Attachment #623223 - Flags: review?(dcamp)
Attachment #623223 - Flags: review?(dcamp) → review+
https://hg.mozilla.org/mozilla-central/rev/1e82a90aeee4
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.