Merge pull request #334 from ctchipps/master

Application Class Extensibility

Author: john-storey-devops

sfdx-source/apex-common/main/classes/fflib_Application.cls

@@ -39,6 +39,11 @@ public virtual class fflib_Application
 		private List<SObjectType> m_objectTypes;
 		private fflib_ISObjectUnitOfWork m_mockUow;
 
+		/**
+		 * Constructs a Unit Of Work factory
+		 **/
+		public UnitOfWorkFactory() { }
+
 		/**
 		 * Constructs a Unit Of Work factory
 		 *
@@ -54,29 +59,27 @@ public virtual class fflib_Application
 		 *   SObjectType list provided in the constructor, returns a Mock implementation
 		 *   if set via the setMock method
 		 **/
-		public fflib_ISObjectUnitOfWork newInstance()
+		public virtual fflib_ISObjectUnitOfWork newInstance()
 		{
 			// Mock?
 			if(m_mockUow!=null)
 				return m_mockUow;
 			return new fflib_SObjectUnitOfWork(m_objectTypes);
 		}
 
-
 		/**
 		 * Returns a new fflib_SObjectUnitOfWork configured with the
 		 *   SObjectType list provided in the constructor, returns a Mock implementation
 		 *   if set via the setMock method
 		 **/
-		public fflib_ISObjectUnitOfWork newInstance(fflib_SObjectUnitOfWork.IDML dml)
+		public virtual fflib_ISObjectUnitOfWork newInstance(fflib_SObjectUnitOfWork.IDML dml)
 		{
 			// Mock?
 			if(m_mockUow!=null)
 				return m_mockUow;
 			return new fflib_SObjectUnitOfWork(m_objectTypes, dml);
 		}
 
-
 		/**
 		 * Returns a new fflib_SObjectUnitOfWork configured with the
 		 *   SObjectType list specified, returns a Mock implementation
@@ -85,7 +88,7 @@ public virtual class fflib_Application
 		 * @remark If mock is set, the list of SObjectType in the mock could be different
 		 *         then the list of SObjectType specified in this method call
 		 **/
