From 9911497290de9239440c4f1080f08a9b217ff0f5 Mon Sep 17 00:00:00 2001 From: Jamie Lennox Date: Wed, 13 Jan 2016 15:06:59 +1100 Subject: [PATCH] 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 --- keystonemiddleware/fixture.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/keystonemiddleware/fixture.py b/keystonemiddleware/fixture.py index a6e921b7..18fbb475 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,