From f7241e8cb949ecf2d86fae2a759a8e1fdb148a26 Mon Sep 17 00:00:00 2001 From: Stephen Kelly Date: Tue, 9 Oct 2012 16:24:52 +0200 Subject: Exclude a QCOMPARE in tst_QListWidget instead of excluding the whole class. The original bug was reported as a different test failing. Looking through the CI logs, that test now always passes, but fastScroll fails unstably. The fastScroll test is already excluded on Mac, but the patch and reason for that predates the public git history. This is a regression since marking the entire test as insignificant, and shows why insignification of tests should be as narrow as possible. Narrow this one now. Task-number: QTBUG-21098 Change-Id: I0d6c22e422af190c9e6331e123db38022af28e4b Reviewed-by: Rohan McGovern --- tests/auto/widgets/itemviews/qlistwidget/tst_qlistwidget.cpp | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'tests/auto/widgets/itemviews/qlistwidget/tst_qlistwidget.cpp') diff --git a/tests/auto/widgets/itemviews/qlistwidget/tst_qlistwidget.cpp b/tests/auto/widgets/itemviews/qlistwidget/tst_qlistwidget.cpp index 79014fcbbf..b882b56735 100644 --- a/tests/auto/widgets/itemviews/qlistwidget/tst_qlistwidget.cpp +++ b/tests/auto/widgets/itemviews/qlistwidget/tst_qlistwidget.cpp @@ -1534,8 +1534,12 @@ void tst_QListWidget::fastScroll() sbar->setValue(sbar->value() + sbar->singleStep()); QApplication::processEvents(); + const QSize actualItemSize = widget.painted.boundingRect().size(); + if (actualItemSize != itemSize) + QEXPECT_FAIL("", "QTBUG-21098", Continue); + // only one item should be repainted, the rest should be scrolled in memory - QCOMPARE(widget.painted.boundingRect().size(), itemSize); + QCOMPARE(actualItemSize, itemSize); } #endif // Q_OS_MAC -- cgit v1.2.3