fix: revert changes in non-relevant files

Author: mariajgrimaldi

openedx_authz/management/commands/enforcement.py

@@ -78,9 +78,7 @@ def handle(self, *args, **options):
         Raises:
             CommandError: If model or policy files are not found or enforcer creation fails.
         """
-        model_file_path = (
-            self._get_file_path("model.conf") or options["model_file_path"]
-        )
+        model_file_path = self._get_file_path("model.conf") or options["model_file_path"]
         policy_file_path = options["policy_file_path"]
 
         if not os.path.isfile(model_file_path):
@@ -95,9 +93,7 @@ def handle(self, *args, **options):
 
         try:
             enforcer = casbin.Enforcer(model_file_path, policy_file_path)
-            self.stdout.write(
-                self.style.SUCCESS("Casbin enforcer created successfully")
-            )
+            self.stdout.write(self.style.SUCCESS("Casbin enforcer created successfully"))
 
             policies = enforcer.get_policy()
             roles = enforcer.get_grouping_policy()
@@ -160,9 +156,7 @@ def _run_interactive_mode(self, enforcer: casbin.Enforcer) -> None:
                 self.stdout.write(self.style.ERROR("Exiting interactive mode..."))
                 break
 
-    def _test_interactive_request(
-        self, enforcer: casbin.Enforcer, user_input: str
-    ) -> None:
+    def _test_interactive_request(self, enforcer: casbin.Enforcer, user_input: str) -> None:
         """Process and test a single enforcement request from user input.
 
         Parses the input string, validates the format, executes the enforcement
