improve tests

ipavlic · Jul 5, 2024 · c82efc8 · c82efc8
1 parent 8d72b3f
commit c82efc8
Show file tree

Hide file tree

Showing 12 changed files with 53 additions and 26 deletions.
diff --git a/force-app/main/default/classes/test/classes/collection/DecimalCollectionTest.cls b/force-app/main/default/classes/test/classes/collection/DecimalCollectionTest.cls
@@ -1,4 +1,4 @@
-@IsTest(IsParallel=true)
+@IsTest(isParallel=true)
 private class DecimalCollectionTest {
 	@IsTest
 	private static void sumShouldReturnSumIfItExists() {

diff --git a/force-app/main/default/classes/test/classes/collection/DoubleCollectionTest.cls b/force-app/main/default/classes/test/classes/collection/DoubleCollectionTest.cls
@@ -1,4 +1,4 @@
-@IsTest(IsParallel=true)
+@IsTest(isParallel=true)
 private class DoubleCollectionTest {
 	@IsTest
 	private static void sumShouldReturnSumIfItExists() {

diff --git a/force-app/main/default/classes/test/classes/collection/SObjectCollectionTest.cls b/force-app/main/default/classes/test/classes/collection/SObjectCollectionTest.cls
@@ -1,4 +1,4 @@
-@IsTest(IsParallel=true)
+@IsTest(isParallel=true)
 private class SObjectCollectionTest {
 
 	static Id firstUserId = TestUtility.getFakeId(User.SObjectType);

diff --git a/force-app/main/default/classes/test/classes/function/MapToObjectTest.cls b/force-app/main/default/classes/test/classes/function/MapToObjectTest.cls
@@ -1,4 +1,4 @@
-@IsTest(IsParallel=true)
+@IsTest(isParallel=true)
 private class MapToObjectTest {
 
 	private class MappingTarget {

diff --git a/force-app/main/default/classes/test/classes/function/MapToSObjectTest.cls b/force-app/main/default/classes/test/classes/function/MapToSObjectTest.cls
@@ -2,42 +2,69 @@
 public with sharing class MapToSObjectTest {
 
 	@IsTest
-	private static void testFieldSetting() {
-		Id oppId = TestUtility.getFakeId(Opportunity.SObjectType);
-		Opportunity opp = new Opportunity(Id = oppId);
+	private static void testSetFieldWithFieldValue() {
+		Opportunity opp = new Opportunity();
 		Task task = (Task) new MapToSObject(Task.SObjectType).setField(Task.Subject, 'Test').call(opp);
 		System.Assert.areEqual('Test', task.Subject);
 	}
-
 	@IsTest
-	private static void testFieldNameSetting() {
-		Id oppId = TestUtility.getFakeId(Opportunity.SObjectType);
-		Opportunity opp = new Opportunity(Id = oppId);
+	private static void testSetFieldWithRelationValue() {
+		Opportunity opp = new Opportunity();
 		Task task = (Task) new MapToSObject(Task.SObjectType).setField('Subject', 'Test').call(opp);
 		System.Assert.areEqual('Test', task.Subject);
 	}
 
 	@IsTest
-	private static void testSimpleFieldMapping() {
-		Id oppId = TestUtility.getFakeId(Opportunity.SObjectType);
-		Opportunity opp = new Opportunity(Id = oppId);
-		Task task = (Task) new MapToSObject(Task.SObjectType).mapField(Task.WhatId, Opportunity.Id).call(opp);
-		System.Assert.areEqual(oppId, task.WhatId);
+	private static void testSetFieldsWithFieldValues() {
+		Opportunity opp = new Opportunity();
+		Task task = (Task) new MapToSObject(Task.SObjectType).setFields(new Map<Schema.SObjectField, Object>{
+				Task.Subject => 'Test'
+		}).call(opp);
+		System.Assert.areEqual('Test', task.Subject);
 	}
 
 	@IsTest
-	private static void testSimpleFieldNameMapping() {
+	private static void testSetFieldsWithPrototype() {
+		Opportunity opp = new Opportunity();
+		Task task = (Task) new MapToSObject(Task.SObjectType).setFields(new Task(Subject = 'Test')).call(opp);
+		System.Assert.areEqual('Test', task.Subject);
+	}
+
+	@IsTest
+	private static void testMapFieldWithField() {
 		Id oppId = TestUtility.getFakeId(Opportunity.SObjectType);
 		Opportunity opp = new Opportunity(Id = oppId);
-		Task task = (Task) new MapToSObject(Task.SObjectType).mapField(Task.WhatId, 'Id').call(opp);
+		Task task = (Task) new MapToSObject(Task.SObjectType).mapField(Task.WhatId, Opportunity.Id).call(opp);
 		System.Assert.areEqual(oppId, task.WhatId);
 	}
 
 	@IsTest
-	private static void testRelationMapping() {
+	private static void testMapFieldWithRelation() {
 		Id accountId = TestUtility.getFakeId(Account.SObjectType);
 		Opportunity opp = new Opportunity(Account = new Account(Id = accountId));
 		Task task = (Task) new MapToSObject(Task.SObjectType).mapField(Task.WhatId, 'Account.Id').call(opp);
 		System.Assert.areEqual(accountId, task.WhatId);
 	}
+
+	@IsTest
+	private static void testMapFieldsWithFields() {
+		Opportunity opp = new Opportunity(Description = 'Test', Amount = 500);
+		Account account = (Account) new MapToSObject(Account.SObjectType).mapFields(new Map<Schema.SObjectfield, Schema.SObjectField>{
+			Account.Description => Opportunity.Description,
+			Account.AnnualRevenue => Opportunity.Amount
+		}).call(opp);
+		System.Assert.areEqual(account.Description, opp.Description);
+		System.Assert.areEqual(account.AnnualRevenue, opp.Amount);
+	}
+
+	@IsTest
+	private static void testMapFieldsWithRelations() {
+		Opportunity opp = new Opportunity(Description = 'Test', Amount = 500);
+		Account account = (Account) new MapToSObject(Account.SObjectType).mapFields(new Map<Schema.SObjectfield, String>{
+				Account.Description => 'Description',
+				Account.AnnualRevenue => 'Amount'
+		}).call(opp);
+		System.Assert.areEqual(account.Description, opp.Description);
+		System.Assert.areEqual(account.AnnualRevenue, opp.Amount);
+	}
 }
diff --git a/force-app/main/default/classes/test/classes/function/ModifySObjectTest.cls b/force-app/main/default/classes/test/classes/function/ModifySObjectTest.cls
@@ -1,4 +1,4 @@
-@IsTest(IsParallel=true)
+@IsTest(isParallel=true)
 private class ModifySObjectTest {
 
 	@IsTest

diff --git a/force-app/main/default/classes/test/classes/stream/SObjectStreamTest.cls b/force-app/main/default/classes/test/classes/stream/SObjectStreamTest.cls
@@ -1,4 +1,4 @@
-@IsTest(IsParallel=true)
+@IsTest(isParallel=true)
 private class SObjectStreamTest {
 	@IsTest
 	private static void testFilter() {

diff --git a/force-app/main/default/classes/test/classes/util/OptionalDecimalTest.cls b/force-app/main/default/classes/test/classes/util/OptionalDecimalTest.cls
@@ -1,4 +1,4 @@
-@IsTest(IsParallel=true)
+@IsTest(isParallel=true)
 private class OptionalDecimalTest {
 	@IsTest
 	private static void optionalCanBeNull() {

diff --git a/force-app/main/default/classes/test/classes/util/OptionalDoubleTest.cls b/force-app/main/default/classes/test/classes/util/OptionalDoubleTest.cls
@@ -1,4 +1,4 @@
-@IsTest(IsParallel=true)
+@IsTest(isParallel=true)
 private class OptionalDoubleTest {
 	@IsTest
 	private static void optionalCanBeNull() {

diff --git a/force-app/main/default/classes/test/classes/util/OptionalSObjectTest.cls b/force-app/main/default/classes/test/classes/util/OptionalSObjectTest.cls
@@ -1,4 +1,4 @@
-@IsTest(IsParallel=true)
+@IsTest(isParallel=true)
 private class OptionalSObjectTest {
 	@IsTest
 	private static void optionalCanBeNull() {

diff --git a/force-app/main/default/classes/test/classes/util/PrimitiveComparerTest.cls b/force-app/main/default/classes/test/classes/util/PrimitiveComparerTest.cls
@@ -1,4 +1,4 @@
-@IsTest(IsParallel=true)
+@IsTest(isParallel=true)
 private class PrimitiveComparerTest {
 	@IsTest
 	private static void nullComparison() {

diff --git a/force-app/main/default/classes/test/classes/util/SObjectFieldReaderTest.cls b/force-app/main/default/classes/test/classes/util/SObjectFieldReaderTest.cls
@@ -1,4 +1,4 @@
-@IsTest(IsParallel=true)
+@IsTest(isParallel=true)
 private class SObjectFieldReaderTest {
 	@IsTest
 	private static void testResolving() {