squash!: Attend PR comments

Author: rodmgwgu

openedx_authz/engine/enforcer.py

@@ -170,7 +170,7 @@ def load_policy_if_needed(cls):
         last_version = PolicyCacheControl.get_version()
 
         if last_version is None:
-            # No timestamp in cache; initialize it
+            # No version in cache control; initialize it
             last_version = uuid4()
             PolicyCacheControl.set_version(last_version)
             logger.info("Initialized policy last modified version in cache control.")

openedx_authz/models/engine.py

@@ -11,6 +11,8 @@ class PolicyCacheControl(models.Model):
     This model can be used to trigger cache invalidation for authorization policies
     by changing the version. Whenever this model is updated, the authorization
     engine should invalidate its cached policies.
+
+    .. no_pii:
     """
 
     version = models.UUIDField(default=uuid4)

openedx_authz/tests/test_enforcer.py

@@ -836,8 +836,8 @@ class TestEnforcerPolicyCacheBehavior(TransactionTestCase):
 
     @patch("openedx_authz.engine.enforcer.libraries_v2_enabled")
     @override_settings(CASBIN_AUTO_LOAD_POLICY_INTERVAL=0)
-    def test_load_policy_if_needed_initializes_cache_timestamp(self, mock_toggle):
-        """Test that load_policy_if_needed initializes cache timestamp on first call.
+    def test_load_policy_if_needed_initializes_cache_version(self, mock_toggle):
+        """Test that load_policy_if_needed initializes cache version on first call.
 
         Expected result:
             - On first call, cache invalidation model is initialized