Added test

framework/default/ortoo-core/default/classes/ApplicationMockRegistrar.cls

@@ -204,7 +204,6 @@ public inherited sharing class ApplicationMockRegistrar
 	 * @param	Type			The type (interface) for which the mock app logic class should be registered
 	 * @return	Amoss_Instance	The controller for the mock Service
 	 */
-	// TODO: test
 	public static Amoss_Instance registerMockAppLogic( Type appLogicType )
 	{
 		Amoss_Instance mockAppLogicController = new Amoss_Instance( appLogicType );

framework/default/ortoo-core/default/classes/services/dml/DmlRecord.cls

@@ -39,7 +39,12 @@ public virtual inherited sharing class DmlRecord
 		return this.recordToDml;
 	}
 
-	// TODO: document / test
+	/**
+	 * Returns the Id that is on the SObject this will perform the DML against.
+	 * Is populated after an insert has been performed
+	 *
+	 * @return	Id	The Id on the SObject
+	 */
 	public Id getSobjectId()
 	{
 		return this.recordToDml.Id;

framework/default/ortoo-core/default/classes/services/dml/tests/DmlRecordTest.cls

@@ -30,6 +30,20 @@ private without sharing class DmlRecordTest
         ortoo_Asserts.assertContains( 'DmlRecord constructor called with a null recordToDml', exceptionMessage, 'constructor, when given null, will throw an exception' );
     }
 
+    @isTest
+    private static void getSobjectId_willReturnTheIdOfTheSobject() // NOPMD: Test method name format
+    {
+		Id accountId = TestIdUtils.generateId( Account.sobjectType );
+
+		DmlRecord dmlRecord = new DmlRecord( new Account( Id = accountId ) );
+
+        Test.startTest();
+			Id got = dmlRecord.getSobjectId();
+        Test.stopTest();
+
+        System.assertEquals( accountId, got, 'getSobjectId, will return the Id of the SObject' );
+    }
+
     @isTest
     private static void addChildContect_whenCalled_willInitialiseADefinerForThatType() // NOPMD: Test method name format
     {