974376 - Marionnette JS tests are reported twice on Travis and locally.

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia, defect)

Description

[Anthony Ricaud (:rik)]

I've first noticed this in bug 939186 comment 9. I've bisected on Travis results and I believe 0c44f595055203484d1e8a293d9724ab609f213f introduced this (bug 968675).

Tim created a pull request with this patch reverted to check that assumption: https://github.com/mozilla-b2g/gaia/pull/16449. For whatever reason, Travis won't start running this :( https://travis-ci.org/mozilla-b2g/gaia/builds/19182840

Comment 1

[Tim Guan-tin Chien [:timdream] (please needinfo)]

(In reply to Anthony Ricaud (:rik) from comment #0)
> I've first noticed this in bug 939186 comment 9. I've bisected on Travis
> results and I believe 0c44f595055203484d1e8a293d9724ab609f213f introduced
> this (bug 968675).
> 
> Tim created a pull request with this patch reverted to check that
> assumption: https://github.com/mozilla-b2g/gaia/pull/16449. For whatever
> reason, Travis won't start running this :(
> https://travis-ci.org/mozilla-b2g/gaia/builds/19182840

It looks like all jobs error'd at npm. I have restarted the job just now; will post the saved log here.

Comment 2

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Attachment: https://s3.amazonaws.com/archive.travis-ci.org/jobs/19182842/log.txt

Comment 3

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Attachment: https://s3.amazonaws.com/archive.travis-ci.org/jobs/19182843/log.txt

Comment 4

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Attachment: https://s3.amazonaws.com/archive.travis-ci.org/jobs/19182845/log.txt

Comment 5

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Attachment: https://s3.amazonaws.com/archive.travis-ci.org/jobs/19182846/log.txt

Comment 6

[Yuren [:yurenju] (AFK ~ Oct. 12) (volunteer mode)]

I can reproduce it in locally.

Comment 7

[Yuren [:yurenju] (AFK ~ Oct. 12) (volunteer mode)]

Attachment: github PR: https://github.com/mozilla-b2g/gaia/pull/16473

root cause is gaia-marionette doesn't execlude build_stage, so actually we have ran integration tests for email, clock, camera twice before.

Comment 8

[Yuren [:yurenju] (AFK ~ Oct. 12) (volunteer mode)]

got green on travis.

https://travis-ci.org/mozilla-b2g/gaia/builds/19240493

Comment 9

[Evan Tseng [:evanxd]]

Comment on attachment 8378825 [details] [review]
github PR: https://github.com/mozilla-b2g/gaia/pull/16473

Hi Yuren,

r+, good job!

The linters and gaia_ui_tests actions are not passed.
But the patch is not related with that.

Comment 10

[Tim Guan-tin Chien [:timdream] (please needinfo)]

What's blocking this from landing to master? We should be able to land this stability improvement.

Comment 11

[Yuren [:yurenju] (AFK ~ Oct. 12) (volunteer mode)]

landed

https://github.com/mozilla-b2g/gaia/commit/11c5cb7b56a0fced1d0ad5f7580cc1e16faad368