diff options
| author | Jamie Lennox <jamielennox@gmail.com> | 2016-01-13 15:06:59 +1100 |
|---|---|---|
| committer | Jamie Lennox <jamielennox@gmail.com> | 2016-01-13 15:10:29 +1100 |
| commit | 9911497290de9239440c4f1080f08a9b217ff0f5 (patch) | |
| tree | 9288c88895064dca099420b57d2a8264509678c8 /keystonemiddleware/fixture.py | |
| parent | 5ba3d06b2063e10cf30dafd3bd6886f8fc24244d (diff) | |
| download | keystonemiddleware-9911497290de9239440c4f1080f08a9b217ff0f5.tar.gz | |
Revert "Stop using private keystoneclient functions"
This reverts commit 96f6668a27c34c7fd49c8a0df160c0789b0aa6ba.
Whilst in theory I agree we don't want to expose this many positional
arguments. This is exactly what positional is supposed to be used for.
I'm happy to have the debate as to where we can make @positional
publically available from but for now (and follow up patch) let's keep
it.
Change-Id: Idd07e49370597b63c0e49b56843266a7014a0e65
Diffstat (limited to 'keystonemiddleware/fixture.py')
| -rw-r--r-- | keystonemiddleware/fixture.py | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/keystonemiddleware/fixture.py b/keystonemiddleware/fixture.py index a6e921b..18fbb47 100644 --- a/keystonemiddleware/fixture.py +++ b/keystonemiddleware/fixture.py @@ -15,6 +15,7 @@ import uuid import fixtures from keystoneauth1 import fixture as client_fixtures +from keystoneclient import utils from oslo_utils import timeutils from keystonemiddleware import auth_token @@ -42,6 +43,7 @@ class AuthTokenFixture(fixtures.Fixture): def tokens(self): return self._token_data.keys() + @utils.positional(1) def add_token_data(self, token_id=None, expires=None, user_id=None, user_name=None, user_domain_id=None, user_domain_name=None, project_id=None, |