From 0ca7a96b1597186484c1886b6c313b60f3ea28f9 Mon Sep 17 00:00:00 2001 From: Konstantin Tsabolov <1006564+tsabolov@users.noreply.github.com> Date: Wed, 22 Nov 2023 10:32:06 +0100 Subject: [PATCH] chore: apply eslint rules to attestation-manager --- apps/attestation-manager/src/common/constants.ts | 2 +- .../src/credentialDef/controller/controller.ts | 3 ++- .../src/issue-credential/controller/controller.ts | 3 ++- .../src/schemas/controller/controller.ts | 3 ++- .../src/schemas/tests/controller.spec.ts | 1 + .../src/userInfo/controller/controller.ts | 4 +++- apps/attestation-manager/src/utils/logger.ts | 13 +------------ .../src/utils/tests/pagination.spec.ts | 2 +- 8 files changed, 13 insertions(+), 18 deletions(-) diff --git a/apps/attestation-manager/src/common/constants.ts b/apps/attestation-manager/src/common/constants.ts index 98b288f..08eedda 100644 --- a/apps/attestation-manager/src/common/constants.ts +++ b/apps/attestation-manager/src/common/constants.ts @@ -1,6 +1,6 @@ export enum LoggerConfig { FILE_PATH = 'logs/log.json', - lOG_DIR = './logs', + LOG_DIR = './logs', } export enum Connection { diff --git a/apps/attestation-manager/src/credentialDef/controller/controller.ts b/apps/attestation-manager/src/credentialDef/controller/controller.ts index c644d0f..d653197 100644 --- a/apps/attestation-manager/src/credentialDef/controller/controller.ts +++ b/apps/attestation-manager/src/credentialDef/controller/controller.ts @@ -1,5 +1,7 @@ import type { ResponseType } from '../../common/response.js'; import type CredentialDefLedgerDto from '../entities/credentialDefLedger-entity.js'; +// eslint-disable-next-line @typescript-eslint/consistent-type-imports +import type { Response } from 'express'; import { Body, @@ -22,7 +24,6 @@ import { ApiTags, } from '@nestjs/swagger'; import { Prisma } from '@prisma/client'; -import { Response } from 'express'; import logger from '../../utils/logger.js'; import CredentialDefDto from '../entities/credentialDef-entity.js'; diff --git a/apps/attestation-manager/src/issue-credential/controller/controller.ts b/apps/attestation-manager/src/issue-credential/controller/controller.ts index c3fb696..a7830f8 100644 --- a/apps/attestation-manager/src/issue-credential/controller/controller.ts +++ b/apps/attestation-manager/src/issue-credential/controller/controller.ts @@ -1,6 +1,8 @@ import type { ResponseType } from '../../common/response.js'; import type CredentialDto from '../entities/credential.entity.js'; import type CredentialStateDto from '../entities/credential.state.entity.js'; +// eslint-disable-next-line @typescript-eslint/consistent-type-imports +import type { Response } from 'express'; import { BadRequestException, @@ -30,7 +32,6 @@ import { ApiTags, } from '@nestjs/swagger'; import { Prisma } from '@prisma/client'; -import { Response } from 'express'; import { Abstraction, diff --git a/apps/attestation-manager/src/schemas/controller/controller.ts b/apps/attestation-manager/src/schemas/controller/controller.ts index 2286104..c24f4ee 100644 --- a/apps/attestation-manager/src/schemas/controller/controller.ts +++ b/apps/attestation-manager/src/schemas/controller/controller.ts @@ -1,4 +1,6 @@ import type { ResponseType } from '../../common/response.js'; +// eslint-disable-next-line @typescript-eslint/consistent-type-imports +import type { Response } from 'express'; import { Body, @@ -21,7 +23,6 @@ import { ApiTags, } from '@nestjs/swagger'; import { Prisma } from '@prisma/client'; -import { Response } from 'express'; import { VersionRegex } from '../../common/constants.js'; import logger from '../../utils/logger.js'; diff --git a/apps/attestation-manager/src/schemas/tests/controller.spec.ts b/apps/attestation-manager/src/schemas/tests/controller.spec.ts index dbda0e8..7093ae8 100644 --- a/apps/attestation-manager/src/schemas/tests/controller.spec.ts +++ b/apps/attestation-manager/src/schemas/tests/controller.spec.ts @@ -1,5 +1,6 @@ import type SchemaDto from '../entities/schema-entity.js'; import type { TestingModule } from '@nestjs/testing'; +import type { Response } from 'express'; import { HttpStatus } from '@nestjs/common'; import { Test } from '@nestjs/testing'; diff --git a/apps/attestation-manager/src/userInfo/controller/controller.ts b/apps/attestation-manager/src/userInfo/controller/controller.ts index 741b6fd..7d30099 100644 --- a/apps/attestation-manager/src/userInfo/controller/controller.ts +++ b/apps/attestation-manager/src/userInfo/controller/controller.ts @@ -1,3 +1,6 @@ +// eslint-disable-next-line @typescript-eslint/consistent-type-imports +import type { Response } from 'express'; + import { BadRequestException, Body, @@ -12,7 +15,6 @@ import { import { ApiBody, ApiOperation, ApiTags } from '@nestjs/swagger'; import { Prisma } from '@prisma/client'; import { isUUID } from 'class-validator'; -import { Response } from 'express'; import { AutoAcceptCredential } from '../../common/constants.js'; import logger from '../../utils/logger.js'; diff --git a/apps/attestation-manager/src/utils/logger.ts b/apps/attestation-manager/src/utils/logger.ts index 218005e..d69767a 100644 --- a/apps/attestation-manager/src/utils/logger.ts +++ b/apps/attestation-manager/src/utils/logger.ts @@ -1,20 +1,9 @@ import { ecsFormat } from '@elastic/ecs-winston-format'; -import { mkdirSync } from 'fs'; import { createLogger, transports, type Logger } from 'winston'; -import { LoggerConfig } from '../common/constants.js'; - -mkdirSync(LoggerConfig.lOG_DIR, { recursive: true }); - const logger: Logger = createLogger({ format: ecsFormat({ convertReqRes: true }), - transports: [ - new transports.Console(), - new transports.File({ - filename: LoggerConfig.FILE_PATH, - }), - // esTransport, - ], + transports: [new transports.Console()], }); logger.on('error', (error) => { diff --git a/apps/attestation-manager/src/utils/tests/pagination.spec.ts b/apps/attestation-manager/src/utils/tests/pagination.spec.ts index d4e7879..2fe06f7 100644 --- a/apps/attestation-manager/src/utils/tests/pagination.spec.ts +++ b/apps/attestation-manager/src/utils/tests/pagination.spec.ts @@ -1,4 +1,4 @@ -import pagination from '../pagination'; +import pagination from '../pagination.js'; describe('Pagination', () => { let pageSize: number; -- GitLab