[KSP] Incorrect generates map binding code for ClassKey when class is generic

[BenSchwab]

This can be reproduced with the following code, when using the KSP proccessor (dagger version 2.48)

@Component(
  modules = [
    MyModuleA::class
  ]
)
interface ApplicationComponent {

  val map: Map<Class<*>, String>

}

@Module
internal object MyModuleA {

  @Provides
  @IntoMap
  @ClassKey(Thing::class)
  fun provideThingValue(): String {
    return "value for Thing"
  }

  @Provides
  @IntoMap
  @ClassKey(GenericThing::class)
  fun provideAbstractThingValue(): String {
    return "value for Thing"
  }

}

class Thing

class GenericThing<T>

This produces the following incorrect method:

 @Override
    public Map<Class<?>, String> getMap() {
      return MapBuilder.<Class<?>, String>newMapBuilder(2).put(Thing.class, MyModuleA_ProvideThingValueFactory.provideThingValue()).put(GenericThing<?>.class, MyModuleA_ProvideAbstractThingValueFactory.provideAbstractThingValue()).build();
    }

Notably GenericThing<?>.class should be GenericThing.class.

Using kapt will generate the correct code:

 @Override
    public Map<Class<?>, String> getMap() {
      return MapBuilder.<Class<?>, String>newMapBuilder(2).put(Thing.class, MyModuleA_ProvideThingValueFactory.provideThingValue()).put(GenericThing.class, MyModuleA_ProvideAbstractThingValueFactory.provideAbstractThingValue()).build();
    }

[bcorso]

@BenSchwab thanks for filing this bug!

Looks like this this is a bug in the code here: https://github.com/google/dagger/blob/master/java/dagger/internal/codegen/binding/AnnotationExpression.java#L115

We should be using the ClassName rather than the TypeName there. I'll send out a fix for that.