wikimedia/mediawiki-extensions-AbuseFilter
1
0
Fork 0
mirror of https://gerrit.wikimedia.org/r/mediawiki/extensions/AbuseFilter.git synced 2024-11-27 15:30:42 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge "Re-enable skipped tests with fixes"

Browse source
This commit is contained in:
jenkins-bot 2023-07-14 14:02:53 +00:00 committed by Gerrit Code Review
parent c0c344a2f9 85639c857a
commit 1f930d3e16
2 changed files with 14 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
tests/phpunit/RCVariableGeneratorTest.php
View file

@ -205,7 +205,6 @@ class RCVariableGeneratorTest extends MediaWikiIntegrationTestCase {
* @covers \MediaWiki\Extension\AbuseFilter\Variables\LazyVariableComputer
*/
public function testAddEditVarsForRow() {
$this->markTestSkipped( 'Temp' );
$timestamp = 1514700000;
MWTimestamp::setFakeTime( $timestamp );
@ -248,11 +247,11 @@ class RCVariableGeneratorTest extends MediaWikiIntegrationTestCase {
'old_wikitext' => $oldText,
'old_size' => strlen( $oldText ),
'old_content_model' => 'wikitext',
'old_links' => [ $oldLink ],
'old_links' => [ "https://wikipedia.org/" ],
'new_wikitext' => $newText,
'new_size' => strlen( $newText ),
'new_content_model' => 'wikitext',
'all_links' => [ $newLink ],
'all_links' => [ "https://en.wikipedia.org/" ],
'timestamp' => (string)$timestamp,
];
foreach ( $expected as $var => $value ) {

25
tests/phpunit/integration/ActionVariablesIntegrationTest.php
View file

@ -154,9 +154,9 @@ class ActionVariablesIntegrationTest extends ApiTestCase {
'added_lines' => [ $new ],
'removed_lines' => [],
'added_lines_pst' => [ $new ],
'all_links' => [ 'https://a.com' ],
'all_links' => [ 'https://a.com/' ],
'old_links' => [],
'added_links' => [ 'https://a.com' ],
'added_links' => [ 'https://a.com/' ],
'removed_links' => [],
],
'params' => [ 'text' => $new, 'summary' => $summary, 'createonly' => true ],
@ -184,10 +184,10 @@ class ActionVariablesIntegrationTest extends ApiTestCase {
'added_lines' => explode( "\n", $new ),
'removed_lines' => [ $old ],
'added_lines_pst' => [ "'''Random'''.", "Some ''special'' chars: àèìòù 名探偵コナン.", '[[Help:PST|PST]] test, [//www.b.com link]' ],
'old_links' => [ 'https://a.com' ],
'all_links' => [ '//www.b.com' ],
'removed_links' => [ 'https://a.com' ],
'added_links' => [ '//www.b.com' ],
'old_links' => [ 'https://a.com/' ],
'all_links' => [ 'https://www.b.com/' ],
'removed_links' => [ 'https://a.com/' ],
'added_links' => [ 'https://www.b.com/' ],
],
'params' => [ 'text' => $new, 'summary' => $summary ],
'oldContent' => new WikitextContent( $old ),
@ -215,10 +215,10 @@ class ActionVariablesIntegrationTest extends ApiTestCase {
'added_lines' => [ $new ],
'removed_lines' => explode( "\n", $old ),
'added_lines_pst' => [ $new ],
'old_links' => [ '//www.b.com' ],
'all_links' => [ 'https://a.com' ],
'removed_links' => [ '//www.b.com' ],
'added_links' => [ 'https://a.com' ],
'old_links' => [ 'https://www.b.com/' ],
'all_links' => [ 'https://a.com/' ],
'removed_links' => [ 'https://www.b.com/' ],
'added_links' => [ 'https://a.com/' ],
],
'params' => [ 'text' => $new, 'summary' => $summary ],
'oldContent' => new WikitextContent( $old ),
@ -264,7 +264,7 @@ class ActionVariablesIntegrationTest extends ApiTestCase {
'new_wikitext' => 'new test https://en.wikipedia.org',
'new_content_model' => 'wikitext',
'old_links' => [],
'all_links' => [ 'https://en.wikipedia.org' ],
'all_links' => [ 'https://en.wikipedia.org/' ],
],
'params' => [
'text' => 'new test https://en.wikipedia.org',
@ -280,7 +280,7 @@ class ActionVariablesIntegrationTest extends ApiTestCase {
'old_content_model' => 'wikitext',
'new_wikitext' => '{"key": "value"}',
'new_content_model' => 'json',
'old_links' => [ 'https://en.wikipedia.org' ],
'old_links' => [ 'https://en.wikipedia.org/' ],
'all_links' => [],
],
'params' => [
@ -301,7 +301,6 @@ class ActionVariablesIntegrationTest extends ApiTestCase {
public function testEditVariables(
array $expected, array $params, Content $oldContent = null
) {
$this->markTestSkipped( 'Temp' );
$varHolder = null;
$this->prepareServices();
$this->setAbuseLoggerFactoryWithEavesdrop( $varHolder );