diff --git a/Lib/test/test_importlib/test_abc.py b/Lib/test/test_importlib/test_abc.py index db36204e3b4e79..54b2da6a3d233a 100644 --- a/Lib/test/test_importlib/test_abc.py +++ b/Lib/test/test_importlib/test_abc.py @@ -157,15 +157,14 @@ class MetaPathFinderDefaultsTests(ABCTestHarness): def test_find_module(self): # Default should return None. - self.assertIsNone(self.ins.find_module('something', None)) + with self.assertWarns(DeprecationWarning): + found = self.ins.find_module('something', None) + self.assertIsNone(found) def test_invalidate_caches(self): # Calling the method is a no-op. self.ins.invalidate_caches() - def test_find_module_warns(self): - with self.assertWarns(DeprecationWarning): - self.ins.find_module('something', None) (Frozen_MPFDefaultTests, Source_MPFDefaultTests @@ -183,7 +182,9 @@ class PathEntryFinderDefaultsTests(ABCTestHarness): SPLIT = make_abc_subclasses(PathEntryFinder) def test_find_loader(self): - self.assertEqual((None, []), self.ins.find_loader('something')) + with self.assertWarns(DeprecationWarning): + found = self.ins.find_loader('something') + self.assertEqual(found, (None, [])) def find_module(self): self.assertEqual(None, self.ins.find_module('something')) @@ -192,9 +193,6 @@ def test_invalidate_caches(self): # Should be a no-op. self.ins.invalidate_caches() - def test_find_loader_warns(self): - with self.assertWarns(DeprecationWarning): - self.ins.find_loader('something') (Frozen_PEFDefaultTests, Source_PEFDefaultTests @@ -324,7 +322,8 @@ def test_no_spec(self): finder = self.finder(None) path = ['a', 'b', 'c'] name = 'blah' - found = finder.find_module(name, path) + with self.assertWarns(DeprecationWarning): + found = finder.find_module(name, path) self.assertIsNone(found) self.assertEqual(name, finder.called_for[0]) self.assertEqual(path, finder.called_for[1]) @@ -333,7 +332,8 @@ def test_spec(self): loader = object() spec = self.util.spec_from_loader('blah', loader) finder = self.finder(spec) - found = finder.find_module('blah', None) + with self.assertWarns(DeprecationWarning): + found = finder.find_module('blah', None) self.assertIs(found, spec.loader) @@ -358,7 +358,8 @@ def find_spec(self, fullname, target=None): def test_no_spec(self): finder = self.finder(None) name = 'blah' - found = finder.find_loader(name) + with self.assertWarns(DeprecationWarning): + found = finder.find_loader(name) self.assertIsNone(found[0]) self.assertEqual([], found[1]) self.assertEqual(name, finder.called_for) @@ -367,7 +368,8 @@ def test_spec_with_loader(self): loader = object() spec = self.util.spec_from_loader('blah', loader) finder = self.finder(spec) - found = finder.find_loader('blah') + with self.assertWarns(DeprecationWarning): + found = finder.find_loader('blah') self.assertIs(found[0], spec.loader) def test_spec_with_portions(self): @@ -375,7 +377,8 @@ def test_spec_with_portions(self): paths = ['a', 'b', 'c'] spec.submodule_search_locations = paths finder = self.finder(spec) - found = finder.find_loader('blah') + with self.assertWarns(DeprecationWarning): + found = finder.find_loader('blah') self.assertIsNone(found[0]) self.assertEqual(paths, found[1])