ApiDelete should have PHPUnit tests for deleting non-file pages
Closed, ResolvedPublic

Description

In MediaWiki core, the ApiDelete module allows deleting pages, is a pretty important module, and should have tests.

If you're not already familiar with PHPUnit tests, https://www.mediawiki.org/wiki/Manual:PHP_unit_testing/Writing_unit_tests has links to some good material on the basics of testing.

I would also recommend looking at other tests in the tests/phpunit/includes/api directory of MediaWiki core for an example of how we currently test API modules. For the purposes of this GCI task, only worry about testing deletion of normal pages, not files, which have special handling.

I'll mentor this for Google-Code-in-2017

Legoktm created this task.Tue, Jan 2, 5:36 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptTue, Jan 2, 5:36 AM
Anomie moved this task from Unsorted to Needs Code on the MediaWiki-API board.Tue, Jan 2, 3:41 PM

Change 404150 had a related patch set uploaded (by Eflyjason; owner: Eflyjason):
[mediawiki/core@master] Add PHPUnit for ApiDelete for deleting non-file pages

https://gerrit.wikimedia.org/r/404150

Change 404150 merged by jenkins-bot:
[mediawiki/core@master] Add PHPUnit tests for ApiDelete

https://gerrit.wikimedia.org/r/404150

Legoktm closed this task as Resolved.Thu, Jan 18, 4:19 PM
Legoktm claimed this task.
Legoktm reassigned this task from Legoktm to eflyjason.Thu, Jan 18, 5:05 PM