Closed Bug 1447494 Opened 4 years ago Closed 4 years ago

Stop importing/using gcli test helpers


(DevTools Graveyard :: Graphic Commandline and Toolbar, enhancement, P2)



(firefox63 fixed)

Firefox 63
Tracking Status
firefox63 --- fixed


(Reporter: ochameau, Assigned: yulia)




(1 file)

This module is referenced in many test related file, whereas it doesn't seem to be used much outside of devtools/client/commandline:

List of references:

It is used in a couple of places to test gcli commands, but this doesn't justify all these imports at first sight.
Product: Firefox → DevTools
I checked this and it looks like it has been fully removed. :ochameau can you confirm?
Flags: needinfo?(poirot.alex)
I may have removed all the callsites in bug 

But the helpers file is still referenced by many *.ini files:
It would be interesting to remove them to ensure we really dropped all usages of it.

Also, this comment seems to suggest we may be able to get rid of this devtools/client/commandline folder completely!
Flags: needinfo?(poirot.alex)
Removed remaining imports of commandline helpers, as they are no longer used anywhere
You would have to update file in devtools/client to dereference the one from commandline:
Assignee: nobody → ystartsev
Comment on attachment 9001242 [details]
Bug 1447494 - remove all instances of commandline/helpers; r=ochameau

Alexandre Poirot [:ochameau] has approved the revision.
Attachment #9001242 - Flags: review+
Pushed by
remove all instances of commandline/helpers; r=ochameau
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
Product: DevTools → DevTools Graveyard
You need to log in before you can comment on or make changes to this bug.