Fixed a bug in the autotest for q3progressbar.

The "standard" test would sometimes fail (Mac OS X)
because extra paint events could be generated, which
would cause paintNumber to be > 1. Comparing it to 1
would fail. This test should be redesigned, I think.

2 files changed, 13 insertions, 3 deletions

diff --git a/tests/auto/bic/tst_bic.cpp b/tests/auto/bic/tst_bic.cpp
index db0c5baa62..82c8dc0594 100644
--- a/tests/auto/bic/tst_bic.cpp
+++ b/tests/auto/bic/tst_bic.cpp
@@ -293,7 +293,7 @@ void tst_Bic::sizesAndVTables()
 
     bool isFailed = false;
 
-    qDebug() << oldLib.arg(libName);
+    //qDebug() << oldLib.arg(libName);
     if (oldLib.isEmpty() || !QFile::exists(oldLib.arg(libName)))
         QSKIP("No platform spec found for this platform/version.", SkipSingle);
 
diff --git a/tests/auto/q3progressbar/tst_q3progressbar.cpp b/tests/auto/q3progressbar/tst_q3progressbar.cpp
index c5485a7ca4..34a95d375e 100644
--- a/tests/auto/q3progressbar/tst_q3progressbar.cpp
+++ b/tests/auto/q3progressbar/tst_q3progressbar.cpp
@@ -93,11 +93,15 @@ class MyCustomProgressBar : public Q3ProgressBar
         void paintEvent(QPaintEvent * event)
         {
             paintNumber++;
+            qDebug() << "PAINT EVENT:" << paintNumber;
             Q3ProgressBar::paintEvent(event);
         }
         int paintNumber;
 };
 
+/*
+  Maybe this test should be redesigned.
+ */
 void tst_Q3ProgressBar::setProgress()
 {
     MyCustomProgressBar * m_progressBar = new MyCustomProgressBar();
@@ -111,15 +115,21 @@ void tst_Q3ProgressBar::setProgress()
     m_progressBar->setProgress(m_progressBar->progress() + 1);
     QCOMPARE(oldValue + 1,m_progressBar->progress());
     QApplication::processEvents();
-    QVERIFY(m_progressBar->paintNumber >= 1); //it might be more than 1 because it is animated
 
+    // It might be > 1 because it is animated.
+    QVERIFY(m_progressBar->paintNumber >= 1);
+    qDebug() << "Animation test: paintNumber =" << m_progressBar->paintNumber;
+    
     //standard case
     m_progressBar->setTotalSteps(3);
     m_progressBar->setProgress(0);
     m_progressBar->paintNumber = 0;
     m_progressBar->setProgress(m_progressBar->progress() + 1);
     QApplication::processEvents();
-    QCOMPARE(m_progressBar->paintNumber,1);
+
+    // It might be > 1 because other events might cause painting.
+    QVERIFY(m_progressBar->paintNumber >= 1);
+    qDebug() << "Standard test: paintNumber =" << m_progressBar->paintNumber;
 }
 
 QTEST_MAIN(tst_Q3ProgressBar)