OKTA-641384: clear token from cache on call to oauth.clear_access_tok…

…en() (#380)

fix oauth.clear_access_token()
 update changelog and version

---------

Co-authored-by: haggrip <[email protected]>

CHANGELOG.md

@@ -1,5 +1,8 @@
 # Okta Python SDK Changelog
 
+## 2.9.5
+* Clear access token from cache on call to OAuth.clear_access_token()
+
 ## 2.9.4
 * Add optional parameter to api_response.next() to include response object as a third tuple value. 
 

okta/__init__.py

@@ -1 +1 @@
-__version__ = '2.9.4'
+__version__ = '2.9.5'

okta/oauth.py

@@ -90,3 +90,5 @@ def clear_access_token(self):
         Clear currently used OAuth access token, probably expired
         """
         self._access_token = None
+        self._request_executor._cache.delete("OKTA_ACCESS_TOKEN")
+        self._request_executor._default_headers.pop("Authorization", None)

tests/unit/test_oauth_clear_access_token.py

@@ -0,0 +1,32 @@
+from okta.oauth import OAuth
+
+"""
+Testing OAuth.clear_access_token
+"""
+class WasCalled:
+    def __init__(self):
+        self.value = False
+
+cache_delete_was_called = WasCalled
+headers_pop_was_called = WasCalled
+
+class mockCache:
+    def delete(token):
+        cache_delete_was_called.value = True
+
+class mockHeaders:
+    def pop(header, ignored):
+        headers_pop_was_called.value = True
+
+class mockRequestExecutor:
+    def __init__(self, cache):
+        self._cache = cache
+        self._default_headers = mockHeaders
+
+def test_oauth_clear_access_token():
+    _mockRequestExecutor = mockRequestExecutor(mockCache)
+    oauth = OAuth(_mockRequestExecutor, {})
+    oauth.clear_access_token()
+    assert cache_delete_was_called.value
+    assert headers_pop_was_called.value
+