From abd78574ade85f167c52184d3fbe51cc41c8396e Mon Sep 17 00:00:00 2001 From: Tim Starling Date: Wed, 28 Sep 2016 10:17:50 +1000 Subject: [PATCH] Remove test for which fails when Extension:Math is installed And add a test for C. Scott asked for this test to be added in CR on Ib48b91238c3eddd6a86b62, but it was never actually executed prior to Ic840678b851d6fc472b834 due to a TestFileReader bug. The nuisance it causes now becomes apparent. As was noted at the time, it is unnecessary because Tidy is not a sanitization pass, is not whitelisted in the Sanitizer. C. Scott said "I was just concerned that the particular means used to hide the mathml from tidy might also hide it from the sanitizer." I don't think that's plausible, but we can test for it anyway by trying to pass elements through the Sanitizer. Change-Id: I5928e6f31d8b80ef384a1c0e6bacfa789cac9770 --- tests/parser/parserTests.txt | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/parser/parserTests.txt b/tests/parser/parserTests.txt index ff1450b5a6..ad96138e15 100644 --- a/tests/parser/parserTests.txt +++ b/tests/parser/parserTests.txt @@ -16711,11 +16711,11 @@ Expansion of multi-line templates in attribute values (bug 6255 sanity check 2) !! end !! test -evil -wiki-tags without Extension:Math enabled +Tags which are hidden from Tidy cannot pass through the Sanitizer !! wikitext - + !! html+tidy -

<math><img src="some evil external link"><script>some_evil_javascript();</script></math>

+

<mw:toc><script>alert();</script></mw:toc>

!! end ### -- 2.20.1