From 63b4c5ee67703ee0a640080180e237311685adc4 Mon Sep 17 00:00:00 2001 From: Vadim Zeitlin Date: Thu, 15 Jul 2021 19:58:28 +0200 Subject: [PATCH] Remove checks for buildbot in the test suite buildbot is not used since a long time, so don't bother checking for it in IsAutomaticTest(). This commit is best viewed ignoring whitespace-only changes. --- tests/test.cpp | 18 ++---------------- 1 file changed, 2 insertions(+), 16 deletions(-) diff --git a/tests/test.cpp b/tests/test.cpp index d5699cffef..0ee15c3b4b 100644 --- a/tests/test.cpp +++ b/tests/test.cpp @@ -391,22 +391,8 @@ extern bool IsAutomaticTest() static int s_isAutomatic = -1; if ( s_isAutomatic == -1 ) { - // Allow setting an environment variable to emulate buildslave user for - // testing. - wxString username; - if ( !wxGetEnv(wxASCII_STR("WX_TEST_USER"), &username) ) - username = wxGetUserId(); - - username.MakeLower(); - s_isAutomatic = username == wxASCII_STR("buildbot") || - username.Matches(wxASCII_STR("sandbox*")); - - // Also recognize various CI environments. - if ( !s_isAutomatic ) - { - s_isAutomatic = wxGetEnv(wxASCII_STR("GITHUB_ACTIONS"), NULL) || - wxGetEnv(wxASCII_STR("APPVEYOR"), NULL); - } + s_isAutomatic = wxGetEnv(wxASCII_STR("GITHUB_ACTIONS"), NULL) || + wxGetEnv(wxASCII_STR("APPVEYOR"), NULL); } return s_isAutomatic == 1;