GCLI needs extra unit tests for recently added code

RESOLVED FIXED in Firefox 15

Status

()

Firefox
Developer Tools: Console
P2
normal
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: jwalker, Assigned: jwalker)

Tracking

unspecified
Firefox 15
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

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

Attachments

(1 attachment, 1 obsolete attachment)

Specifically to answer https://github.com/campd/gcli/pull/31#commitcomment-1208061
Target Milestone: --- → Firefox 15
https://github.com/campd/gcli/pull/32
Created attachment 623119 [details] [diff] [review]
Upload 1

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)
Created attachment 623223 [details] [diff] [review]
Upload 2

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)
Relevant tries:
https://tbpl.mozilla.org/?tree=Try&rev=27b9b61a663e
https://tbpl.mozilla.org/?tree=Try&rev=77b47c8bbc35
https://tbpl.mozilla.org/?tree=Try&rev=1ce36097dc58

Updated

5 years ago
Attachment #623223 - Flags: review?(dcamp) → review+
Try before land:
https://tbpl.mozilla.org/?tree=Try&rev=eba127ab7962

Fx-Team:
https://tbpl.mozilla.org/?tree=Fx-Team&rev=0f08ab4dfcac
Whiteboard: [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/1e82a90aeee4
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.