| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
|
|
|
|
|
|
|
|
| |
The tests weren't using keep_blank_values when calling
urlparse.parse_qs, so any empty values were just ignored and
couldn't be tested properly. With this change, tests can verify
query parameters that have no value (for example, ?no_catalog on
the v3 auth request).
bp auth-token-use-client
Change-Id: Iafcb952c81ca7bd2acab4383687c36ec68a838d2
|
| |
|
|
|
|
|
| |
Left timeutils and strutils in openstack/common since they are used in
openstack/common/apiclient and memorycache.
Change-Id: Idb5f09c159d907dfba84cd1f7501f650318af7d9
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
HTTPretty was great particularly when we were having to deal with both
httplib and requests together. Since that time the library itself has
caused a number of problems including backwards incompatible changes,
broken releases and some dependency issues that make it difficult to
package. There are also issues with memcache tests or anything else
that also tries to use the socket.
requests-mock does a very similar job, with a very similar interface
however it targets only requests and so doesn't have the same socket
issues and will be a much easier dependency on packagers.
keystoneclient is the first of a number of clients that will do the
changeover.
Change-Id: Ida6e5feb71b6ff6662fb24b9fa6535b039c99d96
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fixed most of the errors reported back from hacking 0.9.2.
Specifically:
- E128 continuation line under-indented for visual indent
- E251 unexpected spaces around keyword / parameter equals
- E265 block comment should start with '# '
- H305 imports not grouped correctly
- H307 like imports should be grouped together
- H402 one line docstring needs punctuation
- H904 Wrap long lines in parentheses instead of a backslash
But opted to ignore the following for now:
- E122: continuation line missing indentation or outdented
- H405: multi line docstring summary not separated with an empty line
Change-Id: Ib8e698d85fd598fa91435538657361a1f695ce89
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch adds role assignments list support
to keystoneclient.
Created RoleAssignment resource and RoleAssignmentManager
classes. RoleAssignmentManager only implements the list()
method, the other inherited methods from base.CrudManager
raises a MethodNotImplemented error with customized messages.
This bp is complimented with the OSC part:
https://blueprints.launchpad.net/python-openstackclient/+spec/roles-assignment-list
Change-Id: I164b58b67ff42320238e943ddfa9d0a8aadd0a6d
Implements: blueprint roles-assignment-support
Closes-Bug: #1246310
|
| |
|
|
|
|
|
|
| |
Move DisableModuleFixture to test utilities instead of auth token
middleware tests, so it can be reused for the oauthlib tests.
implements: bp add-oauth-support
Change-Id: I09723a9f0e28e73e1aa31072e514f5fae2173100
|
| |
|
|
|
|
| |
The default value for dict.get is None, no need to specify again.
Change-Id: I9f0204e59bd4c1642586bddfef0c75e0a7730925
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| | |
We don't need vim modelines in each source file, it can be set in
user's vimrc.
Change-Id: Ic7a61430a0a320ce6b0c4518d9f5d988e35f8aae
Closes-Bug: #1229324
|
| |\ \ |
|
| | |/
| |
| |
| |
| |
| |
| |
| | |
This was added in review: https://review.openstack.org/#/c/53500 but
lost in the conversion to using session. Add it back again.
Change-Id: Ia063eb018d3a7da706a02d60df63bfa1be21d147
Related-Bug: #1249891
|
| |/
|
|
|
|
| |
In Python 3, we must decode it before using it.
Change-Id: I473ba7b9732555ef5f9011a50d8684260e1f7f71
|
| |
|
|
|
|
|
| |
Co-Authored-By: Saju Madhavan <sajuptpm@gmail.com>
Closes-Bug: 1047867
Implements: blueprint extensible-crud-manager-operations
Change-Id: I64e11bcf3797eb84e4695605daea9749259d78ec
|
| |
|
|
|
|
|
|
| |
As requested in https://review.openstack.org/#/c/65728/, this "fixes"
assertQueryStringIs so it takes a query string, rather than a dict
of lists, which is a less convenient format in many tests.
Change-Id: Ib4c49ba71f7e12bb5bcb4dfde77974f3418c7854
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| | |
In Python 2, get() is a synonym for getheader(), which does not exist in Python
3.
Closes-Bug: #1267987
Change-Id: I6989a6ea733e0a186e9e68a61aaa8c3e43a198a6
|
| |/
|
|
|
|
|
|
|
|
|
|
| |
The "requests" module tries to do this comparison in raise_for_status():
400 <= self.status_code < 500
with "self.status_code" being "None". In Python2, it's a valid comparison, but
in Python3, it triggers a TypeError. We fix it by giving a valid default value
to "self.status_code".
Change-Id: I9ea9ecfb698546a7f6526464aa528441bb7c1ccd
|
| |
|
|
|
|
|
| |
This will be a fairly common pattern and so should be a helper of its
own.
Change-Id: Ic06add6dfe40c7d3ab88dfdd99a9087fdebd9b67
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
It should be possible to authenticate against the v2 tokens
API with a trust_id, because it suports rescoping an existing
token to a trust, this patch adds client support for this.
Note with the current keystone code it's necessary to pass the
trustor tenant ID when rescoping with a trust where
impersonation==True, e.g:
c = client_v2.Client(username=TRUSTEE_USERNAME,
password=TRUSTEE_USERNAME,
tenant_name=TRUSTEE_TENANT_NAME,
auth_url=OS_AUTH_URL_V2)
c.authenticate(trust_id=trust_i.id, tenant_id=TRUSTOR_TENANT_ID)
Change-Id: I177c41af298b7437e2c6fb437aa9ce9a09773b9d
Closes-Bug: #1231483
|
|
|
This is the suggested location for tests and is adopted by most
projects. As part of this change relative imports to package imports.
Fix all the test running and coverage code to point to the new location.
Change-Id: I01264aed14f396ab9a7242e3e72b71e1bc332675
|
