Opprett kun queueable-jobb når relevante felter på samtalereferatet oppdateres

Author: frodehoen

src/integration/henvendelse-cache/classes/handlers/ConvNoteCacheHandler.cls

@@ -6,11 +6,40 @@ global class ConvNoteCacheHandler extends MyTriggers {
         );
     }
     global override void onAfterUpdate(Map<Id, SObject> triggerOldMap) {
-        //TODO: Sjekk kun for felter som er relevante for cachingen
-        createHenvendelseCacheQueueable(
-            (List<Conversation_Note__c>) records,
-            true
-        );
+        List<String> fieldNamesToCheck = new List<String>{
+            'CRM_Communication_Channel__c',
+            'CRM_Disposal_Datetime__c',
+            'CRM_Incorrectly_Sent__c',
+            'CRM_Is_Read__c',
+            'STO_Sensitive_Information__c',
+            'CRM_Subtheme__c',
+            'CRM_Theme__c',
+            'CRM_Is_Redacted__c',
+            'STO_Sensitive_Information__c'
+        };
+
+        List<Conversation_Note__c> convNotesWithRelevantChanges = new List<Conversation_Note__c>();
+        for (
+            Conversation_Note__c record : (List<Conversation_Note__c>) records
+        ) {
+            Conversation_Note__c recordOld = (Conversation_Note__c) triggerOldMap.get(
+                record.Id
+            );
+
+            if (
+                MyTriggers.hasChangedFields(
+                    fieldNamesToCheck,
+                    record,
+                    recordOld
+                )
+            ) {
+                convNotesWithRelevantChanges.add(record);
+            }
+        }
+        if (convNotesWithRelevantChanges.isEmpty()) {
+            return;
+        }
+        createHenvendelseCacheQueueable(convNotesWithRelevantChanges, true);
     }
 
     private void createHenvendelseCacheQueueable(

src/integration/henvendelse-cache/classes/handlers/tests/ConvNoteCacheHandlerTest.cls

@@ -61,7 +61,7 @@ public class ConvNoteCacheHandlerTest {
     }
 
     @IsTest
-    static void shouldNotCreateQueuableJobWhenConvNoteIsUpdatedByInternalUser() {
+    static void shouldNotCreateQueuableJobWhenNumbeOfApiRetriesIsUpdated() {
         ApiMock.setTestMock(
             HenvendelseCacheCalloutService.SERVICES.DELETE_HENVENDELSE_CACHE_API.name(),
             200,
@@ -78,10 +78,10 @@ public class ConvNoteCacheHandlerTest {
         );
 
         Test.startTest();
-        convNote.CRM_Is_Read__c = true;
+        convNote.CRM_Number_of_API_Retries__c = 7;
         update convNote;
         Test.stopTest();
 
-        Assert.areEqual(2, [SELECT COUNT() FROM AsyncApexJob]);
+        Assert.areEqual(1, [SELECT COUNT() FROM AsyncApexJob]); //Expect one job when creating conv note, but no job should be created when upddating
     }
 }