diff options
| -rw-r--r-- | apps/patchwork/tests/__init__.py | 1 | ||||
| -rw-r--r-- | apps/patchwork/tests/list.py | 38 | ||||
| -rw-r--r-- | apps/patchwork/tests/updates.py | 2 | ||||
| -rw-r--r-- | templates/patchwork/patch-list.html | 2 |
4 files changed, 41 insertions, 2 deletions
diff --git a/apps/patchwork/tests/__init__.py b/apps/patchwork/tests/__init__.py index 8ae271a..d9ed346 100644 --- a/apps/patchwork/tests/__init__.py +++ b/apps/patchwork/tests/__init__.py @@ -28,3 +28,4 @@ from patchwork.tests.registration import * from patchwork.tests.user import * from patchwork.tests.mail_settings import * from patchwork.tests.notifications import * +from patchwork.tests.list import * diff --git a/apps/patchwork/tests/list.py b/apps/patchwork/tests/list.py new file mode 100644 index 0000000..1bdb506 --- /dev/null +++ b/apps/patchwork/tests/list.py @@ -0,0 +1,38 @@ +# Patchwork - automated patch tracking system +# Copyright (C) 2012 Jeremy Kerr <jk@ozlabs.org> +# +# This file is part of the Patchwork package. +# +# Patchwork is free software; you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 2 of the License, or +# (at your option) any later version. +# +# Patchwork is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with Patchwork; if not, write to the Free Software +# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA + +import unittest +from django.test import TestCase +from django.test.client import Client +from patchwork.tests.utils import defaults, create_user, find_in_context +from django.core.urlresolvers import reverse + +class EmptyPatchListTest(TestCase): + + def testEmptyPatchList(self): + """test that we don't output an empty table when there are no + patches present""" + project = defaults.project + defaults.project.save() + url = reverse('patchwork.views.patch.list', + kwargs={'project_id': project.linkname}) + response = self.client.get(url) + self.assertContains(response, 'No patches to display') + self.assertNotContains(response, 'tbody') + diff --git a/apps/patchwork/tests/updates.py b/apps/patchwork/tests/updates.py index ba1cb6d..177ee78 100644 --- a/apps/patchwork/tests/updates.py +++ b/apps/patchwork/tests/updates.py @@ -52,7 +52,7 @@ class MultipleUpdateTest(TestCase): data.update({'archived': 'True'}) self._selectAllPatches(data) response = self.client.post(self.url, data) - self.assertContains(response, self.properties_form_id, + self.assertContains(response, 'No patches to display', status_code = 200) for patch in [Patch.objects.get(pk = p.pk) for p in self.patches]: self.assertTrue(patch.archived) diff --git a/templates/patchwork/patch-list.html b/templates/patchwork/patch-list.html index 770f005..723c4a0 100644 --- a/templates/patchwork/patch-list.html +++ b/templates/patchwork/patch-list.html @@ -134,7 +134,7 @@ </tr> </thead> -{% if page %} +{% if page.paginator.count %} <tbody> {% for patch in page.object_list %} <tr id="patch_row:{{patch.id}}" class="{% cycle 'odd' 'even' %}"> |