diff --git a/management/templates/management/group_list.html b/management/templates/management/group_list.html index 8e868de..8af820a 100644 --- a/management/templates/management/group_list.html +++ b/management/templates/management/group_list.html @@ -27,7 +27,7 @@ {{ group.member_count }}
- {% lucide "pencil" size=14 %} {% trans "Edit" %} + {% lucide "pencil" size=14 %} {% trans "Edit" %}
diff --git a/management/tests.py b/management/tests.py index 5d7c27d..b163e96 100644 --- a/management/tests.py +++ b/management/tests.py @@ -824,6 +824,19 @@ class GroupManagementTests(ManagementTestBase): self.client.force_login(self.make_non_admin_coach()) self.assertEqual(self.club_get("group_list").status_code, 403) + def test_the_lists_edit_button_goes_to_the_group_page_not_straight_to_the_form(self): + # Same as Teams: Edit lands on the overview, which is where the members + # live, and offers its own Edit for the rename form. The list pages that + # do jump straight to a form (Locations, Opponents, Sponsors, Positions, + # Referee levels) have no detail page to land on at all. + group = Group.objects.create(club=self.club, name="Referees") + self.client.force_login(self.admin_user) + + response = self.club_get("group_list") + + self.assertContains(response, reverse("management:group_detail", args=[group.pk])) + self.assertNotContains(response, reverse("management:group_update", args=[group.pk])) + def test_admin_can_create_a_group(self): self.client.force_login(self.admin_user)