Add better logging when failing to load a test.

typ/tests/main_test.py

 # Copyright 2014 Dirk Pranke. All rights reserved.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
 # You may obtain a copy of the License at
 #
 #    http://www.apache.org/licenses/LICENSE-2.0
 #
 # Unless required by applicable law or agreed to in writing, software
 # distributed under the License is distributed on an "AS IS" BASIS,
@@ skipping 340 matching lines @@
                    ret=0, err='',
                    out='foo.pass_test.PassingTest.test_pass\n')
         self.check(['-l', '--path', '../foo', 'pass_test'],
                    files=files, cwd='bar', ret=0, err='',
                    out='pass_test.PassingTest.test_pass\n')
 
     def test_help(self):
         self.check(['--help'], ret=0, rout='.*', err='')
 
     def test_import_failure_missing_file(self):
-        self.check(['-l', 'foo'], ret=1, err='',
-                   rout='Failed to load "foo".*')
+        _, out, _, _ = self.check(['-l', 'foo'], ret=1, err='')
+        self.assertIn('Failed to load "foo" in find_tests', out)

[nednguyen, 2016/09/24 01:51:45]

Can you assert the out message contains the exception about missing file as
well?

[Dirk Pranke, 2016/10/11 23:48:33]

Done.

 
     def test_import_failure_missing_package(self):
         files = {'foo.py': d("""\
                              import unittest
                              import package_that_does_not_exist
 
                              class ImportFailureTest(unittest.TestCase):
                                 def test_case(self):
                                     pass
                              """)}
-        self.check(['-l', 'foo.py'], files=files, ret=1, err='',
-                   rout=('Failed to load "foo.py":'))
+        _, out, _, _ = self.check(['-l', 'foo.py'], files=files, ret=1, err='')
+        self.assertIn('Failed to load "foo.py" in find_tests', out)

[nednguyen, 2016/09/24 01:51:45]

ditto

[Dirk Pranke, 2016/10/11 23:48:33]

Done.

 
     def test_import_failure_no_tests(self):
         files = {'foo.py': 'import unittest'}
-        self.check(['-l', 'foo.bar'], files=files, ret=1, err='',
-                   rout='Failed to load "foo.bar":.*')
+        _, out, _, _ = self.check(['-l', 'foo.bar'], files=files, ret=1, err='')
+        self.assertIn('Failed to load "foo.bar" in find_tests', out)

[Dirk Pranke, 2016/10/11 23:48:33]

I'm not actually sure what I thought I was testing here originally, but I'm
changing this test to assert that we didn't find any tests to run.

 
     def test_import_failure_syntax_error(self):
         files = {'syn_test.py': d("""\
                              import unittest
 
                              class SyntaxErrorTest(unittest.TestCase):
                                  def test_syntax_error_in_test(self):
                                      syntax error
                              """)}
         _, out, _, _ = self.check([], files=files, ret=1, err='')
@@ skipping 364 matching lines @@
                     modules_to_unload.append(k)
             for k in modules_to_unload:
                 del sys.modules[k]
             sys.path = orig_sys_path
 
         return ret, out, err
 
     def test_debugger(self):
         # TODO: this test seems to hang under coverage.
         pass