Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-133340

Modify QQuickTest::showView() to verify that position has been updated

    XMLWordPrintable

Details

    • Task
    • Resolution: Fixed
    • P2: Important
    • None
    • 6.9
    • Testing: QuickTest
    • None
    • Linux/X11
    • 9e9ed403d (dev), 0b4c76d60 (6.9), 372e02da3 (6.8), 7504ae9b9 (tqtc/lts-6.5)

    Description

      QQuickTest::showView() returns a bool if showing a qquickView is successful. A race condition happens on Opensuse where the qquickView is shown but his position and framePosition have not been updated correctly. Events can be sent to the un-updated position making some tests fail.

      Fix this by verifying if the position is done adjusting. If the qquickView does not have a frame, this verification should be skipped.

      Flaky and failing test affected by this on Opensuse:

      Tst_touchMouse::touchCancelWillCancelMousePress

      Tst_touchMouse::touchButtonOnFlickable

      Tst_qquickmouse::doubleTap

      Tst_QQuickMultiPointTouchArea::inFlickable2

      Tst_qquickmousearea::doubleTap

      Tst_qquickmousearea::nestedFlickableStopAtBounds 

      Tst_qquickpinchhandler::scale

       

      Attachments

        Issue Links

          covers

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-132630 tst_qquickmousearea::doubleTap is flaky on opensuse

          • P1: Critical - Urgent and Important, will STOP the release if matched with set FixVersion field. This includes regressions from the last version that are not edge cases; Data loss; Build issues; Flaky tests; All but the most unlikely crashes/lockups/hanging; Serious usability issues and embarrassing bugs & Bugs critical to a deliverable. This is the highest possible priority for requirements.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-132648 tst_TouchMouse::touchButtonOnFlickable is flaky on opensuse

          • P1: Critical - Urgent and Important, will STOP the release if matched with set FixVersion field. This includes regressions from the last version that are not edge cases; Data loss; Build issues; Flaky tests; All but the most unlikely crashes/lockups/hanging; Serious usability issues and embarrassing bugs & Bugs critical to a deliverable. This is the highest possible priority for requirements.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-132941 Top flaky test: tst_QQuickMouseArea::nestedFlickableStopAtBounds

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-133341 flaky test Tst_touchMouse::touchCancelWillCancelMousePress

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-133342 Tst_QQuickMultiPointTouchArea::inFlickable2 is flaky on Opensuse

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-133343 Tst_qquickmousearea::doubleTap flaky on Opensuse

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-133344 Tst_qquickpinchhandler::scale is flaky on Opensuse

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Closed
          For Gerrit Dashboard: QTBUG-133340
          # Subject Branch Project Status CR V
          616654,18 Check if showView uses updated position dev qt/qtdeclarative Status: MERGED +2 0
          624130,2 Check if showView uses updated position 6.9 qt/qtdeclarative Status: MERGED +2 0
          624312,5 Check if showView uses updated position 6.8 qt/qtdeclarative Status: MERGED +2 +1
          625699,2 Check if showView uses updated position tqtc/lts-6.5 qt/tqtc-qtdeclarative Status: MERGED +2 0

          Activity

            People

              fredericl Frederic Lefebvre
              fredericl Frederic Lefebvre
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes