Handles error with string cause

getsentry · Oct 10, 2024 · 1aeb779 · 1aeb779
1 parent c773341
commit 1aeb779
Show file tree

Hide file tree

Showing 2 changed files with 41 additions and 2 deletions.
diff --git a/src/js/integrations/nativelinkederrors.ts b/src/js/integrations/nativelinkederrors.ts
@@ -13,7 +13,7 @@ import type {
  StackFrame,
  StackParser,
 } from '@sentry/types';
-import { isInstanceOf, isPlainObject } from '@sentry/utils';
+import { isInstanceOf, isPlainObject,isString } from '@sentry/utils';
 
 import type { NativeStackFrames } from '../NativeRNSentry';
 import { NATIVE } from '../wrapper';
@@ -103,7 +103,9 @@ function walkErrorTree(
 
  let exception: Exception;
  let exceptionDebugImages: DebugImage[] | undefined;
- if ('stackElements' in linkedError) {
+ if (isString(linkedError)) {
+ exception = { value: linkedError };
+ } else if ('stackElements' in linkedError) {
  // isJavaException
  exception = exceptionFromJavaStackElements(linkedError);
  } else if ('stackReturnAddresses' in linkedError) {

diff --git a/test/integrations/nativelinkederrors.test.ts b/test/integrations/nativelinkederrors.test.ts
@@ -337,6 +337,43 @@ describe('NativeLinkedErrors', () => {
  }),
  );
  });
+
+ it('handles events with a string cause', async () => {
+ const actualEvent = await executeIntegrationFor(
+ {
+ exception: {
+ values: [
+ {
+ type: 'Error',
+ value: 'Captured exception',
+ },
+ ],
+ },
+ },
+ {
+ originalException: createNewError({
+ message: 'Error with string cause',
+ cause: 'string cause',
+ }),
+ },
+ );
+
+ expect(actualEvent).toEqual(
+ expect.objectContaining(<Partial<Event>>{
+ exception: {
+ values: [
+ {
+ type: 'Error',
+ value: 'Captured exception',
+ },
+ {
+ value: 'string cause',
+ },
+ ],
+ },
+ }),
+ );
+ });
 });
 
 function executeIntegrationFor(mockedEvent: Event, mockedHint: EventHint): Event | null {