mojo-apptests: Fix detecting failure when no test ran.

mojo-apptests: Fix detecting failure when no test ran.

BUG=none
R=msw@chromium.org

Committed: https://crrev.com/c36c6282445549860ff359b3a9bc1515a8566e1f
Cr-Commit-Position: refs/heads/master@{#362825}

[sadrul, 2015-12-02 20:38:45]

sadrul@chromium.org changed reviewers:
+ sky@chromium.org

[sadrul, 2015-12-02 20:38:45]

+sky for owner

[sky, 2015-12-02 20:42:29]

LGTM

[sadrul, 2015-12-02 20:49:13]

Hrm, this script is hella confusing. With the first patchset, the
mus_wm_apptests passes, but it still says it fails after running all the
apptests:

Running mojo:clipboard_apptests... [  PASSED  ]
Running mojo:network_service_apptests... [  PASSED  ]
Running mojo:resource_provider_apptests... [  PASSED  ]
Running mojo:mus_apptests... WARNING: tests are forced to gtest_isolated until
http://crbug.com/529487 is fixed
[  PASSED  ]
Running mojo:filesystem_apptests... [  PASSED  ]
Running mojo:html_viewer_apptests... [  PASSED  ]
Running mojo:html_viewer_apptests new frame... [  PASSED  ]
Running mojo:mandoline_browser_apptests... [  PASSED  ]
Running mojo:mash_wm_apptests... [  PASSED  ]
Running mojo:media_apptests... [  PASSED  ]
Running mojo:sql_apptests... [  PASSED  ]
Running mojo:web_view_apptests... [  PASSED  ]
[==========] 167 tests ran.
[  PASSED  ] 166 tests.
[  FAILED  ] 1 tests, listed below:
[  FAILED  ] mojo:mash_wm_apptests

So I have made some more changes in the script, so now it still says things pass
at the end. PTAL?

[sky, 2015-12-02 21:10:37]

LGTM - I reverted my patch to disable, so it isn't critical to land this asap,
but it should be fixed.

[sadrul, 2015-12-02 22:38:39]

The CQ bit was checked by sadrul@chromium.org

[commit-bot: I haz the power, 2015-12-02 22:40:38]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1499463002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1499463002/20001

[commit-bot: I haz the power, 2015-12-03 00:09:04]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-03 00:09:48]

Description was changed from

==========
mojo-apptests: Fix detecting failure when no test ran.

BUG=none
R=msw@chromium.org
==========

to

==========
mojo-apptests: Fix detecting failure when no test ran.

BUG=none
R=msw@chromium.org

Committed: https://crrev.com/c36c6282445549860ff359b3a9bc1515a8566e1f
Cr-Commit-Position: refs/heads/master@{#362825}
==========

[commit-bot: I haz the power, 2015-12-03 00:09:49]

Patchset 2 (id:??) landed as
https://crrev.com/c36c6282445549860ff359b3a9bc1515a8566e1f
Cr-Commit-Position: refs/heads/master@{#362825}

[sky, 2015-12-03 04:36:52]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/1495783002/ by sky@chromium.org.

The reason for reverting is: Speculative revert as cq is failing in an unusual
way when running mojo_apptests:
'module' object has no attribute 'should_start_xvfb'

from
https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium....