fix(repetition-assembler): value to null merge in arrays

cypress/integration/repetitionAssembler.test.ts

@@ -117,6 +117,68 @@ describe('Merging of the repetition and the original event payloads at the overv
     expect(result).to.deep.equal(assembledRepetition);
   });
 
+  it('(array of objects) should replace null in original if the repetition event has value', () => {
+    originalEvent = {
+      ...originalEvent,
+      backtrace: [
+        {
+          ...baseEvent.backtrace[0],
+          sourceCode: null
+        }
+      ],
+    };
+
+    repetitionEvent = {
+      ...repetitionEvent,
+      backtrace: [
+        {
+          ...baseEvent.backtrace[0],
+          sourceCode: [
+            {
+              content: '111',
+              line: 4
+            },
+            {
+              content: '222',
+              line: 5
+            },
+            {
+              content: '333',
+              line: 6
+            }
+          ]
+        }
+      ],
+    };
+
+    assembledRepetition = {
+      ...baseEvent,
+      backtrace: [
+        {
+          ...baseEvent.backtrace[0],
+          sourceCode: [
+            {
+              content: '111',
+              line: 4
+            },
+            {
+              content: '222',
+              line: 5
+            },
+            {
+              content: '333',
+              line: 6
+            }
+          ]
+        }
+      ],
+    };
+
+    const result = repetitionAssembler(originalEvent, repetitionEvent);
+
+    expect(result).to.deep.equal(assembledRepetition);
+  });
+
   it('should take the repetition field value instead of the original event value in the array of objects', () => {
     originalEvent = {
       ...originalEvent,
@@ -229,4 +291,4 @@ describe('Merging of the repetition and the original event payloads at the overv
 
     expect(result).to.deep.equal(assembledRepetition);
   });
-});
+});

src/types/events.ts

@@ -252,7 +252,7 @@ export interface HawkEventBacktraceFrame {
   /**
    * Part of source code file near the called line
    */
-  sourceCode: BacktraceSourceCode[];
+  sourceCode: BacktraceSourceCode[] | null;
 
   /**
    * Function name extracted from current stack frame

src/utils.ts

@@ -182,8 +182,19 @@ export function repetitionAssembler(originalEvent: HawkEventPayload, repetition:
       return originalParam;
     }
 
-    if (typeof repetitionParam === 'object' && typeof originalEvent === 'object') {
-      return repetitionAssembler(originalParam, repetitionParam);
+
+    if (typeof repetitionParam === 'object' && typeof originalParam === 'object') {
+      /**
+       * If original event has null but repetition has some value, we need to return repetition value
+       */
+      if (originalParam === null) {
+        return repetitionParam;
+      /**
+       * Otherwise, we need to recursively merge original and repetition values
+       */
+      } else {
+        return repetitionAssembler(originalParam, repetitionParam);
+      }
     }
 
     return repetitionParam;