summaryrefslogtreecommitdiff
path: root/unotest
diff options
context:
space:
mode:
authorMike Kaganski <mike.kaganski@collabora.com>2021-04-01 16:54:44 +0300
committerMike Kaganski <mike.kaganski@collabora.com>2021-04-01 17:39:54 +0200
commit6dfc7354d7100effa380ec7f6e368a2c0ec92232 (patch)
treee25d2cb3b9bf213f68d9ae8602af74aeb6f72a7e /unotest
parent2611b5c25551c38d047d8be058177bc7fbfc672d (diff)
Exclude tests based on device color depth, not RDP
After 5c9ba1f47d00ed10960b59928befd68f6c020b15, the failures on Jenkins (see 21191d0d8953a3ca6eac6022c0a14a87fe9c5e2a) made it apparent that (at least some of) Jenkins builds also run in RDP sessions. Since the tests excluded in commit 9c6142ec26a0ba61b1cf58d1e6bf0b5376394bcd never failed in Jenkins builds before, it is wrong to exclude all those tests in all RDP sessions: our CI would not test those on Windows. In the meanwhile, I discovered that the system that failed the tests actually had 16-bit color depth, despite RDP being configured to use 32-bit colors; that was the reason why the colors were modified on roundtrip. So it is better to test the actual problem to exclude tests. This reimplements the check that was introduced in commit 9c6142ec26a0ba61b1cf58d1e6bf0b5376394bcd to test default virtual device color depth. Change-Id: I329a3e2d8eca21732c77dcacf15394d1246b2e18 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/113460 Tested-by: Mike Kaganski <mike.kaganski@collabora.com> Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'unotest')
-rw-r--r--unotest/source/cpp/bootstrapfixturebase.cxx13
1 files changed, 0 insertions, 13 deletions
diff --git a/unotest/source/cpp/bootstrapfixturebase.cxx b/unotest/source/cpp/bootstrapfixturebase.cxx
index a8affd272674..5c5b3bcc3a9c 100644
--- a/unotest/source/cpp/bootstrapfixturebase.cxx
+++ b/unotest/source/cpp/bootstrapfixturebase.cxx
@@ -13,10 +13,6 @@
#include <comphelper/processfactory.hxx>
#include <basic/sbstar.hxx>
-#if defined _WIN32
-#include <systools/win32/uwinapi.h>
-#endif
-
using namespace ::com::sun::star;
// NB. this constructor is called before any tests are run, once for each
@@ -36,13 +32,4 @@ void test::BootstrapFixtureBase::setUp()
void test::BootstrapFixtureBase::tearDown() { StarBASIC::DetachAllDocBasicItems(); }
-bool test::BootstrapFixtureBase::isWindowsRDP()
-{
-#if defined _WIN32
- return GetSystemMetrics(SM_REMOTESESSION) != 0;
-#else
- return false;
-#endif
-}
-
/* vim:set shiftwidth=4 softtabstop=4 expandtab: */