From faae3adbb98521d039c574fd3ed417f54a72374d Mon Sep 17 00:00:00 2001 From: "Eric V. Smith" Date: Wed, 27 Jun 2012 15:26:26 -0400 Subject: Changed importlib tests to use assertIs, assertIsInstance, etc., instead of just assertTrue. --- Lib/importlib/test/builtin/test_loader.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'Lib/importlib/test/builtin/test_loader.py') diff --git a/Lib/importlib/test/builtin/test_loader.py b/Lib/importlib/test/builtin/test_loader.py index ec126c9ccc9..357ec2621e2 100644 --- a/Lib/importlib/test/builtin/test_loader.py +++ b/Lib/importlib/test/builtin/test_loader.py @@ -18,10 +18,10 @@ class LoaderTests(abc.LoaderTests): def verify(self, module): """Verify that the module matches against what it should have.""" - self.assertTrue(isinstance(module, types.ModuleType)) + self.assertIsInstance(module, types.ModuleType) for attr, value in self.verification.items(): self.assertEqual(getattr(module, attr), value) - self.assertTrue(module.__name__ in sys.modules) + self.assertIn(module.__name__, sys.modules) load_module = staticmethod(lambda name: machinery.BuiltinImporter.load_module(name)) @@ -49,7 +49,7 @@ class LoaderTests(abc.LoaderTests): with util.uncache(builtin_util.NAME): module1 = self.load_module(builtin_util.NAME) module2 = self.load_module(builtin_util.NAME) - self.assertTrue(module1 is module2) + self.assertIs(module1, module2) def test_unloadable(self): name = 'dssdsdfff' @@ -74,12 +74,12 @@ class InspectLoaderTests(unittest.TestCase): def test_get_code(self): # There is no code object. result = machinery.BuiltinImporter.get_code(builtin_util.NAME) - self.assertTrue(result is None) + self.assertIs(result, None) def test_get_source(self): # There is no source. result = machinery.BuiltinImporter.get_source(builtin_util.NAME) - self.assertTrue(result is None) + self.assertIs(result, None) def test_is_package(self): # Cannot be a package. -- cgit v1.2.3