cleanup

Author: mydea

packages/core/src/client.ts

@@ -81,16 +81,14 @@ interface DoNotSendEventError {
   [DO_NOT_SEND_EVENT_SYMBOL]: true;
 }
 
-/** Exported for tests only. */
-export function _makeInternalError(message: string): InternalError {
+function _makeInternalError(message: string): InternalError {
   return {
     message,
     [INTERNAL_ERROR_SYMBOL]: true,
   };
 }
 
-/** Exported for tests only. */
-export function _makeDoNotSendEventError(message: string): DoNotSendEventError {
+function _makeDoNotSendEventError(message: string): DoNotSendEventError {
   return {
     message,
     [DO_NOT_SEND_EVENT_SYMBOL]: true,

packages/core/test/lib/client.test.ts

@@ -12,7 +12,6 @@ import {
   withMonitor,
 } from '../../src';
 import type { BaseClient, Client } from '../../src/client';
-import { _makeInternalError } from '../../src/client';
 import * as integrationModule from '../../src/integration';
 import type { Envelope, ErrorEvent, Event, SpanJSON, TransactionEvent } from '../../src/types-hoist';
 import * as loggerModule from '../../src/utils-hoist/logger';
@@ -1294,7 +1293,7 @@ describe('Client', () => {
         expect(beforeSend).toHaveBeenCalled();
         expect(TestClient.instance!.event).toBeUndefined();
         expect(loggerWarnSpy).toBeCalledWith(
-          _makeInternalError('before send for type `error` must return `null` or a valid event.').message,
+          'before send for type `error` must return `null` or a valid event.',
         );
       }
     });
@@ -1315,7 +1314,7 @@ describe('Client', () => {
         expect(beforeSendTransaction).toHaveBeenCalled();
         expect(TestClient.instance!.event).toBeUndefined();
         expect(loggerWarnSpy).toBeCalledWith(
-          _makeInternalError('before send for type `transaction` must return `null` or a valid event.').message,
+          'before send for type `transaction` must return `null` or a valid event.',
         );
       }
     });
@@ -1688,9 +1687,9 @@ describe('Client', () => {
         originalException: exception,
       });
       expect(loggerWarnSpy).toBeCalledWith(
-        _makeInternalError(
+
           `Event processing pipeline threw an error, original event will not be sent. Details have been sent as a new event.\nReason: ${exception}`,
-        ).message,
+        ,
       );
     });