test: Fix fingerprint tests for core (#1745)

* test: Fix fingerprint tests for core

* test: Fix unit tests with new tslib build

Author: kamilogorek

CHANGELOG.md

@@ -4,7 +4,7 @@
 
 ## 4.3.1
 
-- [browser] ref: Breadcrumbs will no be logged only to a max object depth of 2
+- [browser] ref: Breadcrumbs will now be logged only to a max object depth of 2
 - [core] feat: Filter internal Sentry errors from transports/sdk
 - [core] ref: Better fingerprint handling
 - [node] ref: Expose Parsers functions

packages/browser/test/karma/karma.unit.config.js

@@ -14,8 +14,15 @@ module.exports = config => {
     karmaTypescriptConfig: {
       tsconfig: 'tsconfig.json',
       compilerOptions: {
-        declaration: false,
         allowJs: true,
+        declaration: false,
+        paths: {
+          '@sentry/utils/*': ['../../utils/src/*'],
+          '@sentry/core': ['../../core/src'],
+          '@sentry/hub': ['../../hub/src'],
+          '@sentry/types': ['../../types/src'],
+          '@sentry/minimal': ['../../minimal/src'],
+        },
       },
       bundlerOptions: {
         sourceMap: true,

packages/core/test/lib/base.test.ts

@@ -174,11 +174,10 @@ describe('BaseClient', () => {
           values: [
             {
               type: 'Error',
-              value: 'random error',
+              value: 'test exception',
             },
           ],
         },
-        message: 'Error: test exception',
       });
     });
 
@@ -188,6 +187,7 @@ describe('BaseClient', () => {
       await client.captureMessage('test message', undefined, undefined, scope);
       expect(TestBackend.instance!.event).toEqual({
         event_id: '42',
+        fingerprint: ['test message'],
         message: 'test message',
       });
     });
@@ -215,6 +215,7 @@ describe('BaseClient', () => {
       expect(TestBackend.instance!.event!.message).toBe('message');
       expect(TestBackend.instance!.event).toEqual({
         event_id: '42',
+        fingerprint: ['message'],
         message: 'message',
       });
     });
@@ -229,6 +230,7 @@ describe('BaseClient', () => {
       expect(TestBackend.instance!.event!).toEqual({
         environment: 'env',
         event_id: '42',
+        fingerprint: ['message'],
         message: 'message',
       });
     });
@@ -242,6 +244,7 @@ describe('BaseClient', () => {
       await client.captureEvent({ message: 'message' }, undefined, scope);
       expect(TestBackend.instance!.event!).toEqual({
         event_id: '42',
+        fingerprint: ['message'],
         message: 'message',
         release: 'v1.0.0',
       });
@@ -255,6 +258,7 @@ describe('BaseClient', () => {
       expect(TestBackend.instance!.event!).toEqual({
         breadcrumbs: [{ message: 'breadcrumb' }],
         event_id: '42',
+        fingerprint: ['message'],
         message: 'message',
       });
     });
@@ -268,6 +272,7 @@ describe('BaseClient', () => {
       expect(TestBackend.instance!.event!).toEqual({
         breadcrumbs: [{ message: '2' }],
         event_id: '42',
+        fingerprint: ['message'],
         message: 'message',
       });
     });
@@ -282,6 +287,7 @@ describe('BaseClient', () => {
       expect(TestBackend.instance!.event!).toEqual({
         event_id: '42',
         extra: { b: 'b' },
+        fingerprint: ['message'],
         message: 'message',
         tags: { a: 'a' },
         user: { id: 'user' },

packages/core/test/mocks/backend.ts

@@ -29,12 +29,11 @@ export class TestBackend extends BaseBackend<TestOptions> {
       exception: {
         values: [
           {
-            type: 'Error',
-            value: 'random error',
+            type: exception.name,
+            value: exception.message,
           },
         ],
       },
-      message: String(exception),
     };
   }