Last modified: 2014-10-12 18:19:15 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T67727, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 65727 - run phpunit (on jenkins) on EditFilterMergedContent handling code in EditEntity
run phpunit (on jenkins) on EditFilterMergedContent handling code in EditEntity
Status: NEW
Product: MediaWiki extensions
Classification: Unclassified
WikidataRepo (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: Wikidata bugs
u=dev c=infrastructure p=0
:
Depends on: 65726
Blocks:
  Show dependency treegraph
 
Reported: 2014-05-24 17:10 UTC by Aude
Modified: 2014-10-12 18:19 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Aude 2014-05-24 17:10:10 UTC
if we want tests to run on EditFilterMergedContent hook handling code (and catch issues with interaction w/ AbuseFilter and SpamBlacklist), then we should have those installed in the Wikidata Jenkins builds.
Comment 1 Aude 2014-05-24 17:11:15 UTC
running phpunit on my dev wiki (with those extensions present), caught bug 65726 but wikidata jenkins did not fail due to not executing the relevant code
Comment 2 Aude 2014-06-19 15:18:30 UTC
this would still be nice to do, but a workaround is to ensure the hook gets run regardless of if it is registered.
Comment 3 Gerrit Notification Bot 2014-06-19 15:19:57 UTC
Change 140722 had a related patch set uploaded by Aude:
Force EditFilterMergedHook to be registered in EditEntity

https://gerrit.wikimedia.org/r/140722
Comment 4 Gerrit Notification Bot 2014-08-19 10:37:26 UTC
Change 140722 abandoned by Aude:
Force EditFilterMergedHook to be registered in EditEntity

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

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links