Page MenuHomePhabricator

Improve test coverage
Open, LowPublic

Description

The parser test files should be updated. Further special cases should be added.

Event Timeline

Change 455061 had a related patch set uploaded (by MGChecker; owner: MGChecker):
[mediawiki/extensions/Variables@master] Add further parser tests

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

@Dinoguy1000 I would like you to review this patch, since it's mostly about variable behavior in obscure edge cases. Are you interested? Feel free to propose further tests!

It looks good to me, though TBH the testcases you have there already cover behaviors that I've never used myself; I try to avoid using the weirder behaviors of the extension, so I don't really have anything to add to the tests.

Change 455061 merged by jenkins-bot:
[mediawiki/extensions/Variables@master] Add further parser tests

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

MGChecker lowered the priority of this task from High to Low.

Further tests, perhaps for internal functions as well, would be nice, but aren't a priority right now.