Reorganize template scraper method order, placing abstract methods ne…

…arest the top of the class definition

recipe_scrapers/_abstract.py

@@ -156,6 +156,7 @@ def ratings(self):
         raise NotImplementedError("This should be implemented.")
 
     def author(self):
+        # question: should we make this a required field (abstractmethod)?
         raise NotImplementedError("This should be implemented.")
 
     def cuisine(self):

templates/scraper.py

@@ -8,21 +8,12 @@ class Template(AbstractScraper):
     def host(cls):
         return "example.com"
 
-    def author(self):
-        return self.schema.author()
-
     def title(self):
         return self.schema.title()
 
-    def category(self):
-        return self.schema.category()
-
     def total_time(self):
         return self.schema.total_time()
 
-    def yields(self):
-        return self.schema.yields()
-
     def image(self):
         return self.schema.image()
 
@@ -32,6 +23,16 @@ def ingredients(self):
     def instructions(self):
         return self.schema.instructions()
 
+    def author(self):
+        # question: should we make this a required field?
+        return self.schema.author()
+
+    def category(self):
+        return self.schema.category()
+
+    def yields(self):
+        return self.schema.yields()
+
     def ratings(self):
         return self.schema.ratings()
 

templates/test_scraper.py

@@ -11,21 +11,12 @@ class TestTemplateScraper(ScraperTest):
     def test_host(self):
         self.assertEqual("example.com", self.harvester_class.host())
 
-    def test_author(self):
-        self.assertEqual(None, self.harvester_class.author())
-
     def test_title(self):
         self.assertEqual(None, self.harvester_class.title())
 
-    def test_category(self):
-        self.assertEqual(None, self.harvester_class.category())
-
     def test_total_time(self):
         self.assertEqual(None, self.harvester_class.total_time())
 
-    def test_yields(self):
-        self.assertEqual(None, self.harvester_class.yields())
-
     def test_image(self):
         self.assertEqual(None, self.harvester_class.image())
 
@@ -35,6 +26,16 @@ def test_ingredients(self):
     def test_instructions(self):
         self.assertEqual(None, self.harvester_class.instructions())
 
+    def test_author(self):
+        # should we make this a required field?
+        self.assertEqual(None, self.harvester_class.author())
+
+    def test_category(self):
+        self.assertEqual(None, self.harvester_class.category())
+
+    def test_yields(self):
+        self.assertEqual(None, self.harvester_class.yields())
+
     def test_ratings(self):
         self.assertEqual(None, self.harvester_class.ratings())