fix: Warn if there are multiple registrations (#5)

* fix: Warn if there are multiple registrations

* Fix build props

Author: glennawatson

src/Directory.Build.props

@@ -7,10 +7,10 @@
     <Authors>Glenn Watson</Authors>
     <Copyright>Copyright (c) 2021 ReactiveUI Association Inc</Copyright>
     <PackageLicenseExpression>MIT</PackageLicenseExpression>
-    <PackageProjectUrl>https://github.com/reactivemarbles/ObservableEvents</PackageProjectUrl>
     <PackageDescription>Generates source for Splat dependency injection registrations with constructor and property injection.</PackageDescription>
     <Owners>glennawatson</Owners>
     <PackageTags>system.reactive;splat;dependency injection</PackageTags>
+    <PackageProjectUrl>https://github.com/reactivemarbles/Splat.DI.SourceGenerator</PackageProjectUrl>
     <PackageReleaseNotes>https://github.com/reactivemarbles/Splat.DI.SourceGenerator/releases</PackageReleaseNotes>
     <RepositoryUrl>https://github.com/reactivemarbles/Splat.DI.SourceGenerator</RepositoryUrl>
     <RepositoryType>git</RepositoryType>

src/Splat.DependencyInjection.SourceGenerator.Tests/RegisterLazySingletonTests.InterfaceRegisteredMultipleTimes_contractParameter=.00.verified.txt

@@ -0,0 +1,13 @@
+{
+  Diagnostics: [
+    {
+      Id: SPLATDI006,
+      Title: Interface has been registered before,
+      Severity: Warning,
+      WarningLevel: 1,
+      Location: : (11,12)-(11,76),
+      MessageFormat: {0} has been registered in multiple places,
+      Category: Compiler
+    }
+  ]
+}

src/Splat.DependencyInjection.SourceGenerator.Tests/RegisterLazySingletonTests.InterfaceRegisteredMultipleTimes_contractParameter=.01.verified.cs

@@ -0,0 +1,86 @@
+//HintName: Splat.DI.Extensions.SourceGenerated.cs
+
+// <auto-generated />
+namespace Splat
+{
+    /// <summary>
+    /// Extension methods for the Splat DI source generator.
+    /// </summary>
+    internal static partial class SplatRegistrations
+    {
+        /// <summary>
+        /// Registers a class with its concrete class.
+        /// </summary>
+        public static void Register<TInterface, TConcrete>()
+        {
+        }
+
+        /// <summary>
+        /// Registers a class with its concrete class.
+        /// </summary>
+        /// <param name="contract">Optional contract.</param>
+        public static void Register<TInterface, TConcrete>(string contract)
+        {
+        }
+
+        /// <summary>
+        /// Registers a class with its concrete class.
+        /// </summary>
+        public static void RegisterLazySingleton<TInterface, TConcrete>()
+        {
+        }
+
+        /// <summary>
+        /// Registers a class with its concrete class.
+        /// </summary>
+        /// <param name="mode">The threading mode.</param>
+        public static void RegisterLazySingleton<TInterface, TConcrete>(System.Threading.LazyThreadSafetyMode mode)
+        {
+        }
+
+        /// <summary>
+        /// Registers a class with its concrete class.
+        /// </summary>
+        /// <param name="contract">Optional contract.</param>
+        public static void RegisterLazySingleton<TInterface, TConcrete>(string contract)
+        {
+        }
+
+        /// <summary>
+        /// Registers a class with its concrete class.
+        /// </summary>
+        /// <param name="contract">Optional contract.</param>
+        /// <param name="mode">The threading mode.</param>
+        public static void RegisterLazySingleton<TInterface, TConcrete>(string contract, System.Threading.LazyThreadSafetyMode mode)
+        {
+        }
+    }
+
+    /// <summary>
+    /// Makes the property get added by the DI engine.
+    /// </summary>
+    [System.AttributeUsage(System.AttributeTargets.Property)]
+    internal class DependencyInjectionPropertyAttribute : System.Attribute
+    {
+        /// <summary>
+        /// Initializes a new instance of the <see cref="DependencyInjectionPropertyAttribute"/> class.
+        /// </summary>
+        public DependencyInjectionPropertyAttribute()
+        {
+        }
+    }
+
+    /// <summary>
+    /// Makes this the constructor used by the DI engine.
+    /// </summary>
+    [System.AttributeUsage(System.AttributeTargets.Constructor)]
+    internal class DependencyInjectionConstructorAttribute : System.Attribute
+    {
+        /// <summary>
+        /// Initializes a new instance of the <see cref="DependencyInjectionConstructorAttribute"/> class.
+        /// </summary>
+        public DependencyInjectionConstructorAttribute()
+        {
+        }
+    }
+}

src/Splat.DependencyInjection.SourceGenerator.Tests/RegisterLazySingletonTests.InterfaceRegisteredMultipleTimes_contractParameter=.02.verified.cs

@@ -0,0 +1,20 @@
+//HintName: Splat.DI.Extensions.Registrations.SourceGenerated.cs
+namespace Splat
+{
+    internal static partial class SplatRegistrations
+    {
+        static SplatRegistrations()
+        {
+            {
+                global::System.Lazy<Test.ITest> lazy = new global::System.Lazy<Test.ITest>(() => new global::Test.TestConcrete1());
+                Splat.Locator.CurrentMutable.Register(() => lazy, typeof(global::System.Lazy<Test.ITest>));
+                Splat.Locator.CurrentMutable.Register(() => lazy.Value, typeof(global::Test.ITest));
+            }
+            {
+                global::System.Lazy<Test.ITest> lazy = new global::System.Lazy<Test.ITest>(() => new global::Test.TestConcrete2());
+                Splat.Locator.CurrentMutable.Register(() => lazy, typeof(global::System.Lazy<Test.ITest>));
+                Splat.Locator.CurrentMutable.Register(() => lazy.Value, typeof(global::Test.ITest));
+            }
+        }
+    }
+}

src/Splat.DependencyInjection.SourceGenerator.Tests/RegisterLazySingletonTests.InterfaceRegisteredMultipleTimes_contractParameter=Test1.00.verified.txt

@@ -0,0 +1,13 @@
+{
+  Diagnostics: [
+    {
+      Id: SPLATDI006,
+      Title: Interface has been registered before,
+      Severity: Warning,
+      WarningLevel: 1,
+      Location: : (11,12)-(11,83),
+      MessageFormat: {0} has been registered in multiple places,
+      Category: Compiler
+    }
+  ]
+}

src/Splat.DependencyInjection.SourceGenerator.Tests/RegisterLazySingletonTests.InterfaceRegisteredMultipleTimes_contractParameter=Test1.01.verified.cs

@@ -0,0 +1,86 @@
+//HintName: Splat.DI.Extensions.SourceGenerated.cs
+
+// <auto-generated />
+namespace Splat
+{
+    /// <summary>
+    /// Extension methods for the Splat DI source generator.
+    /// </summary>
+    internal static partial class SplatRegistrations
+    {
+        /// <summary>
+        /// Registers a class with its concrete class.
+        /// </summary>
+        public static void Register<TInterface, TConcrete>()
+        {
+        }
+
+        /// <summary>
+        /// Registers a class with its concrete class.
+        /// </summary>
+        /// <param name="contract">Optional contract.</param>
+        public static void Register<TInterface, TConcrete>(string contract)
+        {
+        }
+
+        /// <summary>
+        /// Registers a class with its concrete class.
+        /// </summary>
+        public static void RegisterLazySingleton<TInterface, TConcrete>()
+        {
+        }
+
+        /// <summary>
+        /// Registers a class with its concrete class.
+        /// </summary>
+        /// <param name="mode">The threading mode.</param>
+        public static void RegisterLazySingleton<TInterface, TConcrete>(System.Threading.LazyThreadSafetyMode mode)
+        {
+        }
+
+        /// <summary>
+        /// Registers a class with its concrete class.
+        /// </summary>
+        /// <param name="contract">Optional contract.</param>
+        public static void RegisterLazySingleton<TInterface, TConcrete>(string contract)
+        {
+        }
+
+        /// <summary>
+        /// Registers a class with its concrete class.
+        /// </summary>
+        /// <param name="contract">Optional contract.</param>
+        /// <param name="mode">The threading mode.</param>
+        public static void RegisterLazySingleton<TInterface, TConcrete>(string contract, System.Threading.LazyThreadSafetyMode mode)
+        {
+        }
+    }
+
+    /// <summary>
+    /// Makes the property get added by the DI engine.
+    /// </summary>
+    [System.AttributeUsage(System.AttributeTargets.Property)]
+    internal class DependencyInjectionPropertyAttribute : System.Attribute
+    {
+        /// <summary>
+        /// Initializes a new instance of the <see cref="DependencyInjectionPropertyAttribute"/> class.
+        /// </summary>
+        public DependencyInjectionPropertyAttribute()
+        {
+        }
+    }
+
+    /// <summary>
+    /// Makes this the constructor used by the DI engine.
+    /// </summary>
+    [System.AttributeUsage(System.AttributeTargets.Constructor)]
+    internal class DependencyInjectionConstructorAttribute : System.Attribute
+    {
+        /// <summary>
+        /// Initializes a new instance of the <see cref="DependencyInjectionConstructorAttribute"/> class.
+        /// </summary>
+        public DependencyInjectionConstructorAttribute()
+        {
+        }
+    }
+}

src/Splat.DependencyInjection.SourceGenerator.Tests/RegisterLazySingletonTests.InterfaceRegisteredMultipleTimes_contractParameter=Test1.02.verified.cs

@@ -0,0 +1,20 @@
+//HintName: Splat.DI.Extensions.Registrations.SourceGenerated.cs
+namespace Splat
+{
+    internal static partial class SplatRegistrations
+    {
+        static SplatRegistrations()
+        {
+            {
+                global::System.Lazy<Test.ITest> lazy = new global::System.Lazy<Test.ITest>(() => new global::Test.TestConcrete1());
+                Splat.Locator.CurrentMutable.Register(() => lazy, typeof(global::System.Lazy<Test.ITest>), "Test1");
+                Splat.Locator.CurrentMutable.Register(() => lazy.Value, typeof(global::Test.ITest), "Test1");
+            }
+            {
+                global::System.Lazy<Test.ITest> lazy = new global::System.Lazy<Test.ITest>(() => new global::Test.TestConcrete2());
+                Splat.Locator.CurrentMutable.Register(() => lazy, typeof(global::System.Lazy<Test.ITest>), "Test1");
+                Splat.Locator.CurrentMutable.Register(() => lazy.Value, typeof(global::Test.ITest), "Test1");
+            }
+        }
+    }
+}

src/Splat.DependencyInjection.SourceGenerator.Tests/RegisterLazySingletonTests.InterfaceRegisteredMultipleTimes_contractParameter=Test2.00.verified.txt

@@ -0,0 +1,13 @@
+{
+  Diagnostics: [
+    {
+      Id: SPLATDI006,
+      Title: Interface has been registered before,
+      Severity: Warning,
+      WarningLevel: 1,
+      Location: : (11,12)-(11,83),
+      MessageFormat: {0} has been registered in multiple places,
+      Category: Compiler
+    }
+  ]
+}

src/Splat.DependencyInjection.SourceGenerator.Tests/RegisterLazySingletonTests.InterfaceRegisteredMultipleTimes_contractParameter=Test2.01.verified.cs

@@ -0,0 +1,86 @@
+//HintName: Splat.DI.Extensions.SourceGenerated.cs
+
+// <auto-generated />
+namespace Splat
+{
+    /// <summary>
+    /// Extension methods for the Splat DI source generator.
+    /// </summary>
+    internal static partial class SplatRegistrations
+    {
+        /// <summary>
+        /// Registers a class with its concrete class.
+        /// </summary>
+        public static void Register<TInterface, TConcrete>()
+        {
+        }
+
+        /// <summary>
+        /// Registers a class with its concrete class.
+        /// </summary>
+        /// <param name="contract">Optional contract.</param>
+        public static void Register<TInterface, TConcrete>(string contract)
+        {
+        }
+
+        /// <summary>
+        /// Registers a class with its concrete class.
+        /// </summary>
+        public static void RegisterLazySingleton<TInterface, TConcrete>()
+        {
+        }
+
+        /// <summary>
+        /// Registers a class with its concrete class.
+        /// </summary>
+        /// <param name="mode">The threading mode.</param>
+        public static void RegisterLazySingleton<TInterface, TConcrete>(System.Threading.LazyThreadSafetyMode mode)
+        {
+        }
+
+        /// <summary>
+        /// Registers a class with its concrete class.
+        /// </summary>
+        /// <param name="contract">Optional contract.</param>
+        public static void RegisterLazySingleton<TInterface, TConcrete>(string contract)
+        {
+        }
+
+        /// <summary>
+        /// Registers a class with its concrete class.
+        /// </summary>
+        /// <param name="contract">Optional contract.</param>
+        /// <param name="mode">The threading mode.</param>
+        public static void RegisterLazySingleton<TInterface, TConcrete>(string contract, System.Threading.LazyThreadSafetyMode mode)
+        {
+        }
+    }
+
+    /// <summary>
+    /// Makes the property get added by the DI engine.
+    /// </summary>
+    [System.AttributeUsage(System.AttributeTargets.Property)]
+    internal class DependencyInjectionPropertyAttribute : System.Attribute
+    {
+        /// <summary>
+        /// Initializes a new instance of the <see cref="DependencyInjectionPropertyAttribute"/> class.
+        /// </summary>
+        public DependencyInjectionPropertyAttribute()
+        {
+        }
+    }
+
+    /// <summary>
+    /// Makes this the constructor used by the DI engine.
+    /// </summary>
+    [System.AttributeUsage(System.AttributeTargets.Constructor)]
+    internal class DependencyInjectionConstructorAttribute : System.Attribute
+    {
+        /// <summary>
+        /// Initializes a new instance of the <see cref="DependencyInjectionConstructorAttribute"/> class.
+        /// </summary>
+        public DependencyInjectionConstructorAttribute()
+        {
+        }
+    }
+}

src/Splat.DependencyInjection.SourceGenerator.Tests/RegisterLazySingletonTests.InterfaceRegisteredMultipleTimes_contractParameter=Test2.02.verified.cs

@@ -0,0 +1,20 @@
+//HintName: Splat.DI.Extensions.Registrations.SourceGenerated.cs
+namespace Splat
+{
+    internal static partial class SplatRegistrations
+    {
+        static SplatRegistrations()
+        {
+            {
+                global::System.Lazy<Test.ITest> lazy = new global::System.Lazy<Test.ITest>(() => new global::Test.TestConcrete1());
+                Splat.Locator.CurrentMutable.Register(() => lazy, typeof(global::System.Lazy<Test.ITest>), "Test2");
+                Splat.Locator.CurrentMutable.Register(() => lazy.Value, typeof(global::Test.ITest), "Test2");
+            }
+            {
+                global::System.Lazy<Test.ITest> lazy = new global::System.Lazy<Test.ITest>(() => new global::Test.TestConcrete2());
+                Splat.Locator.CurrentMutable.Register(() => lazy, typeof(global::System.Lazy<Test.ITest>), "Test2");
+                Splat.Locator.CurrentMutable.Register(() => lazy.Value, typeof(global::Test.ITest), "Test2");
+            }
+        }
+    }
+}