diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2021-04-01 16:54:44 +0300 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2021-04-01 18:00:49 +0200 |
| commit | 7f2c6fff796f86d685aa20018ae72375011b1367 (patch) | |
| tree | c8be3c829f1850a25f91a5981bf81bc9079e6f82 /unotest | |
| parent | db749f20bf499353fa662020c10563f83080a043 (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/+/113466
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.cxx | 13 |
1 files changed, 0 insertions, 13 deletions
diff --git a/unotest/source/cpp/bootstrapfixturebase.cxx b/unotest/source/cpp/bootstrapfixturebase.cxx index 5eb5b91dcaf0..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() const -{ -#if defined _WIN32 - return GetSystemMetrics(SM_REMOTESESSION); -#else - return false; -#endif -} - /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ |