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

Consolidate test helpers

    XMLWordPrintable

Details

    • Task
    • Resolution: Done
    • Not Evaluated
    • None
    • None
    • Quick: Controls 2
    • None

    Description

      Before qtquickcontrols2 was merged into qtdeclarative, it had a bunch of test helpers, some of which were duplicated from qtdeclarative (such as QQmlTestMessageHandler). We should try to remove any duplicates and move the qtquickcontrols2 helpers to tests/auto/shared. In the process, we can make a library out of that code, as currently there is a whole lot of unnecessary compilation going on as a result of tests including the sources rather than linking:

      mitch@mitch-ubuntu-20:~/dev/qt-dev/qtdeclarative$ git grep util.h -- tests/auto/*.cpp | wc -l
270

      Attachments

        Issue Links

          relates to

          Task - A task that needs to be done. QTBUG-95173 Merge qtquickcontrols2 into qtdeclarative

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

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-95756 tst_QQmlImport::removeDynamicPlugin() is flaky on macOS

          • 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; 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
          For Gerrit Dashboard: QTBUG-95621
          # Subject Branch Project Status CR V
          363532,42 Consolidate test helpers into private libraries dev qt/qtdeclarative Status: MERGED -1 0
          370739,4 Consolidate test helpers into private libraries 6.2 qt/qtdeclarative Status: MERGED +2 0

          Activity

            People

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

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes