Skip to content

Commit 7f304d0

Browse files
jtojnarj0k3r
authored andcommitted
tests: Check encoding was preserved in testHtmlLang
The fix introduced in efbbc86 alongside this test also manipulates `meta[charset]` but we were not checking if it does not break encoding.
1 parent 109a226 commit 7f304d0

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

tests/ReadabilityTest.php

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -493,21 +493,21 @@ public function dataForHtmlLang()
493493
{
494494
return [
495495
'meta' => [
496-
'<html lang="fr"><head><meta charset="utf-8"></head><body><article>' . str_repeat('<p>This is the awesome content :)</p>', 7) . '</article></body></html>',
496+
'<html lang="fr"><head><meta charset="utf-8"></head><body><article>' . str_repeat('<p>Tous les êtres humains naissent libres et égaux en dignité et en droits. Ils sont doués de raison et de conscience et doivent agir les uns envers les autres dans un esprit de fraternité.</p>', 7) . '</article></body></html>',
497497
'fr',
498498
],
499499
'head' => [
500-
'<html lang="fr"><head><title>Foo</title></head><body><article>' . str_repeat('<p>This is the awesome content :)</p>', 7) . '</article></body></html>',
500+
'<html lang="fr"><head><title>Foo</title></head><body><article>' . str_repeat('<p>Tous les êtres humains naissent libres et égaux en dignité et en droits. Ils sont doués de raison et de conscience et doivent agir les uns envers les autres dans un esprit de fraternité.</p>', 7) . '</article></body></html>',
501501
'fr',
502502
],
503503
'headless' => [
504-
'<html lang="fr"><body><article>' . str_repeat('<p>This is the awesome content :)</p>', 7) . '</article></body></html>',
504+
'<html lang="fr"><body><article>' . str_repeat('<p>Tous les êtres humains naissent libres et égaux en dignité et en droits. Ils sont doués de raison et de conscience et doivent agir les uns envers les autres dans un esprit de fraternité.</p>', 7) . '</article></body></html>',
505505
'fr',
506506
// tidy would add <head> tag.
507507
false,
508508
],
509509
'fragment' => [
510-
'<article>' . str_repeat('<p>This is the awesome content :)</p>', 7) . '</article>',
510+
'<article>' . str_repeat('<p>Tous les êtres humains naissent libres et égaux en dignité et en droits. Ils sont doués de raison et de conscience et doivent agir les uns envers les autres dans un esprit de fraternité.</p>', 7) . '</article>',
511511
'',
512512
// tidy would add <html>.
513513
false,
@@ -526,6 +526,8 @@ public function testHtmlLang($html, $lang, $useTidy = true)
526526
$this->assertTrue($res);
527527
$this->assertInstanceOf(\DOMDocument::class, $readability->dom);
528528
$this->assertSame($lang, $readability->dom->documentElement->getAttribute('lang'));
529+
$this->assertInstanceOf('Readability\JSLikeHTMLElement', $readability->getContent());
530+
$this->assertStringContainsString('êtres', $readability->getContent()->getInnerHtml());
529531
}
530532

531533
private function getReadability($html, $url = null, $parser = 'libxml', $useTidy = true)

0 commit comments

Comments
 (0)