diff options
| author | Morgan Jones <morgan@parelastic.com> | 2016-05-25 12:29:10 -0400 |
|---|---|---|
| committer | Morgan Jones <morgan@parelastic.com> | 2016-05-26 16:14:37 -0400 |
| commit | ea0e472edd194243392fd51a5af3a3a77101fddd (patch) | |
| tree | f2346c3b4e00c63b67cc623663a1f6c2e44c4970 /troveclient/tests/test_backups.py | |
| parent | f7031a7e9af98fe2fb952e357155cd858d3bf440 (diff) | |
| download | python-troveclient-ea0e472edd194243392fd51a5af3a3a77101fddd.tar.gz | |
Fix troveclient to support Mistral
Mistral gets confused by Trove's (aguably wrong) inclusion
of a member called 'items' in the Pagenated object that
Trove returns as the result of 'list' client methods.
This change changes Pagenated to inherit from the 'list'
class so that the items method is not required (and has
the additional benefit of just generally being a better
implementation of a list type result).
Change-Id: I683120451f69f07f131e6fa422c082f85735b196
Closes-bug: 1585705
Depends-On: Id674ae57bfcdc5e09bde1e323a614b3a03a7cad3
Diffstat (limited to 'troveclient/tests/test_backups.py')
| -rw-r--r-- | troveclient/tests/test_backups.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/troveclient/tests/test_backups.py b/troveclient/tests/test_backups.py index a7dd085..f6401a0 100644 --- a/troveclient/tests/test_backups.py +++ b/troveclient/tests/test_backups.py @@ -87,7 +87,8 @@ class BackupManagerTest(testtools.TestCase): def test_copy(self): create_mock = mock.Mock() self.backups._create = create_mock - args = {'name': 'test_backup', 'backup': '1'} + args = {'name': 'test_backup', 'instance': 'foo', + 'backup': '1'} body = {'backup': args} self.backups.create(**args) create_mock.assert_called_with('/backups', body, 'backup') |