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

Reduce default QTRY_* timeout

    XMLWordPrintable

Details

    • Task
    • Resolution: Unresolved
    • Not Evaluated
    • None
    • 7.0 (Next Major Release)
    • Testing: qtestlib
    • None

    Description

      As explained in QTBUG-81979 and its patches (in particular, this comment from the maintainer), 5 seconds (doubled by Qt Test, so effectively 10 seconds) is too long. For most tests, 1 second is plenty and would save developers a lot of time, especially for tests involving many QTRY_* checks where it's not clear which test is failing if the test was killed before the check finished (to avoid waiting the full 10 seconds).

      Changing this in Qt 6 would break a lot of users' tests, so we should do it in Qt 7. After https://codereview.qt-project.org/c/qt/qtbase/+/648330, they can also increase the timeout with a one-liner for each test case if it's then too short.

      Attachments

        Issue Links

          For Gerrit Dashboard: QTBUG-138160
          # Subject Branch Project Status CR V

          Activity

            People

              macadder Jason McDonald
              mitch_curtis Mitch Curtis
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:

                Gerrit Reviews

                  There is 1 open Gerrit change