split test

Masterminds · Jan 11, 2023 · 585278e · 585278e
1 parent 4d14e8c
commit 585278e
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 6 deletions.
diff --git a/src/HTML5/Parser/DOMTreeBuilder.php b/src/HTML5/Parser/DOMTreeBuilder.php
@@ -567,7 +567,7 @@ public function text($data)
             // Per '8.2.5.4.3 The "before head" insertion mode' the characters
             // " \t\n\r\f" should be ignored .
             $dataTmp = trim($data, " \t\n\r\f");
-            if (! empty($dataTmp)) {
+            if (!empty($dataTmp)) {
                 $this->startTag('head');
                 $this->endTag('head');
                 $this->startTag('body');

diff --git a/test/HTML5/Html5Test.php b/test/HTML5/Html5Test.php
@@ -507,7 +507,7 @@ public function testTagOmission($input, $expected)
 
         $out = $this->html5->saveHTML($doc);
 
-        $this->assertRegExp("|" . preg_quote($expected, "|") . "|", $out);
+        $this->assertRegExp('|' . preg_quote($expected, '|') . '|', $out);
     }
 
     /**
@@ -534,7 +534,7 @@ public function tagOmissionProvider()
                 '<head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head>
 <body>
 <br>
-</body>'
+</body>',
             ),
         );
     }

diff --git a/test/HTML5/Parser/DOMTreeBuilderTest.php b/test/HTML5/Parser/DOMTreeBuilderTest.php
@@ -457,14 +457,17 @@ public function testText()
         $data = $wrapper->childNodes->item(0);
         $this->assertEquals(XML_TEXT_NODE, $data->nodeType);
         $this->assertEquals('test', $data->data);
+    }
 
+    public function testTextBeforeHeadNotAllowed()
+    {
         // The DomTreeBuilder has special handling for text when in before head mode.
-        $html = '<!DOCTYPE html><html>
-    Foo<head></head><body></body></html>';
+        $html = '<!DOCTYPE html><html>Foo<head></head><body></body></html>';
         $doc = $this->parse($html);
-        $this->assertEquals('Line 0, Col 0: Unexpected text. Ignoring: Foo', $this->errors[0]);
+        $this->assertEquals('Line 0, Col 0: Unexpected head tag outside of head context.', $this->errors[0]);
         $headElement = $doc->documentElement->firstChild;
         $this->assertEquals('head', $headElement->tagName);
+        $this->assertXmlStringEqualsXmlString($doc, '<html xmlns="http://www.w3.org/1999/xhtml"><head/><body>Foo<head/><body/></body></html>');
     }
 
     public function testParseErrors()