Simplify tests

SFDO-Community · Nov 27, 2024 · d33aa47 · d33aa47
1 parent a866534
commit d33aa47
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 24 deletions.
diff --git a/dlrs/main/classes/RollupCalculateJobTest.cls b/dlrs/main/classes/RollupCalculateJobTest.cls
@@ -261,17 +261,14 @@ private class RollupCalculateJobTest {
 
     RollupCalculateJob job = new RollupCalculateJob(rollupCfg.Id, 'Id != NULL');
     String jobId;
-    System.runAs(new User(Id = UserInfo.getUserId())) {
-      Test.startTest();
-      Assert.areEqual(
-        true,
-        Utilities.userHasCustomPermission(permName),
-        'Expected user to have ' + permName
-      );
-      // go into runAs because we need to get the perms recalculated
-      jobId = Database.executeBatch(job);
-      Test.stopTest();
-    }
+    Test.startTest();
+    Assert.isTrue(
+      Utilities.userHasCustomPermission(permName),
+      'Expected user to have ' + permName
+    );
+    // go into runAs because we need to get the perms recalculated
+    jobId = Database.executeBatch(job);
+    Test.stopTest();
 
     AsyncApexJob asyncJob = [
       SELECT Id, Status, JobItemsProcessed, TotalJobItems

diff --git a/dlrs/main/classes/RollupJobTest.cls b/dlrs/main/classes/RollupJobTest.cls
@@ -251,17 +251,14 @@ private class RollupJobTest {
     insert items;
 
     String jobId;
-    System.runAs(new User(Id = UserInfo.getUserId())) {
-      Test.startTest();
-      Assert.areEqual(
-        true,
-        Utilities.userHasCustomPermission(permName),
-        'Expected user to have ' + permName
-      );
-      // go into runAs because we need to get the perms recalculated
-      jobId = Database.executeBatch(new RollupJob());
-      Test.stopTest();
-    }
+    Test.startTest();
+    Assert.isTrue(
+      Utilities.userHasCustomPermission(permName),
+      'Expected user to have ' + permName
+    );
+    // go into runAs because we need to get the perms recalculated
+    jobId = Database.executeBatch(new RollupJob());
+    Test.stopTest();
 
     AsyncApexJob asyncJob = [
       SELECT Id, Status, JobItemsProcessed, TotalJobItems

diff --git a/dlrs/main/classes/UtilitiesTest.cls b/dlrs/main/classes/UtilitiesTest.cls
@@ -44,8 +44,7 @@ public class UtilitiesTest {
       if (String.isNotBlank(perm.NamespacePrefix)) {
         permName = perm.NamespacePrefix + '__' + perm.DeveloperName;
       }
-      Assert.areEqual(
-        true,
+      Assert.isTrue(
         Utilities.userHasCustomPermission(permName),
         'Expected user to have ' + permName
       );