Page MenuHomePhabricator

Remove redundant DefaultDrySiteTestCase
Closed, ResolvedPublic

Description

DefaultDrySiteTestCase only sets dry = True and provides no additional functionality. This is redundant since DefaultSiteTestCase already supports the dry class attribute. Remove DefaultDrySiteTestCase and update the DefaultSiteTestCase documentation to mention that dry = True can be used to run tests in offline mode.

Update tests accordingly: https://codesearch.wmcloud.org/pywikibot/?q=DefaultDrySiteTestCase&files=&excludeFiles=&repos=

Event Timeline

Xqt triaged this task as Low priority.

Change #1323806 had a related patch set uploaded (by Xqt; author: Xqt):

[pywikibot/core@master] Remove DefaultDrySiteTest and replace it's usage wir dry = True

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

Change #1323806 merged by jenkins-bot:

[pywikibot/core@master] Remove DefaultDrySiteTest and replace it's usage wir dry = True

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