mirror of
https://github.com/python/cpython.git
synced 2025-10-17 04:08:28 +00:00
gh-119584: Fix test_import Failed Assertion (gh-119623)
The fix in gh-119561 introduced an assertion that doesn't hold true if any of the three new test extension modules are loaded more than once. This is fine normally but breaks if the new test_check_state_first() is run more than once, which happens for refleak checking and with the regrtest --forever flag. We fix that here by clearing each of the three modules after loading them. We also tweak a check in _modules_by_index_check().
This commit is contained in:
parent
0bd0d4072a
commit
ae7b17673f
3 changed files with 8 additions and 2 deletions
|
@ -2887,12 +2887,15 @@ class SinglephaseInitTests(unittest.TestCase):
|
|||
|
||||
self.assertIs(reloaded.snapshot.cached, reloaded.module)
|
||||
|
||||
@unittest.skipIf(_testinternalcapi is None, "requires _testinternalcapi")
|
||||
def test_check_state_first(self):
|
||||
for variant in ['', '_with_reinit', '_with_state']:
|
||||
name = f'{self.NAME}{variant}_check_cache_first'
|
||||
with self.subTest(name):
|
||||
mod = self._load_dynamic(name, self.ORIGIN)
|
||||
self.assertEqual(mod.__name__, name)
|
||||
sys.modules.pop(name, None)
|
||||
_testinternalcapi.clear_extension(name, self.ORIGIN)
|
||||
|
||||
# Currently, for every single-phrase init module loaded
|
||||
# in multiple interpreters, those interpreters share a
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue