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.
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.
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Resolved | daniel | T262443 Do not hit actual urls during phpunit tests | |||
Resolved | Physikerwelt | T265646 Rewrite Math's MathCoverageTest to not try to make HTTP requests |
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
@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!