Created test for testHandleAutoLeadScoring. Updated TestDataFactory. Fixed issue in LeadTriggerHandler

Author: ManoolK

force-app/main/default/classes/LeadTriggerHandler.cls

@@ -76,21 +76,18 @@ public with sharing class LeadTriggerHandler {
 	 */
 	public static void handleAutoLeadScoring(List<Lead> leadsToScore) {
 		for (Lead ld : leadsToScore) {
-			Integer score = 10;
+			Integer score = 0;
 
 			// Check and add points based on the specified conditions
-			if (ld.LeadSource == 'Website' && ld.Email != null) {
-				score = 3;
+			if (ld.LeadSource == 'Web' && ld.Email != null) {
+				score += 3;
 			}
-
 			if (ld.Phone != null) {
-				score = 5;
+				score += 5;
 			}
-
 			if (ld.Industry == 'Technology') {
-				score = 10;
+				score += 10;
 			}
-
 			ld.Lead_Score__c = score; // Set the computed score back to the lead
 		}
 	}

force-app/main/default/classes/LeadTriggerHandlerTest.cls

@@ -82,4 +82,38 @@ private class LeadTriggerHandlerTest {
             Assert.areEqual('Assistant', lead.Title, 'Expected Title \'Assistant\'');
         }
     }
+
+    /*
+    * LeadTriggerHandler.handleTitleNormalization update test
+    */
+    @isTest
+    static void testHandleAutoLeadScoring() {
+        // Prepare the test data
+        List<Map<String, Object>> params = new List<Map<String, Object>>();
+        params.add(new Map<String, Object>{'Lead_Score__c' => 20, 'LeadSource' => null}); // 0
+        params.add(new Map<String, Object>{'LeadSource' => 'Other', 'Email' => '[email protected]'}); // 0
+        params.add(new Map<String, Object>{'LeadSource' => 'Web', 'Email' => '[email protected]'}); // 3
+        params.add(new Map<String, Object>{'Phone' => '(908)345-1234', 'Industry' => 'Government'}); // 5
+        params.add(new Map<String, Object>{'LeadSource' => 'Web', 'Email' => '[email protected]', 'Phone' => '(908)345-2345'}); // 8
+        params.add(new Map<String, Object>{'Lead_Score__c' => 10, 'Email' => '[email protected]', 'Industry' => 'Technology'}); // 10
+        params.add(new Map<String, Object>{'LeadSource' => 'Web', 'Email' => '[email protected]', 'Industry' => 'Technology'}); // 13
+        params.add(new Map<String, Object>{'LeadSource' => 'Web', 'Phone' => '(908)346-1234', 'Industry' => 'Technology'}); // 15
+        params.add(new Map<String, Object>{'LeadSource' => 'Web', 'Email' => '[email protected]', 
+                                           'Phone' => '(908)346-1234', 'Industry' => 'Technology'}); // 18
+
+        List<Lead> leadsToScore = TestDataFactory.createLeadsByParams(params, false);
+        
+        // Perform the test
+        Test.startTest();
+        LeadTriggerHandler.handleAutoLeadScoring(leadsToScore);
+        Test.stopTest();
+
+        // Assert that the Score calculates correctly
+        List<Integer> scoreVariants = new List<Integer>{0, 0, 3, 5, 8, 10, 13, 15, 18};
+        for (Integer i = 0; i < leadsToScore.size(); i++) {
+            Assert.isTrue(leadsToScore[i].Lead_Score__c <= 18, 'Lead score shouldn\'t be more than 18');
+            Assert.areEqual(scoreVariants[i], leadsToScore[i].Lead_Score__c, 'Score has not correctly calculated');
+        }
+        
+    }
 }

force-app/main/default/classes/TestDataFactory.cls

@@ -18,4 +18,23 @@ public class TestDataFactory {
 
         return leadsToInsert;
     }
+
+    public static List<Lead> createLeadsByParams(List<Map<String, Object>> params, Boolean doInsert) {
+        List<Lead> leads = new List<Lead>();
+        for (Integer i = 0; i < params.size(); i++) {
+            Lead newLead = new Lead();
+            newLead.LastName = 'Test Lead ' + i;
+            newLead.Company = 'Test Company';
+            Map<String, Object> locParams = params[i];
+            for (String key : locParams.keySet()) {
+                newLead.put(key, locParams.get(key));
+            }
+            leads.add(newLead);
+        }
+        if (doInsert) {
+            Database.insert(leads);
+        }
+
+        return leads;
+    }
 }