@@ -180,11 +174,7 @@ def _test_interactive_request(
         try:
             parts = [part.strip() for part in user_input.split()]
             if len(parts) != 3:
-                self.stdout.write(
-                    self.style.ERROR(
-                        f"✗ Invalid format. Expected 3 parts, got {len(parts)}"
-                    )
-                )
+                self.stdout.write(self.style.ERROR(f"✗ Invalid format. Expected 3 parts, got {len(parts)}"))
                 self.stdout.write("Format: subject action scope")
                 self.stdout.write("Example: user:alice act:read org:OpenedX")
                 return
@@ -193,13 +183,9 @@ def _test_interactive_request(
             result = enforcer.enforce(subject, action, scope)
 
             if result:
-                self.stdout.write(
-                    self.style.SUCCESS(f"✓ ALLOWED: {subject} {action} {scope}")
-                )
+                self.stdout.write(self.style.SUCCESS(f"✓ ALLOWED: {subject} {action} {scope}"))
             else:
-                self.stdout.write(
-                    self.style.ERROR(f"✗ DENIED: {subject} {action} {scope}")
-                )
+                self.stdout.write(self.style.ERROR(f"✗ DENIED: {subject} {action} {scope}"))
 
         except (ValueError, IndexError, TypeError) as e:
             self.stdout.write(self.style.ERROR(f"✗ Error processing request: {str(e)}"))

openedx_authz/tests/test_commands.py

@@ -41,10 +41,7 @@ def test_requires_policy_file_argument(self):
         with self.assertRaises(CommandError) as ctx:
             call_command("enforcement")
 
-        self.assertEqual(
-            "Error: the following arguments are required: --policy-file-path",
-            str(ctx.exception),
-        )
+        self.assertEqual("Error: the following arguments are required: --policy-file-path", str(ctx.exception))
 
     def test_policy_file_not_found_raises(self):
         """Test that command errors when the provided policy file does not exist."""
@@ -55,18 +52,13 @@ def test_policy_file_not_found_raises(self):
 
         self.assertEqual(f"Policy file not found: {non_existent}", str(ctx.exception))
 
-    @patch.object(
-        EnforcementCommand, "_get_file_path", return_value="invalid/path/model.conf"
-    )
+    @patch.object(EnforcementCommand, "_get_file_path", return_value="invalid/path/model.conf")
     def test_model_file_not_found_raises(self, mock_get_file_path: Mock):
         """Test that command errors when the provided model file does not exist."""
         with self.assertRaises(CommandError) as ctx:
             call_command("enforcement", policy_file_path=self.policy_file_path.name)
 
-        self.assertEqual(
-            f"Model file not found: {mock_get_file_path.return_value}",
-            str(ctx.exception),
-        )
+        self.assertEqual(f"Model file not found: {mock_get_file_path.return_value}", str(ctx.exception))
 
     @patch("openedx_authz.management.commands.enforcement.casbin.Enforcer")
     def test_error_creating_enforcer_raises(self, mock_enforcer_cls: Mock):
@@ -76,16 +68,11 @@ def test_error_creating_enforcer_raises(self, mock_enforcer_cls: Mock):
         with self.assertRaises(CommandError) as ctx:
             call_command("enforcement", policy_file_path=self.policy_file_path.name)
 
-        self.assertEqual(
-            "Error creating Casbin enforcer: Enforcer creation error",
-            str(ctx.exception),
-        )
+        self.assertEqual("Error creating Casbin enforcer: Enforcer creation error", str(ctx.exception))
 
     @patch("openedx_authz.management.commands.enforcement.casbin.Enforcer")
     @patch.object(EnforcementCommand, "_run_interactive_mode")
-    def test_successful_run_prints_summary(
-        self, mock_run_interactive: Mock, mock_enforcer_cls: Mock
-    ):
+    def test_successful_run_prints_summary(self, mock_run_interactive: Mock, mock_enforcer_cls: Mock):
         """
         Test successful command execution with policy file and interactive mode.
         When files exist, command should create enforcer, print counts, and call interactive loop.
@@ -102,11 +89,7 @@ def test_successful_run_prints_summary(
         mock_enforcer.get_named_grouping_policy.return_value = action_grouping
         mock_enforcer_cls.return_value = mock_enforcer
 
-        call_command(
-            "enforcement",
-            policy_file_path=self.policy_file_path.name,
-            stdout=self.buffer,
-        )
+        call_command("enforcement", policy_file_path=self.policy_file_path.name, stdout=self.buffer)
 
         output = self.buffer.getvalue()
         self.assertIn("Casbin Interactive Enforcement", output)
@@ -122,17 +105,10 @@ def test_run_interactive_mode_displays_help(self):
             self.command._run_interactive_mode(self.enforcer)
 
         self.assertIn("Interactive Mode", self.buffer.getvalue())
-        self.assertIn(
-            "Test custom enforcement requests interactively.", self.buffer.getvalue()
-        )
-        self.assertIn(
-            "Enter 'quit', 'exit', or 'q' to exit the interactive mode.",
-            self.buffer.getvalue(),
-        )
+        self.assertIn("Test custom enforcement requests interactively.", self.buffer.getvalue())
+        self.assertIn("Enter 'quit', 'exit', or 'q' to exit the interactive mode.", self.buffer.getvalue())
         self.assertIn("Format: subject action scope", self.buffer.getvalue())
-        self.assertIn(
-            "Example: user:alice act:read org:OpenedX", self.buffer.getvalue()
-        )
+        self.assertIn("Example: user:alice act:read org:OpenedX", self.buffer.getvalue())
 
     def test_run_interactive_mode_maintains_interactive_loop(self):
         """Test that the interactive mode maintains the interactive loop."""
@@ -211,9 +187,7 @@ def test_interactive_request_error(self, exception: Exception):
         """Test that `_test_interactive_request` handles processing errors."""
         self.enforcer.enforce.side_effect = exception
 
-        self.command._test_interactive_request(
-            self.enforcer, "user:alice act:read org:OpenedX"
-        )
+        self.command._test_interactive_request(self.enforcer, "user:alice act:read org:OpenedX")
 
         error_output = self.buffer.getvalue()
         self.assertIn(f"✗ Error processing request: {str(exception)}", error_output)