]> git.ipfire.org Git - thirdparty/patchwork.git/commitdiff
tests: Hardcode expected values
authorStephen Finucane <stephen@that.guru>
Thu, 6 Sep 2018 14:53:09 +0000 (15:53 +0100)
committerStephen Finucane <stephen@that.guru>
Wed, 17 Oct 2018 17:37:54 +0000 (18:37 +0100)
Compare against known values to ensure bugs introduced in the function
are caught.

Signed-off-by: Stephen Finucane <stephen@that.guru>
Reviewed-by: Daniel Axtens <dja@axtens.net>
patchwork/tests/test_series.py

index bb44e39d540c1ad88deed3147e58ff21d27928aa..ff3412e6e8ff6308479f8691745d584ad3ba52de 100644 (file)
@@ -623,10 +623,6 @@ class SeriesNameTestCase(TestCase):
     def _parse_mail(self, mail):
         return parser.parse_mail(mail, self.project.listid)
 
-    @staticmethod
-    def _format_name(cover):
-        return models.Series._format_name(cover)
-
     def test_cover_letter(self):
         """Cover letter name set as series name.
 
@@ -642,7 +638,7 @@ class SeriesNameTestCase(TestCase):
         mbox = self._get_mbox('base-cover-letter.mbox')
 
         cover = self._parse_mail(mbox[0])
-        cover_name = self._format_name(cover)
+        cover_name = 'A sample series'
         self.assertEqual(cover.series.first().name, cover_name)
 
         self._parse_mail(mbox[1])
@@ -697,7 +693,7 @@ class SeriesNameTestCase(TestCase):
         self.assertEqual(patch.series.first().name, patch.name)
 
         cover = self._parse_mail(mbox[2])
-        self.assertEqual(cover.series.first().name, self._format_name(cover))
+        self.assertEqual(cover.series.first().name, 'A sample series')
 
         mbox.close()