2013-08-06 13:36:45 -03:00
|
|
|
# Licensed under the Apache License, Version 2.0 (the "License");
|
|
|
|
# you may not use this file except in compliance with the License.
|
|
|
|
# You may obtain a copy of the License at
|
|
|
|
#
|
|
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
|
|
#
|
|
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
|
|
# distributed under the License is distributed on an "AS IS" BASIS,
|
|
|
|
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
|
|
|
|
# implied.
|
|
|
|
# See the License for the specific language governing permissions and
|
|
|
|
# limitations under the License.
|
|
|
|
|
2014-04-14 11:53:10 -03:00
|
|
|
import argparse
|
2012-12-24 21:49:34 -06:00
|
|
|
import re
|
2012-05-21 16:32:35 -04:00
|
|
|
import sys
|
|
|
|
|
2012-12-24 21:49:34 -06:00
|
|
|
import fixtures
|
2014-08-15 17:37:58 -04:00
|
|
|
import mock
|
2015-02-12 16:28:35 -06:00
|
|
|
import pkg_resources
|
2014-07-11 09:31:05 +10:00
|
|
|
import requests_mock
|
2015-02-12 16:28:35 -06:00
|
|
|
import requests
|
2013-06-12 08:28:17 -05:00
|
|
|
from six import moves
|
2012-12-24 21:49:34 -06:00
|
|
|
from testtools import matchers
|
|
|
|
|
2016-01-27 15:09:45 -06:00
|
|
|
import cinderclient
|
2013-07-15 16:25:40 +00:00
|
|
|
from cinderclient import exceptions
|
2015-02-12 16:28:35 -06:00
|
|
|
from cinderclient import auth_plugin
|
2014-04-14 11:53:10 -03:00
|
|
|
from cinderclient import shell
|
2015-02-12 16:28:35 -06:00
|
|
|
from cinderclient.tests.unit.test_auth_plugins import mock_http_request
|
|
|
|
from cinderclient.tests.unit.test_auth_plugins import requested_headers
|
2015-03-06 09:55:26 -07:00
|
|
|
from cinderclient.tests.unit.fixture_data import keystone_client
|
2015-01-07 14:29:48 -08:00
|
|
|
from cinderclient.tests.unit import utils
|
2014-08-15 17:37:58 -04:00
|
|
|
import keystoneclient.exceptions as ks_exc
|
2014-06-25 10:04:57 -07:00
|
|
|
from keystoneclient.exceptions import DiscoveryFailure
|
2012-05-21 16:32:35 -04:00
|
|
|
|
|
|
|
|
|
|
|
class ShellTest(utils.TestCase):
|
|
|
|
|
2012-12-24 21:49:34 -06:00
|
|
|
FAKE_ENV = {
|
|
|
|
'OS_USERNAME': 'username',
|
|
|
|
'OS_PASSWORD': 'password',
|
|
|
|
'OS_TENANT_NAME': 'tenant_name',
|
2015-06-22 19:53:34 -07:00
|
|
|
'OS_AUTH_URL': 'http://no.where/v2.0',
|
2012-12-24 21:49:34 -06:00
|
|
|
}
|
|
|
|
|
2012-05-21 16:32:35 -04:00
|
|
|
# Patch os.environ to avoid required auth info.
|
2015-02-12 16:28:35 -06:00
|
|
|
def make_env(self, exclude=None, include=None):
|
2014-08-15 17:37:58 -04:00
|
|
|
env = dict((k, v) for k, v in self.FAKE_ENV.items() if k != exclude)
|
2015-02-12 16:28:35 -06:00
|
|
|
env.update(include or {})
|
2014-08-15 17:37:58 -04:00
|
|
|
self.useFixture(fixtures.MonkeyPatch('os.environ', env))
|
|
|
|
|
2012-05-21 16:32:35 -04:00
|
|
|
def setUp(self):
|
2012-12-24 21:49:34 -06:00
|
|
|
super(ShellTest, self).setUp()
|
|
|
|
for var in self.FAKE_ENV:
|
|
|
|
self.useFixture(fixtures.EnvironmentVariable(var,
|
|
|
|
self.FAKE_ENV[var]))
|
2012-05-21 16:32:35 -04:00
|
|
|
|
|
|
|
def shell(self, argstr):
|
|
|
|
orig = sys.stdout
|
|
|
|
try:
|
2013-06-12 08:28:17 -05:00
|
|
|
sys.stdout = moves.StringIO()
|
2014-04-14 11:53:10 -03:00
|
|
|
_shell = shell.OpenStackCinderShell()
|
2012-05-21 16:32:35 -04:00
|
|
|
_shell.main(argstr.split())
|
|
|
|
except SystemExit:
|
|
|
|
exc_type, exc_value, exc_traceback = sys.exc_info()
|
2014-02-26 15:59:14 +09:00
|
|
|
self.assertEqual(0, exc_value.code)
|
2012-05-21 16:32:35 -04:00
|
|
|
finally:
|
|
|
|
out = sys.stdout.getvalue()
|
|
|
|
sys.stdout.close()
|
|
|
|
sys.stdout = orig
|
|
|
|
|
|
|
|
return out
|
|
|
|
|
|
|
|
def test_help_unknown_command(self):
|
|
|
|
self.assertRaises(exceptions.CommandError, self.shell, 'help foofoo')
|
|
|
|
|
|
|
|
def test_help(self):
|
|
|
|
required = [
|
2012-12-24 21:49:34 -06:00
|
|
|
'.*?^usage: ',
|
2014-02-03 13:39:15 -06:00
|
|
|
'.*?(?m)^\s+create\s+Creates a volume.',
|
|
|
|
'.*?(?m)^Run "cinder help SUBCOMMAND" for help on a subcommand.',
|
2012-05-21 16:32:35 -04:00
|
|
|
]
|
2012-11-08 23:33:14 +01:00
|
|
|
help_text = self.shell('help')
|
|
|
|
for r in required:
|
2012-12-24 21:49:34 -06:00
|
|
|
self.assertThat(help_text,
|
2013-01-16 15:45:29 +02:00
|
|
|
matchers.MatchesRegex(r, re.DOTALL | re.MULTILINE))
|
2012-05-21 16:32:35 -04:00
|
|
|
|
|
|
|
def test_help_on_subcommand(self):
|
|
|
|
required = [
|
2012-12-24 21:49:34 -06:00
|
|
|
'.*?^usage: cinder list',
|
2014-02-03 13:39:15 -06:00
|
|
|
'.*?(?m)^Lists all volumes.',
|
2012-05-21 16:32:35 -04:00
|
|
|
]
|
2012-11-08 23:33:14 +01:00
|
|
|
help_text = self.shell('help list')
|
|
|
|
for r in required:
|
2012-12-24 21:49:34 -06:00
|
|
|
self.assertThat(help_text,
|
2013-01-16 15:45:29 +02:00
|
|
|
matchers.MatchesRegex(r, re.DOTALL | re.MULTILINE))
|
2014-04-14 11:53:10 -03:00
|
|
|
|
2014-07-11 09:31:05 +10:00
|
|
|
def register_keystone_auth_fixture(self, mocker, url):
|
|
|
|
mocker.register_uri('GET', url,
|
|
|
|
text=keystone_client.keystone_request_callback)
|
2014-06-25 10:04:57 -07:00
|
|
|
|
2014-07-11 09:31:05 +10:00
|
|
|
@requests_mock.Mocker()
|
|
|
|
def test_version_discovery(self, mocker):
|
2014-06-25 10:04:57 -07:00
|
|
|
_shell = shell.OpenStackCinderShell()
|
|
|
|
|
|
|
|
os_auth_url = "https://WrongDiscoveryResponse.discovery.com:35357/v2.0"
|
2014-07-11 09:31:05 +10:00
|
|
|
self.register_keystone_auth_fixture(mocker, os_auth_url)
|
2014-06-25 10:04:57 -07:00
|
|
|
self.assertRaises(DiscoveryFailure, _shell._discover_auth_versions,
|
|
|
|
None, auth_url=os_auth_url)
|
|
|
|
|
|
|
|
os_auth_url = "https://DiscoveryNotSupported.discovery.com:35357/v2.0"
|
2014-07-11 09:31:05 +10:00
|
|
|
self.register_keystone_auth_fixture(mocker, os_auth_url)
|
2014-06-25 10:04:57 -07:00
|
|
|
v2_url, v3_url = _shell._discover_auth_versions(
|
|
|
|
None, auth_url=os_auth_url)
|
|
|
|
self.assertEqual(v2_url, os_auth_url, "Expected v2 url")
|
2015-08-09 15:57:00 +09:00
|
|
|
self.assertIsNone(v3_url, "Expected no v3 url")
|
2014-06-25 10:04:57 -07:00
|
|
|
|
|
|
|
os_auth_url = "https://DiscoveryNotSupported.discovery.com:35357/v3.0"
|
2014-07-11 09:31:05 +10:00
|
|
|
self.register_keystone_auth_fixture(mocker, os_auth_url)
|
2014-06-25 10:04:57 -07:00
|
|
|
v2_url, v3_url = _shell._discover_auth_versions(
|
|
|
|
None, auth_url=os_auth_url)
|
|
|
|
self.assertEqual(v3_url, os_auth_url, "Expected v3 url")
|
2015-08-09 15:57:00 +09:00
|
|
|
self.assertIsNone(v2_url, "Expected no v2 url")
|
2014-06-25 10:04:57 -07:00
|
|
|
|
2015-08-19 15:48:27 -05:00
|
|
|
@requests_mock.Mocker()
|
|
|
|
def list_volumes_on_service(self, count, mocker):
|
|
|
|
os_auth_url = "http://multiple.service.names/v2.0"
|
|
|
|
mocker.register_uri('POST', os_auth_url + "/tokens",
|
|
|
|
text=keystone_client.keystone_request_callback)
|
|
|
|
mocker.register_uri('GET',
|
|
|
|
"http://cinder%i.api.com/v2/volumes/detail"
|
|
|
|
% count, text='{"volumes": []}')
|
|
|
|
self.make_env(include={'OS_AUTH_URL': os_auth_url,
|
|
|
|
'CINDER_SERVICE_NAME': 'cinder%i' % count})
|
|
|
|
_shell = shell.OpenStackCinderShell()
|
|
|
|
_shell.main(['list'])
|
|
|
|
|
|
|
|
def test_cinder_service_name(self):
|
|
|
|
# Failing with 'No mock address' means we are not
|
|
|
|
# choosing the correct endpoint
|
|
|
|
for count in range(1, 4):
|
|
|
|
self.list_volumes_on_service(count)
|
|
|
|
|
2015-08-19 11:36:33 -05:00
|
|
|
@mock.patch('keystoneclient.adapter.Adapter.get_token',
|
|
|
|
side_effect=ks_exc.ConnectionRefused())
|
|
|
|
@mock.patch('keystoneclient.discover.Discover',
|
|
|
|
side_effect=ks_exc.ConnectionRefused())
|
2014-08-15 17:37:58 -04:00
|
|
|
@mock.patch('sys.stdin', side_effect=mock.MagicMock)
|
|
|
|
@mock.patch('getpass.getpass', return_value='password')
|
2015-08-19 11:36:33 -05:00
|
|
|
def test_password_prompted(self, mock_getpass, mock_stdin, mock_discover,
|
|
|
|
mock_token):
|
2014-08-15 17:37:58 -04:00
|
|
|
self.make_env(exclude='OS_PASSWORD')
|
2015-08-19 11:36:33 -05:00
|
|
|
_shell = shell.OpenStackCinderShell()
|
|
|
|
self.assertRaises(ks_exc.ConnectionRefused, _shell.main, ['list'])
|
2014-08-15 17:37:58 -04:00
|
|
|
mock_getpass.assert_called_with('OS Password: ')
|
|
|
|
|
2015-02-12 16:28:35 -06:00
|
|
|
@mock.patch.object(requests, "request")
|
|
|
|
@mock.patch.object(pkg_resources, "iter_entry_points")
|
|
|
|
def test_auth_system_not_keystone(self, mock_iter_entry_points,
|
|
|
|
mock_request):
|
|
|
|
"""Test that we can authenticate using the auth plugin system."""
|
|
|
|
non_keystone_auth_url = "http://non-keystone-url.com/v2.0"
|
|
|
|
|
|
|
|
class MockEntrypoint(pkg_resources.EntryPoint):
|
|
|
|
def load(self):
|
|
|
|
return FakePlugin
|
|
|
|
|
|
|
|
class FakePlugin(auth_plugin.BaseAuthPlugin):
|
|
|
|
def authenticate(self, cls, auth_url):
|
|
|
|
cls._authenticate(auth_url, {"fake": "me"})
|
|
|
|
|
|
|
|
def get_auth_url(self):
|
|
|
|
return non_keystone_auth_url
|
|
|
|
|
|
|
|
mock_iter_entry_points.side_effect = lambda _t: [
|
|
|
|
MockEntrypoint("fake", "fake", ["FakePlugin"])]
|
|
|
|
|
|
|
|
mock_request.side_effect = mock_http_request()
|
|
|
|
|
|
|
|
# Tell the shell we wish to use our 'fake' auth instead of keystone
|
|
|
|
# and the auth plugin will provide the auth url
|
|
|
|
self.make_env(exclude="OS_AUTH_URL",
|
|
|
|
include={'OS_AUTH_SYSTEM': 'fake'})
|
|
|
|
# This should fail as we have not setup a mock response for 'list',
|
|
|
|
# however auth should have been called
|
|
|
|
_shell = shell.OpenStackCinderShell()
|
|
|
|
self.assertRaises(KeyError, _shell.main, ['list'])
|
|
|
|
|
|
|
|
headers = requested_headers(_shell.cs)
|
|
|
|
token_url = _shell.cs.client.auth_url + "/tokens"
|
|
|
|
self.assertEqual(non_keystone_auth_url + "/tokens", token_url)
|
|
|
|
|
2015-07-10 22:28:49 +08:00
|
|
|
mock_request.assert_any_call(
|
2015-02-12 16:28:35 -06:00
|
|
|
"POST",
|
|
|
|
token_url,
|
|
|
|
headers=headers,
|
|
|
|
data='{"fake": "me"}',
|
|
|
|
allow_redirects=True,
|
|
|
|
**self.TEST_REQUEST_BASE)
|
|
|
|
|
2016-01-27 15:09:45 -06:00
|
|
|
@mock.patch.object(cinderclient.client.HTTPClient, 'authenticate',
|
|
|
|
side_effect=exceptions.Unauthorized('No'))
|
|
|
|
# Easiest way to make cinderclient use httpclient is a None session
|
|
|
|
@mock.patch.object(cinderclient.shell.OpenStackCinderShell,
|
|
|
|
'_get_keystone_session', return_value=None)
|
|
|
|
def test_http_client_insecure(self, mock_authenticate, mock_session):
|
|
|
|
self.make_env(include={'CINDERCLIENT_INSECURE': True})
|
|
|
|
|
|
|
|
_shell = shell.OpenStackCinderShell()
|
|
|
|
|
|
|
|
# This "fails" but instantiates the client.
|
|
|
|
self.assertRaises(exceptions.CommandError, _shell.main, ['list'])
|
|
|
|
|
|
|
|
self.assertEqual(False, _shell.cs.client.verify_cert)
|
|
|
|
|
2014-04-14 11:53:10 -03:00
|
|
|
|
|
|
|
class CinderClientArgumentParserTest(utils.TestCase):
|
2014-06-25 10:04:57 -07:00
|
|
|
|
2014-04-14 11:53:10 -03:00
|
|
|
def test_ambiguity_solved_for_one_visible_argument(self):
|
|
|
|
parser = shell.CinderClientArgumentParser(add_help=False)
|
|
|
|
parser.add_argument('--test-parameter',
|
|
|
|
dest='visible_param',
|
|
|
|
action='store_true')
|
|
|
|
parser.add_argument('--test_parameter',
|
|
|
|
dest='hidden_param',
|
|
|
|
action='store_true',
|
|
|
|
help=argparse.SUPPRESS)
|
|
|
|
|
|
|
|
opts = parser.parse_args(['--test'])
|
|
|
|
|
2014-06-25 10:04:57 -07:00
|
|
|
# visible argument must be set
|
2014-04-14 11:53:10 -03:00
|
|
|
self.assertTrue(opts.visible_param)
|
|
|
|
self.assertFalse(opts.hidden_param)
|
|
|
|
|
|
|
|
def test_raise_ambiguity_error_two_visible_argument(self):
|
|
|
|
parser = shell.CinderClientArgumentParser(add_help=False)
|
|
|
|
parser.add_argument('--test-parameter',
|
|
|
|
dest="visible_param1",
|
|
|
|
action='store_true')
|
|
|
|
parser.add_argument('--test_parameter',
|
|
|
|
dest="visible_param2",
|
|
|
|
action='store_true')
|
|
|
|
|
|
|
|
self.assertRaises(SystemExit, parser.parse_args, ['--test'])
|
|
|
|
|
|
|
|
def test_raise_ambiguity_error_two_hidden_argument(self):
|
|
|
|
parser = shell.CinderClientArgumentParser(add_help=False)
|
|
|
|
parser.add_argument('--test-parameter',
|
|
|
|
dest="hidden_param1",
|
|
|
|
action='store_true',
|
|
|
|
help=argparse.SUPPRESS)
|
|
|
|
parser.add_argument('--test_parameter',
|
|
|
|
dest="hidden_param2",
|
|
|
|
action='store_true',
|
|
|
|
help=argparse.SUPPRESS)
|
|
|
|
|
|
|
|
self.assertRaises(SystemExit, parser.parse_args, ['--test'])
|