Page MenuHomePhabricator

Improve tests for an expected failure in reflinks_tests
Closed, ResolvedPublic

Description

expectedFailure does not recognize whether an expected failure occurred. It ignores every fault inside the test.

  • For test_namespace_none there should be a test against raising TypeError when namespace is None and this exception is really expected or the XmlDumpPageGenerator must be able to handle that parameter successfully.
  • For test_namespace_string_id the XmlDumpPageGenerator should be able to accept namepaces as string or if this is to be disarded, test should expect an empty list instead of just ignore failing.
  • test_start_with_underscore: XmlDumpPageGenerator should be improved to allow underscored start parameters
  • test_start_prefix: XmlDumpPageGenerator should be improved to allow prefix start parameters
  • xml generator for other test with expectedFailure decorator in TestReferencesBotConstructor should be improved. I guess the problems above solves these tests too.

Event Timeline

Change 283401 had a related patch set uploaded (by Xqt):
[bugfix] Bugfixes for for XMLDumpOldPageGenerator

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

Change 283401 merged by jenkins-bot:
[bugfix] Bugfixes for XMLDumpOldPageGenerator

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

Xqt claimed this task.
Xqt triaged this task as Medium priority.