Update tests with new package name

Author: homebysix

.coveragerc

@@ -1,5 +1,5 @@
 [run]
-include = pre_commit_hooks/*
+include = pre_commit_macadmin_hooks/*
 
 [report]
 exclude_lines =

tests/test_check_jamf_extension_attributes.py

@@ -20,7 +20,7 @@ def test_argument_parser(self):
         self.assertEqual(args.valid_shebangs, ["#!/bin/bash"])
 
     @mock.patch(
-        "pre_commit_hooks.check_jamf_extension_attributes.validate_shebangs",
+        "pre_commit_macadmin_hooks.check_jamf_extension_attributes.validate_shebangs",
         return_value=True,
     )
     def test_valid_file(self, mock_validate):
@@ -32,7 +32,7 @@ def test_valid_file(self, mock_validate):
         self.assertEqual(retval, 0)
 
     @mock.patch(
-        "pre_commit_hooks.check_jamf_extension_attributes.validate_shebangs",
+        "pre_commit_macadmin_hooks.check_jamf_extension_attributes.validate_shebangs",
         return_value=True,
     )
     def test_missing_result_tags(self, mock_validate):
@@ -46,7 +46,7 @@ def test_missing_result_tags(self, mock_validate):
         mock_print.assert_any_call(f"{tf.name}: missing <result> and/or </result> tags")
 
     @mock.patch(
-        "pre_commit_hooks.check_jamf_extension_attributes.validate_shebangs",
+        "pre_commit_macadmin_hooks.check_jamf_extension_attributes.validate_shebangs",
         return_value=False,
     )
     def test_invalid_shebang(self, mock_validate):
@@ -60,7 +60,7 @@ def test_invalid_shebang(self, mock_validate):
         mock_print.assert_any_call(f"{tf.name}: does not start with a valid shebang")
 
     @mock.patch(
-        "pre_commit_hooks.check_jamf_extension_attributes.validate_shebangs",
+        "pre_commit_macadmin_hooks.check_jamf_extension_attributes.validate_shebangs",
         return_value=True,
     )
     def test_multiple_files_mixed_results(self, mock_validate):
@@ -81,7 +81,7 @@ def test_multiple_files_mixed_results(self, mock_validate):
         )
 
     @mock.patch(
-        "pre_commit_hooks.check_jamf_extension_attributes.validate_shebangs",
+        "pre_commit_macadmin_hooks.check_jamf_extension_attributes.validate_shebangs",
         return_value=True,
     )
     def test_no_filenames(self, mock_validate):

tests/test_check_jamf_scripts.py

@@ -10,7 +10,8 @@ class TestCheckJamfScripts(unittest.TestCase):
     def setUp(self):
         # Patch validate_shebangs for controlled behavior
         patcher = mock.patch(
-            "pre_commit_hooks.check_jamf_scripts.validate_shebangs", return_value=True
+            "pre_commit_macadmin_hooks.check_jamf_scripts.validate_shebangs",
+            return_value=True,
         )
         self.mock_validate = patcher.start()
         self.addCleanup(patcher.stop)

tests/test_check_munki_pkgsinfo.py

@@ -22,7 +22,7 @@ def setUp(self):
         ]
         self.patchers = []
         for func in patcher_list:
-            p = mock.patch(f"pre_commit_hooks.util.{func}", return_value=True)
+            p = mock.patch(f"pre_commit_macadmin_hooks.util.{func}", return_value=True)
             self.patchers.append(p)
             p.start()
         self.addCleanup(lambda: [p.stop() for p in self.patchers])
@@ -72,7 +72,7 @@ def test_valid_pkginfo_returns_zero(self):
     # def test_missing_required_key_returns_one(self):
     #     # Patch validate_required_keys to return False
     #     with mock.patch(
-    #         "pre_commit_hooks.util.validate_required_keys", return_value=False
+    #         "pre_commit_macadmin_hooks.util.validate_required_keys", return_value=False
     #     ):
     #         pkginfo = {
     #             "name": "foo",
@@ -264,7 +264,9 @@ def test_require_pkg_blocking_apps_missing_returns_one(self):
 
     def test_script_with_invalid_shebang_returns_one(self):
         # Patch validate_shebangs to return False
-        with mock.patch("pre_commit_hooks.util.validate_shebangs", return_value=False):
+        with mock.patch(
+            "pre_commit_macadmin_hooks.util.validate_shebangs", return_value=False
+        ):
             pkginfo = {
                 "description": "desc",
                 "name": "foo",

tests/test_check_munkiadmin_scripts.py

@@ -10,7 +10,7 @@ class TestCheckMunkiadminScripts(unittest.TestCase):
     def setUp(self):
         # Patch validate_shebangs for all tests
         patcher = mock.patch(
-            "pre_commit_hooks.check_munkiadmin_scripts.validate_shebangs",
+            "pre_commit_macadmin_hooks.check_munkiadmin_scripts.validate_shebangs",
             return_value=True,
         )
         self.mock_validate_shebangs = patcher.start()

tests/test_check_outset_scripts.py

@@ -15,7 +15,8 @@ def test_build_argument_parser(self):
         self.assertEqual(args.filenames, ["file1", "file2"])
 
     @mock.patch(
-        "pre_commit_hooks.check_outset_scripts.validate_shebangs", return_value=True
+        "pre_commit_macadmin_hooks.check_outset_scripts.validate_shebangs",
+        return_value=True,
     )
     def test_main_executable_and_valid_shebang(self, mock_validate):
         with tempfile.NamedTemporaryFile(delete=False) as tf:
@@ -33,7 +34,8 @@ def test_main_executable_and_valid_shebang(self, mock_validate):
             os.unlink(filename)
 
     @mock.patch(
-        "pre_commit_hooks.check_outset_scripts.validate_shebangs", return_value=False
+        "pre_commit_macadmin_hooks.check_outset_scripts.validate_shebangs",
+        return_value=False,
     )
     def test_main_invalid_shebang(self, mock_validate):
         with tempfile.NamedTemporaryFile(delete=False) as tf:
@@ -53,7 +55,8 @@ def test_main_invalid_shebang(self, mock_validate):
             os.unlink(filename)
 
     @mock.patch(
-        "pre_commit_hooks.check_outset_scripts.validate_shebangs", return_value=True
+        "pre_commit_macadmin_hooks.check_outset_scripts.validate_shebangs",
+        return_value=True,
     )
     def test_main_not_executable(self, mock_validate):
         with tempfile.NamedTemporaryFile(delete=False) as tf:
@@ -72,7 +75,8 @@ def test_main_not_executable(self, mock_validate):
             os.unlink(filename)
 
     @mock.patch(
-        "pre_commit_hooks.check_outset_scripts.validate_shebangs", return_value=False
+        "pre_commit_macadmin_hooks.check_outset_scripts.validate_shebangs",
+        return_value=False,
     )
     def test_main_not_executable_and_invalid_shebang(self, mock_validate):
         with tempfile.NamedTemporaryFile(delete=False) as tf:

tests/test_forbid_autopkg_overrides.py

@@ -25,7 +25,9 @@ def test_build_argument_parser_help_message(self):
         self.assertIn("Filenames to check.", help_text)
         self.assertIn("usage:", help_text)
 
-    @mock.patch("pre_commit_hooks.forbid_autopkg_overrides.load_autopkg_recipe")
+    @mock.patch(
+        "pre_commit_macadmin_hooks.forbid_autopkg_overrides.load_autopkg_recipe"
+    )
     def test_main_recipe_missing_required_key(self, mock_load):
         # Simulate a recipe missing the "Process" key
         mock_load.return_value = {"Input": "value"}
@@ -37,7 +39,9 @@ def test_main_recipe_missing_required_key(self, mock_load):
         finally:
             os.unlink(filename)
 
-    @mock.patch("pre_commit_hooks.forbid_autopkg_overrides.load_autopkg_recipe")
+    @mock.patch(
+        "pre_commit_macadmin_hooks.forbid_autopkg_overrides.load_autopkg_recipe"
+    )
     def test_main_recipe_with_required_key(self, mock_load):
         # Simulate a recipe with the "Process" key
         mock_load.return_value = {"Process": [], "Input": "value"}
@@ -49,7 +53,9 @@ def test_main_recipe_with_required_key(self, mock_load):
         finally:
             os.unlink(filename)
 
-    @mock.patch("pre_commit_hooks.forbid_autopkg_overrides.load_autopkg_recipe")
+    @mock.patch(
+        "pre_commit_macadmin_hooks.forbid_autopkg_overrides.load_autopkg_recipe"
+    )
     def test_main_recipe_load_returns_none(self, mock_load):
         # Simulate load_autopkg_recipe returning None (invalid file)
         mock_load.return_value = None
@@ -66,7 +72,9 @@ def test_build_argument_parser(self):
         args = parser.parse_args(["file1", "file2"])
         self.assertEqual(args.filenames, ["file1", "file2"])
 
-    @mock.patch("pre_commit_hooks.forbid_autopkg_overrides.load_autopkg_recipe")
+    @mock.patch(
+        "pre_commit_macadmin_hooks.forbid_autopkg_overrides.load_autopkg_recipe"
+    )
     def test_main_multiple_files(self, mock_load):
         # First file is valid, second is missing required key
         mock_load.side_effect = [{"Process": []}, {"Input": "value"}]
@@ -81,7 +89,9 @@ def test_main_multiple_files(self, mock_load):
             os.unlink(filenames[0])
             os.unlink(filenames[1])
 
-    @mock.patch("pre_commit_hooks.forbid_autopkg_overrides.load_autopkg_recipe")
+    @mock.patch(
+        "pre_commit_macadmin_hooks.forbid_autopkg_overrides.load_autopkg_recipe"
+    )
     def test_main_no_filenames(self, mock_load):
         # No files to check, should return 0
         result = target.main([])

tests/test_forbid_autopkg_trust_info.py

@@ -10,21 +10,27 @@ def test_build_argument_parser(self):
         args = parser.parse_args(["file1", "file2"])
         self.assertEqual(args.filenames, ["file1", "file2"])
 
-    @mock.patch("pre_commit_hooks.forbid_autopkg_trust_info.load_autopkg_recipe")
+    @mock.patch(
+        "pre_commit_macadmin_hooks.forbid_autopkg_trust_info.load_autopkg_recipe"
+    )
     def test_main_no_files(self, mock_load):
         # No files provided, should return 0
         ret = target.main([])
         self.assertEqual(ret, 0)
         mock_load.assert_not_called()
 
-    @mock.patch("pre_commit_hooks.forbid_autopkg_trust_info.load_autopkg_recipe")
+    @mock.patch(
+        "pre_commit_macadmin_hooks.forbid_autopkg_trust_info.load_autopkg_recipe"
+    )
     def test_main_file_with_no_recipe(self, mock_load):
         mock_load.return_value = None
         ret = target.main(["fakefile"])
         self.assertEqual(ret, 1)
         mock_load.assert_called_once_with("fakefile")
 
-    @mock.patch("pre_commit_hooks.forbid_autopkg_trust_info.load_autopkg_recipe")
+    @mock.patch(
+        "pre_commit_macadmin_hooks.forbid_autopkg_trust_info.load_autopkg_recipe"
+    )
     def test_main_file_with_trust_info(self, mock_load):
         mock_load.return_value = {"ParentRecipeTrustInfo": {}}
         with mock.patch("builtins.print") as mock_print:
@@ -34,13 +40,17 @@ def test_main_file_with_trust_info(self, mock_load):
                 "recipe_with_trust: trust info in recipe"
             )
 
-    @mock.patch("pre_commit_hooks.forbid_autopkg_trust_info.load_autopkg_recipe")
+    @mock.patch(
+        "pre_commit_macadmin_hooks.forbid_autopkg_trust_info.load_autopkg_recipe"
+    )
     def test_main_file_without_trust_info(self, mock_load):
         mock_load.return_value = {"Input": {}}
         ret = target.main(["recipe_without_trust"])
         self.assertEqual(ret, 0)
 
-    @mock.patch("pre_commit_hooks.forbid_autopkg_trust_info.load_autopkg_recipe")
+    @mock.patch(
+        "pre_commit_macadmin_hooks.forbid_autopkg_trust_info.load_autopkg_recipe"
+    )
     def test_main_multiple_files(self, mock_load):
         # First file: valid, second: missing, third: has trust info
         mock_load.side_effect = [{"Input": {}}, None, {"ParentRecipeTrustInfo": {}}]

tests/test_util.py

@@ -1,6 +1,6 @@
 """test_util.py
 
-Unit tests for the shared/utility functions in pre_commit_hooks.util module.
+Unit tests for the shared/utility functions in pre_commit_macadmin_hooks.util module.
 """
 
 import json