Skip to content

Commit

Permalink
[ServiceFactoryCreation] Rename from ServiceFactoryPair to match Depe…
Browse files Browse the repository at this point in the history
…ndencyContainerCreation.
  • Loading branch information
Aleksbgbg committed Jun 20, 2019
1 parent accadd0 commit 33bbf53
Show file tree
Hide file tree
Showing 4 changed files with 19 additions and 19 deletions.
8 changes: 4 additions & 4 deletions Wingman.Tests/Integration/ServiceFactoryTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -19,11 +19,11 @@ public ServiceFactoryTests()

_dependencyRegistrar = dependencyContainerCreation.Registrar;

ServiceFactoryPair serviceFactoryPair = ServiceFactoryFactory.Create(dependencyContainerCreation.Registrar,
dependencyContainerCreation.Retriever);
ServiceFactoryCreation serviceFactoryCreation = ServiceFactoryFactory.Create(dependencyContainerCreation.Registrar,
dependencyContainerCreation.Retriever);

_serviceFactoryRegistrar = serviceFactoryPair.Registrar;
_serviceFactory = serviceFactoryPair.Factory;
_serviceFactoryRegistrar = serviceFactoryCreation.Registrar;
_serviceFactory = serviceFactoryCreation.Factory;
}

[Fact]
Expand Down
8 changes: 4 additions & 4 deletions Wingman/Bootstrapper/BootstrapperBase.cs
Original file line number Diff line number Diff line change
Expand Up @@ -63,16 +63,16 @@ protected sealed override object GetInstance(Type service, string key)

protected sealed override void Configure()
{
ServiceFactoryPair serviceFactoryPair = ServiceFactoryFactory.Create(_dependencyRegistrar, _dependencyRetriever);
ServiceFactoryCreation serviceFactoryCreation = ServiceFactoryFactory.Create(_dependencyRegistrar, _dependencyRetriever);

RegisterCommonDependencies(serviceFactoryPair.Factory);
RegisterCommonDependencies(serviceFactoryCreation.Factory);

RegisterViewModels(_dependencyRegistrar);
CheckRootViewModelRegistered();

RegisterServices(_dependencyRegistrar);
RegisterFactoryViewModels(serviceFactoryPair.Registrar);
RegisterFactoryServices(serviceFactoryPair.Registrar);
RegisterFactoryViewModels(serviceFactoryCreation.Registrar);
RegisterFactoryServices(serviceFactoryCreation.Registrar);
}

/// <summary> Override to register ViewModels in the provided dependency registrar. Required as ViewModels are always needed. </summary>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
namespace Wingman.ServiceFactory
{
public class ServiceFactoryPair
public class ServiceFactoryCreation
{
public ServiceFactoryPair(IServiceFactoryRegistrar registrar, IServiceFactory factory)
public ServiceFactoryCreation(IServiceFactoryRegistrar registrar, IServiceFactory factory)
{
Registrar = registrar;
Factory = factory;
Expand Down
18 changes: 9 additions & 9 deletions Wingman/ServiceFactory/ServiceFactoryFactory.cs
Original file line number Diff line number Diff line change
Expand Up @@ -7,18 +7,18 @@
/// <summary> Factory that invokes the internal constructor of <see cref="ServiceFactory"/> and <see cref="ServiceFactoryRegistrar"/>. </summary>
public static class ServiceFactoryFactory
{
public static ServiceFactoryPair Create(IDependencyRegistrar dependencyRegistrar, IDependencyRetriever dependencyRetriever)
public static ServiceFactoryCreation Create(IDependencyRegistrar dependencyRegistrar, IDependencyRetriever dependencyRetriever)
{
RetrievalStrategyStore retrievalStrategyStore = new RetrievalStrategyStore();

return new ServiceFactoryPair(new ServiceFactoryRegistrar(dependencyRegistrar,
new RetrievalStrategyFactory(dependencyRetriever,
new ArgumentBuilderFactory(dependencyRetriever),
new ConstructorMapFactory(new ConstructorFactory())
),
retrievalStrategyStore
),
new ServiceFactory(retrievalStrategyStore)
return new ServiceFactoryCreation(new ServiceFactoryRegistrar(dependencyRegistrar,
new RetrievalStrategyFactory(dependencyRetriever,
new ArgumentBuilderFactory(dependencyRetriever),
new ConstructorMapFactory(new ConstructorFactory())
),
retrievalStrategyStore
),
new ServiceFactory(retrievalStrategyStore)
);
}
}
Expand Down

0 comments on commit 33bbf53

Please sign in to comment.