Fix params oeder to correspond to real signature:
assertEqual(expected, actual)
Change-Id: Iad65accc86a2d1f7df7284596e1bbceeb091d39e
Closes-Bug: #1277104
The structure of the tests directories should mimic the code repository,
therefore this patch is moving the ironicclient/common tests to
ironicclient/tests/unit/common.
Change-Id: I1c63a72bc53df59f0618dae59e30c80c66c5e0d9
When logging the curl command or the http response we should obfuscate
the secrets.
Change-Id: Iaf8aef795782b5c4608f3a2be6887d0429176250
Partial-Bug: 1530972
The test case of listing nodes with provisioning state is incomplete.
Add a missed test case and fix a typo in the existing one.
Change-Id: Ide5249984bcecca001347dd18bbd07a31caf44af
Closes-Bug: #1531410
Also make use of sphinx-quickstart for reno conf.py as previously
it was copypasted from glance and changed manually.
Change-Id: I0efac1cb5181dbf71ef8e50da969a646f5577e11
Replace assertEqual(None, *) with assertIsNone in tests to have
more clear messages in case of failure.
Change-Id: I109bef1afbbdd3a2cf25e9fb7133509516910891
Closes-bug: #1280522
This patch removes `MANIFEST.in` file as pbr generates a sensible
manifest from git files and some standard files and it removes
the need for an explicit `MANIFEST.in` file.
Change-Id: I8b89fc5c1ec1eedafb3e5a83e6acc74cfb9719e2
Blatantly plagiarized from I1acc2deadc01421baec256352b35a646676445e5
This commit switches the tox command to use ostestr instead of calling
testr through setup.py. The primary advantage here is that it uses the
subunit-trace output filter. ostestr primarily exists as a replacement
for pretty_tox.sh bash scripts which spread like a plague through
OpenStack projects after Nova copied it from Tempest. (although ostestr
also provides some other useful features) Granted ironic wasn't using
that, but this commit makes the switch to enable getting the
subunit-trace output which is generally useful.
Note that this dramatically increases output while running tests.
However, test failures are still at the bottom of the output, so it
shouldn't cause much pain, if any.
Change-Id: Ia373b9b4857824644e99a2c8d27e64018cb84b78
"Python 3.3 support is being dropped since OpenStack Liberty."
written in following URL.
https://wiki.openstack.org/wiki/Python3
And already the infra team and the oslo team are dropping py33
support from their projects.
Since we rely on oslo for a lot of our work, and depend on infra
for our CI, we should drop py33 support too.
Change-Id: I73a53c2c16b3bc8d7cf2c2e491e3be4c4d007828
Closes-Bug: #1526170
without changing the API
This commit does refactoring regarding the base.Manager and the
specific managers that inherit from it (Chassis, Node, Port and
Driver).
The specific managers had a lot of duplicate code in methods like:
get, create, delete and update, they were literally equal, what does
not make sense once we already had the base.Manager inheritance to
avoid this type of problem.
This change moves the duplicate code to the parent (base.Manager) and
make the specific managers simpler by making the base.Manager more
useful regarding code reuse for these methods.
Change-Id: Ic77a86196d5bcd956f0a3ae517200824a4d70aa8
Closes-Bug: #1517631
node-port-list accecpts node_name but help say it only accecpt
node_uuid. Correct it and add related tests.
Change-Id: I37f7d505e249156e6a428c3b8d2648b43fcc3df4
The exceptions.from_response() from the common.apiclient.exceptions
module takes 'method' as it's second param.
Co-Authored-By: Lucas Alvares Gomes <lucasagomes@gmail.com>
Closes-Bug: #1520528
Change-Id: I7d59c2f6dcdf181f561f38f95b460e556e8b6d9e
Tests for the Ironic CLI node-set-power-state command that checks that
node can be set to 'on', 'off' or 'reboot' power states
Change-Id: Ib396d19280f62c656f5dc0458880c466395cdc2f
Implements: blueprint ironic-client-advanced-functional-tests