ensure pass $this->dom->documentElement on dom->saveHTML()

samsonasik · samsonasik · commit 7f842989526e · 2020-07-16T07:42:44.000+07:00
diff --git a/system/Test/DOMParser.php b/system/Test/DOMParser.php
@@ -76,7 +76,7 @@ public function __construct()
 	 */
 	public function getBody(): string
 	{
-		return $this->dom->saveHTML();
+		return $this->dom->saveHTML($this->dom->documentElement);
 	}
 
 	/**
@@ -142,8 +142,8 @@ public function see(string $search = null, string $element = null): bool
 		// If Element is null, we're just scanning for text
 		if (is_null($element))
 		{
-			$content = $this->dom->saveHTML();
-			return mb_strpos($content, $search) !== false;
+			$content = $this->dom->saveHTML($this->dom->documentElement);
+			return strpos($content, $search) !== false;
 		}
 
 		$result = $this->doXPath($search, $element);
diff --git a/tests/system/Test/DOMParserTest.php b/tests/system/Test/DOMParserTest.php
@@ -66,14 +66,25 @@ public function testParseSelectorWithAttribute()
 		$this->assertEquals(['href' => 'http://example.com'], $selector['attr']);
 	}
 
-	public function testSeeText()
+	public function provideText()
+	{
+		return [
+			['Hello World'],
+			['Hellö Wörld'],
+		];
+	}
+
+	/**
+	 * @dataProvider provideText
+	 */
+	public function testSeeText($text)
 	{
 		$dom = new DOMParser();
 
-		$html = '<html><body><h1>Hello World</h1></body></html>';
+		$html = '<html><body><h1>' . $text . '</h1></body></html>';
 		$dom->withString($html);
 
-		$this->assertTrue($dom->see('Hello World'));
+		$this->assertTrue($dom->see($text));
 	}
 
 	public function testSeeHTML()

Original file line number	Diff line number	Diff line change
`@@ -76,7 +76,7 @@ public function __construct()`
`76`	`76`	`*/`
`77`	`77`	`public function getBody(): string`
`78`	`78`	`{`
`79`		`- return $this->dom->saveHTML();`
	`79`	`+ return $this->dom->saveHTML($this->dom->documentElement);`
`80`	`80`	`}`
`81`	`81`
`82`	`82`	`/**`
`@@ -142,8 +142,8 @@ public function see(string $search = null, string $element = null): bool`
`142`	`142`	`// If Element is null, we're just scanning for text`
`143`	`143`	`if (is_null($element))`
`144`	`144`	`{`
`145`		`- $content = $this->dom->saveHTML();`
`146`		`- return mb_strpos($content, $search) !== false;`
	`145`	`+ $content = $this->dom->saveHTML($this->dom->documentElement);`
	`146`	`+ return strpos($content, $search) !== false;`
`147`	`147`	`}`
`148`	`148`
`149`	`149`	`$result = $this->doXPath($search, $element);`