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

qmlls: more test coverage for qualified names

    XMLWordPrintable

Details

    • Task
    • Resolution: Unresolved
    • P2: Important
    • 6.8
    • None
    • QML: Tooling
    • None

    Description

      Lately, the Dom around qualified names changed and all features that existed before it might not have been adapted to the new qualified names.

      One should test if various qmlls features still work correctly by adding more testcases to:

      • find definition,
      • find type definition,
      • find Base object,
      • find definition from location and
      • resolve expression type
      • (that is almost everything except findUsages more or less)

      tests.

      The idea here is to go through the various Dom structures that changed to be able to support qualified names and see if the various qmlls features (still) work correctly on them.

      List of Dom structures to go through:

      • The left hand side of Bindings, e.g. someItem.property: 42
      • Property and signal definitions, e.g. signal mySignal() or property int myProperty
      • type annotations, e.g. function f(x: QQ.Item) {} or function f(): QQ.Item {}
      • Qml Component name in instantiations, e.g. QQ.Item {} in default and normal bindings
      • Qml Component name in definitions, e.g. component MyIC: QQ.Item {}
      • Qml Component type in definitions, e.g. component MyIC: QQ.Item {}
      • Qualified names in JS snippets, like return someItem.property

      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:
              1 Start watching this issue

              Dates

                Created:
                Updated:

                Gerrit Reviews

                  There are no open Gerrit changes