solemnwarning/wxwidgets
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 4

Add a wait to wxGrid UI unit tests too

Browse source 
Wait for a bit before checking for the expected event as a single
wxYield() call might apparently not be enough under load, as this test
sporadically fails in AppVeyor CI builds even though it passes locally.
This commit is contained in:
Vadim Zeitlin 2023-10-03 15:59:52 +02:00
parent 068bac2599
commit 3d07de66b1
1 changed files with 9 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

12
tests/controls/gridtest.cpp
View file

@ -608,7 +608,9 @@ TEST_CASE_METHOD(GridTestCase, "Grid::SortClick", "[grid]")
wxYield();
sim.MouseClick();
wxYield();
WaitFor("mouse click to be processed", [&]() {
return sort.GetCount() != 0;
});
CHECK(sort.GetCount() == 1);
#endif
@ -642,7 +644,9 @@ TEST_CASE_METHOD(GridTestCase, "Grid::Size", "[grid]")
wxYield();
sim.MouseUp();
wxYield();
WaitFor("mouse release to be processed", [&]() {
return colsize.GetCount() != 0;
});
CHECK(colsize.GetCount() == 1);
@ -651,7 +655,9 @@ TEST_CASE_METHOD(GridTestCase, "Grid::Size", "[grid]")
sim.MouseDragDrop(pt.x, pt.y, pt.x, pt.y + 50);
wxYield();
WaitFor("mouse drag to be processed", [&]() {
return rowsize.GetCount() != 0;
});
CHECK(rowsize.GetCount() == 1);
#endif