delta/openstack/python-troveclient.git/troveclient/tests/test_base.py, branch master

Use unittest.mock instead of third party mock

2020-04-18T16:59:46+00:00

Now that we no longer support py27, we can use the standard library
unittest.mock module instead of the third party mock lib.

Change-Id: Ib83e626f51e6e01898bb7931c1660a1a548500f3
Signed-off-by: Sean McGinnis

Update hacking for Python3

2020-03-31T11:33:41+00:00

The repo is Python 3 now, so update hacking to version 3.0 which
supports Python 3.

Fix problems found.

Change-Id: I5da442bb2f1e634e6b3fe37a6ea3820cea1db96f

Fix show instance with integer name NotFound

2018-01-16T08:00:29+00:00

Fix trove cli 'show' instance with integer name error

Change-Id: Ic6a943b4d5980017915dd2c0361636877376c5cb
Closes-bug: #1740015

move old apiclient code out of openstack/common

2016-11-10T15:02:08+00:00

As part of the first community-wide goal [1], teams were asked
to remove the openstack/common package of their projects
if one existed. This was a byproduct of the old oslo-incubator
form of syncing common functionality.

The package, apiclient, was moved to a top level location
and cliutils was moved to the common module. There are no oslo
specific libraries, the recommended solution is to move it
in tree and maintain it there.

Specifically in this patch, I also rearranged some of the
imports to make them match up with the hacking guidelines
while I was re-ordering them for the refactor anyway.

Lastly, the apiclient/fake_client.py was deleted since it
was not used.

[1] http://governance.openstack.org/goals/ocata/remove-incubated-oslo-code.html

Change-Id: Ib7b691240b819c9c91b100ca47ffaba75b281c78

Fix troveclient to support Mistral

2016-05-26T20:14:37+00:00

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

Merge "Change assertTrue(isinstance()) by optimal assert"

2016-01-08T20:28:16+00:00

Change assertTrue(isinstance()) by optimal assert

2016-01-08T07:10:56+00:00

Some of tests use different method of assertTrue(isinstance(A, B)),
The correct way is to use assertIsInstance(A,B) provided by testtools

Closes-bug: #1268480
Change-Id: I718b86e83b93a4e778710b9de895f2796a9b6d61

Replace assertEqual(None, *) with assertIsNone in tests

2015-12-17T14:02:34+00:00

Replace assertEqual(None, *) with assertIsNone in tests to have
more clear messages in case of failure.

Change-Id: Idd271edb45dafed10aad255267ff0b9722cabe2d
Closes-bug: #1280522

Fixes unit-test in troveclient

2015-04-08T09:11:13+00:00

1. Renames tes___eq__ to be test___eq__,
   because earlier method was not being called.
2. Corrects the test-code in test___eq__,
   for the failures it gave when it started running.
3. Removes some unused code from tests,
   there were few lines/methods which were not used by tests
   just removed/updated them.

Change-Id: I90aa8cb9bb9f5d15697f00df0ba1defa0ef7be59
Closes-Bug: #1441518

Correct order of parameters to assertEqual

2015-02-23T17:48:39+00:00

The correct order of parameters to assertEqual is (expected,
actual). It makes the assertion message more meaningful when there is
a failure.

Change-Id: I1ab6933d2eee336b41f6ab791e32ad6eaa004a6b
Partial-Bug: #1277104