Page MenuHomePhabricator

Increase test coverage of Codex Special:Block
Open, Needs TriagePublic5 Estimated Story Points

Description

At the beginning of the project, Jest (our testing framework) was only freshly added to Core. Since then, we've gotten better at writing these kinds of tests, and I think now that the codebase has matured it's time to be more deliberate in our test coverage.

Actionables

Event Timeline

Change #1128595 had a related patch set uploaded (by MusikAnimal; author: MusikAnimal):

[mediawiki/core@master] jest: add coverage report and increase coverage

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

MusikAnimal changed the point value for this task from 3 to 5.Mar 18 2025, 4:56 AM

Change #1128946 had a related patch set uploaded (by MusikAnimal; author: MusikAnimal):

[mediawiki/core@master] jest: add snapshot tests for mediawiki.special.block

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

Change #1128595 merged by jenkins-bot:

[mediawiki/core@master] jest: add coverage report and increase coverage

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

MusikAnimal updated the task description. (Show Details)

I never did add the snapshot tests. They were tedious to review and made me question how important they are. Perhaps we should do it only for sub-components, or something.

Anyway I'm going to unlick this.

Change #1128946 abandoned by MusikAnimal:

[mediawiki/core@master] jest: add snapshot tests for mediawiki.special.block

Reason:

unclear need

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