sw/qa/extras/tiledrendering/tiledrendering2.cxx | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
New commits:
commit 56664a5b6fffa8b4bd993663374acc94fe1b7fba
Author: Miklos Vajna <vmik...@collabora.com>
AuthorDate: Mon Dec 2 08:52:54 2024 +0100
Commit: Miklos Vajna <vmik...@collabora.com>
CommitDate: Mon Dec 2 10:22:58 2024 +0100
CppunitTest_sw_tiledrendering2: make this more stable, try 2
Similar to commit d248d8f5271c380e51af9482b6724d471e2cb929
(CppunitTest_sw_tiledrendering2: try to make this more stable,
2024-11-20), but note that the problematic assert at e.g.
<https://ci.libreoffice.org/job/gerrit_linux_gcc_release/178813/console>
is before the early return, so move the early assert below the early
return.
Change-Id: I0ed9168840defcc567e2e2fa8262bdf32489b54a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/177652
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmik...@collabora.com>
diff --git a/sw/qa/extras/tiledrendering/tiledrendering2.cxx
b/sw/qa/extras/tiledrendering/tiledrendering2.cxx
index f628042a447d..767aee6d15a6 100644
--- a/sw/qa/extras/tiledrendering/tiledrendering2.cxx
+++ b/sw/qa/extras/tiledrendering/tiledrendering2.cxx
@@ -74,12 +74,12 @@ CPPUNIT_TEST_FIXTURE(SwTiledRenderingTest,
testStatusBarPageNumber)
pWrtShell2->GetView().GetViewFrame().GetBindings().GetTimer().Invoke();
// Then make sure the page number in view 2 is correct:
- CPPUNIT_ASSERT_EQUAL(static_cast<size_t>(1),
aView2.m_aStateChanges.size());
// FIXME this should not happen, but it does from time to time.
if (!aView2.m_aStateChanges.empty())
{
return;
}
+ CPPUNIT_ASSERT_EQUAL(static_cast<size_t>(1),
aView2.m_aStateChanges.size());
// Without the accompanying fix in place, this test would have failed with:
// - Expected: .uno:StatePageNumber=Page 2 of 2
// - Actual : .uno:StatePageNumber=Page 1 of 2
