From f09d866ab9672b99c6379733d1530a6952317a48 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Miroslav=20Bajto=C5=A1?= Date: Thu, 13 Dec 2018 17:32:50 +0100 Subject: [PATCH] chore: fix tslint violations The rule "no-unused-variable" is messing up with TypeScript compiler state and as a result, other rules don't detect certain violations. In this commit, I am fixing few violations I discovered when I have temporarily disabled no-unused-variable rule. --- .../repository/test/acceptance/has-one.relation.acceptance.ts | 2 +- packages/rest/src/rest.server.ts | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/packages/repository/test/acceptance/has-one.relation.acceptance.ts b/packages/repository/test/acceptance/has-one.relation.acceptance.ts index a242ecc1cef4..18313c6d324e 100644 --- a/packages/repository/test/acceptance/has-one.relation.acceptance.ts +++ b/packages/repository/test/acceptance/has-one.relation.acceptance.ts @@ -55,7 +55,7 @@ describe('hasOne relation', () => { const address = await controller.createCustomerAddress(existingCustomerId, { street: '123 test avenue', }); - expect( + await expect( controller.createCustomerAddress(existingCustomerId, { street: '456 test street', }), diff --git a/packages/rest/src/rest.server.ts b/packages/rest/src/rest.server.ts index f83bb747d596..564cdf9d6ce4 100644 --- a/packages/rest/src/rest.server.ts +++ b/packages/rest/src/rest.server.ts @@ -465,7 +465,8 @@ export class RestServer extends Context implements Server, HttpServerLike { if (config.disabled) { debug('Redirect to swagger-ui was disabled by configuration.'); - return next(); + next(); + return; } debug('Redirecting to swagger-ui from %j.', request.originalUrl);