Page MenuHomePhabricator

Remove (slow) Cite parser tests that aren't needed any more
Open, Needs TriagePublic

Description

We started writing PHPUnit tests for the Cite extension (some being integration test with MediaWiki, some being pure unit tests). Before, the extension was exclusively tested with parser tests.

At the end of this project, we would like to review the parser tests and:

  • Delete duplicates that actually test the exact same code paths.
  • Delete (slow) parser tests for code that is sufficiently covered by PHPUnit tests.

Details

Related Gerrit Patches:
mediawiki/extensions/Cite : masterAdd missing test cases for follow="…"

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptNov 13 2019, 9:10 AM

Change 552482 had a related patch set uploaded (by Thiemo Kreuz (WMDE); owner: Thiemo Kreuz (WMDE)):
[mediawiki/extensions/Cite@master] Add missing test cases for follow="…"

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

Change 552482 merged by jenkins-bot:
[mediawiki/extensions/Cite@master] Add missing test cases for follow="…"

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