Page MenuHomePhabricator

PHP Fatal error: Cannot declare class HookRunnerTest, because the name is already in use
Closed, ResolvedPublic

Description

It seems unnamespaced HookRunnerTest is used in CentralNotice, FlaggedRevs, and AdminLinks (from a quick search), so if any of these extensions are used together in tests it causes them to fail. This seems to be the issue with Cosmos, which tests load AdminLinks and FlaggedRevs causing tests to continuously fail due to it.

Event Timeline

Change 1002600 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/CentralNotice@master] Rename HookRunnerTest

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

Change 1002601 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/FlaggedRevs@master] Rename HookRunnerTest

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

Change 1002602 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/AdminLinks@master] Rename HookRunnerTest

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

Change 1002601 merged by jenkins-bot:

[mediawiki/extensions/FlaggedRevs@master] Rename HookRunnerTest

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

Change 1002602 merged by jenkins-bot:

[mediawiki/extensions/AdminLinks@master] Rename HookRunnerTest

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

Change 1002600 merged by jenkins-bot:

[mediawiki/extensions/CentralNotice@master] Rename HookRunnerTest

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

Umherirrender claimed this task.
Umherirrender removed a project: Patch-For-Review.

Change #1052668 had a related patch set uploaded (by Jforrester; author: Jforrester):

[mediawiki/extensions/CentralNotice@wmf_deploy] Merge branch 'master' into wmf_deploy

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

Change #1052668 merged by jenkins-bot:

[mediawiki/extensions/CentralNotice@wmf_deploy] Merge branch 'master' into wmf_deploy

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