Merge pull request #115 from Lullabot/skip-internet-tests

Add the ability to skip tests that require an internet connection

tests/AmpTest.php

@@ -23,12 +23,14 @@
  */
 class AmpTest extends PHPUnit_Framework_TestCase
 {
-    /** @var AMP|null */
+    /** @var AMP */
     protected $amp = null;
+    protected $skip_internet = false;
 
     public function setup()
     {
         $this->amp = new AMP();
+        $this->skip_internet = getenv('AMP_TEST_SKIP_INTERNET');
     }
 
     /**
@@ -46,6 +48,10 @@ public function testFiles($test_filename, $fragment)
             // An out file does not exist, skip this test
             $this->markTestSkipped("$test_filename.out file does not exist. Skipping test.");
         }
+
+        if (!empty($this->skip_internet) && !empty($options['requires_internet'])) {
+            $this->markTestSkipped("Skipping test as it requires internet and AMP_TEST_SKIP_INTERNET environment variable is set.");
+        }
         $this->assertEquals($expected_output, $output);
     }
 

tests/test-data/fragment-html/facebook-non-iframe-fragment.html.options.json

@@ -0,0 +1,4 @@
+{
+    "_readme" : "requires_internet is just for information for the test runner and has no significance for the functioning of library",
+    "requires_internet": "true"
+}

tests/test-data/fragment-html/instagram-fragment.html.options.json

@@ -0,0 +1,4 @@
+{
+  "_readme" : "requires_internet is just for information for the test runner and has no significance for the functioning of library",
+  "requires_internet": "true"
+}

tests/test-data/fragment-html/sample-html-fragment.html.options.json

@@ -0,0 +1,4 @@
+{
+  "_readme" : "requires_internet is just for information for the test runner and has no significance for the functioning of library",
+  "requires_internet": "true"
+}