Page MenuHomePhabricator

Improve test coverage on Special:Investigate code
Closed, ResolvedPublic3 Estimated Story Points

Description

As reported by https://doc.wikimedia.org/cover-extensions/CheckUser/src/Investigate/index.html, the code coverage over Investigate related code is low.

Increasing it is important to ensure that the large changes needed to implement T329189: Add read new support to Special:Investigate for event table migration do not cause a Regression.

Acceptance criteria

Event Timeline

Change #1018273 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/CheckUser@master] Improve test coverage for Investigate related code.

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

Change #1018750 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/CheckUser@master] Improve test coverage on Investigate Utilities classes

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

Change #1018751 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/CheckUser@master] Add more tests for TokenManager service

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

Change #1018752 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/CheckUser@master] Mark ChangeService as covered by tests for subclasses

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

Change #1018753 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/CheckUser@master] Test PagerFactory implementations and TimelineRowFormatterFactory

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

Change #1018754 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/CheckUser@master] Add integration tests for TimelinePager

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

Change #1018755 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/CheckUser@master] Test the TimelineRowFormatter class

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

Dreamy_Jazz set the point value for this task to 3.Apr 10 2024, 6:17 PM

Change #1018767 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/CheckUser@master] Expand SpecialInvestigate integration tests

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

Change #1018750 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Improve test coverage on Investigate Utilities classes

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

Change #1018752 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Mark ChangeService as covered by tests for subclasses

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

Change #1018753 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Test PagerFactory implementations and TimelineRowFormatterFactory

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

Change #1018754 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Add integration tests for TimelinePager

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

Change #1018751 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Add more tests for TokenManager service

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

Change #1018755 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Test the TimelineRowFormatter class

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

Change #1018273 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Start on integration tests for SpecialInvestigate

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

Change #1020899 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/CheckUser@master] Expand tests for the ComparePager

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

Change #1020899 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Expand tests for the ComparePager

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

Change #1018767 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Expand SpecialInvestigate integration tests

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

Dreamy_Jazz updated the task description. (Show Details)