Running a non-existing test suite doesn't fail

[Cris Dywan]

This has tripped me up before and finally I realized what the issue is:

> autopilot run -v dialer_app
11:51:55.164 INFO run:64 - Autopilot Source Version: 1.4.0
Autopilot Package Version: 1.4+14.04.20140213-0ubuntu1
could not import package dialer_app: No module named dialer_app
Loading tests from:

Tests running...

Ran 0 tests in 0.000s
OK

There's no test package there and the tests I want to run don't exist. Yet I get a OK there, no error.

You may argue that it's "obvious" but believe me it isn't.

[Thomi Richards (thomir-deactivatedaccount)]

Agreed - it should print an error message. It does exit non-0 though, so it's not totally broken :)

[Corey Goldberg (coreygoldberg)]

fixed in linked branch.. waiting for final review and landing. (CI is currently failing for an unrelated reason. waiting for resolution on that to see when this will land.)

[Launchpad Janitor (janitor)]

This bug was fixed in the package autopilot - 1.4+14.04.20140408-0ubuntu1

---------------
autopilot (1.4+14.04.20140408-0ubuntu1) trusty; urgency=low

  [ Thomi Richards ]
  * Fix flake8 errors in trunk.
  * Make the make-coverage.sh script exit non-0 when tests fail.
  * Don't abort print_tree when we catch the StateNotFoundError
    exception. (LP: #1276672)
  * Tweak the vis tool so the splitter between the tree widget and the
    properties pane cannot be completely dragged to one side or the
    other. (LP: #1281360)
  * Fix functional test that was very dependant on timing. (LP:
    #1301005)
  * Make logger objects within autopilot private.

  [ Nicholas Skaggs ]
  * Don't abort print_tree when we catch the StateNotFoundError
    exception. (LP: #1276672)

  [ Corey Goldberg ]
  * When no tests are found, display "Did not find any tests" and exit,
    instead of invoking the runner. (LP: #1282995)
  * Fix functional test that was very dependant on timing. (LP:
    #1301005)

  [ nskaggs ]
  * Don't abort print_tree when we catch the StateNotFoundError
    exception. (LP: #1276672)

  [ Max Brustkern ]
  * Update documentation to reflect new terminology (LP:1288054) (LP:
    #1288054)
  * Make logger objects within autopilot private.
 -- Ubuntu daily release <email address hidden> Tue, 08 Apr 2014 00:29:56 +0000