wikimedia/mediawiki-extensions-DiscussionTools
1
0
Fork 0
mirror of https://gerrit.wikimedia.org/r/mediawiki/extensions/DiscussionTools synced 2024-11-28 02:00:57 +00:00
Code Issues Packages Projects Releases Wiki Activity
mediawiki-extensions-Discus.../tests/cases/formattedreply.json

99 lines
3.7 KiB
JSON
Raw Normal View History

Add tests for CommentFormatter I haven't really reviewed the outputs, but at least a) they don't crash b) they will fail if the output suddenly changes (which could cause problems due to caching). Bug: T252555 Change-Id: I1bbcbc5dd17ce1e24b3622062f5e8df4baf5f389
2020-10-19 20:59:55 +00:00
[
{
"name": "plwiki oldparser",
Tests: Add original titles to test cases These are not used for anything yet, but soon the parser will want to know the title of the page it is parsing. Change-Id: I02fa5d63fae78f3e92032d93bc27ac5c744faecb
2022-01-11 15:45:55 +00:00
"title": "Wikipedia:Kawiarenka/Artykuły dyskusja/Archiwum/2018-październik",
Add tests for CommentFormatter I haven't really reviewed the outputs, but at least a) they don't crash b) they will fail if the output suddenly changes (which could cause problems due to caching). Bug: T252555 Change-Id: I1bbcbc5dd17ce1e24b3622062f5e8df4baf5f389
2020-10-19 20:59:55 +00:00
"dom": "cases/pl-big-oldparser/pl-big-oldparser.html",
"expected": "cases/pl-big-oldparser/pl-big-oldparser-formattedreply.html",
"config": "../data/plwiki-config.json",
"data": "../data/plwiki-data.json"
},
{
"name": "enwiki oldparser",
Tests: Add original titles to test cases These are not used for anything yet, but soon the parser will want to know the title of the page it is parsing. Change-Id: I02fa5d63fae78f3e92032d93bc27ac5c744faecb
2022-01-11 15:45:55 +00:00
"title": "Wikipedia:Village pump (technical)/Archive 175",
Add tests for CommentFormatter I haven't really reviewed the outputs, but at least a) they don't crash b) they will fail if the output suddenly changes (which could cause problems due to caching). Bug: T252555 Change-Id: I1bbcbc5dd17ce1e24b3622062f5e8df4baf5f389
2020-10-19 20:59:55 +00:00
"dom": "cases/en-big-oldparser/en-big-oldparser.html",
"expected": "cases/en-big-oldparser/en-big-oldparser-formattedreply.html",
"config": "../data/enwiki-config.json",
"data": "../data/enwiki-data.json"
},
{
"name": "arwiki no-paragraph oldparser",
Tests: Add original titles to test cases These are not used for anything yet, but soon the parser will want to know the title of the page it is parsing. Change-Id: I02fa5d63fae78f3e92032d93bc27ac5c744faecb
2022-01-11 15:45:55 +00:00
"title": "نقاش المستخدم:Khaled",
Add tests for CommentFormatter I haven't really reviewed the outputs, but at least a) they don't crash b) they will fail if the output suddenly changes (which could cause problems due to caching). Bug: T252555 Change-Id: I1bbcbc5dd17ce1e24b3622062f5e8df4baf5f389
2020-10-19 20:59:55 +00:00
"dom": "cases/ar-no-paragraph-oldparser/ar-no-paragraph-oldparser.html",
"expected": "cases/ar-no-paragraph-oldparser/ar-no-paragraph-oldparser-formattedreply.html",
"config": "../data/arwiki-config.json",
"data": "../data/arwiki-data.json"
},
{
"name": "frwiki unsigned oldparser",
Tests: Add original titles to test cases These are not used for anything yet, but soon the parser will want to know the title of the page it is parsing. Change-Id: I02fa5d63fae78f3e92032d93bc27ac5c744faecb
2022-01-11 15:45:55 +00:00
"title": "Discussion:Le Monde",
Add tests for CommentFormatter I haven't really reviewed the outputs, but at least a) they don't crash b) they will fail if the output suddenly changes (which could cause problems due to caching). Bug: T252555 Change-Id: I1bbcbc5dd17ce1e24b3622062f5e8df4baf5f389
2020-10-19 20:59:55 +00:00
"dom": "cases/fr-unsigned-oldparser/fr-unsigned-oldparser.html",
"expected": "cases/fr-unsigned-oldparser/fr-unsigned-oldparser-formattedreply.html",
"config": "../data/frwiki-config.json",
"data": "../data/frwiki-data.json"
},
Add tests covering it.wp unsigned comment template Bug: T268178 Bug: T268589 Change-Id: Icd353275b3b8191eddbff4d72984a8c2c8e6cdd6
2020-11-25 00:06:56 +00:00
{
"name": "itwiki unsigned oldparser",
Tests: Add original titles to test cases These are not used for anything yet, but soon the parser will want to know the title of the page it is parsing. Change-Id: I02fa5d63fae78f3e92032d93bc27ac5c744faecb
2022-01-11 15:45:55 +00:00
"title": "Aiuto:Sportello informazioni",
Add tests covering it.wp unsigned comment template Bug: T268178 Bug: T268589 Change-Id: Icd353275b3b8191eddbff4d72984a8c2c8e6cdd6
2020-11-25 00:06:56 +00:00
"dom": "cases/it-unsigned-oldparser/it-unsigned-oldparser.html",
"expected": "cases/it-unsigned-oldparser/it-unsigned-oldparser-formattedreply.html",
"config": "../data/itwiki-config.json",
"data": "../data/itwiki-data.json"
},
Broken test cases for comments with double signatures Bug: T275934 Change-Id: I00edc63b723053e7ac7d26ce5fc1dde4b824380d
2021-02-26 21:24:31 +00:00
{
"name": "Double signature 1",
Tests: Add original titles to test cases These are not used for anything yet, but soon the parser will want to know the title of the page it is parsing. Change-Id: I02fa5d63fae78f3e92032d93bc27ac5c744faecb
2022-01-11 15:45:55 +00:00
"title": "Wikipedia:Requests for comment/User names",
Broken test cases for comments with double signatures Bug: T275934 Change-Id: I00edc63b723053e7ac7d26ce5fc1dde4b824380d
2021-02-26 21:24:31 +00:00
"dom": "cases/double-signature1/double-signature1.html",
"expected": "cases/double-signature1/double-signature1-formattedreply.html",
"config": "../data/enwiki-config.json",
"data": "../data/enwiki-data.json"
},
{
"name": "Double signature 2",
Tests: Add original titles to test cases These are not used for anything yet, but soon the parser will want to know the title of the page it is parsing. Change-Id: I02fa5d63fae78f3e92032d93bc27ac5c744faecb
2022-01-11 15:45:55 +00:00
"title": "Wikipedia:Articles for deletion/Equal Education",
Broken test cases for comments with double signatures Bug: T275934 Change-Id: I00edc63b723053e7ac7d26ce5fc1dde4b824380d
2021-02-26 21:24:31 +00:00
"dom": "cases/double-signature2/double-signature2.html",
"expected": "cases/double-signature2/double-signature2-formattedreply.html",
"config": "../data/enwiki-config.json",
"data": "../data/enwiki-data.json"
},
Broken test cases for a big mess of <font> tags Bug: T271385 Change-Id: I26b08923583593f40a372ef6614524f69781a87a
2021-01-08 19:32:29 +00:00
{
"name": "Mess caused by unclosed <font> tags in wikitext",
Tests: Add original titles to test cases These are not used for anything yet, but soon the parser will want to know the title of the page it is parsing. Change-Id: I02fa5d63fae78f3e92032d93bc27ac5c744faecb
2022-01-11 15:45:55 +00:00
"title": "Overleg:Unclosed font",
Broken test cases for a big mess of <font> tags Bug: T271385 Change-Id: I26b08923583593f40a372ef6614524f69781a87a
2021-01-08 19:32:29 +00:00
"dom": "cases/unclosed-font/unclosed-font.html",
"expected": "cases/unclosed-font/unclosed-font-formattedreply.html",
"config": "../data/nlwiki-config.json",
"data": "../data/nlwiki-data.json"
},
Fix detecting decorative comment frames with whitespace As a result of 0fc71f60cd464e9cf54c4048985971953503ce51, "empty" text nodes (containing only whitespace) at the end of the comment may be inside the comment's range, and trying to ignore them caused the ranges not to match and the frame not to be detected. Now the code works whether they're inside the comment's range or not. Add a test case for wrapped discussion comments with HTML comments and with whitespace. Bug: T250126 Bug: T268407 Change-Id: I2217ff5a635fd1c9c9e803f46795b1bfb3d17535
2020-12-08 22:46:11 +00:00
{
"name": "Reply inserted inside/outside various wrapper elements",
Tests: Add original titles to test cases These are not used for anything yet, but soon the parser will want to know the title of the page it is parsing. Change-Id: I02fa5d63fae78f3e92032d93bc27ac5c744faecb
2022-01-11 15:45:55 +00:00
"title": "Talk:Wrappers",
Fix detecting decorative comment frames with whitespace As a result of 0fc71f60cd464e9cf54c4048985971953503ce51, "empty" text nodes (containing only whitespace) at the end of the comment may be inside the comment's range, and trying to ignore them caused the ranges not to match and the frame not to be detected. Now the code works whether they're inside the comment's range or not. Add a test case for wrapped discussion comments with HTML comments and with whitespace. Bug: T250126 Bug: T268407 Change-Id: I2217ff5a635fd1c9c9e803f46795b1bfb3d17535
2020-12-08 22:46:11 +00:00
"dom": "cases/wrappers/wrappers.html",
"expected": "cases/wrappers/wrappers-formattedreply.html",
"config": "../data/enwiki-config.json",
"data": "../data/enwiki-data.json"
},
Test case for several bugs from it.wp Servers as another test case for partially transcluded comments, and a test case for comment start markers placed outside of paragraphs. Bug: T298051 Bug: T298096 Change-Id: Id07d2f57708c037578cb653c83848490c9a15fc6
2021-12-21 14:40:17 +00:00
{
"name": "Many comment start markers outside of paragraphs",
Tests: Add original titles to test cases These are not used for anything yet, but soon the parser will want to know the title of the page it is parsing. Change-Id: I02fa5d63fae78f3e92032d93bc27ac5c744faecb
2022-01-11 15:45:55 +00:00
"title": "Wikipedia:Vandalismi in corso",
Test case for several bugs from it.wp Servers as another test case for partially transcluded comments, and a test case for comment start markers placed outside of paragraphs. Bug: T298051 Bug: T298096 Change-Id: Id07d2f57708c037578cb653c83848490c9a15fc6
2021-12-21 14:40:17 +00:00
"dom": "cases/transclusions4/transclusions4.html",
"expected": "cases/transclusions4/transclusions4-formattedreply.html",
"config": "../data/itwiki-config.json",
"data": "../data/itwiki-data.json"
},
Add tests for CommentFormatter I haven't really reviewed the outputs, but at least a) they don't crash b) they will fail if the output suddenly changes (which could cause problems due to caching). Bug: T252555 Change-Id: I1bbcbc5dd17ce1e24b3622062f5e8df4baf5f389
2020-10-19 20:59:55 +00:00
{
"name": "Signatures in funny places",
Tests: Add original titles to test cases These are not used for anything yet, but soon the parser will want to know the title of the page it is parsing. Change-Id: I02fa5d63fae78f3e92032d93bc27ac5c744faecb
2022-01-11 15:45:55 +00:00
"title": "Talk:Signatures funny",
Add tests for CommentFormatter I haven't really reviewed the outputs, but at least a) they don't crash b) they will fail if the output suddenly changes (which could cause problems due to caching). Bug: T252555 Change-Id: I1bbcbc5dd17ce1e24b3622062f5e8df4baf5f389
2020-10-19 20:59:55 +00:00
"dom": "cases/signatures-funny/signatures-funny.html",
"expected": "cases/signatures-funny/signatures-funny-formattedreply.html",
"config": "../data/enwiki-config.json",
"data": "../data/enwiki-data.json"
CommentFormatter: Fix inserting placeholder heading marker This code expected $container->firstChild to be a <div class="mw-parser-output">, but that element is not present when we're running on HTML to be saved in parser cache. We ended up inserting the marker inside whatever node was the first on the page, and if it was a <style> element, both our marker and the styles would be lost when serializing, like in 6c7a0ca9a2c1e40d5de90b88f79ba2646d647f80. When we're running on final HTML, the marker will now be outside of <div class="mw-parser-output">, but that seems to be fine. Only early versions of I4e60fdbc098c1a74757d6e60fec6bcf8e5db37c1 had problems with that (see comments on patchset 41), but it works now. The added test case also covers the fix for T274709. Bug: T275440 Change-Id: I38d45dd8686919be51e1d307ded12b0afe185eb5
2021-02-22 22:02:41 +00:00
},
{
Tests: Add original titles to test cases These are not used for anything yet, but soon the parser will want to know the title of the page it is parsing. Change-Id: I02fa5d63fae78f3e92032d93bc27ac5c744faecb
2022-01-11 15:45:55 +00:00
"name": "For parser cache",
"title": "Talk:For parser cache",
CommentFormatter: Fix inserting placeholder heading marker This code expected $container->firstChild to be a <div class="mw-parser-output">, but that element is not present when we're running on HTML to be saved in parser cache. We ended up inserting the marker inside whatever node was the first on the page, and if it was a <style> element, both our marker and the styles would be lost when serializing, like in 6c7a0ca9a2c1e40d5de90b88f79ba2646d647f80. When we're running on final HTML, the marker will now be outside of <div class="mw-parser-output">, but that seems to be fine. Only early versions of I4e60fdbc098c1a74757d6e60fec6bcf8e5db37c1 had problems with that (see comments on patchset 41), but it works now. The added test case also covers the fix for T274709. Bug: T275440 Change-Id: I38d45dd8686919be51e1d307ded12b0afe185eb5
2021-02-22 22:02:41 +00:00
"dom": "cases/for-parser-cache/for-parser-cache.html",
"expected": "cases/for-parser-cache/for-parser-cache-formattedreply.html",
"config": "../data/enwiki-config.json",
"data": "../data/enwiki-data.json"
Add tests for CommentFormatter I haven't really reviewed the outputs, but at least a) they don't crash b) they will fail if the output suddenly changes (which could cause problems due to caching). Bug: T252555 Change-Id: I1bbcbc5dd17ce1e24b3622062f5e8df4baf5f389
2020-10-19 20:59:55 +00:00
}
]
Reference in a new issue Copy permalink