summaryrefslogtreecommitdiff
path: root/openstackclient/tests
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Add unit tests for "server pause" command."Jenkins2015-11-191-0/+57
|\
| * Add unit tests for "server pause" command.Tang Chen2015-11-201-0/+57
| | | | | | | | | | | | | | | | This patch adds unit tests for "server pause" command, including one and multiple servers. Change-Id: If5551e77d7dd4f7f48c6ee4a7f80f8313817f492 Implements: blueprint osc-unit-test-framework-improvement
* | Merge "Introduce random server faking mechanism."Jenkins2015-11-191-0/+70
|\ \ | |/
| * Introduce random server faking mechanism.Tang Chen2015-11-191-0/+70
| | | | | | | | | | | | | | | | | | | | | | This patch introduces a new server faking mechanism to support multiple servers faking. Server names and ids can be generated randomly, and use APIs in class FakeServer to get one or more servers. Change-Id: Ic54f3bf7c77294dc7dfb9acdbf4a721eb5eef6af Implements: blueprint osc-unit-test-framework-improvement
* | Merge "Enable FakeResource to fake methods."Jenkins2015-11-191-1/+26
|\ \ | |/
| * Enable FakeResource to fake methods.Tang Chen2015-11-191-1/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use MagicMock to fake a method in FakeResource. A new function: add_method(name, return_value) is added to FakeResource. The caller specifies method @name and @return_value, the function will add an attribute with @name, which is a callable MagicMock object whose return value is @return_value. When user access the attribute with a (), @return_value will be returned by MagicMock, which looks like a function call. Change-Id: I12eb876cbebab064773df7b5dd612de69bbf3f01 Implements: blueprint osc-unit-test-framework-improvement
* | Allow error status to be specifiedMark Vanderwiel2015-11-181-0/+40
|/ | | | | | | | For some apis, heat, the error status is "failed". This patch changes the wait_for_status method to allow for the error status to be passed in the same way as the success status. Change-Id: I20db4051d3f5611a4b13fe23ea8798b82a40da81
* Merge "better format remote IDs for identity providers"Jenkins2015-11-152-7/+8
|\
| * better format remote IDs for identity providersSteve Martinelli2015-11-122-7/+8
| | | | | | | | | | | | | | | | | | remote-ids are a list, and we should format these values as such, rather than python representations of lists/arrays. Closes-Bug: 1478995 Change-Id: Ia6ced0fab2435b8cb486822c676c0dee32613abe
* | Merge "Split the vol_id from a dev mapping"Jenkins2015-11-132-0/+83
|\ \
| * | Split the vol_id from a dev mappingxiexs2015-11-112-0/+83
| |/ | | | | | | | | | | | | Add a split into the dev mapping to get the right vol_id. Change-Id: I1a7bf6351491b1321c5ca0fa4a27f29825400eaf Closes-Bug: #1514394
* | Add capability to update description of an IdPlin-hua-cheng2015-11-121-0/+48
| | | | | | | | | | Change-Id: I854067642bbfde6fdf84b22b9cc1de8afc7767c0 Closes-Bug: #1515815
* | Merge "Add project-name/-id validation for the OSC "openstack quota set""Jenkins2015-11-121-6/+20
|\ \
| * | Add project-name/-id validation for the OSC "openstack quota set"xiexs2015-11-041-6/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | The quota info would be set into DB, even though the project actually does not exist. This patch tried to add a validation to forbid this undesirable behavior. Change-Id: Ia2d8c96527820e25b074e6486d3f39c5ad7eae60 Closes-Bug: #1512638
* | | Merge "Fix the bug of "openstack console log show""Jenkins2015-11-121-0/+55
|\ \ \ | |_|/ |/| |
| * | Fix the bug of "openstack console log show"xiexs2015-11-091-0/+55
| | | | | | | | | | | | | | | | | | | | | | | | The behaviors are inconsistent while different negative line numbers specified. Change-Id: I2573f3e789f5603c896758971830ffc0b94c5e2b Closes-Bug: #1512263
* | | Merge "Add --owner to `image create`"Jenkins2015-11-121-2/+4
|\ \ \
| * | | Add --owner to `image create`NiallBunting2015-11-061-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds --owner to `image create`. This is backwards compatable with v1. Change-Id: I9e79cf880c91a1386419db729818d23dfe632179 Depends-On: I8d572a070bbb04dccdd051b8e0ad199c5754746e
* | | | Merge "Fix issue when displaying image_member"Jenkins2015-11-112-3/+9
|\ \ \ \
| * | | | Fix issue when displaying image_memberlin-hua-cheng2015-10-232-3/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | image_member doesn't have a _info attribute, glanceclient returns warlock object instead of a Resource object. Change-Id: If6e7c4bd404454bd6cbe8c111879c1afa1380211 Closes-Bug: #1509054
* | | | | Merge "Add compute service delete"Jenkins2015-11-112-0/+58
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | Add compute service deleteAlex Schultz2015-10-162-0/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change adds 'compute service delete <service>' to the openstack client. This is the equivalent of 'nova service-delete <id>' Change-Id: I69ef1cac72cbe125c2114f8e958e22350a70f367 Closes-Bug: #1503510
* | | | | Use fake server name instead of id when testing "server_name" param.Tang Chen2015-11-071-11/+11
| |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "server create" command takes a server name parameter. The server id is generated by openstack. When we intended to pass a server name to the unit tests of "server create" command, we passed server id. It won't be any problem because the fake server id is also a string, but we should pass a fake server name because we have a string to fake one. Change-Id: I9944f0ea2a6c457e4fad8215a54778bca08965ab
* | | | Import the module not the classSean Perry2015-11-041-5/+5
| |_|/ |/| | | | | | | | | | | | | | As requested during review 226922. Change-Id: Ic5222141e247ce33cf5dbee66667cee3040e1cc3
* | | Merge "Mark arguments for 'credential' commands as required"Jenkins2015-11-042-0/+116
|\ \ \
| * | | Mark arguments for 'credential' commands as requiredSean Perry2015-09-302-0/+116
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | According to the [1], 'user_id', 'type', and 'blob' are all required arguments for 'credential set' but the code treats them as optional. Set the 'required' flag and remove logic supporting missing arguments. [1]: https://github.com/openstack/keystone-specs/blob/master/api/v3/identity-api-v3.rst#credentials-v3credentials "spec" Change-Id: I597c9616ad744385fc6dd92379feb03daec54458 Closes-Bug: #1418837
* | | | Merge "Trivial clean up: do not use plural form in command arguments."Jenkins2015-11-021-3/+3
|\ \ \ \
| * | | | Trivial clean up: do not use plural form in command arguments.Tang Chen2015-10-241-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It is a general rule that we don't use plural form for the name of command arguments. But class DeleteServer() is still using "servers". So use "server instead". Change-Id: I2d76de14ec34b88547b9f728b41e9bd93b2a22c5
* | | | | Merge "Rename context.py to logs.py"Jenkins2015-10-301-32/+32
|\ \ \ \ \
| * | | | | Rename context.py to logs.pyTerry Howe2015-10-301-32/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | At one point this file contained the context for logging, but the reason for its existence is now for logging. Implements: blueprint logging-migration Change-Id: I4ba42bbef97b09d31236ac8c01b6fb23827b8bee
* | | | | | Merge "remove url from v3 regions"Jenkins2015-10-292-75/+6
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | remove url from v3 regionsSteve Martinelli2015-10-212-75/+6
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the parameter "url" was removed from Keystone, it was only added for one release as part of an experimental support for adding service providers. BackwardsIncompatibleImpact Closes-Bug: 1506841 Change-Id: I7a62fbf1d9bfa8e6dd8d619e98c32b9860348d2e
* | | | | Merge "Support pagination params for flavor list"Jenkins2015-10-231-5/+15
|\ \ \ \ \ | |_|_|_|/ |/| | | |
| * | | | Support pagination params for flavor listliyingjun2015-10-211-5/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Missing 'marker' and 'limit' params for `openstack flavor list` shell command. It would be nice to have this when there are many flavors. Closes-bug: #1505874 Change-Id: I088ac5d24f0d7595f5cbce14f063e296a449eb26
* | | | | Merge "Move session and fixtures to keystoneauth1"Jenkins2015-10-207-7/+7
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Move session and fixtures to keystoneauth1Steve Martinelli2015-10-147-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | in an effort to start consuming keystoneauth1, we can move our sesssion and fixture code over to use keystoneauth1 instead of keystoneclient. Change-Id: Ibcbd588ce2e3d864f87ff1eb6e1c3c071a1e06f6
* | | | | Merge "Follow-on for volume list - add tests, clean help"Jenkins2015-10-193-10/+340
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Follow-on for volume list - add tests, clean helpDean Troyer2015-10-163-10/+340
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add volume list tests for v1 (a copy of the v2 tests) * Converts volume v2 tests to use Identity v3 so domains can be tested * Add volume list (v2) tests for new options * Re-orders volume list options (both v1 and v2) to match * MArks the new volume list (v2) options as v2-only in doc Change-Id: I2181b2c48cfde2147d7d0ef135322df8a81e7ce8
* | | | Merge "Fix up object-store show commands"Jenkins2015-10-143-13/+1
|\ \ \ \ | |/ / / |/| | |
| * | | Fix up object-store show commandsSteve Martinelli2015-10-103-13/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1) Change metadata to appear under a common 'properties' key, and use the utility to format them, this applied to object, account and container. 2) Clean up container and object output, which were setting the x-container-meta-owner property, but this is metadata only for the container, so it's pointless to have, removed it. 3) Container show was showing read/write ACLs and sync stuff, but these are not being returned by my swift by default, so I moved these to be checks, so we don't clutter the output. Change-Id: Ife7521fe9c2724035b06963c118bd6016ba2f5b5
* | | | Merge "Add test for role list --inherited"Jenkins2015-10-121-0/+33
|\ \ \ \ | |/ / / |/| | |
| * | | Add test for role list --inheritedDean Troyer2015-10-071-0/+33
| | | | | | | | | | | | | | | | Change-Id: I216ab6c8ac903720ec67870a5171ae57a8f293aa
* | | | Merge "image set should not show the resource"Jenkins2015-10-082-14/+3
|\ \ \ \
| * | | | image set should not show the resourceSteve Martinelli2015-10-062-14/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the rest of OSC set commands do not show the resource after it has been updated. unless the update fails then we report back a failure, otherwise the user should assume everything went fine. Change-Id: I2bd4188450c3853b4a1bc25f80fc9450cda32bdd
* | | | | Merge "Evaluate --inherited in role list"Jenkins2015-10-081-0/+5
|\ \ \ \ \ | | |/ / / | |/| | |
| * | | | Evaluate --inherited in role listRudolf Vriend2015-10-061-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the --inherited option was not being passed into keystoneclient Closes-Bug: #1502822 Change-Id: I48170dc67b23cc9b0665b1e0f38118eea952f131
* | | | | Merge "Add tags to `image set`"Jenkins2015-10-073-6/+98
|\ \ \ \ \ | | |/ / / | |/| | |
| * | | | Add tags to `image set`NiallBunting2015-10-063-6/+98
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds --tag to the v2 version of `image set`. This is another step to compatability between the osc image api. Added merge of tags into existing tags and handling duplicates, and tests for same. Co-Authored-By: Steve Martinelli <stevemar@ca.ibm.com> Change-Id: Ie800fcbf8bbc0978c54ace3278750a18023e8ce4
* | | | | Merge "Clean up Image v2 image set command"Jenkins2015-10-061-33/+174
|\ \ \ \ \ | |/ / / / | | / / / | |/ / / |/| | |
| * | | Clean up Image v2 image set commandDean Troyer2015-10-061-33/+174
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make the Image v2 image set command meet at the intersection of the v1 image set command and the v2 image create command: * Add visibility to the deadopts list and remove the option * Put the options in the same order as v1 image set * Make the help text match * Add --properties * Move the additional options that do not appear in either v1 image set or v2 image create after --property as they are really pre-defined properties * Add tests for v2 image set to match v1 and then some * Put the SetImage class in v2/image.py in alphabetical order Change-Id: I102b914e8ad09a014f6fdd846c5766b6c2eaadb8
View Lorry Controller Status