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

QuickTest cannot double-click TapHandler

    XMLWordPrintable

Details

    • 0f94430a0f (qt/qtbase/dev) 0f94430a0f (qt/tqtc-qtbase/dev)

    Description

      When using mouseDoubleClickSequence(), or simulating a double click via press-release-press-release, in a QuickTest the TapHandler does not register the double click. It emits two tapped() signals, and doubleTapped is not emitted.

      I did some digging of the code to see if I could understand what was happening, and work around it.
      The issue stems from the two press events being too far apart. TapHandler wants the presses to be < 0.4 seconds apart. However, the helper in QuickTest for sending mouse events explicitly stops this from happening:

      case MouseRelease:
                me = QMouseEvent(QEvent::MouseButtonRelease, pos, window->mapToGlobal(pos), button, {}, stateKey);
                me.setTimestamp(++lastMouseTimestamp);
                lastMouseTimestamp += 500; // avoid double clicks being generated
                break;

      qtdeclarative/src/imports/testlib/quicktestevent.cpp:199

      Thus making it impossible to perform a double click action with a TapHandler.

      mouseDoubleClickSequence() is a shortcut for press-release-press-doubleclick-release. Thus the timestamps would be 1; 2; 503; 504; 505.

      Attachments

        1. main.cpp
          0.1 kB
        2. taptest.pro
          0.1 kB
        3. tst_taptest.qml
          0.7 kB

        Issue Links

          is required for

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-107264 TapHandler.singleTapped is emitted immediately before we know whether a double-tap will occur

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

          Task - A task that needs to be done. QTBUG-95421 need the ability to delay (increment timestamps of) touch events, as with mouse events

          • Not Evaluated - Not yet evaluated/prioritized
          • Reported

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-129794 QTest::mouseClick() and mouseDClick() don't actually apply the delay to each event in the sequence

          • P3: Somewhat important - Should be fixed, but doesn't affect the release in any way.
          • Closed
          resulted in

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-77389 tst_qquicktextedit::mouseSelection has become flaky on OpenSuse

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Closed
          For Gerrit Dashboard: QTBUG-102441
          # Subject Branch Project Status CR V
          436963,6 testlib: make it possible to test double-clicks with discrete events dev qt/qtbase Status: MERGED +2 0
          438990,2 tst_qquicktextedit: test double- and triple-clicks with delays only dev qt/qtdeclarative Status: MERGED +2 0

          Activity

            People

              srutledg Shawn Rutledge
              samuelshaw Samuel Shaw
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes