Types on J.FieldAccess aren't completely accurate

[traceyyoshima]

Comparing the types from the Java compiler with the PSI-based parser shows incorrect type attribution:

class A {
    public Integer id = 0;
}
class B extends A {
    Integer getId() {
        return super.id;
    }
}

J.FieldAccess#type: JavaType.Class with signature: java.lang.Integer
J.FieldAccess#target#type: JavaType.Class with signature: A
J.FieldAccess#target#fieldType: JavaType.Variable with signature: B{name=super,type=A}

open class A {
    val id: Int = 0
}
class B : A() {
    fun get(): Int {
        return super.id
    }
}

J.FieldAccess#type: JavaType.Class with signature: kotlin.int
J.FieldAccess#target#type: JavaType.Class with signature: A
J.FieldAccess#target#fieldType: null

The issue is caused by visitSimpleNameExpression not having context to the necessary types and might cause incorrect types in many other places. We don't test the type of every identifier, which might be necessary now that we've switched to the PSI. This issue might not exist in the IR, but we'd need to check.