diff options
author | Gage Hugo <gagehugo@gmail.com> | 2019-09-24 11:34:37 -0500 |
---|---|---|
committer | Colleen Murphy <colleen@gazlene.net> | 2020-05-14 19:46:59 +0000 |
commit | 247227f3f367be37db3292ecae130e3e02f18063 (patch) | |
tree | 06b79a702f1c7e87f4be864ee2c8092e79ca7b95 | |
parent | 40cbb7bebd50276412daa1981ff5a7c7b3b899a5 (diff) | |
download | keystone-247227f3f367be37db3292ecae130e3e02f18063.tar.gz |
Change time faking for totp test
This change moves the time mocking from using freezegun to
using oslo.utils TimeFixture for the unit test
test_with_passcode_in_previous_windows_extended, which was
occasionally failing with 401 errors due to
the totp creation time not properly be faked with
4 extended windows.
Closes-Bug: #1843464
Change-Id: I3aefd99907fbc2d03538c9814f7279b282715679
(cherry picked from commit 6525203c1af9ecdf7c23af3f25e45f0db9f6fed2)
-rw-r--r-- | keystone/tests/unit/test_v3_auth.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/keystone/tests/unit/test_v3_auth.py b/keystone/tests/unit/test_v3_auth.py index 6d6c6f649..278d7f6e7 100644 --- a/keystone/tests/unit/test_v3_auth.py +++ b/keystone/tests/unit/test_v3_auth.py @@ -5306,9 +5306,9 @@ class TestAuthTOTP(test_v3.RestfulTestCase): secret = creds[-1]['blob'] past = datetime.datetime.utcnow() - datetime.timedelta(minutes=2) - with freezegun.freeze_time(past): - auth_data = self._make_auth_data_by_id( - totp._generate_totp_passcodes(secret)[0]) + self.useFixture(fixture.TimeFixture(past)) + auth_data = self._make_auth_data_by_id( + totp._generate_totp_passcodes(secret)[0]) # Stop the clock otherwise there is a chance of auth failure due to # getting a different TOTP between the call here and the call in the |