Import alias flagged as unused import

[gabrielittner]

Something like import a.b.c.MyClass as SomeOtherName is marked as an unused import after upgrading to 0.34.0.

[shashachu]

@gabrielittner this is a regression in 0.34.0?

[gabrielittner]

Yes. 0.33 was working without this issue on the same code

[shashachu]

@gabrielittner could you provide a more complete code snippet? it looks like the rule attempts to handle import aliases (and tests that have import aliases are passing), so I want to understand what broke.

[gabrielittner]

I will try to put together something reproducible tomorrow

[shashachu]

@gabrielittner I'm going to revert the changes to the unused imports rule that we made in 0.34.0, and I suspect it will fix your issue, but when you have a repro case let me know and I can test.

[shashachu]

@gabrielittner if you can provide your repro case, #535 is attempting to fix the regressions in 0.34.0, so we want to make sure your case is covered.

[k3muri84]

i also have this case.
import com.product....JavaClass as AnotherName in a Kotlin unit test in our case
will be removed with 0.34 and worked with 0.33.
I might also try to put something together in case @gabrielittner doesn't have something already.

[sowmyav24]

Similar example was provided in the PR review. It has been added as a test case and fixed now.

[romtsn]

Fixed via #535