Cleanup after latest PR

j0k3r · Jun 8, 2018 · ed87be6 · ed87be6
1 parent 4ab3b31
commit ed87be6
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/Graby.php b/src/Graby.php
@@ -252,7 +252,8 @@ private function doFetchContent($url)
         }
 
         $html = $this->convert2Utf8($response['body'], $response['all_headers']);
-        // Removing empty nodes
+
+        // Remove empty nodes (except iframe)
         $re = '/<([^>\s]+)[^iframe|>]*>(?:\s*(?:<br \/>|&nbsp;|&thinsp;|&ensp;|&emsp;|&#8201;|&#8194;|&#8195;)\s*)*<\/\1>/m';
         $html = preg_replace($re, '', $html);
 

diff --git a/tests/GrabyTest.php b/tests/GrabyTest.php
@@ -1444,13 +1444,14 @@ public function testEmptyNodesRemoved()
             [
                 'content-type' => 'text/html',
             ],
-            Stream::factory(file_get_contents(__DIR__ . '/fixtures/sites/framablog.test'))
+            Stream::factory(__DIR__ . '/fixtures/sites/framablog.test')
         );
         $client = new Client();
         $client->getEmitter()->attach(new Mock([$response]));
 
         $graby = new Graby();
         $res = $graby->fetchContent('https://framablog.org/2017/12/02/avancer-ensemble-vers-la-contribution/');
+
         // The initial treatment was encapsulating the content into the empty node
         // So we don't want to see that again
         $this->assertNotContains('<figure><p>Après un <em>icebreaker</em>', $res['html']);