Merge pull request #4950 from umbraco/v8/feature/4590-resolvable-ifactory

IFactory is resolvable

src/Umbraco.Core/Composing/Composition.cs

@@ -133,7 +133,11 @@ namespace Umbraco.Core.Composing
 
             Configs.RegisterWith(_register);
 
-            return _register.CreateFactory();
+            IFactory factory = null;
+            // ReSharper disable once AccessToModifiedClosure -- on purpose
+            _register.Register(_ => factory, Lifetime.Singleton);
+            factory = _register.CreateFactory();
+            return factory;
         }
 
         /// <summary>

src/Umbraco.Tests/Composing/CompositionTests.cs

@@ -0,0 +1,53 @@
+using System;
+using Moq;
+using NUnit.Framework;
+using Umbraco.Core;
+using Umbraco.Core.Cache;
+using Umbraco.Core.Composing;
+using Umbraco.Core.Logging;
+
+namespace Umbraco.Tests.Composing
+{
+    [TestFixture]
+    public class CompositionTests
+    {
+        [Test]
+        public void FactoryIsResolvable()
+        {
+            Func<IFactory, IFactory> factoryFactory = null;
+
+            var mockedRegister = Mock.Of<IRegister>();
+            var mockedFactory = Mock.Of<IFactory>();
+
+            // the mocked register creates the mocked factory
+            Mock.Get(mockedRegister)
+                .Setup(x => x.CreateFactory())
+                .Returns(mockedFactory);
+
+            // the mocked register can register a factory factory
+            Mock.Get(mockedRegister)
+                .Setup(x => x.Register(It.IsAny<Func<IFactory, IFactory>>(), Lifetime.Singleton))
+                .Callback<Func<IFactory, IFactory>, Lifetime>((ff, lt) => factoryFactory = ff);
+
+            // the mocked factory can invoke the factory factory
+            Mock.Get(mockedFactory)
+                .Setup(x => x.GetInstance(typeof(IFactory)))
+                .Returns(() => factoryFactory?.Invoke(mockedFactory));
+
+            var logger = new ProfilingLogger(Mock.Of<ILogger>(), Mock.Of<IProfiler>());
+            var typeLoader = new TypeLoader(Mock.Of<IAppPolicyCache>(), "", logger);
+            var composition = new Composition(mockedRegister, typeLoader, logger, Mock.Of<IRuntimeState>());
+
+            // create the factory, ensure it is the mocked factory
+            var factory = composition.CreateFactory();
+            Assert.AreSame(mockedFactory, factory);
+
+            // ensure we can get an IFactory instance,
+            // meaning that it has been properly registered
+
+            var resolved = factory.GetInstance<IFactory>();
+            Assert.IsNotNull(resolved);
+            Assert.AreSame(factory, resolved);
+        }
+    }
+}

src/Umbraco.Tests/Umbraco.Tests.csproj

@@ -118,6 +118,7 @@
     <Compile Include="Cache\SnapDictionaryTests.cs" />
     <Compile Include="Clr\ReflectionUtilitiesTests.cs" />
     <Compile Include="Collections\OrderedHashSetTests.cs" />
+    <Compile Include="Composing\CompositionTests.cs" />
     <Compile Include="Composing\LightInjectValidation.cs" />
     <Compile Include="Composing\ContainerConformingTests.cs" />
     <Compile Include="CoreThings\CallContextTests.cs" />