-		public fflib_ISObjectUnitOfWork newInstance(List<SObjectType> objectTypes)
+		public virtual fflib_ISObjectUnitOfWork newInstance(List<SObjectType> objectTypes)
 		{
 			// Mock?
 			if(m_mockUow!=null)
@@ -101,17 +104,16 @@ public virtual class fflib_Application
 		 * @remark If mock is set, the list of SObjectType in the mock could be different
 		 *         then the list of SObjectType specified in this method call
 		 **/
-		public fflib_ISObjectUnitOfWork newInstance(List<SObjectType> objectTypes, fflib_SObjectUnitOfWork.IDML dml)
+		public virtual fflib_ISObjectUnitOfWork newInstance(List<SObjectType> objectTypes, fflib_SObjectUnitOfWork.IDML dml)
 		{
 			// Mock?
 			if(m_mockUow!=null)
 				return m_mockUow;
 			return new fflib_SObjectUnitOfWork(objectTypes, dml);
 		}
 
-
 		@TestVisible
-		private void setMock(fflib_ISObjectUnitOfWork mockUow)
+		protected virtual void setMock(fflib_ISObjectUnitOfWork mockUow)
 		{
 			m_mockUow = mockUow;
 		}
@@ -126,6 +128,11 @@ public virtual class fflib_Application
 
 		protected Map<Type, Object> m_serviceInterfaceTypeByMockService;
 
+		/**
+		 * Constructs a simple Service Factory
+		 **/
+		public ServiceFactory() { }
+
 		/**
 		 * Constructs a simple Service Factory, 
 		 *   using a Map of Apex Interfaces to Apex Classes implementing the interface
@@ -162,7 +169,7 @@ public virtual class fflib_Application
 		}
 
 		@TestVisible
-		private virtual void setMock(Type serviceInterfaceType, Object serviceImpl)
+		protected virtual void setMock(Type serviceInterfaceType, Object serviceImpl)
 		{
 			m_serviceInterfaceTypeByMockService.put(serviceInterfaceType, serviceImpl);
 		}
@@ -176,6 +183,11 @@ public virtual class fflib_Application
 		protected Map<SObjectType, Type> m_sObjectBySelectorType;
 		protected Map<SObjectType, fflib_ISObjectSelector> m_sObjectByMockSelector;
 
+		/**
+		 * Constructs a simple Selector Factory
+		 **/
+		public SelectorFactory() { }
+
 		/**
 		 * Consturcts a Selector Factory linking SObjectType's with Apex Classes implement the fflib_ISObjectSelector interface
 		 *   Note that the factory does not check the given Apex Classes implement the interface
@@ -218,7 +230,7 @@ public virtual class fflib_Application
 		 * @param recordIds The SObject record Ids, must be all the same SObjectType
 		 * @exception Is thrown if the record Ids are not all the same or the SObjectType is not registered
 		 **/
-		public List<SObject> selectById(Set<Id> recordIds)
+		public virtual List<SObject> selectById(Set<Id> recordIds)
 		{
 			// No point creating an empty Domain class, nor can we determine the SObjectType anyway
 			if(recordIds==null || recordIds.size()==0)
@@ -246,7 +258,7 @@ public virtual class fflib_Application
 		 * @param relatedRecords used to extract the related record Ids, e.g. Opportunity records
 		 * @param relationshipField field in the passed records that contains the relationship records to query, e.g. Opportunity.AccountId
 		 **/
-		public List<SObject> selectByRelationship(List<SObject> relatedRecords, SObjectField relationshipField)
+		public virtual List<SObject> selectByRelationship(List<SObject> relatedRecords, SObjectField relationshipField)
 		{
 			Set<Id> relatedIds = new Set<Id>();
 			for(SObject relatedRecord : relatedRecords)
@@ -259,7 +271,7 @@ public virtual class fflib_Application
 		}
 
 		@TestVisible
-		private virtual void setMock(fflib_ISObjectSelector selectorInstance)
+		protected virtual void setMock(fflib_ISObjectSelector selectorInstance)
 		{
 			m_sObjectByMockSelector.put(selectorInstance.sObjectType(), selectorInstance);
 		} 
@@ -276,6 +288,11 @@ public virtual class fflib_Application
 
 		protected Map<Object, fflib_IDomain> mockDomainByObject;
 
+        /**
+		 * Constructs a Domain factory
+		 **/
+		public DomainFactory() { }
+
 		/**
 		 * Constructs a Domain factory, using an instance of the Selector Factory
 		 *   and a map of Apex classes implementing fflib_ISObjectDomain by SObjectType
@@ -396,18 +413,18 @@ public virtual class fflib_Application
 		}
 
 		@TestVisible
-		private virtual void setMock(fflib_ISObjectDomain mockDomain)
+		protected virtual void setMock(fflib_ISObjectDomain mockDomain)
 		{
 			mockDomainByObject.put((Object) mockDomain.sObjectType(), (fflib_IDomain) mockDomain);
 		}
 
 		@TestVisible
-		private virtual void setMock(fflib_IDomain mockDomain)
+		protected virtual void setMock(fflib_IDomain mockDomain)
 		{
 			mockDomainByObject.put(mockDomain.getType(), mockDomain);
 		}
 
-		private Map<Object, Type> getConstructorTypeByObject(Map<SObjectType, Type> constructorTypeBySObjectType)
+		protected virtual Map<Object, Type> getConstructorTypeByObject(Map<SObjectType, Type> constructorTypeBySObjectType)
 		{
 			Map<Object, Type> result = new Map<Object, Type>();
 			for (SObjectType sObjectType : constructorTypeBySObjectType.keySet())