Add SObjectNotNullTest, improve ModifySObjectTest test, replace isPar…

…allel with IsParallel
ipavlic · Jul 5, 2024 · 3f5172d · 3f5172d
1 parent 03b4630
commit 3f5172d
Show file tree

Hide file tree

Showing 14 changed files with 56 additions and 13 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
@@ -1,4 +1,4 @@
-@IsTest(isParallel=true)
+@IsTest(IsParallel=true)
 public with sharing class MapToSObjectTest {
 
 	@IsTest

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,7 +1,33 @@
-@IsTest(isParallel=true)
+@IsTest(IsParallel=true)
 private class ModifySObjectTest {
+
+	@IsTest
+	private static void stringDefinedFieldIsUsed() {
+		Opportunity opp = new Opportunity();
+		new ModifySObject().setField('Amount', 1000).call(opp);
+		System.Assert.areEqual(1000, opp.Amount);
+	}
+
+	@IsTest
+	private static void fieldTokenDefinedFieldIsUsed() {
+		Opportunity opp = new Opportunity();
+		new ModifySObject().setField(Opportunity.Amount, 1000).call(opp);
+		System.Assert.areEqual(1000, opp.Amount);
+	}
+
+	@IsTest
+	private static void fieldTokenDefinedFieldAreUsed() {
+		Opportunity opp = new Opportunity();
+		new ModifySObject().setFields(new Map<SObjectField, Object>{
+			Opportunity.Name => 'Test',
+			Opportunity.Amount => 1000
+		}).call(opp);
+		System.Assert.areEqual(1000, opp.Amount);
+		System.Assert.areEqual('Test', opp.Name);
+	}
+
 	@IsTest
-	private static void definedFieldsAreUsed() {
+	private static void prototypeDefinedFieldsAreUsed() {
 		Opportunity opp = new Opportunity();
 		new ModifySObject().setFields(new Opportunity(Amount = 1000, Name = 'Test')).call(opp);
 		System.Assert.areEqual(1000, opp.Amount);

diff --git a/force-app/main/default/classes/test/classes/function/SObjectNotNullTest.cls b/force-app/main/default/classes/test/classes/function/SObjectNotNullTest.cls
@@ -0,0 +1,12 @@
+@IsTest(IsParallel=true)
+private class SObjectNotNullTest {
+	@IsTest
+	private static void nullSObjectReturnsFalse() {
+		Assert.areEqual(false, new SObjectNotNull().call(null));
+	}
+
+	@IsTest
+	private static void nonNullSObjectReturnsTrue() {
+		Assert.areEqual(true, new SObjectNotNull().call(new Opportunity()));
+	}
+}
diff --git a/force-app/main/default/classes/test/classes/function/SObjectNotNullTest.cls-meta.xml b/force-app/main/default/classes/test/classes/function/SObjectNotNullTest.cls-meta.xml
@@ -0,0 +1,5 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<ApexClass xmlns="http://soap.sforce.com/2006/04/metadata">
+    <apiVersion>56.0</apiVersion>
+    <status>Active</status>
+</ApexClass>
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() {