Avoid custom menu label tests clashing with existing test assertions

pull/10361/head
Sage Abdullah 2023-04-13 15:28:45 +01:00 zatwierdzone przez Matt Westcott
rodzic 7ba2098d1a
commit a51e7ae75a
2 zmienionych plików z 8 dodań i 5 usunięć

Wyświetl plik

@ -862,8 +862,8 @@ class TestMenuItemRegistration(BaseSnippetViewSetTests):
self.model = DraftStateModel
menu_items = settings_menu.render_component(self.request)
item = menu_items[0]
self.assertEqual(item.name, "draft-state-models")
self.assertEqual(item.label, "Draft State Models")
self.assertEqual(item.name, "publishables")
self.assertEqual(item.label, "Publishables")
self.assertEqual(item.icon_name, "snippet")
self.assertEqual(item.url, self.get_url("list"))
@ -906,7 +906,7 @@ class TestMenuItemRegistration(BaseSnippetViewSetTests):
item = [
item
for item in menu_items
if item.name in {"fullfeatured", "revisables", "draft-state-models"}
if item.name in {"fullfeatured", "revisables", "publishables"}
]
self.assertEqual(len(item), 0)
@ -930,8 +930,8 @@ class TestMenuItemRegistration(BaseSnippetViewSetTests):
menu_items = settings_menu.render_component(self.request)
item = menu_items[0]
self.assertEqual(item.name, "draft-state-models")
self.assertEqual(item.label, "Draft State Models")
self.assertEqual(item.name, "publishables")
self.assertEqual(item.label, "Publishables")
self.assertEqual(item.icon_name, "snippet")
self.assertEqual(item.url, self.get_url("list"))

Wyświetl plik

@ -304,6 +304,9 @@ class DraftStateModelViewSet(SnippetViewSet):
search_fields = ["text"]
search_backend_name = None
add_to_settings_menu = True
# Don't use "Draft" as the menu label,
# as it may cause incorrect assertion counts in tests
menu_label = "Publishables"
# Ensure that the menu item is placed first
menu_order = -999999