ref(core): Log debug message when capturing error events

Author: mydea

packages/core/src/baseclient.ts

@@ -713,6 +713,16 @@ export abstract class BaseClient<O extends ClientOptions> implements Client<O> {
    * @param scope
    */
   protected _captureEvent(event: Event, hint: EventHint = {}, scope?: Scope): PromiseLike<string | undefined> {
+    if (DEBUG_BUILD && isErrorEvent(event)) {
+      logger.log(
+        `Captured error event \`${
+          (event.exception && event.exception.values && event.exception.values[0] && event.exception.values[0].value) ||
+          event.message ||
+          '<unknown>'
+        }\``,
+      );
+    }
+
     return this._processEvent(event, hint, scope).then(
       finalEvent => {
         return finalEvent.event_id;

packages/core/test/lib/baseclient.test.ts

@@ -366,6 +366,22 @@ describe('BaseClient', () => {
       // `captureException` should bail right away this second time around and not get as far as calling this again
       expect(clientEventFromException).toHaveBeenCalledTimes(1);
     });
+
+    test('captures logger message', () => {
+      const logSpy = jest.spyOn(loggerModule.logger, 'log').mockImplementation(() => undefined);
+
+      const options = getDefaultTestClientOptions({ dsn: PUBLIC_DSN });
+      const client = new TestClient(options);
+
+      client.captureException(new Error('test error here'));
+      client.captureException({});
+
+      expect(logSpy).toHaveBeenCalledTimes(2);
+      expect(logSpy).toBeCalledWith('Captured error event `test error here`');
+      expect(logSpy).toBeCalledWith('Captured error event `<unknown>`');
+
+      logSpy.mockRestore();
+    });
   });
 
   describe('captureMessage', () => {
@@ -442,6 +458,20 @@ describe('BaseClient', () => {
         }),
       );
     });
+
+    test('captures logger message', () => {
+      const logSpy = jest.spyOn(loggerModule.logger, 'log').mockImplementation(() => undefined);
+
+      const options = getDefaultTestClientOptions({ dsn: PUBLIC_DSN });
+      const client = new TestClient(options);
+
+      client.captureMessage('test error here');
+
+      expect(logSpy).toHaveBeenCalledTimes(1);
+      expect(logSpy).toBeCalledWith('Captured error event `test error here`');
+
+      logSpy.mockRestore();
+    });
   });
 
   describe('captureEvent() / prepareEvent()', () => {
@@ -1658,6 +1688,22 @@ describe('BaseClient', () => {
         message: 'hello',
       });
     });
+
+    test('captures logger message', () => {
+      const logSpy = jest.spyOn(loggerModule.logger, 'log').mockImplementation(() => undefined);
+
+      const options = getDefaultTestClientOptions({ dsn: PUBLIC_DSN });
+      const client = new TestClient(options);
+
+      client.captureEvent({ message: 'hello' });
+      // transactions are ignored and not logged
+      client.captureEvent({ type: 'transaction', message: 'hello 2' });
+
+      expect(logSpy).toHaveBeenCalledTimes(1);
+      expect(logSpy).toBeCalledWith('Captured error event `hello`');
+
+      logSpy.mockRestore();
+    });
   });
 
   describe('integrations', () => {