Merge "Add tests for site processor init"

tests/unit/site_processors/test_site_processor.py

@@ -61,6 +61,31 @@ class TestSiteProcessor(unittest.TestCase):
       site_type:{{ undefined_param }}
     ...""")
 
+    @mock.patch(
+        'spyglass.data_extractor.models.SiteDocumentData',
+        spec=models.SiteDocumentData)
+    def test___init__(self, SiteDocumentData):
+        site_data = SiteDocumentData()
+        _out_dir = mkdtemp()
+        obj = SiteProcessor(site_data, _out_dir, force_write=False)
+        self.assertEqual(site_data, obj.site_data)
+        self.assertEqual(_out_dir, obj.manifest_dir)
+        self.assertFalse(obj.force_write)
+
+    @mock.patch(
+        'spyglass.site_processors.site_processor.site_document_data_factory',
+        autospec=True,
+        return_value='success')
+    def test___init___use_site_document_data_factory(
+            self, site_document_data_factory):
+        site_data = {}
+        _out_dir = mkdtemp()
+        obj = SiteProcessor(site_data, _out_dir, force_write=False)
+        site_document_data_factory.assert_called_once_with(site_data)
+        self.assertEqual('success', obj.site_data)
+        self.assertEqual(_out_dir, obj.manifest_dir)
+        self.assertFalse(obj.force_write)
+
     @mock.patch(
         'spyglass.data_extractor.models.SiteDocumentData',
         spec=models.SiteDocumentData)

tox.ini

@@ -89,6 +89,6 @@ commands =
   pipenv install --dev
   bash -c 'PATH=$PATH:~/.local/bin; pytest --cov=spyglass \
       --cov-report html:cover --cov-report xml:cover/coverage.xml \
-      --cov-report term --cov-fail-under 84 tests/'
+      --cov-report term --cov-fail-under 88 tests/'
 whitelist_externals =
   bash