refactor: Remove unused variables

Author: KShivendu

supertokens_python/recipe/session/constants.py

@@ -34,7 +34,6 @@
 available_token_transfer_methods: List[TokenTransferMethod] = ["cookie", "header"]
 
 JWKCacheMaxAgeInMs = 6 * 1000  # 6s
-JWKRequestCooldownInMs = 500  # 0.5s
 protected_props = [
     "sub",
     "iat",

supertokens_python/recipe/session/jwks.py

@@ -19,7 +19,7 @@
 
 from jwt import PyJWK, PyJWKSet
 
-from .constants import JWKCacheMaxAgeInMs, JWKRequestCooldownInMs
+from .constants import JWKCacheMaxAgeInMs
 
 from supertokens_python.utils import RWMutex, RWLockContext, get_timestamp_ms
 from supertokens_python.querier import Querier
@@ -38,8 +38,7 @@ class JWKSConfigType(TypedDict):
 
 
 class CachedKeys:
-    def __init__(self, path: str, keys: List[PyJWK]):
-        self.path = path
+    def __init__(self, keys: List[PyJWK]):
         self.keys = keys
         self.last_refresh_time = get_timestamp_ms()
 
@@ -121,9 +120,8 @@ def get_latest_keys(kid: Optional[str] = None) -> List[PyJWK]:
                 last_error = e
 
             if cached_jwks is not None:  # we found a valid JWKS
-                cached_keys = CachedKeys(path, cached_jwks)
+                cached_keys = CachedKeys(cached_jwks)
                 log_debug_message("Returning JWKS from fetch")
                 return cached_keys.keys
 
     raise last_error
-

tests/test_utils.py

@@ -112,22 +112,26 @@ class BankAccount:
     def __init__(self):
         self.balance = 0
         self.mutex = RWMutex()
+        self.deposit_count = 0
+        self.withdraw_count = 0
 
     def deposit(self, amount: int):
         self.mutex.lock()
         self.balance += amount
+        self.deposit_count += 1
         self.mutex.unlock()
 
     def withdraw(self, amount: int):
         self.mutex.lock()
         self.balance -= amount
+        self.withdraw_count += 1
         self.mutex.unlock()
 
-    def get_balance(self):
+    def get_stats(self):
         self.mutex.r_lock()
         balance = self.balance
         self.mutex.r_unlock()
-        return balance
+        return balance, (self.deposit_count, self.withdraw_count)
 
 
 def test_rw_mutex_writes():
@@ -140,8 +144,9 @@ def test_rw_mutex_writes():
         threads.append(t)
 
     def balance_is_valid():
-        balance = account.get_balance()
-        assert balance % 5 == 0 and balance >= 0
+        balance, (deposit_count, widthdraw_count) = account.get_stats()
+        expected_balance = 10 * deposit_count - 5 * widthdraw_count
+        assert balance == expected_balance
 
     # Create 15 balance checking threads
     for _ in range(15):
@@ -164,4 +169,5 @@ def balance_is_valid():
     # Check account balance
     expected_balance = 10 * 10  # 10 threads depositing 10 each
     expected_balance -= 10 * 5  # 10 threads withdrawing 5 each
-    assert account.get_balance() == expected_balance, "Incorrect account balance"
+    actual_balance, _ = account.get_stats()
+    assert actual_balance == expected_balance, "Incorrect account balance"