refactor(): update to the latest version of prettier

.prettierrc

@@ -1,4 +1,5 @@
 {
   "singleQuote": true,
+  "arrowParens": "avoid",
   "trailingComma": "all"
 }

integration/hello-world/e2e/exceptions.spec.ts

@@ -18,10 +18,7 @@ describe('Error messages', () => {
   });
 
   it(`/GET`, () => {
-    return request(server)
+    return request(server).get('/sync').expect(HttpStatus.BAD_REQUEST).expect({
-      .get('/sync')
-      .expect(HttpStatus.BAD_REQUEST)
-      .expect({
       statusCode: 400,
       error: 'Bad Request',
       message: 'Integration test',
@@ -29,10 +26,7 @@ describe('Error messages', () => {
   });
 
   it(`/GET (Promise/async)`, () => {
-    return request(server)
+    return request(server).get('/async').expect(HttpStatus.BAD_REQUEST).expect({
-      .get('/async')
-      .expect(HttpStatus.BAD_REQUEST)
-      .expect({
       statusCode: 400,
       error: 'Bad Request',
       message: 'Integration test',

integration/hello-world/e2e/exclude-middleware-fastify.spec.ts

@@ -78,16 +78,11 @@ describe('Exclude middleware (fastify)', () => {
     ).createNestApplication<NestFastifyApplication>(new FastifyAdapter());
 
     await app.init();
-    await app
+    await app.getHttpAdapter().getInstance().ready();
-      .getHttpAdapter()
-      .getInstance()
-      .ready();
   });
 
   it(`should exclude "/test" endpoint`, () => {
-    return request(app.getHttpServer())
+    return request(app.getHttpServer()).get('/test').expect(200, RETURN_VALUE);
-      .get('/test')
-      .expect(200, RETURN_VALUE);
   });
 
   it(`should not exclude "/test2" endpoint`, () => {

integration/hello-world/e2e/exclude-middleware.spec.ts

@@ -77,9 +77,7 @@ describe('Exclude middleware', () => {
   });
 
   it(`should exclude "/test" endpoint`, () => {
-    return request(app.getHttpServer())
+    return request(app.getHttpServer()).get('/test').expect(200, RETURN_VALUE);
-      .get('/test')
-      .expect(200, RETURN_VALUE);
   });
 
   it(`should not exclude "/test2" endpoint`, () => {

integration/hello-world/e2e/express-instance.spec.ts

@@ -20,10 +20,7 @@ describe('Hello world (express instance)', () => {
   });
 
   it(`/GET`, () => {
-    return request(server)
+    return request(server).get('/hello').expect(200).expect('Hello world!');
-      .get('/hello')
-      .expect(200)
-      .expect('Hello world!');
   });
 
   it(`/GET (Promise/async)`, () => {
@@ -41,10 +38,7 @@ describe('Hello world (express instance)', () => {
   });
 
   it(`/GET { host: ":tenant.example.com" } not matched`, () => {
-    return request(server)
+    return request(server).get('/host').expect(404).expect({
-      .get('/host')
-      .expect(404)
-      .expect({
       statusCode: 404,
       error: 'Not Found',
       message: 'Cannot GET /host',

integration/hello-world/e2e/express-multiple.spec.ts

@@ -29,10 +29,7 @@ describe('Hello world (express instance with multiple applications)', () => {
   });
 
   it(`/GET`, () => {
-    return request(server)
+    return request(server).get('/hello').expect(200).expect('Hello world!');
-      .get('/hello')
-      .expect(200)
-      .expect('Hello world!');
   });
 
   it(`/GET (app2)`, () => {

integration/hello-world/e2e/guards.spec.ts

@@ -37,8 +37,6 @@ describe('Guards', () => {
     app = (await createTestModule(new AuthGuard())).createNestApplication();
 
     await app.init();
-    return request(app.getHttpServer())
+    return request(app.getHttpServer()).get('/hello').expect(401);
-      .get('/hello')
-      .expect(401);
   });
 });

integration/hello-world/e2e/interceptors.spec.ts

@@ -77,9 +77,7 @@ describe('Interceptors', () => {
     ).createNestApplication();
 
     await app.init();
-    return request(app.getHttpServer())
+    return request(app.getHttpServer()).get('/hello').expect(200, RETURN_VALUE);
-      .get('/hello')
-      .expect(200, RETURN_VALUE);
   });
 
   it(`should map response`, async () => {

integration/hello-world/e2e/local-pipes.spec.ts

@@ -18,10 +18,7 @@ describe('Hello world (default adapter)', () => {
   });
 
   it(`host=example.com should execute locally injected pipe by HelloController`, () => {
-    return request(server)
+    return request(server).get('/hello/local-pipe/1').expect(200).expect({
-      .get('/hello/local-pipe/1')
-      .expect(200)
-      .expect({
       id: '1',
     });
   });
@@ -39,10 +36,7 @@ describe('Hello world (default adapter)', () => {
   });
 
   it(`should return 404 for mismatched host`, () => {
-    return request(server)
+    return request(server).get('/host/local-pipe/1').expect(404).expect({
-      .get('/host/local-pipe/1')
-      .expect(404)
-      .expect({
       error: 'Not Found',
       message: 'Cannot GET /host/local-pipe/1',
       statusCode: 404,

integration/hello-world/e2e/middleware-class.spec.ts

@@ -51,9 +51,7 @@ describe('Middleware (class)', () => {
   });
 
   it(`forRoutes(*)`, () => {
-    return request(app.getHttpServer())
+    return request(app.getHttpServer()).get('/hello').expect(200, RETURN_VALUE);
-      .get('/hello')
-      .expect(200, RETURN_VALUE);
   });
 
   afterEach(async () => {

integration/hello-world/e2e/middleware.spec.ts

@@ -57,15 +57,11 @@ describe('Middleware', () => {
   });
 
   it(`forRoutes(*)`, () => {
-    return request(app.getHttpServer())
+    return request(app.getHttpServer()).get('/hello').expect(200, RETURN_VALUE);
-      .get('/hello')
-      .expect(200, RETURN_VALUE);
   });
 
   it(`forRoutes(TestController)`, () => {
-    return request(app.getHttpServer())
+    return request(app.getHttpServer()).get('/test').expect(200, SCOPED_VALUE);
-      .get('/test')
-      .expect(200, SCOPED_VALUE);
   });
 
   it(`forRoutes(tests/*)`, () => {

integration/microservices/e2e/broadcast-mqtt.spec.ts

@@ -33,9 +33,7 @@ describe('MQTT transport', () => {
   });
 
   it(`Broadcast (2 subscribers)`, () => {
-    return request(server)
+    return request(server).get('/broadcast').expect(200, '2');
-      .get('/broadcast')
-      .expect(200, '2');
   });
 
   afterEach(async () => {

integration/microservices/e2e/broadcast-nats.spec.ts

@@ -33,9 +33,7 @@ describe('NATS transport', () => {
   });
 
   it(`Broadcast (2 subscribers)`, () => {
-    return request(server)
+    return request(server).get('/broadcast').expect(200, '2');
-      .get('/broadcast')
-      .expect(200, '2');
   });
 
   afterEach(async () => {

integration/microservices/e2e/broadcast-redis.spec.ts

@@ -33,9 +33,7 @@ describe('REDIS transport', () => {
   });
 
   it(`Broadcast (2 subscribers)`, () => {
-    return request(server)
+    return request(server).get('/broadcast').expect(200, '2');
-      .get('/broadcast')
-      .expect(200, '2');
   });
 
   afterEach(async () => {

integration/microservices/e2e/orders-grpc.spec.ts

@@ -129,11 +129,7 @@ describe('Advanced GRPC transport', () => {
     callHandler.on('error', (err: any) => {
       // We want to fail only on real errors while Cancellation error
       // is expected
-      if (
+      if (String(err).toLowerCase().indexOf('cancelled') === -1) {
-        String(err)
-          .toLowerCase()
-          .indexOf('cancelled') === -1
-      ) {
         fail('gRPC Stream error happened, error: ' + err);
       }
     });
@@ -165,11 +161,7 @@ describe('Advanced GRPC transport', () => {
     callHandler.on('error', (err: any) => {
       // We want to fail only on real errors while Cancellation error
       // is expected
-      if (
+      if (String(err).toLowerCase().indexOf('cancelled') === -1) {
-        String(err)
-          .toLowerCase()
-          .indexOf('cancelled') === -1
-      ) {
         fail('gRPC Stream error happened, error: ' + err);
       }
     });

integration/microservices/e2e/sum-grpc.spec.ts

@@ -63,11 +63,7 @@ describe('GRPC transport', () => {
     callHandler.on('error', (err: any) => {
       // We want to fail only on real errors while Cancellation error
       // is expected
-      if (
+      if (String(err).toLowerCase().indexOf('cancelled') === -1) {
-        String(err)
-          .toLowerCase()
-          .indexOf('cancelled') === -1
-      ) {
         fail('gRPC Stream error happened, error: ' + err);
       }
     });
@@ -89,11 +85,7 @@ describe('GRPC transport', () => {
     callHandler.on('error', (err: any) => {
       // We want to fail only on real errors while Cancellation error
       // is expected
-      if (
+      if (String(err).toLowerCase().indexOf('cancelled') === -1) {
-        String(err)
-          .toLowerCase()
-          .indexOf('cancelled') === -1
-      ) {
         fail('gRPC Stream error happened, error: ' + err);
       }
     });

integration/microservices/e2e/sum-kafka.spec.ts

@@ -106,17 +106,11 @@ describe('Kafka transport', () => {
     user: newUser,
   };
   it(`/POST (sync command create user)`, () => {
-    return request(server)
+    return request(server).post('/user').send(userDto).expect(200);
-      .post('/user')
-      .send(userDto)
-      .expect(200);
   });
 
   it(`/POST (sync command create business`, () => {
-    return request(server)
+    return request(server).post('/business').send(businessDto).expect(200);
-      .post('/business')
-      .send(businessDto)
-      .expect(200);
   });
 
   it(`/POST (sync command create user) Concurrency Test`, async () => {
@@ -124,12 +118,7 @@ describe('Kafka transport', () => {
     for (let concurrencyKey = 0; concurrencyKey < 100; concurrencyKey++) {
       const innerUserDto = JSON.parse(JSON.stringify(userDto));
       innerUserDto.name += `+${concurrencyKey}`;
-      promises.push(
+      promises.push(request(server).post('/user').send(userDto).expect(200));
-        request(server)
-          .post('/user')
-          .send(userDto)
-          .expect(200),
-      );
     }
     await Promise.all(promises);
   });

integration/microservices/e2e/sum-nats.spec.ts

@@ -75,9 +75,7 @@ describe('NATS transport', () => {
   });
 
   it(`/GET (exception)`, () => {
-    return request(server)
+    return request(server).get('/exception').expect(200, {
-      .get('/exception')
-      .expect(200, {
       message: 'test',
       status: 'error',
     });

integration/microservices/e2e/sum-rpc.spec.ts

@@ -76,9 +76,7 @@ describe('RPC transport', () => {
   });
 
   it(`/POST (pattern not found)`, () => {
-    return request(server)
+    return request(server).post('/?command=test').expect(500);
-      .post('/?command=test')
-      .expect(500);
   });
 
   it(`/POST (event notification)`, done => {

packages/core/errors/exceptions/unknown-element.exception.ts

@@ -3,8 +3,9 @@ import { RuntimeException } from './runtime.exception';
 export class UnknownElementException extends RuntimeException {
   constructor(name?: string) {
     super(
-      `Nest could not find ${name ||
+      `Nest could not find ${
-        'given'} element (this provider does not exist in the current context)`,
+        name || 'given'
+      } element (this provider does not exist in the current context)`,
     );
   }
 }

packages/core/errors/messages.ts

@@ -109,8 +109,9 @@ export const INVALID_CLASS_SCOPE_MESSAGE = (
   text: TemplateStringsArray,
   name: string | undefined,
 ) =>
-  `${name ||
+  `${
-    'This class'} is marked as a scoped provider. Request and transient-scoped providers can't be used in combination with "get()" method. Please, use "resolve()" instead.`;
+    name || 'This class'
+  } is marked as a scoped provider. Request and transient-scoped providers can't be used in combination with "get()" method. Please, use "resolve()" instead.`;
 
 export const INVALID_MIDDLEWARE_CONFIGURATION = `An invalid middleware configuration has been passed inside the module 'configure()' method.`;
 export const UNKNOWN_REQUEST_MAPPING = `An invalid controller has been detected. Perhaps, one of your controllers is missing @Controller() decorator.`;

packages/core/nest-application-context.ts

@@ -138,12 +138,7 @@ export class NestApplicationContext implements INestApplicationContext {
     }
 
     signals = signals
-      .map((signal: string) =>
+      .map((signal: string) => signal.toString().toUpperCase().trim())
-        signal
-          .toString()
-          .toUpperCase()
-          .trim(),
-      )
       // filter out the signals which is already listening to
       .filter(signal => !this.activeShutdownSignals.includes(signal));
 

packages/core/scanner.ts

@@ -313,7 +313,9 @@ export class DependenciesScanner {
     if (!providersKeys.includes(type as string)) {
       return this.container.addProvider(provider as any, token);
     }
-    const providerToken = `${type as string} (UUID: ${randomStringGenerator()})`;
+    const providerToken = `${
+      type as string
+    } (UUID: ${randomStringGenerator()})`;
 
     let scope = (provider as ClassProvider | FactoryProvider).scope;
     if (isNil(scope) && (provider as ClassProvider).useClass) {

packages/microservices/server/server-grpc.ts

@@ -240,9 +240,7 @@ export class ServerGrpc extends Server implements CustomTransportStrategy {
       call.on('data', (m: any) => req.next(m));
       call.on('error', (e: any) => {
         // Check if error means that stream ended on other end
-        const isCancelledError = String(e)
+        const isCancelledError = String(e).toLowerCase().indexOf('cancelled');
-          .toLowerCase()
-          .indexOf('cancelled');
 
         if (isCancelledError) {
           call.end();

sample/01-cats-app/e2e/cats/cats.e2e-spec.ts

@@ -23,10 +23,7 @@ describe('Cats', () => {
   });
 
   it(`/GET cats`, () => {
-    return request(app.getHttpServer())
+    return request(app.getHttpServer()).get('/cats').expect(200).expect({
-      .get('/cats')
-      .expect(200)
-      .expect({
       data: catsService.findAll(),
     });
   });