polish(shared): Clean up browser.ts

Author: dschom

libs/shared/sentry-browser/src/lib/browser.ts

@@ -35,6 +35,7 @@ export function captureException(err: Error) {
         );
       }
     });
+
     Sentry.captureException(err);
   });
 }

libs/shared/sentry/src/lib/browser.spec.ts

@@ -56,7 +56,8 @@ describe('sentry/browser', () => {
     it('works without request url', () => {
       const data = {
         key: 'value',
-      } as Sentry.Event;
+        type: undefined,
+      } as Sentry.ErrorEvent;
 
       const resultData = sentryWrapper.beforeSend(config, data, {});
 
@@ -65,13 +66,14 @@ describe('sentry/browser', () => {
 
     it('fingerprints errno', () => {
       const data = {
+        type: undefined,
         request: {
           url: 'https://example.com',
         },
         tags: {
           errno: '100',
         },
-      } as Sentry.Event;
+      } as Sentry.ErrorEvent;
 
       const resultData = sentryWrapper.beforeSend(config, data, {});
       expect(resultData?.fingerprint?.[0]).toEqual('errno100');
@@ -100,7 +102,7 @@ describe('sentry/browser', () => {
 
     it('will return null from before send when disabled', () => {
       sentryWrapper.disable();
-      expect(sentryWrapper.beforeSend({}, {}, {})).toBeNull();
+      expect(sentryWrapper.beforeSend({}, { type: undefined }, {})).toBeNull();
     });
   });
 });

libs/shared/sentry/src/lib/browser.ts

@@ -34,7 +34,7 @@ export function captureException(err: Error) {
         );
       }
     });
-    _Sentry.captureException(err);
+    Sentry.captureException(err);
   });
 }
 
@@ -67,7 +67,7 @@ export function enable() {
  */
 export function beforeSend(
   opts: SentryConfigOpts,
-  event: Sentry.Event,
+  event: Sentry.ErrorEvent,
   _hint?: Sentry.EventHint
 ) {
   if (sentryEnabled === false) {