tests: gate more networked tests behind SKIP_DOWNLOADER_TESTS

Author: jlu5

tests/TestGoodreadsSource.py

@@ -15,11 +15,12 @@
 # with this program.  If not, see <http://www.gnu.org/licenses/>.
 ### END LICENSE
 
+import os
 import unittest
 
 from jumble.Jumble import Jumble
 
-
+@unittest.skipIf(os.getenv("SKIP_DOWNLOADER_TESTS"), "Skipping downloader tests (SKIP_DOWNLOADER_TESTS is set)")
 class TestGoodreadsSource(unittest.TestCase):
     def test_get_for_author(self):
         p = Jumble(["variety/plugins/builtin"])

tests/TestImageFetcher.py

@@ -14,6 +14,7 @@
 # You should have received a copy of the GNU General Public License along
 # with this program.  If not, see <http://www.gnu.org/licenses/>.
 ### END LICENSE
+import os
 import os.path
 import shutil
 import unittest
@@ -23,6 +24,7 @@
 
 
 class TestImageFetcher(unittest.TestCase):
+    @unittest.skipIf(os.getenv("SKIP_DOWNLOADER_TESTS"), "Skipping downloader tests (SKIP_DOWNLOADER_TESTS is set)")
     def test_fetch(self):
         target_folder = "/tmp/variety/ImageFetcher"
         shutil.rmtree(target_folder, ignore_errors=True)

tests/TestQuotationsPageSource.py

@@ -15,11 +15,12 @@
 # with this program.  If not, see <http://www.gnu.org/licenses/>.
 ### END LICENSE
 
+import os
 import unittest
 
 from jumble.Jumble import Jumble
 
-
+@unittest.skipIf(os.getenv("SKIP_DOWNLOADER_TESTS"), "Skipping downloader tests (SKIP_DOWNLOADER_TESTS is set)")
 class TestQuotationsPageSource(unittest.TestCase):
     def test_get_random(self):
         p = Jumble(["variety/plugins/builtin"])

tests/TestUtil.py

@@ -173,6 +173,7 @@ def f(i):
 
         self.assertEqual([20, 30], list(Util.safe_map(f, [1, 5, 20, 10, 30, 4])))
 
+    @unittest.skipIf(os.getenv("SKIP_DOWNLOADER_TESTS"), "Skipping networked tests (SKIP_DOWNLOADER_TESTS is set)")
     def test_fetch(self):
         resp = Util.fetch("//google.com")
         self.assertTrue(len(resp) > 0)
@@ -198,6 +199,7 @@ def test_is_animated_gif(self):
         self.assertTrue(Util.is_animated_gif("animated.gif"))
         self.assertFalse(Util.is_animated_gif("not-animated.gif"))
 
+    @unittest.skipIf(os.getenv("SKIP_DOWNLOADER_TESTS"), "Skipping downloader tests (SKIP_DOWNLOADER_TESTS is set)")
     def test_is_dead_or_not_image(self):
         self.assertTrue(Util.is_dead_or_not_image(None))
         self.assertTrue(Util.is_dead_or_not_image("not a URL"))