Improved tests

navbuilder/forms.py

@@ -17,6 +17,3 @@ class Meta:
             "title", "slug", "position", "menu", "parent", "target",
             "link_content_type", "link_object_id"
         ]
-
-    def clean_parent(self):
-        pass

navbuilder/tests/test_admin.py

@@ -2,9 +2,12 @@
 from django.test import TestCase
 from django.test.client import Client
 
+from navbuilder.tests.test_base import load_fixtures
+
 
 class AdminTestCase(TestCase):
     def setUp(self):
+        load_fixtures(self)
         self.client = Client()
         self.editor = get_user_model().objects.create(
             username="editor",
@@ -21,8 +24,20 @@ def test_admin(self):
         self.assertEqual(response.status_code, 200)
 
     def test_admin_menu(self):
+        response = self.client.get("/admin/navbuilder/menu/")
+        self.assertEqual(response.status_code, 200)
+
         response = self.client.get("/admin/navbuilder/menu/add/")
         self.assertEqual(response.status_code, 200)
 
+    def test_admin_menuitem(self):
+        response = self.client.get("/admin/navbuilder/menuitem/")
+        self.assertEqual(response.status_code, 200)
+        self.assertContains(response, self.menuitem.title)
+        self.assertContains(response, self.menuitem.link.get_absolute_url())
+
+        response = self.client.get("/admin/navbuilder/menuitem/add/")
+        self.assertEqual(response.status_code, 200)
+
     def tearDown(self):
         self.client.logout()

navbuilder/tests/test_models.py

@@ -8,26 +8,18 @@ def setUp(self):
         load_fixtures(self)
 
     def test_link(self):
-
-        # ensure the data was saved correctly
         for key, value in self.link_data.items():
             self.assertEqual(getattr(self.link, key), value)
 
     def test_menu(self):
-
-        # ensure the data was saved correctly
         for key, value in self.menu_data.items():
             self.assertEqual(getattr(self.menu, key), value)
+        self.assertEqual(unicode(self.menu), self.menu.title)
 
     def test_menuitem(self):
-
-        # ensure the data was saved correctly
         for key, value in self.menuitem_data.items():
             self.assertEqual(getattr(self.menuitem, key), value)
-
-        # ensure the sub menu item data was saved correctly
         for key, value in self.sub_menuitem_data.items():
             self.assertEqual(getattr(self.sub_menuitem, key), value)
-
-        # ensure the parent menu item is accessible
         self.assertEqual(self.sub_menuitem.parent, self.menuitem)
+        self.assertEqual(unicode(self.menuitem), self.menuitem.title)