The parser test files should be updated. Further special cases should be added.
Description
Description
Details
Details
Subject | Repo | Branch | Lines +/- | |
---|---|---|---|---|
Add further parser tests | mediawiki/extensions/Variables | master | +127 -3 |
Event Timeline
Comment Actions
Change 455061 had a related patch set uploaded (by MGChecker; owner: MGChecker):
[mediawiki/extensions/Variables@master] Add further parser tests
Comment Actions
@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!
Comment Actions
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.
Comment Actions
Change 455061 merged by jenkins-bot:
[mediawiki/extensions/Variables@master] Add further parser tests
Comment Actions
Further tests, perhaps for internal functions as well, would be nice, but aren't a priority right now.