Lint K2 UAST: UCallableReferenceExpression reports wrong qualifierType [325107804]

Fixed

Bug

Status Update

No update yet.

Description

za...@gmail.com

created issue #1

Feb 14, 2024 07:01AM

When using useK2Uast in Lint (version 8.4.0-alph09), I noticed a test start failing in this PR for a case where we expect to report errors on some callable references.

The detector attempts to lint against Dispatchers::IO reference calls (amongst others), but with K2 UAST enabled it for some reason reports this source string as java.lang.String::IO. Dispatchers is not a java.lang.String :)

To run: hardcode the param to setUseK2Uast() in BaseSlackLintTest then run the RawDispatchersUsageDetectorTest.simple test.

Comments

ar...@google.com <ar...@google.com> Feb 15, 2024 01:14AM

Assigned to an...@google.com.

js...@google.com <js...@google.com> #2Feb 15, 2024 05:21AM

Reassigned to js...@google.com.

This was broken by 3bca759a5ff08352de831bb1e9b61b1ec2b3362d.
Fix (pending) is I2c2dc7b600603ee430fd0d91b23d52ea8aa29ca9.

js...@google.com <js...@google.com> #3Feb 15, 2024 08:08AM

Accepted by js...@google.com.

Almost 2 months later and this is still broken

js...@google.com <js...@google.com> #4Feb 15, 2024 07:04PM

Marked as fixed.

Since there is no progression, I wanted to share our quick-fix for the issue.

#sdkmanager --package_file=${PATH_WORKSPACE}/packages
while read p; do echo "y" | sdkmanager "${p}"; done <${PATH_WORKSPACE}/packages

za...@gmail.com <za...@gmail.com> #5Feb 17, 2024 01:26AM

jb...@google.com What is the update on this?

an...@google.com <an...@google.com> #6Feb 22, 2024 10:07PM

What is the status of this item?

za...@gmail.com <za...@gmail.com> #7Mar 4, 2024 12:55AM

This has been fixed on master today (internal ref: ag/2945015) and will be available in the next SDK release.

js...@google.com <js...@google.com> #8Mar 4, 2024 02:20AM

Any ETA on next release?