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

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

      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.

        For Gerrit Dashboard: QTQAINFRA-7156
        # Subject Branch Project Status CR V

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

              Created:
              Updated:
              Resolved:

                There are no open Gerrit changes