Give CompositeTypeHandler higher precedence over getUserCompositeType

[m-Peter]

Closes onflow/cadence-tools#172

Description

When running the following script, in the Flow Emulator:

pub fun main(): Type? {
    return CompositeType("flow.AccountContractAdded")
}

The following error is returned:

error: [Error Code: 1054] location (flow) is not a valid location: expecting an AddressLocation, but other location types are passed
 --> b2b8424d1ef6de8f49bd006c8b7cbdba4a98c3be6e5834fb4ed1b281655293fe:2:4
  |
2 |     return CompositeType("flow.AccountContractAdded")

After the proposed change, the script runs successfully:

Result: Type<flow.AccountContractAdded>()

Note: This seemed to work in the test suite of the cadence-tools/test repository, however, it didn't work when I tried it in an actual Cadence test file.

---

• Targeted PR against master branch
• Linked to Github issue with discussion and accepted design OR link to spec that describes this work
• Code follows the standards mentioned here
• Updated relevant documentation
• Re-reviewed Files changed in the Github PR explorer
• Added appropriate labels

[codecov]

Codecov Report

Merging #2717 (5051efd) into master (af119ed) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #2717   +/-   ##
=======================================
  Coverage   79.12%   79.12%           
=======================================
  Files         333      333           
  Lines       78306    78306           
=======================================
  Hits        61962    61962           
  Misses      14047    14047           
  Partials     2297     2297           

Flag	Coverage Δ
unittests	79.12% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed	Coverage Δ
runtime/interpreter/interpreter.go	89.72% <100.00%> (ø)

[SupunS]

LGTM!

[turbolent]

Makes sense to ask the handler first, if available, and then check for a user-defined type otherwise 👍