jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/510520 )

Change subject: Skip TestLoginLogout if it is running on Appveyor ......................................................................

Skip TestLoginLogout if it is running on Appveyor

APPVEYOR environment variable may be 'True' or 'true'

Bug: T222508 Change-Id: I38e8af3921815ae45d8912bfa35f59fcd9a1b36f --- M tests/site_tests.py 1 file changed, 1 insertion(+), 1 deletion(-)

Approvals: Dvorapa: Looks good to me, approved jenkins-bot: Verified

diff --git a/tests/site_tests.py b/tests/site_tests.py index 2db780d..77b572f 100644 --- a/tests/site_tests.py +++ b/tests/site_tests.py @@ -3707,7 +3707,7 @@

"""Test for login and logout methods."""

- @unittest.skipIf(os.environ.get('APPVEYOR', 'false') == 'true', + @unittest.skipIf(os.environ.get('APPVEYOR', 'false') in ('true', 'True'), 'No user defined for APPVEYOR tests') def test_login_logout(self): """Validate login and logout methods by toggling the state."""