Fix --enable options on commands

The --enable option on commands is ignored when the arguments are parsed.
This is related to the --enable-beta-commands option. Renaming the option
to --os-beta-command fixes the problem.

There's no need to handle backwards compatibility for the option name
change because there hasn't been an OSC release yet with beta commands.

Change-Id: I0327ba8a2058858a83e9a42e231470ed733cc834
Closes-Bug: #1588384

Author: rtheis

doc/source/command-beta.rst

@@ -33,7 +33,7 @@ example list
 List examples
 
 .. caution:: This is a beta command and subject to change.
-             Use global option ``--enable-beta-commands`` to
+             Use global option ``--os-beta-command`` to
              enable this command.
 
 .. program:: example list
@@ -52,21 +52,17 @@ The command help must label the command as a beta.
         """Display example details
 
            (Caution: This is a beta command and subject to change.
-            Use global option --enable-beta-commands to enable
+            Use global option --os-beta-command to enable
             this command)
         """
 
 Implementation
 --------------
 
 The command must raise a ``CommandError`` exception if beta commands
-are not enabled via ``--enable-beta-commands`` global option.
+are not enabled via ``--os-beta-command`` global option.
 
 .. code-block:: python
 
     def take_action(self, parsed_args):
-        if not self.app.options.enable_beta_commands:
-            msg = _('Caution: This is a beta command and subject to '
-                    'change. Use global option --enable-beta-commands '
-                    'to enable this command.')
-            raise exceptions.CommandError(msg)
+        self.validate_os_beta_command_enabled()

doc/source/command-objects/network-segment.rst

@@ -15,7 +15,7 @@ network segment list
 List network segments
 
 .. caution:: This is a beta command and subject to change.
-             Use global option ``--enable-beta-commands`` to
+             Use global option ``--os-beta-command`` to
              enable this command.
 
 .. program:: network segment list
@@ -39,7 +39,7 @@ network segment show
 Display network segment details
 
 .. caution:: This is a beta command and subject to change.
-             Use global option ``--enable-beta-commands`` to
+             Use global option ``--os-beta-command`` to
              enable this command.
 
 .. program:: network segment show

doc/source/man/openstack.rst

@@ -132,6 +132,9 @@ OPTIONS
     This key should be the value of one of the HMAC keys defined in the
     configuration files of OpenStack services to be traced.
 
+:option:`--os-beta-command`
+    Enable beta commands which are subject to change
+
 :option:`--log-file` <LOGFILE>
     Specify a file to log output. Disabled by default.
 
@@ -144,9 +147,6 @@ OPTIONS
 :option:`--debug`
     Show tracebacks on errors and set verbosity to debug
 
-:option:`--enable-beta-commands`
-    Enable beta commands which are subject to change
-
 COMMANDS
 ========
 

functional/tests/network/v2/test_network_segment.py

@@ -34,7 +34,7 @@ def setUpClass(cls):
 
         # Get the segment for the network.
         opts = cls.get_show_opts(['ID', 'Network'])
