Issue #12599: Be more strict in accepting None vs. a false-like object

in importlib.

Thanks to PJE for pointing out the issue and Nick Coghlan for filing
the bug.

1 files changed, 15 insertions, 1 deletions

diff --git a/Lib/importlib/test/test_util.py b/Lib/importlib/test/test_util.py
index c7cdad1..b035d65 100644
--- a/Lib/importlib/test/test_util.py
+++ b/Lib/importlib/test/test_util.py
@@ -65,9 +65,23 @@ class ModuleForLoaderTests(unittest.TestCase):
         self.assertEqual(wrapped.__name__, fxn.__name__)
         self.assertEqual(wrapped.__qualname__, fxn.__qualname__)
 
-class SetPackageTests(unittest.TestCase):
+    def test_false_module(self):
+        # If for some odd reason a module is considered false, still return it
+        # from sys.modules.
+        class FalseModule(types.ModuleType):
+            def __bool__(self): return False
+
+        name = 'mod'
+        module = FalseModule(name)
+        with test_util.uncache(name):
+            self.assertFalse(module)
+            sys.modules[name] = module
+            given = self.return_module(name)
+            self.assertTrue(given is module)
 
 
+class SetPackageTests(unittest.TestCase):
+
     """Tests for importlib.util.set_package."""
 
     def verify(self, module, expect):