Inject: Kotlin support for sets and map

Hi,

Currently we have to explicity specify java.util.Set and java.util.Map for kotlin bindings when using @ProvidesIntoSet or bindIntoSet

It would be great if we could support Kotlin Sets and Maps or at least this limitation is documented:

Works:

class Test @Inject constructor(val set: java.util.Set<Hej>)

Breaks:

class Test @Inject constructor(val set: Set<Hej>)

[eduard-vasinskyi]

@richarddd Thank you for providing a test!

The problem was with Kotlin’s declaration-site variance Set in Kotlin is declared as Set<out E>, which leads to Set<Container> dependency result in Set<? extends Container>

We have fixed a type resolution in the master branch. You can try it out by installing the latest version of ActiveJ from the master branch locally (using ./install.sh script from project root) and then changing the ActiveJ dependency version in pom.xml to 5.0-SNAPSHOT.

This will be part of a final ActiveJ v5.0 release

[eduard-vasinskyi]

@richarddd Feel free to reopen this if the issue still persists.