-        raw_output = cls.openstack('--enable-beta-commands '
+        raw_output = cls.openstack('--os-beta-command '
                                    'network segment list '
                                    ' --network ' + cls.NETWORK_NAME +
                                    ' ' + opts)
@@ -48,13 +48,13 @@ def tearDownClass(cls):
 
     def test_network_segment_list(self):
         opts = self.get_list_opts(['ID'])
-        raw_output = self.openstack('--enable-beta-commands '
+        raw_output = self.openstack('--os-beta-command '
                                     'network segment list' + opts)
         self.assertIn(self.NETWORK_SEGMENT_ID, raw_output)
 
     def test_network_segment_show(self):
         opts = self.get_show_opts(['network_id'])
-        raw_output = self.openstack('--enable-beta-commands '
+        raw_output = self.openstack('--os-beta-command '
                                     'network segment show ' +
                                     self.NETWORK_SEGMENT_ID + opts)
         self.assertEqual(self.NETWORK_ID + "\n", raw_output)

openstackclient/common/command.py

@@ -20,6 +20,9 @@
 from cliff import show
 import six
 
+from openstackclient.common import exceptions
+from openstackclient.i18n import _
+
 
 class CommandMeta(abc.ABCMeta):
 
@@ -37,6 +40,13 @@ def run(self, parsed_args):
         self.log.debug('run(%s)', parsed_args)
         return super(Command, self).run(parsed_args)
 
+    def validate_os_beta_command_enabled(self):
+        if not self.app.options.os_beta_command:
+            msg = _('Caution: This is a beta command and subject to '
+                    'change. Use global option --os-beta-command '
+                    'to enable this command.')
+            raise exceptions.CommandError(msg)
+
 
 class Lister(Command, lister.Lister):
     pass

openstackclient/network/v2/network_segment.py

@@ -16,7 +16,6 @@
 # TODO(rtheis): Add description and name properties when support is available.
 
 from openstackclient.common import command
-from openstackclient.common import exceptions
 from openstackclient.common import utils
 from openstackclient.i18n import _
 
@@ -25,7 +24,7 @@ class ListNetworkSegment(command.Lister):
     """List network segments
 
        (Caution: This is a beta command and subject to change.
-        Use global option --enable-beta-commands to enable
+        Use global option --os-beta-command to enable
         this command)
     """
 
@@ -46,11 +45,7 @@ def get_parser(self, prog_name):
         return parser
 
     def take_action(self, parsed_args):
-        if not self.app.options.enable_beta_commands:
-            msg = _('Caution: This is a beta command and subject to '
-                    'change. Use global option --enable-beta-commands '
-                    'to enable this command.')
-            raise exceptions.CommandError(msg)
+        self.validate_os_beta_command_enabled()
 
         network_client = self.app.client_manager.network
 
@@ -94,7 +89,7 @@ class ShowNetworkSegment(command.ShowOne):
     """Display network segment details
 
        (Caution: This is a beta command and subject to change.
-        Use global option --enable-beta-commands to enable
+        Use global option --os-beta-command to enable
         this command)
     """
 
@@ -108,11 +103,7 @@ def get_parser(self, prog_name):
         return parser
 
     def take_action(self, parsed_args):
-        if not self.app.options.enable_beta_commands:
-            msg = _('Caution: This is a beta command and subject to '
-                    'change. Use global option --enable-beta-commands '
-                    'to enable this command.')
-            raise exceptions.CommandError(msg)
+        self.validate_os_beta_command_enabled()
 
         client = self.app.client_manager.network
         obj = client.find_segment(

openstackclient/shell.py

@@ -251,7 +251,7 @@ def build_option_parser(self, description, version):
             help="Print API call timing info",
         )
         parser.add_argument(
-            '--enable-beta-commands',
+            '--os-beta-command',
             action='store_true',
             help="Enable beta commands which are subject to change",
         )

openstackclient/tests/common/test_command.py

@@ -15,6 +15,8 @@
 import mock
 
 from openstackclient.common import command
+from openstackclient.common import exceptions
+from openstackclient.tests import fakes as test_fakes
 from openstackclient.tests import utils as test_utils
 
 
@@ -31,3 +33,16 @@ def test_command_has_logger(self):
         self.assertTrue(hasattr(cmd, 'log'))
         self.assertEqual('openstackclient.tests.common.test_command.'
                          'FakeCommand', cmd.log.name)
+
+    def test_validate_os_beta_command_enabled(self):
+        cmd = FakeCommand(mock.Mock(), mock.Mock())
+        cmd.app = mock.Mock()
+        cmd.app.options = test_fakes.FakeOptions()
+
+        # No exception is raised when enabled.
+        cmd.app.options.os_beta_command = True
+        cmd.validate_os_beta_command_enabled()
+
+        cmd.app.options.os_beta_command = False
+        self.assertRaises(exceptions.CommandError,
+                          cmd.validate_os_beta_command_enabled)

openstackclient/tests/fakes.py

@@ -99,7 +99,7 @@ def __init__(self, _stdout, _log):
 
 class FakeOptions(object):
     def __init__(self, **kwargs):
-        self.enable_beta_commands = False
+        self.os_beta_command = False
 
 
 class FakeClient(object):

openstackclient/tests/network/v2/test_network_segment.py

@@ -25,7 +25,7 @@ def setUp(self):
         super(TestNetworkSegment, self).setUp()
 
         # Enable beta commands.
-        self.app.options.enable_beta_commands = True
+        self.app.options.os_beta_command = True
 
         # Get a shortcut to the network client
         self.network = self.app.client_manager.network
@@ -89,7 +89,7 @@ def test_list_no_option(self):
         self.assertEqual(self.data, list(data))
 
     def test_list_no_beta_commands(self):
-        self.app.options.enable_beta_commands = False
+        self.app.options.os_beta_command = False
         parsed_args = self.check_parser(self.cmd, [], [])
         self.assertRaises(exceptions.CommandError, self.cmd.take_action,
                           parsed_args)
@@ -174,7 +174,7 @@ def test_show_no_beta_commands(self):
         verifylist = [
             ('network_segment', self._network_segment.id),
         ]
-        self.app.options.enable_beta_commands = False
+        self.app.options.os_beta_command = False
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
         self.assertRaises(exceptions.CommandError, self.cmd.take_action,
                           parsed_args)