Page MenuHomePhabricator

Rewrite Math's MathCoverageTest to not try to make HTTP requests
Closed, ResolvedPublic

Description

These are now skipped, but instead the tests should be re-written so that use a mock service (and pass).

Follow-up to rEMAT2ae8fcb7b5dd: MathCoverageTest: Skip all tests for now as HTTP requests are banned.

Event Timeline

Physikerwelt moved this task from Doing to Inbox on the Math board.

As discussed yesterdasy with @daniel the tests will be split into unit tests and rendering tests that will only run on the mathoid side. One needs to consider a mechanism to regenerate fake responses if mathoid changes.

Change 661186 had a related patch set uploaded (by Physikerwelt; owner: Physikerwelt):
[mediawiki/extensions/Math@master] Reenable Math tests

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

@Physikerwelt: Removing task assignee as this open task has been assigned for more than two years - See the email sent to task assignee on Feburary 22nd, 2023.
Please assign this task to yourself again if you still realistically [plan to] work on this task - it would be welcome! :)
If this task has been resolved in the meantime, or should not be worked on by anybody ("declined"), please update its task status via "Add Action… 🡒 Change Status".
Also see https://www.mediawiki.org/wiki/Bug_management/Assignee_cleanup for tips how to best manage your individual work in Phabricator. Thanks!

Physikerwelt claimed this task.

This was completed two years ago;-)