976152 - Move test-agent off of npm

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::TestAgent, defect)

Description

[Gareth Aye [:gaye] (back from PTO)]

:kgrandon did some excellent work to fetch our package.json dependencies in gaia off of npm. We are still somewhat dependent on npm in random places (ie test-agent) which is still hurting us (for example https://travis-ci.org/mozilla-b2g/gaia/jobs/19500054).

Comment 1

[Kevin Grandon :kgrandon]

Should the test-agent modules simply be bundled into the current dependency list? Current dependencies are: 

"test-agent": "0.18.0",
"marionette-client": "0.5.4",
"b2g-scripts": "0.3.0",
"chai": "0.5.3",
"mocha": "~1.7",
"xpcwindow": "0.4.4"

Comment 2

[James Lal [:lightsofapollo]]

FTR- we only need these

"test-agent": "0.18.0",
"chai": "0.5.3",
"mocha": "~1.7"

(and mocha/chai could be omitted they are checked into the tree for browser usage anyway)

Comment 3

[Gareth Aye [:gaye] (back from PTO)]

Attachment: Link to Github pull-request: https://github.com/mozilla-b2g/gaia-node-modules/pull/2

Comment 4

[Gareth Aye [:gaye] (back from PTO)]

:lightsofapollo gave me a shrug+ since :kgrandon mia

Comment 5

[Julien Wajsberg [:julienw]]

Cc Yuren and Ricky so that they know what happens here.

As long as Travis is happy with this change, then it should work. But it's not easy to check with the new gaia-node-modules.

Comment 6

[Gareth Aye [:gaye] (back from PTO)]

Attachment: Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/16581

Comment 7

[Gareth Aye [:gaye] (back from PTO)]

Ok well travis is good and I :lightsofapollo was onboard for this change, so I'll merge.

Comment 8

[Gareth Aye [:gaye] (back from PTO)]

https://github.com/mozilla-b2g/gaia/commit/0477d1e32cfac32167aedcf9dbc078df30b5669a

Comment 9

[Julien Wajsberg [:julienw]]

gaia-node-modules master: 3ea87743d2c3a285e023bca6f7441bd7bc66aa32