69c865695f
Reasons: - code should be pythonicaly clean, that is why number of ignored rules should reduced Changes: - E125, F811, H102, H103, F201, H23, H302, F841, H301, H702, H703 rules are now enabled Change-Id: Ibf4025162244d3c2f1278b49a76ec1527a729042
147 lines
4.8 KiB
Python
147 lines
4.8 KiB
Python
# vim: tabstop=4 shiftwidth=4 softtabstop=4
|
|
|
|
# Copyright 2011 OpenStack Foundation
|
|
# Copyright 2013 Rackspace Hosting
|
|
# Copyright 2013 Hewlett-Packard Development Company, L.P.
|
|
# All Rights Reserved.
|
|
#
|
|
# 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.
|
|
|
|
import testtools
|
|
import mock
|
|
|
|
from troveclient.v1 import users
|
|
from troveclient import base
|
|
|
|
"""
|
|
Unit tests for users.py
|
|
"""
|
|
|
|
|
|
class UserTest(testtools.TestCase):
|
|
def setUp(self):
|
|
super(UserTest, self).setUp()
|
|
self.orig__init = users.User.__init__
|
|
users.User.__init__ = mock.Mock(return_value=None)
|
|
self.user = users.User()
|
|
|
|
def tearDown(self):
|
|
super(UserTest, self).tearDown()
|
|
users.User.__init__ = self.orig__init
|
|
|
|
def test___repr__(self):
|
|
self.user.name = "user-1"
|
|
self.assertEqual('<User: user-1>', self.user.__repr__())
|
|
|
|
|
|
class UsersTest(testtools.TestCase):
|
|
def setUp(self):
|
|
super(UsersTest, self).setUp()
|
|
self.orig__init = users.Users.__init__
|
|
users.Users.__init__ = mock.Mock(return_value=None)
|
|
self.users = users.Users()
|
|
self.users.api = mock.Mock()
|
|
self.users.api.client = mock.Mock()
|
|
|
|
self.orig_base_getid = base.getid
|
|
base.getid = mock.Mock(return_value="instance1")
|
|
|
|
def tearDown(self):
|
|
super(UsersTest, self).tearDown()
|
|
users.Users.__init__ = self.orig__init
|
|
base.getid = self.orig_base_getid
|
|
|
|
def _get_mock_method(self):
|
|
self._resp = mock.Mock()
|
|
self._body = None
|
|
self._url = None
|
|
|
|
def side_effect_func(url, body=None):
|
|
self._body = body
|
|
self._url = url
|
|
return (self._resp, body)
|
|
|
|
return mock.Mock(side_effect=side_effect_func)
|
|
|
|
def _build_fake_user(self, name, hostname=None, password=None,
|
|
databases=None):
|
|
return {
|
|
'name': name,
|
|
'password': password if password else 'password',
|
|
'host': hostname,
|
|
'databases': databases if databases else [],
|
|
}
|
|
|
|
def test_create(self):
|
|
self.users.api.client.post = self._get_mock_method()
|
|
self._resp.status_code = 200
|
|
user = self._build_fake_user('user1')
|
|
|
|
self.users.create(23, [user])
|
|
self.assertEqual('/instances/23/users', self._url)
|
|
self.assertEqual({"users": [user]}, self._body)
|
|
|
|
# Even if host isn't supplied originally,
|
|
# the default is supplied.
|
|
del user['host']
|
|
self.users.create(23, [user])
|
|
self.assertEqual('/instances/23/users', self._url)
|
|
user['host'] = '%'
|
|
self.assertEqual({"users": [user]}, self._body)
|
|
|
|
# If host is supplied, of course it's put into the body.
|
|
user['host'] = '127.0.0.1'
|
|
self.users.create(23, [user])
|
|
self.assertEqual({"users": [user]}, self._body)
|
|
|
|
# Make sure that response of 400 is recognized as an error.
|
|
user['host'] = '%'
|
|
self._resp.status_code = 400
|
|
self.assertRaises(Exception, self.users.create, 12, [user])
|
|
|
|
def test_delete(self):
|
|
self.users.api.client.delete = self._get_mock_method()
|
|
self._resp.status_code = 200
|
|
self.users.delete(27, 'user1')
|
|
self.assertEqual('/instances/27/users/user1', self._url)
|
|
self._resp.status_code = 400
|
|
self.assertRaises(Exception, self.users.delete, 34, 'user1')
|
|
|
|
def test__list(self):
|
|
def side_effect_func(self, val):
|
|
return val
|
|
|
|
key = 'key'
|
|
body = mock.Mock()
|
|
body.get = mock.Mock(
|
|
return_value=[{'href': 'http://test.net/test_file',
|
|
'rel': 'next'}]
|
|
)
|
|
body.__getitem__ = mock.Mock(return_value=["test-value"])
|
|
|
|
resp = mock.Mock()
|
|
resp.status_code = 200
|
|
self.users.resource_class = mock.Mock(side_effect=side_effect_func)
|
|
self.users.api.client.get = mock.Mock(return_value=(resp, body))
|
|
self.assertEqual(["test-value"], self.users._list('url', key).items)
|
|
|
|
self.users.api.client.get = mock.Mock(return_value=(resp, None))
|
|
self.assertRaises(Exception, self.users._list, 'url', None)
|
|
|
|
def test_list(self):
|
|
def side_effect_func(path, user, limit, marker):
|
|
return path
|
|
|
|
self.users._list = mock.Mock(side_effect=side_effect_func)
|
|
self.assertEqual('/instances/instance1/users', self.users.list(1))
|