Update CHANGELOG.md for new namespace command and enhance CLI tests for namespace handling

Author: grongierisc

CHANGELOG.md

@@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Add `iop --update` command to update a production from the command line
 - New info displayed in `iop --help` command showing current namespace
 - `iop --status` command now shows if production needs update, if so a message is displayed
+- Command line option `--namespace` alone will show the current namespace
 
 ### Fixed
 - Fix issue with boolean `response_required` parameter in `send_request_async` method of BusinessProcess

src/iop/_cli.py

@@ -61,7 +61,29 @@ def __init__(self, args: CommandArgs):
             # set environment variable IRISNAMESPACE
             os.environ['IRISNAMESPACE'] = self.args.namespace
 
+    def _has_primary_command(self) -> bool:
+        return any([
+            self.args.default,
+            self.args.list,
+            self.args.start,
+            self.args.stop,
+            self.args.kill,
+            self.args.restart,
+            self.args.status,
+            self.args.test,
+            self.args.version,
+            self.args.export,
+            self.args.migrate,
+            self.args.log,
+            self.args.init,
+            self.args.update,
+        ])
+
     def execute(self) -> None:
+        if self.args.namespace == 'not_set' and not self._has_primary_command():
+            print(os.getenv('IRISNAMESPACE', 'not set'))
+            return
+
         command_type = self._determine_command_type()
         command_handlers = {
             CommandType.DEFAULT: self._handle_default,

src/tests/test_cli.py

@@ -23,6 +23,28 @@ def test_help_and_basic_commands(self):
             self.assertEqual(cm.exception.code, 0)
             self.assertIn('Namespace:', fake_out.getvalue())
 
+    def test_namespace_prints_current(self):
+        """Test namespace display when no value is provided."""
+        with patch.dict(os.environ, {"IRISNAMESPACE": "TESTNS"}, clear=True):
+            with patch('sys.stdout', new=StringIO()) as fake_out:
+                with self.assertRaises(SystemExit) as cm:
+                    main(['-n'])
+                self.assertEqual(cm.exception.code, 0)
+                self.assertEqual(fake_out.getvalue().strip(), 'TESTNS')
+
+    def test_namespace_with_value_prints_help(self):
+        """Test namespace assignment prints help when no other command is provided."""
+        with patch.dict(os.environ, {}, clear=True):
+            with patch('iop._director._Director.get_default_production') as mock_default:
+                mock_default.return_value = 'UnitTest.Production'
+                with patch('sys.stdout', new=StringIO()) as fake_out:
+                    with self.assertRaises(SystemExit) as cm:
+                        main(['-n', 'MyNS'])
+                    self.assertEqual(cm.exception.code, 0)
+                    output = fake_out.getvalue()
+                    self.assertIn('usage:', output)
+                    self.assertIn('Namespace: MyNS', output)
+
     def test_default_settings(self):
         """Test default production settings."""
         # Test with name