summaryrefslogtreecommitdiff
path: root/openstackclient/tests
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | Replace 'MagicMock' with 'Mock'qtang2016-09-1429-63/+63
| |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | Change-Id: I7aeceede6bd3cb88cf04f398454f9758dbee20f1 Closes-Bug: #1475722
* | | | | | Support fetching network project default quotaRui Chen2016-09-132-0/+20
| |_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Neutron server and openstacksdk had supported to fetch network project default quota, this patch add the CLI support in openstackclient. Change-Id: If0ef74c268c41a866c62156da0603a40ae4e6e31 Closes-Bug: #1204956 Depends-On: I6a4e2a146351dd1e7d652442511f1ef2c279da42
* | | | | Merge "Display hypervisor information without uptime"Jenkins2016-09-121-0/+70
|\ \ \ \ \
| * | | | | Display hypervisor information without uptimeHironori Shiina2016-09-111-0/+70
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some virt drivers such as ironic virt driver doesn't implement a method to get host uptime. For such drivers, hypervisor show command displays no information although these drivers provides other host information. This patch fixes the command to display hypervisor information in case where a virt driver doesn't provide host uptime by ignoring a HTTPNotImplemented exception. Change-Id: I7bcca5862cd9c05aadaf6192cb80aa651cd77cad Closes-Bug: 1612065
* | | | | | Merge "Fix regular expression for uptime in hypervisor show"Jenkins2016-09-121-0/+8
|\ \ \ \ \ \ | |/ / / / /
| * | | | | Fix regular expression for uptime in hypervisor showHironori Shiina2016-09-111-0/+8
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Hypervisor show command has a few bugs as follows. - It doesn't trim an extra whitespace in the head of uptime information. - It doesn't display uptime information when the number of user is 1. This patch fixes the regular expression to match uptime information. Change-Id: Ic2f7fd9a9274466717084a0886f95f78e98a9007 Closes-Bug: 1611809
* | | | | Merge "Set quota "per_volume_gigabytes", "backup_gigabytes" and "backups""Jenkins2016-09-121-0/+22
|\ \ \ \ \
| * | | | | Set quota "per_volume_gigabytes", "backup_gigabytes" and "backups"Rui Chen2016-09-121-0/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "per_volume_gigabytes", "backup_gigabytes" and "backups" items can be shown in "openstack quota show" command, but can't be updated by "openstack quota set". This patch fix the issue. Change-Id: I47db5a69d4e4ef6e140f2735257c83e1fb052760 Closes-Bug: #1609767
* | | | | | Merge "Add functional tests for unset multiple volume type props"Jenkins2016-09-122-0/+30
|\ \ \ \ \ \
| * | | | | | Add functional tests for unset multiple volume type propsRui Chen2016-09-122-0/+30
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | CinderClient bug/1596511 cause that OSC can't unset multiple properties of volume type, obviously no functional test case cover the part of logic in OSC, that make the issue raising, the patch add functional test cases for this scenario. Change-Id: I42cf9ac8cc72ccc2f1208926d8faf7b80ee2d288 Partial-Bug: #1596511 Depends-On: I60378a32cdc52aacdf869d69b246dec7eb6cdb77
* | | | | | Merge "Trivial: Rename ListTransferRequests to ListTransferRequest"Jenkins2016-09-122-2/+2
|\ \ \ \ \ \
| * | | | | | Trivial: Rename ListTransferRequests to ListTransferRequestTang Chen2016-09-122-2/+2
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The class name should use plural. Change-Id: I2de2489ff9aa60c2d1bf12743cbd41f2091739ca
* | | | | | Merge "Unit tests of quota command refactor"Jenkins2016-09-122-44/+112
|\ \ \ \ \ \ | | |/ / / / | |/| | | |
| * | | | | Unit tests of quota command refactorRui Chen2016-09-122-44/+112
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Do refactor for quota command related unit tests, remove useless code, add new tests to cover '--volume-type' and '--class' option in "quota set" command and add the volume quota fake object. Change-Id: Iaf214740e98db1bfb4c739e810bac1c5ba6e9625
* | | | | Merge "remove duplicate unit test"Jenkins2016-09-121-286/+0
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | remove duplicate unit testSteve Martinelli2016-09-101-286/+0
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | looks like this test snuck in during the refactor: $ diff openstackclient/tests/volume/v1/test_service.py \ openstackclient/tests/unit/volume/v1/test_service.py 17c17 < from openstackclient.tests.volume.v1 import fakes as service_fakes --- > from openstackclient.tests.unit.volume.v1 import fakes as service_fakes Change-Id: I769b2d39f28cfaf65e4027c785b0ddbbefa26aea
* | | | Merge "Add unit test for volume type in volume v1"Jenkins2016-09-122-0/+421
|\ \ \ \ | |_|/ / |/| | |
| * | | Add unit test for volume type in volume v1Huanxuan Ao2016-09-112-0/+421
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add unit test for volume type commands (create/delete/show/list/set/unset) in volume v1 Change-Id: I6ff1f1c7482bd0b4bfec5b4a1496807b722fa047
* | | | Implement "volume transfer request delete" commandHuanxuan Ao2016-09-116-0/+348
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add "volume transfer request delete" command in volume v1 and v2. Also add the unit tests, docs, release note and functional tests Change-Id: Ic3d375bc8df3312fac53c1800d75f48376b8c91c Implements: bp cinder-command-support Co-Authored-By: Sheel Rana <ranasheel2000@gmail.com>
* | | | Implement "volume transfer request create" commandHuanxuan Ao2016-09-114-6/+152
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | Add "volume transfer request create" command in volume v1 and v2. Also add the unit tests, docs, release note and functional tests Change-Id: If362df1acf214efdf6ba129cd917d33eb54e1030 Implements: bp cinder-command-support Co-Authored-By: Sheel Rana <ranasheel2000@gmail.com>
* | | Merge "Error handling of multi REST API calls for "snapshot set" command"Jenkins2016-09-111-0/+49
|\ \ \
| * | | Error handling of multi REST API calls for "snapshot set" commandHuanxuan Ao2016-09-111-0/+49
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | Support multi REST API calls error handling for "snapshot set" command follow the rule in doc/source/command-errors.rst. Also add a unit test for testing the error handling Change-Id: I0c6214271bc54a25b051c0a62438c3344c8b51d7
* | | Merge "Add "--limit" and "--marker" options to "volume list" command"Jenkins2016-09-112-0/+112
|\ \ \
| * | | Add "--limit" and "--marker" options to "volume list" commandHuanxuan Ao2016-09-112-0/+112
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | Add ``--limit`` option to ``volume list`` command in volume v1, add ``--limit`` and ``--marker`` options to ``volume list`` command in volume v2. Change-Id: I327a252aa83ed84166da99cf6aa80334e0e6dd44 Partial-Bug: #1612484
* | | Support error handling for delete commands in volume v1Huanxuan Ao2016-09-113-15/+321
|/ / | | | | | | | | | | | | | | | | Some delete commands in volume v1 support multi delete but do not support error handling, this patch fixes them, and this patch also refactor (or add new) unit tests for some delete commands in volume v1. Change-Id: Ia8177698f8733cfe75ea0ff00eee8fdc0820f62e
* | unwedge the gateSteve Martinelli2016-09-091-4/+4
|/ | | | | | | | since changing the infra job to call the new bash script location, the functional tests now have a permission error: py.error.EACCES: [Permission denied]: mkdir('/opt/stack/new/python-openstackclient/.tox',) Change-Id: Ibe7057c99feac952d80156fb996d0193e1b0e497
* move all functional tests to tests moduleSteve Martinelli2016-09-0979-0/+4653
| | | | | | | | | | | | | | functional tests should be grouped with other tests (unit and integration tests). as part of this commit the "common" module was renamed to just "base", this was done for simplicity. the post_test_hook.sh file was also copied to the functional module since it should live there. a separate change to the infra repo will be made to call the new location, once that is merged we can remove the old one (a new change will also be posted for that) Needed-By: I49d54f009021d65c1ae49faf6b3f0a7acdadd7b3 Change-Id: Ie8c334f6223373b8e06df8bd8466500d2a2c8ede
* move unit tests to new "unit" test moduleSteve Martinelli2016-09-08125-205/+491
| | | | | | | | this will better isolate the unit tests from the functional tests. unfortunately, the "integration" tests had to be lumped into the "unit" tests since we need the separation in testr.conf Change-Id: Ifd12198c1f90e4e3c951c73bfa1884ab300d8ded
* Merge "format token expires time to prevent json loading datetime data"Jenkins2016-09-084-7/+7
|\
| * format token expires time to prevent json loading datetime dataSteve Martinelli2016-09-054-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | When output to shell, the token issue time is fine; however when selecting the json formatter (via the --format json) option, an exception is raised when formatting the dataetime data. Rather than pass in the datetime data, we should format the data with the ISO 8601 formatting. Closes-Bug: 1619937 Change-Id: Iffebb2d5413fabfd283dfa94fc560fc37270f9dd
* | Merge "Add Subnet service-types to subnets"Jenkins2016-09-082-2/+93
|\ \
| * | Add Subnet service-types to subnetsBrian Haley2016-08-192-2/+93
| | | | | | | | | | | | | | | | | | | | | | | | Add '--service-type' to subnet arguments to support Subnet service-types. Change-Id: I215d83e4d4cf53e03fa35041c5e41a328641b3a9 Partially-implements: blueprint service-subnets
* | | Merge "Clean up FakeClient classes in volume fakes"Jenkins2016-09-066-78/+10
|\ \ \
| * | | Clean up FakeClient classes in volume fakesHuanxuan Ao2016-08-236-78/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Put FakeTransferClient and FakeServiceClient classes into FakeVolumeClient class, I think we need not to create FakeClient classes for every objects, we can use only FakeVolumeClient for all volume objects. This can reduce repetition, unified codes and make codes look cleaner and easier. Change-Id: I2729ef0e80166f4d49ccd9a48b653e2f215e3bfe
* | | | Merge "Use assertIn(A, B) instead of assertTrue(A in B )"Jenkins2016-09-062-8/+8
|\ \ \ \
| * | | | Use assertIn(A, B) instead of assertTrue(A in B )Bin Zhou2016-09-022-8/+8
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | Developers should use assertIn(A, B) instead of assertTrue(A in B ). TrivialFix Change-Id: I0d7c542272315590536c427a8865e49660361e55
* | | | Merge "Add "volume service set" command"Jenkins2016-09-062-0/+290
|\ \ \ \ | |/ / / |/| | |
| * | | Add "volume service set" commandHuanxuan Ao2016-08-172-0/+290
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add "volume service set" command in volume v1 and v2 (v1 is the same as v2) to disable or enable volume service. Change-Id: Ibb2db7e93b24cb2e0d2a7c28b6fd8bcc851b8d2f Closes-Bug: #1613597
* | | | Rearrange integration testsDean Troyer2016-08-315-195/+411
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move the integration tests into their final home... * Create tests/integ to hold the integration tests * Split tests/test_shell_integ.py into tests/integ/base.py and tests/integ/cli/test_shell.py * Rename TestXXXXInteg classes to TestIntegXXXX * Adds tests/integ/cli/test_project.py for some simple project argument tests Change-Id: I2cdd340d1d446d61784eae35dd5aa09d40d5899d
* | | | Fix auth prompt brokennessDean Troyer2016-08-291-0/+58
| |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We start by fixing this in the already-present OSC_Config class so OSC can move forward. This change needs to get ported down into os-client-config in the near future, maybe even soon enough to make the client library freeze this week. * Add the pw-func argument to the OSC_Config (or OpenStackConfig) __init__() * When looping through the auth options from the KSA plugin look for any that have a prompt defined and do not have a value already, so ask for one. Closes-bug: #1617384 Change-Id: Ic86d56b8a6844516292fb74513712b486fec4442
* | | Merge "Integ test cleanup"Jenkins2016-08-211-42/+91
|\ \ \
| * | | Integ test cleanupDean Troyer2016-08-181-42/+91
| | | | | | | | | | | | | | | | Change-Id: Ie58a7bec569421097e92a7ddf3cb164fc3f07413
* | | | Merge "Fix errors in volume set/unset image properties unit tests"Jenkins2016-08-191-3/+3
|\ \ \ \
| * | | | Fix errors in volume set/unset image properties unit testsHuanxuan Ao2016-08-111-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These errors may caused by copy-paste, we should specify a fake return value for get() method but not create() method. Create() mothod will not called in set/unset command. After specifing the return value of get(), we can use the fake volume in the assert. Change-Id: I1e0df4c28ece373168788c396e9082d565e36cc7
* | | | | Merge "Add support for domain specific roles"Jenkins2016-08-183-3/+336
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Add support for domain specific rolesHenry Nash2016-08-103-3/+336
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | A role entity can now be specified as domain specific. Closes-bug: #1606105 Change-Id: I564cf3da1d61f5bfcf85be591480d2f5c8d694a0
* | | | Merge "Add shell integration test"Jenkins2016-08-181-0/+561
|\ \ \ \
| * | | | Add shell integration testDean Troyer2016-08-181-0/+561
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These run next to unit tests, but unlike unit tests, they test the dependent libraries also. This is to detect incompatible breakage in those dependencies. The tests provide CLI-level input and verify the API calls being made via requests using requests_mock so the entire stack is tested. It is possible we want to run these separate from the unit tests. They do not belong in the functional tests as they do not require a functional cloud for testing. Depends-on: I426548376fc7d3cdb36501310dafd8c44d22ae30 Change-Id: I356956fcc4ff35191a6ad6a085b75cf370434b09
* | | | | Fix up last-minute imports to use osc-libDean Troyer2016-08-181-1/+2
| |_|_|/ |/| | | | | | | | | | | Change-Id: I1ed2983cf574ebd565eeac4f8199fbc3a2e29c8e
* | | | Merge "Gate-unbreaking combo review"Jenkins2016-08-182-74/+9
|\ \ \ \ | |/ / / | | | / | |_|/ |/| |
View Lorry Controller Status