Reorganize/rename hook function variables for clarity

Author: brodriguez8774

django_expanded_test_cases/test_cases/integration_test_case.py

@@ -101,9 +101,13 @@ def __init__(self, *args, **kwargs):
         # Initialize some check values.
         # Initialize these to true. Then the default implementation of each hook function sets to False.
         # Presumably if at least one hook is implemented, then it will stay True.
-        self._implemented_auth_setup_hook = True
-        self._implemented_pre_assert_hook = True
-        self._implemented_post_assert_hook = True
+
+        class HookClass:
+            auth_setup_is_okay = True
+            pre_assert_is_okay = True
+            post_assert_is_okay = True
+
+        self.hook_checks = HookClass()
 
     @classmethod
     def setUpClass(cls, *args, debug_print=None, **kwargs):
@@ -2141,7 +2145,7 @@ def _assertResponse__pre_builtin_tests(
 
         # Set class variable to track that hook is not implemented.
         # Still using the default implementation that does nothing.
-        self._implemented_pre_assert_hook = False
+        self.hook_checks.pre_assert_is_okay = False
 
     def _assertResponse__post_builtin_tests(
         self,
@@ -2180,7 +2184,7 @@ def _assertResponse__post_builtin_tests(
 
         # Set class variable to track that hook is not implemented.
         # Still using the default implementation that does nothing.
-        self._implemented_post_assert_hook = False
+        self.hook_checks.post_assert_is_okay = False
 
     def _get_login_user__extra_user_auth_setup(
         self,
@@ -2205,7 +2209,7 @@ def _get_login_user__extra_user_auth_setup(
 
         # Set class variable to track that hook is not implemented.
         # Still using the default implementation that does nothing.
-        self._implemented_auth_setup_hook = False
+        self.hook_checks.auth_setup_is_okay = False
 
         return user
 
@@ -2255,8 +2259,8 @@ def _hook_function_warning_check_if_statement(self, user):
             # User is anonymous. Skip auth hook.
             if (
                 # Comment to prevent "black" formatting.
-                not self._implemented_pre_assert_hook
-                and not self._implemented_post_assert_hook
+                not self.hook_checks.pre_assert_is_okay
+                and not self.hook_checks.post_assert_is_okay
             ):
                 should_raise_warning = True
 
@@ -2265,9 +2269,9 @@ def _hook_function_warning_check_if_statement(self, user):
 
             if (
                 # Comment to prevent "black" formatting.
-                not self._implemented_pre_assert_hook
-                and not self._implemented_post_assert_hook
-                and not self._implemented_auth_setup_hook
+                not self.hook_checks.pre_assert_is_okay
+                and not self.hook_checks.post_assert_is_okay
+                and not self.hook_checks.auth_setup_is_okay
             ):
                 should_raise_warning = True
 

tests/django_expanded_test_cases/tests/test_integration_case/test_integration_hooks.py

@@ -31,14 +31,14 @@ def test_hook_is_called(self):
         """Verifies hook function is called and changes state as expected."""
 
         # Verify initial state of class variable.
-        self.assertEqual(True, self._implemented_pre_assert_hook)
+        self.assertEqual(True, self.hook_checks.pre_assert_is_okay)
 
         # Call a basic assertResponse.
         # We don't even care if it works, this is only to trigger the hook function.
         self.assertResponse('')
 
         # Verify variable changed after call. This variable is only changed in the hook function.
-        self.assertEqual(False, self._implemented_pre_assert_hook)
+        self.assertEqual(False, self.hook_checks.pre_assert_is_okay)
 
 
 class TestIntegrationHooks__CanAccessPostBuiltinHook(IntegrationTestCase):
@@ -51,14 +51,14 @@ def test_hook_is_called(self):
         """Verifies hook function is called and changes state as expected."""
 
         # Verify initial state of class variable.
-        self.assertEqual(True, self._implemented_post_assert_hook)
+        self.assertEqual(True, self.hook_checks.post_assert_is_okay)
 
         # Call a basic assertResponse.
         # We don't even care if it works, this is only to trigger the hook function.
         self.assertResponse('')
 
         # Verify variable changed after call. This variable is only changed in the hook function.
-        self.assertEqual(False, self._implemented_post_assert_hook)
+        self.assertEqual(False, self.hook_checks.post_assert_is_okay)
 
 
 class TestIntegrationHooks__CanAccessAuthSetupHook(IntegrationTestCase):
@@ -74,7 +74,7 @@ def test_hook_is_called(self):
         """
 
         # Verify initial state of class variable.
-        self.assertEqual(True, self._implemented_auth_setup_hook)
+        self.assertEqual(True, self.hook_checks.auth_setup_is_okay)
 
         # Call a basic assertResponse.
         # We don't even care if it works, this is only to trigger the hook function.
@@ -84,7 +84,7 @@ def test_hook_is_called(self):
         )
 
         # Verify variable changed after call. This variable is only changed in the hook function.
-        self.assertEqual(False, self._implemented_auth_setup_hook)
+        self.assertEqual(False, self.hook_checks.auth_setup_is_okay)
 
 
 # endregion Verify Hook Function Access