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

tst_qmlls_utils: adapt tests to changes in qmlls

    XMLWordPrintable

Details

    • 9182ef97f (dev), ec03be43c (6.7)

    Description

      Currently, it seems that tst_qmlls_utils is testing the qmlls utils with different options that qmlls is using.

      This is problematic, because:

      • tst_qmlls_utils tests codepaths that qmlls does not use
      • The corresponding functionalities, for example "go to type definition", might work in tst_qmlls_utils but stop working in qmlls (for example, due to changes in the Dom construction) and there will be no test to warn about this case

      In fact, due to, for example https://codereview.qt-project.org/c/qt/qtdeclarative/+/526805/10, "go to type definition" and "go to definition" stopped working. Same for lots of other commits extending the Dom and breaking the code in qmllsutils.

      Therefore, make sure to construct the Dom in tst_qmlls_utils in the same way as qmlls does it, that means including the DomCreationOptions that were added after those tst_qmlls_utils tests were written!

       

      Once that is done, tend to all tests that are failing now and adapt them to the new Dom structure (for example the change of id's to scriptidentiferexpressions or types to FieldMemberExpressions).

      Attachments

        Issue Links

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

          Activity

            People

              sami.shalayel Sami Shalayel
              sami.shalayel Sami Shalayel
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes