Rudy Lu [:rudyl] (inactive after 2015/7/3) Assignee
	Description • 10 years ago

+++ This bug was initially created as a clone of Bug #997045 +++

When working on Bug #997045, I found that I could not get the travis to pass the tests for marionette-apps, when I update the dependency of marionette-js-client to 1.1.6.

So, I fall back to trying to update the dependency to 1.1.5 and encountered a similar issue here.
Please refer to the travis job,
https://travis-ci.org/mozilla-b2g/marionette-apps/builds/23410030

	Rudy Lu [:rudyl] (inactive after 2015/7/3) Assignee
	Comment 1 • 10 years ago

This patch is to increase the timeout in waitForApp test, and seems it could work for travis to pass.

Evan, Gareth,

Do you think this is good to merge?
Thanks.

	Evan Tseng [:evanxd]
	Comment 2 • 10 years ago

Hi Rudy,
Nice work.
Just investigating the timeout change.
Thanks.

	Gareth Aye [:gaye] (back from PTO)
	Comment 3 • 10 years ago

I think marionette-client should really be made into a "peer dependency"

	Gareth Aye [:gaye] (back from PTO)
	Comment 4 • 10 years ago

Otherwise this works for me.

	Rudy Lu [:rudyl] (inactive after 2015/7/3) Assignee
	Comment 5 • 10 years ago

Comment on attachment 8409568 [details] [review]
Patch V1 - pull request 26

Patch updated to use peer dependency as suggested.
Evan, could you review this?

Thanks.

	Evan Tseng [:evanxd]
	Comment 6 • 10 years ago

For the timeout issue, we should fix that at Bug 1000681.

	Evan Tseng [:evanxd]
	Comment 7 • 10 years ago

Hi Rudy,
I think we should update dependency of marionette-js-client to 1.1.7 with `"marionette-client": "1.1.7"`. Please refer to https://github.com/mozilla-b2g/marionette-apps/pull/26/files#r11937058.

	Rudy Lu [:rudyl] (inactive after 2015/7/3) Assignee
	Comment 8 • 10 years ago

At the end, I think we should not update the dependency to 1.1.5 since it does not require that.
So, just modify this to peer dependency and keep it as 1.0.0.
Update the bug title as well.

	Rudy Lu [:rudyl] (inactive after 2015/7/3) Assignee
	Comment 9 • 10 years ago

Landed,
https://github.com/mozilla-b2g/marionette-apps/commit/12e79a9c4fb862aef34350400240433d3c4269ec

Thanks for the feedback and review.