Fix error assertions

src/evaluator.ts

@@ -75,7 +75,7 @@ export type ExpressionErrorResponse = ErrorResponse & {
 }
 
 export class ExpressionError extends EvaluationError<ExpressionErrorResponse> {
-    constructor(response: ExpressionErrorResponse) {
+    public constructor(response: ExpressionErrorResponse) {
         super(response);
 
         Object.setPrototypeOf(this, ExpressionError.prototype);

test/channel/guaranteedChannel.test.ts

@@ -54,7 +54,7 @@ describe('A guaranteed channel', () => {
         // Invalid acknowledge stamp
         sandboxChannel.notify('pong_stamp');
 
-        await expect(promise).rejects.toThrowError('Maximum confirmation time reached.');
+        await expect(promise).rejects.toThrow('Maximum confirmation time reached.');
     });
 
     test('should stop waiting for confirmation if the channel is closed in the meanwhile', async () => {

test/channel/queuedChannel.test.ts

@@ -101,7 +101,7 @@ describe('A queued channel', () => {
         };
         const channel = new QueuedChannel(outputChannel, new CapacityRestrictedQueue(new InMemoryQueue('foo'), 1));
 
-        await expect(channel.publish('bar')).rejects.toThrowError(new Error('The queue is full.'));
+        await expect(channel.publish('bar')).rejects.toThrow('The queue is full.');
     });
 
     test('should fail to publish messages if the channel is closed', async () => {

test/channel/socketChannel.test.ts

@@ -32,7 +32,7 @@ describe('A socket channel', () => {
 
         await channel.close();
 
-        await expect(channel.publish('bar')).rejects.toThrowError('Channel has been closed.');
+        await expect(channel.publish('bar')).rejects.toThrow('Channel has been closed.');
     });
 
     test('should fail to publish messages if an error occurs in the meanwhile', async () => {
@@ -187,7 +187,7 @@ describe('A socket channel', () => {
 
         const channel = new SocketChannel({url: url, connectionTimeout: 100});
 
-        await expect(channel.publish('timeout')).rejects.toThrowError('Maximum connection timeout reached.');
+        await expect(channel.publish('timeout')).rejects.toThrow('Maximum connection timeout reached.');
 
         expect(close).toHaveBeenCalledWith(1000, 'Maximum connection timeout reached.');
     });
@@ -199,7 +199,7 @@ describe('A socket channel', () => {
         await channel.publish('open connection');
         await server.connected;
 
-        await expect(channel.close()).rejects.toThrowError('Maximum close timeout reached.');
+        await expect(channel.close()).rejects.toThrow('Maximum close timeout reached.');
     });
 
     test('should close connection with error', async () => {
@@ -222,7 +222,7 @@ describe('A socket channel', () => {
 
         server.error();
 
-        await expect(channel.publish('foo')).rejects.toThrowError('Connection has been closed, reason');
+        await expect(channel.publish('foo')).rejects.toThrow('Connection has been closed, reason');
         await expect(logger.error).toHaveBeenCalledWith('Connection error.');
     });
 });

test/evaluator.test.ts

@@ -169,7 +169,8 @@ describe('An evaluator', () => {
 
         const promise = evaluator.evaluate(expression);
 
-        await expect(promise).rejects.toThrowError(new EvaluationError(response));
+        await expect(promise).rejects.toThrow(EvaluationError);
+        await expect(promise).rejects.toEqual(expect.objectContaining({response: response}));
     });
 
     test.each([
@@ -216,7 +217,8 @@ describe('An evaluator', () => {
 
             const promise = evaluator.evaluate(expression);
 
-            await expect(promise).rejects.toThrowError(new ExpressionError(response));
+            await expect(promise).rejects.toThrow(ExpressionError);
+            await expect(promise).rejects.toEqual(expect.objectContaining({response: response}));
         },
     );
 
@@ -280,7 +282,8 @@ describe('An evaluator', () => {
 
         const promise = evaluator.evaluate(expression);
 
-        await expect(promise).rejects.toThrowError(new EvaluationError(response));
+        await expect(promise).rejects.toThrow(EvaluationError);
+        await expect(promise).rejects.toEqual(expect.objectContaining({response: response}));
     });
 });
 

test/schemas/contextSchemas.test.ts

@@ -10,7 +10,7 @@ describe('The token schema', () => {
             tokenScopeSchema.validate(value);
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test('should not allow values other than the defined', () => {

test/schemas/ecommerceSchemas.test.ts

@@ -51,7 +51,7 @@ describe('The product details schema', () => {
             productDetails.validate(value);
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([
@@ -157,7 +157,7 @@ describe('The cart item schema', () => {
             cartItem.validate(value);
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([
@@ -304,7 +304,7 @@ describe('The cart schema', () => {
             cart.validate(value);
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([
@@ -508,7 +508,7 @@ describe('The order item schema', () => {
             orderItem.validate(value);
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([
@@ -680,7 +680,7 @@ describe('The order schema', () => {
             order.validate(value);
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([

test/schemas/evaluationSchemas.test.ts

@@ -18,7 +18,7 @@ describe('The evaluation option schema', () => {
             optionsSchema.validate(value);
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([

test/schemas/eventSchemas.test.ts

@@ -49,7 +49,7 @@ describe('The "cartModified" payload schema', () => {
 
     test('should not allow %s', () => {
         function validate(): void {
-            expect((): void => cartModified.validate({})).not.toThrowError(Error);
+            expect((): void => cartModified.validate({})).not.toThrow(Error);
         }
 
         expect(validate).toThrow(Error);

test/schemas/operationSchemas.test.ts

@@ -44,7 +44,7 @@ describe('An add operation schema', () => {
             addOperation.validate({path: 'foo', value: value});
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([
@@ -98,7 +98,7 @@ describe('A set operation schema', () => {
             setOperation.validate({path: 'foo', value: value});
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([
@@ -152,7 +152,7 @@ describe('A combine operation schema', () => {
             combineOperation.validate({path: 'foo', value: value});
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([
@@ -201,7 +201,7 @@ describe('A merge operation schema', () => {
             mergeOperation.validate({path: 'foo', value: value});
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([
@@ -251,7 +251,7 @@ describe('An increment operation schema', () => {
             incrementOperation.validate({path: 'foo', value: value});
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([
@@ -290,7 +290,7 @@ describe('A decrement operation schema', () => {
             decrementOperation.validate({path: 'foo', value: value});
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([

test/schemas/sdkFacadeSchemas.test.ts

@@ -23,7 +23,7 @@ describe('The SDK facade configuration schema', () => {
             configurationSchema.validate(value);
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([

test/schemas/sdkSchemas.test.ts

@@ -16,7 +16,7 @@ describe('The event metadata schema', () => {
             eventMetadataSchema.validate(value);
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([
@@ -91,7 +91,7 @@ describe('The SDK configuration schema', () => {
             configurationSchema.validate(value);
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([

test/schemas/tokenSchema.test.ts

@@ -36,7 +36,7 @@ describe('The token schema', () => {
             tokenSchema.validate(value);
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([

test/schemas/userSchema.test.ts

@@ -98,7 +98,7 @@ describe('The user profile schema', () => {
             userProfileSchema.validate(value);
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([

test/sdk.test.ts

@@ -41,10 +41,10 @@ describe('A SDK', () => {
 
     test('should validate the specified configuration', () => {
         expect(() => Sdk.init('' as unknown as Configuration))
-            .toThrowError('The configuration must be a key-value map.');
+            .toThrow('The configuration must be a key-value map.');
 
         expect(() => Sdk.init({} as Configuration))
-            .toThrowError('Invalid configuration');
+            .toThrow('Invalid configuration');
     });
 
     test('should be initialized with the specified app ID', () => {

test/tracker.test.ts

@@ -215,7 +215,7 @@ describe('A tracker', () => {
             sinceTime: 0,
         };
 
-        await expect(tracker.track(event)).rejects.toThrowError('The tracker is suspended.');
+        await expect(tracker.track(event)).rejects.toThrow('The tracker is suspended.');
 
         tracker.unsuspend();
 

test/validation/arrayType.test.ts

@@ -28,7 +28,7 @@ describe('An array type', () => {
             type.validate(value);
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([

test/validation/booleanType.test.ts

@@ -21,8 +21,8 @@ describe('A boolean type', () => {
     test('should allow boolean values', () => {
         const type = new BooleanType();
 
-        expect((): void => type.validate(true)).not.toThrowError(Error);
-        expect((): void => type.validate(false)).not.toThrowError(Error);
+        expect((): void => type.validate(true)).not.toThrow(Error);
+        expect((): void => type.validate(false)).not.toThrow(Error);
     });
 
     test.each([

test/validation/jsonType.test.ts

@@ -27,7 +27,7 @@ describe('A JSON array type', () => {
             schema.validate(value);
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([
@@ -81,7 +81,7 @@ describe('A JSON object type', () => {
             schema.validate(value);
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([
@@ -137,7 +137,7 @@ describe('A JSON primitive type', () => {
             new JsonPrimitiveType().validate(value);
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([
@@ -189,7 +189,7 @@ describe('A JSON value type', () => {
             new JsonType().validate(value);
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([

test/validation/nullType.test.ts

@@ -21,7 +21,7 @@ describe('A null type', () => {
     test('should allow a null value', () => {
         const type = new NullType();
 
-        expect((): void => type.validate(null)).not.toThrowError(Error);
+        expect((): void => type.validate(null)).not.toThrow(Error);
     });
 
     test.each([

test/validation/numberType.test.ts

@@ -28,7 +28,7 @@ describe('A number type', () => {
             type.validate(value);
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([

test/validation/objectType.test.ts

@@ -51,7 +51,7 @@ describe('An object type', () => {
             type.validate(value);
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([

test/validation/stringType.test.ts

@@ -46,7 +46,7 @@ describe('A string type', () => {
             new StringType({format: format}).validate(value);
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([
@@ -101,7 +101,7 @@ describe('A string type', () => {
             type.validate(value);
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([

test/validation/unionType.test.ts

@@ -29,7 +29,7 @@ describe('An union type', () => {
             new UnionType(new StringType(), new NumberType()).validate(value);
         }
 
-        expect(validate).not.toThrowError(Error);
+        expect(validate).not.toThrow(Error);
     });
 
     test.each([