Uploaded image for project: 'Qt Quality Assurance Infrastructure'
  1. Qt Quality Assurance Infrastructure
  2. QTQAINFRA-7156

Make new usages of qWait() throw a Hint in sanity bot

    XMLWordPrintable

Details

    • Task
    • Resolution: Done
    • P3: Somewhat important
    • None
    • None
    • Sanity Bot
    • None
    • ae7616a01 (master)

    Description

      Considered an anti-pattern, using qWait() in a test can contribute to flakiness unless the test is created specifically with time-based actions in mind.

      While we should probably have a goal of removing qWait() usages from tests, we should at least begin discouraging new usages for now. Since there may be legitimate uses for this, we should limit sanity complaints to "Hint". This can be upgraded to a -1 if we find that most new usages are not well justified.

      Attachments

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

          People

            daniel.smith Daniel Smith
            daniel.smith Daniel Smith
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes