diff options
| author | Jenkins <jenkins@review.openstack.org> | 2015-05-29 21:03:03 +0000 |
|---|---|---|
| committer | Gerrit Code Review <review@openstack.org> | 2015-05-29 21:03:03 +0000 |
| commit | d9b7d13edf526b566dbada3245a6a2d9681be850 (patch) | |
| tree | 657dca3f1b5d8396eef0334cd937c9b0f16a5701 /openstackclient/tests | |
| parent | 3ee99dc02a025b6d8d48aa55dbf5af1bed8642f9 (diff) | |
| parent | 2c4b87869be7afb26d6190bb4fd3301eb4061604 (diff) | |
| download | python-openstackclient-d9b7d13edf526b566dbada3245a6a2d9681be850.tar.gz | |
Merge "Add cli tests for --verify and friends"
Diffstat (limited to 'openstackclient/tests')
| -rw-r--r-- | openstackclient/tests/test_shell.py | 54 |
1 files changed, 53 insertions, 1 deletions
diff --git a/openstackclient/tests/test_shell.py b/openstackclient/tests/test_shell.py index a3250f17..a37d2ac8 100644 --- a/openstackclient/tests/test_shell.py +++ b/openstackclient/tests/test_shell.py @@ -527,13 +527,65 @@ class TestShellCli(TestShell): super(TestShellCli, self).tearDown() os.environ = self.orig_env - def test_shell_args(self): + def test_shell_args_no_options(self): _shell = make_shell() with mock.patch("openstackclient.shell.OpenStackShell.initialize_app", self.app): fake_execute(_shell, "list user") self.app.assert_called_with(["list", "user"]) + def test_shell_args_ca_options(self): + _shell = make_shell() + + # NOTE(dtroyer): The commented out asserts below are the desired + # behaviour and will be uncommented when the + # handling for --verify and --insecure is fixed. + + # Default + fake_execute(_shell, "list user") + self.assertIsNone(_shell.options.verify) + self.assertIsNone(_shell.options.insecure) + self.assertEqual('', _shell.options.os_cacert) + self.assertTrue(_shell.verify) + + # --verify + fake_execute(_shell, "--verify list user") + self.assertTrue(_shell.options.verify) + self.assertIsNone(_shell.options.insecure) + self.assertEqual('', _shell.options.os_cacert) + self.assertTrue(_shell.verify) + + # --insecure + fake_execute(_shell, "--insecure list user") + self.assertIsNone(_shell.options.verify) + self.assertTrue(_shell.options.insecure) + self.assertEqual('', _shell.options.os_cacert) + self.assertFalse(_shell.verify) + + # --os-cacert + fake_execute(_shell, "--os-cacert foo list user") + self.assertIsNone(_shell.options.verify) + self.assertIsNone(_shell.options.insecure) + self.assertEqual('foo', _shell.options.os_cacert) + self.assertTrue(_shell.verify) + + # --os-cacert and --verify + fake_execute(_shell, "--os-cacert foo --verify list user") + self.assertTrue(_shell.options.verify) + self.assertIsNone(_shell.options.insecure) + self.assertEqual('foo', _shell.options.os_cacert) + self.assertTrue(_shell.verify) + + # --os-cacert and --insecure + # NOTE(dtroyer): This really is a bogus combination, the default is + # to follow the requests.Session convention and let + # --os-cacert override --insecure + fake_execute(_shell, "--os-cacert foo --insecure list user") + self.assertIsNone(_shell.options.verify) + self.assertTrue(_shell.options.insecure) + self.assertEqual('foo', _shell.options.os_cacert) + self.assertTrue(_shell.verify) + def test_default_env(self): flag = "" kwargs = { |