squash!: Format code, bump version

Author: rodmgwgu

CHANGELOG.rst

@@ -14,7 +14,7 @@ Change Log
 Unreleased
 **********
 
-0.15.1 - 2025-11-12
+0.16.0 - 2025-11-13
 ********************
 
 Added

openedx_authz/__init__.py

@@ -4,6 +4,6 @@
 
 import os
 
-__version__ = "0.15.1"
+__version__ = "0.16.0"
 
 ROOT_DIRECTORY = os.path.dirname(os.path.abspath(__file__))

openedx_authz/engine/enforcer.py

@@ -149,8 +149,7 @@ def configure_enforcer_auto_save_and_load(cls):
         Returns:
             None
         """
-        auto_load_policy_interval = getattr(
-            settings, "CASBIN_AUTO_LOAD_POLICY_INTERVAL", 0)
+        auto_load_policy_interval = getattr(settings, "CASBIN_AUTO_LOAD_POLICY_INTERVAL", 0)
         auto_save_policy = getattr(settings, "CASBIN_AUTO_SAVE_POLICY", True)
 
         # TODO: remove autoload in favor of cache invalidation?
@@ -166,7 +165,7 @@ def load_policy_if_needed(cls):
         """Load policy if the last load timestamp indicates it's needed.
 
         This method checks if the policy needs to be reloaded comparing
-        the last load timestamp with the last modified timestamp in cache 
+        the last load timestamp with the last modified timestamp in cache
         and reloads it if necessary.
 
         Returns:
@@ -179,13 +178,9 @@ def load_policy_if_needed(cls):
         if last_modified_timestamp is None:
             # No timestamp in cache; initialize it
             cache.set(cls.CACHE_KEY, current_timestamp, None)
-            logger.info(
-                f">>>> Initialized policy last modified timestamp in cache. {current_timestamp}")
+            logger.info(f">>>> Initialized policy last modified timestamp in cache. {current_timestamp}")
 
-        if (
-            cls._last_policy_load_timestamp is None or
-            last_modified_timestamp > cls._last_policy_load_timestamp
-        ):
+        if cls._last_policy_load_timestamp is None or last_modified_timestamp > cls._last_policy_load_timestamp:
             # Policy has been modified since last load; reload it
             cls._enforcer.load_policy()
             cls._last_policy_load_timestamp = current_timestamp
@@ -263,13 +258,11 @@ def _initialize_enforcer(cls) -> SyncedEnforcer:
             # issues when the app is not fully loaded (e.g., while pulling translations, etc.).
             initialize_enforcer(db_alias)
         except Exception as e:
-            logger.error(
-                f"Failed to initialize Casbin enforcer with DB alias '{db_alias}': {e}")
+            logger.error(f"Failed to initialize Casbin enforcer with DB alias '{db_alias}': {e}")
             raise
 
         adapter = ExtendedAdapter()
         enforcer = SyncedEnforcer(settings.CASBIN_MODEL, adapter)
-        enforcer.add_function("is_staff_or_superuser",
-                              is_admin_or_superuser_check)
+        enforcer.add_function("is_staff_or_superuser", is_admin_or_superuser_check)
 
         return enforcer

openedx_authz/models/core.py

@@ -44,6 +44,7 @@ def get_registry(cls) -> dict[str, type["BaseRegistryModel"]]:
         """
         return cls._registry
 
+
 class ScopeManager(models.Manager):
     """Custom manager for Scope model that handles polymorphic behavior."""
 

openedx_authz/settings/common.py

@@ -31,6 +31,8 @@ def plugin_settings(settings):
     # Set default CASBIN_AUTO_LOAD_POLICY_INTERVAL if not already set.
     # This setting defines how often (in seconds) the Casbin enforcer should
     # automatically reload policies from the database.
+    # By default, we set it to 0, which disables the auto-reload.
+    # As it shouldn't be needed thanks to cache invalidation.
     if not hasattr(settings, "CASBIN_AUTO_LOAD_POLICY_INTERVAL"):
         settings.CASBIN_AUTO_LOAD_POLICY_INTERVAL = 0
 

openedx_authz/tests/api/test_roles.py

@@ -968,7 +968,7 @@ def test_assign_role_creates_extended_casbin_rule(self):
         new_count = ExtendedCasbinRule.objects.count()
         self.assertEqual(new_count, initial_count + 1)
 
-        extended_rule = ExtendedCasbinRule.objects.order_by('-id').first()
+        extended_rule = ExtendedCasbinRule.objects.order_by("-id").first()
         self.assertIsNotNone(extended_rule)
         self.assertIsNotNone(extended_rule.casbin_rule)
         self.assertIsNotNone(extended_rule.subject)

openedx_authz/tests/test_enforcer.py

@@ -825,6 +825,7 @@ def test_multiple_get_enforcer_calls_preserve_auto_save(self, mock_toggle):
             AuthzEnforcer.get_enforcer()
             self.assertTrue(AuthzEnforcer.is_auto_save_enabled())
 
+
 class TestEnforcerPolicyCacheBehavior(TransactionTestCase):
     """Test cases for enforcer policy cache behavior.