Merge pull request #61 from yandex/wp/fix-51

Fix: Tweak alias's equals/hashcode, so they don't include owner.

Author: Fedor Ihnatkevich

core/graph/impl/src/main/kotlin/bindings/AliasBindingImpl.kt

@@ -41,14 +41,12 @@ internal class AliasBindingImpl(
 
     override fun equals(other: Any?): Boolean {
         return this === other || (other is AliasBinding &&
-                source == other.source && target == other.target &&
-                owner == other.owner)
+                source == other.source && target == other.target)
     }
 
     override fun hashCode(): Int {
         var result = target.hashCode()
         result = 31 * result + source.hashCode()
-        result = 31 * result + owner.hashCode()
         return result
     }
 

testing/tests/src/test/kotlin/CoreBindingsTest.kt

@@ -1041,5 +1041,33 @@ class CoreBindingsTest(
 
         compileRunAndValidate()
     }
+
+    @Test
+    fun `@Binds alias is allowed to be duplicated`() {
+        givenKotlinSource("test.TestCase", """
+            import com.yandex.yatagan.*
+            import javax.inject.*
+
+            class TestClass @Inject constructor()
+
+            @Module interface MyModule {
+                @Binds fun binds(i: TestClass): Any
+                @Binds fun binds2(i: TestClass): Any
+            }
+
+            @Component(modules = [MyModule::class])
+            interface TestComponent {
+                val obj: Any
+                fun createSub(): SubComponent
+            }
+
+            @Component(isRoot = false, modules = [MyModule::class])
+            interface SubComponent {
+                val obj: Any
+            }
+        """.trimIndent())
+
+        compileRunAndValidate()
+    }
 }