tests: Fix test_module for Python 3.14

Python 3.14 changed argparse to lazily import _colorize during
ArgumentParser.__init__, triggering a chain of imports
(_colorize -> dataclasses -> inspect -> tokenize) that ends with
`from builtins import open`. The class-level mock.patch.dict on
sys.modules with clear=True removed builtins from sys.modules,
causing this import to fail with ImportError.

Narrow the mock scope to wrap only the take_action call rather
than the entire test class, so get_parser/argparse initialization
runs with an unpatched sys.modules while take_action still
iterates the controlled set of fake modules.

Assisted-By: claude-code
Change-Id: I0e1a2b3c4d5e6f7a8b9c0d1e2f3a4b5c6d7e8f9a
Signed-off-by: Jay Faulkner <jay@jvf.cc>

Author: jayofdoom

openstackclient/tests/unit/common/test_module.py

@@ -128,11 +128,6 @@ def test_command_list_with_group(self):
         self.assertEqual(datalist, tuple(data))
 
 
-@mock.patch.dict(
-    'openstackclient.common.module.sys.modules',
-    values=MODULES,
-    clear=True,
-)
 class TestModuleList(utils.TestCommand):
     def setUp(self):
         super().setUp()
@@ -150,7 +145,12 @@ def test_module_list_no_options(self):
         # In base command class Lister in cliff, abstract method take_action()
         # returns a tuple containing the column names and an iterable
         # containing the data to be listed.
-        columns, data = self.cmd.take_action(parsed_args)
+        with mock.patch.dict(
+            'openstackclient.common.module.sys.modules',
+            values=MODULES,
+            clear=True,
+        ):
+            columns, data = self.cmd.take_action(parsed_args)
 
         # Output xxxclient and openstacksdk, but not regular module, like: zlib
         self.assertIn(module_name_1, columns)
@@ -177,7 +177,12 @@ def test_module_list_all(self):
         # In base command class Lister in cliff, abstract method take_action()
         # returns a tuple containing the column names and an iterable
         # containing the data to be listed.
-        columns, data = self.cmd.take_action(parsed_args)
+        with mock.patch.dict(
+            'openstackclient.common.module.sys.modules',
+            values=MODULES,
+            clear=True,
+        ):
+            columns, data = self.cmd.take_action(parsed_args)
 
         # Output xxxclient, openstacksdk and regular module, like: zlib
         self.assertIn(module_name_1, columns)