From 373c8fb79a3da904a93b419d687d391823c163d3 Mon Sep 17 00:00:00 2001 From: Luke Clarke Date: Fri, 11 Aug 2023 14:47:33 +1000 Subject: [PATCH] Redundant old tests remove - no longer needed as those URL functions have been removed --- NearBeach/tests/test_object_data.py | 12 ++++++------ .../tests/test_redirection_of_non_logged_in_users.py | 12 ++++++------ .../tests/tests_user_permissions/test_admin_user.py | 7 ++++--- 3 files changed, 16 insertions(+), 15 deletions(-) diff --git a/NearBeach/tests/test_object_data.py b/NearBeach/tests/test_object_data.py index 591755ff1..090b3cb26 100644 --- a/NearBeach/tests/test_object_data.py +++ b/NearBeach/tests/test_object_data.py @@ -51,8 +51,8 @@ def test_incorrect_destination_data(self): "bug_list", "customer_list", "customer_list_all", - "group_list", - "group_list_all", + # "group_list", + # "group_list_all", "note_list", "object_link_list", "query_bug_client", @@ -61,7 +61,7 @@ def test_incorrect_destination_data(self): "remove_user", "tag_list", "user_list", - "user_list_all", + # "user_list_all", ] # Loop through each url to test to make sure the decorator is applied @@ -120,13 +120,13 @@ def test_correct_object_data__without_forms(self): "bug_list", "customer_list", "customer_list_all", - "group_list", - "group_list_all", + # "group_list", + # "group_list_all", "note_list", "object_link_list", "tag_list", "user_list", - "user_list_all", + # "user_list_all", ] # Loop through each url to test to make sure the decorator is applied diff --git a/NearBeach/tests/test_redirection_of_non_logged_in_users.py b/NearBeach/tests/test_redirection_of_non_logged_in_users.py index f9c7c4056..b99a1d200 100644 --- a/NearBeach/tests/test_redirection_of_non_logged_in_users.py +++ b/NearBeach/tests/test_redirection_of_non_logged_in_users.py @@ -196,14 +196,14 @@ def test_objects_get(self): c.get(reverse("bug_list", args=["project", 1])), c.get(reverse("customer_list", args=["project", 1])), c.get(reverse("customer_list_all", args=["project", 1])), - c.get(reverse("group_list", args=["project", 1])), - c.get(reverse("group_list_all", args=["project", 1])), + # c.get(reverse("group_list", args=["project", 1])), + # c.get(reverse("group_list_all", args=["project", 1])), c.get(reverse("link_list", args=["project", 1, "task"])), c.get(reverse("note_list", args=["project", 1])), c.get(reverse("object_link_list", args=["project", 1])), c.get(reverse("query_bug_client", args=["project", 1])), c.get(reverse("user_list", args=["project", 1])), - c.get(reverse("user_list_all", args=["project", 1])), + # c.get(reverse("user_list_all", args=["project", 1])), c.get(reverse("lead_user_list")), ] @@ -227,14 +227,14 @@ def test_objects_post(self): c.post(reverse("bug_list", args=["project", 1])), c.post(reverse("customer_list", args=["project", 1])), c.post(reverse("customer_list_all", args=["project", 1])), - c.post(reverse("group_list", args=["project", 1])), - c.post(reverse("group_list_all", args=["project", 1])), + # c.post(reverse("group_list", args=["project", 1])), + # c.post(reverse("group_list_all", args=["project", 1])), c.post(reverse("link_list", args=["project", 1, "task"])), c.post(reverse("note_list", args=["project", 1])), c.post(reverse("object_link_list", args=["project", 1])), c.post(reverse("query_bug_client", args=["project", 1])), c.post(reverse("user_list", args=["project", 1])), - c.post(reverse("user_list_all", args=["project", 1])), + # c.post(reverse("user_list_all", args=["project", 1])), c.post(reverse("lead_user_list")), ] diff --git a/NearBeach/tests/tests_user_permissions/test_admin_user.py b/NearBeach/tests/tests_user_permissions/test_admin_user.py index 4512f4da9..ab5e167f8 100644 --- a/NearBeach/tests/tests_user_permissions/test_admin_user.py +++ b/NearBeach/tests/tests_user_permissions/test_admin_user.py @@ -80,13 +80,14 @@ def test_basic_permissions_as_admin(self): URLTest("bug_list", ["project", 1], {}, 200, "POST"), URLTest("customer_list", ["project", 1], {}, 200, "POST"), URLTest("customer_list_all", ["project", 1], {}, 200, "POST"), - URLTest("group_list", ["project", 1], {}, 200, "POST"), - URLTest("group_list_all", ["project", 1], {}, 200, "POST"), + # URLTest("group_list", ["project", 1], {}, 200, "POST"), + # URLTest("group_list_all", ["project", 1], {}, 200, "POST"), URLTest("note_list", ["project", 1], {}, 200, "POST"), URLTest("object_link_list", ["project", 1], {}, 200, "POST"), URLTest("tag_list", ["project", 1], {}, 200, "POST"), URLTest("user_list", ["project", 1], {}, 200, "POST"), - URLTest("user_list_all", ["project", 1], {}, 200, "POST"), + # URLTest("user_list_all", ["project", 1], {}, 200, "POST"), + URLTest("group_and_user_data", ["project", 1], {}, 200, "POST"), ] # Loop through each url to test to make sure the decorator is applied