diff options
| author | Chih-Hsuan Yen <[email protected]> | 2023-12-15 13:12:26 +0800 |
|---|---|---|
| committer | Chih-Hsuan Yen <[email protected]> | 2023-12-18 22:46:35 +0800 |
| commit | f1a9ac9b159127e8eb7c301e9a646eabb188e1af (patch) | |
| tree | c7353b1bad7040c31c97645eed868d60375d2c15 | |
| parent | 67012f9dac7de22615b72be93fa360f53fefe3ec (diff) | |
sanitizer: add a test to make sure <figure> is intact
Somehow with the old approach, `<figure>` is rearranged into `<head>`,
and the latter is stripped by `Sanitizer::strip_harmful_tags()` (see
[1]). The issue is fixed by [2]. Here I added a test for the regression.
[1] https://community.tt-rss.org/t/unexpected-behavior-with-figure-tag/6244
[2] https://gitlab.tt-rss.org/tt-rss/tt-rss/-/commit/67012f9dac7de22615b72be93fa360f53fefe3ec
| -rw-r--r-- | tests/integration/SanitizerTest.php | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/tests/integration/SanitizerTest.php b/tests/integration/SanitizerTest.php index 1981079f0..421d046af 100644 --- a/tests/integration/SanitizerTest.php +++ b/tests/integration/SanitizerTest.php @@ -9,5 +9,12 @@ final class SanitizerTest extends TestCase { Sanitizer::sanitize('<p>中文</p>') ); } + + public function test_sanitize_keep_figure(): void { + $this->assertEquals( + '<figure>Content</figure>', + Sanitizer::sanitize('<figure>Content</figure>') + ); + } } |