Do no longer skip multi-phase ext mod init test cases

Author: fangerer

graalpython/lib-python/3/test/test_importlib/extension/test_loader.py

@@ -104,7 +104,6 @@ def setUp(self):
     # Handling failure on reload is the up to the module.
     test_state_after_failure = None
 
-    @impl_detail("GR-26683 multi-phase init not supported", graalvm=False)
     def test_module(self):
         '''Test loading an extension module'''
         with util.uncache(self.name):
@@ -119,7 +118,6 @@ def test_module(self):
             self.assertIsInstance(module.__loader__,
                                   self.machinery.ExtensionFileLoader)
 
-    @impl_detail("GR-26683 multi-phase init not supported", graalvm=False)
     def test_functionality(self):
         '''Test basic functionality of stuff defined in an extension module'''
         with util.uncache(self.name):
@@ -140,7 +138,6 @@ def test_functionality(self):
             self.assertEqual(module.int_const, 1969)
             self.assertEqual(module.str_const, 'something different')
 
-    @impl_detail("GR-26683 multi-phase init not supported", graalvm=False)
     def test_reload(self):
         '''Test that reload didn't re-set the module's attributes'''
         with util.uncache(self.name):
@@ -149,7 +146,6 @@ def test_reload(self):
             importlib.reload(module)
             self.assertIs(ex_class, module.Example)
 
-    @impl_detail("GR-26683 multi-phase init not supported", graalvm=False)
     def test_try_registration(self):
         '''Assert that the PyState_{Find,Add,Remove}Module C API doesn't work'''
         module = self.load_module()
@@ -175,15 +171,13 @@ def load_module_by_name(self, fullname):
         loader.exec_module(module)
         return module
 
-    @impl_detail("GR-26683 multi-phase init not supported", graalvm=False)
     def test_load_submodule(self):
         '''Test loading a simulated submodule'''
         module = self.load_module_by_name('pkg.' + self.name)
         self.assertIsInstance(module, types.ModuleType)
         self.assertEqual(module.__name__, 'pkg.' + self.name)
         self.assertEqual(module.str_const, 'something different')
 
-    @impl_detail("GR-26683 multi-phase init not supported", graalvm=False)
     def test_load_short_name(self):
         '''Test loading module with a one-character name'''
         module = self.load_module_by_name('x')
@@ -192,7 +186,6 @@ def test_load_short_name(self):
         self.assertEqual(module.str_const, 'something different')
         self.assertNotIn('x', sys.modules)
 
-    @impl_detail("GR-26683 multi-phase init not supported", graalvm=False)
     def test_load_twice(self):
         '''Test that 2 loads result in 2 module objects'''
         module1 = self.load_module_by_name(self.name)
@@ -213,7 +206,6 @@ def test_unloadable_nonascii(self):
             self.load_module_by_name(name)
         self.assertEqual(cm.exception.name, name)
 
-    @impl_detail("GR-26683 multi-phase init not supported", graalvm=False)
     def test_nonmodule(self):
         '''Test returning a non-module object from create works'''
         name = self.name + '_nonmodule'
@@ -222,7 +214,6 @@ def test_nonmodule(self):
         self.assertEqual(mod.three, 3)
 
     # issue 27782
-    @impl_detail("GR-26683 multi-phase init not supported", graalvm=False)
     def test_nonmodule_with_methods(self):
         '''Test creating a non-module object with methods defined'''
         name = self.name + '_nonmodule_with_methods'
@@ -231,15 +222,13 @@ def test_nonmodule_with_methods(self):
         self.assertEqual(mod.three, 3)
         self.assertEqual(mod.bar(10, 1), 9)
 
-    @impl_detail("GR-26683 multi-phase init not supported", graalvm=False)
     def test_null_slots(self):
         '''Test that NULL slots aren't a problem'''
         name = self.name + '_null_slots'
         module = self.load_module_by_name(name)
         self.assertIsInstance(module, types.ModuleType)
         self.assertEqual(module.__name__, name)
 
-    @impl_detail("GR-26683 multi-phase init not supported", graalvm=False)
     def test_bad_modules(self):
         '''Test SystemError is raised for misbehaving extensions'''
         for name_base in [
@@ -264,7 +253,6 @@ def test_bad_modules(self):
                 with self.assertRaises(SystemError):
                     self.load_module_by_name(name)
 
-    @impl_detail("GR-26683 multi-phase init not supported", graalvm=False)
     def test_nonascii(self):
         '''Test that modules with non-ASCII names can be loaded'''
         # punycode behaves slightly differently in some-ASCII and no-ASCII