Page MenuHomePhabricator

Test to validate deletion log entries and ArticleDeleteComplete hook performers
Open, Needs TriagePublic

Description

Followup from T188479#4009190

Log entries in Special:Log/delete should have a test to valide the performer.

Via #mediawiki: <Sveta> Does mediawiki have a unit test for this?
Someone does need to do that. Multiple tests could have caught this - validating that the log entry inserted had the right performer, checking that the ArticleDeleteComplete hook got the right deleter, etc.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 2 2018, 7:34 PM

Change 415477 had a related patch set uploaded (by Greg Grossmeier; owner: Jforrester):
[mediawiki/core@master] tests: Assert that the user who deleted a page is logged as such

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

Jdforrester-WMF renamed this task from Test to validate lot entries and ArticleDeleteComplete hook performers to Test to validate deletion log entries and ArticleDeleteComplete hook performers.Mar 2 2018, 7:47 PM
Jdforrester-WMF added a subscriber: Jdforrester-WMF.

Change 415477 merged by jenkins-bot:
[mediawiki/core@master] tests: Assert that the user who deleted a page is logged as such

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