From ff44a6567e9f6606af78c7bc7a1ffc358ea14148 Mon Sep 17 00:00:00 2001 From: Philip Molares Date: Mon, 12 Apr 2021 17:06:59 +0200 Subject: [PATCH] Tests: Removed all eslint-disable lines As we now disable what we don't need in the eslint config, we don't need this anymore. Signed-off-by: Philip Molares --- src/auth/auth.service.spec.ts | 10 ++-------- src/config/utils.spec.ts | 6 ------ src/history/history.service.spec.ts | 6 ------ src/media/media.service.spec.ts | 6 ------ src/notes/notes.service.spec.ts | 6 ------ src/permissions/permissions.service.spec.ts | 6 ------ test/private-api/history.e2e-spec.ts | 5 ----- test/private-api/media.e2e-spec.ts | 5 ----- test/private-api/notes.e2e-spec.ts | 5 ----- test/public-api/me.e2e-spec.ts | 5 ----- test/public-api/media.e2e-spec.ts | 5 ----- test/public-api/notes.e2e-spec.ts | 5 ----- 12 files changed, 2 insertions(+), 68 deletions(-) diff --git a/src/auth/auth.service.spec.ts b/src/auth/auth.service.spec.ts index 0aaa41ae3..7dbb63602 100644 --- a/src/auth/auth.service.spec.ts +++ b/src/auth/auth.service.spec.ts @@ -4,12 +4,6 @@ * SPDX-License-Identifier: AGPL-3.0-only */ -/* eslint-disable -@typescript-eslint/no-unsafe-call, -@typescript-eslint/no-unsafe-member-access, -@typescript-eslint/no-unsafe-return, -@typescript-eslint/require-await */ - import { Test, TestingModule } from '@nestjs/testing'; import { AuthService } from './auth.service'; import { PassportModule } from '@nestjs/passport'; @@ -204,8 +198,8 @@ describe('AuthService', () => { service.validateToken(`${authToken.keyId}`), ).rejects.toBeInstanceOf(TokenNotValidError); }); - it('the secret is too long', () => { - void expect( + it('the secret is too long', async () => { + await expect( service.validateToken(`${authToken.keyId}.${'a'.repeat(73)}`), ).rejects.toBeInstanceOf(TokenNotValidError); }); diff --git a/src/config/utils.spec.ts b/src/config/utils.spec.ts index 72d5196a5..c611c53cf 100644 --- a/src/config/utils.spec.ts +++ b/src/config/utils.spec.ts @@ -4,12 +4,6 @@ * SPDX-License-Identifier: AGPL-3.0-only */ -/* eslint-disable -@typescript-eslint/no-unsafe-call, -@typescript-eslint/no-unsafe-member-access, -@typescript-eslint/no-unsafe-return, -@typescript-eslint/require-await */ - import { replaceAuthErrorsWithEnvironmentVariables, toArrayConfig, diff --git a/src/history/history.service.spec.ts b/src/history/history.service.spec.ts index 985215f99..e77f33fbd 100644 --- a/src/history/history.service.spec.ts +++ b/src/history/history.service.spec.ts @@ -4,12 +4,6 @@ * SPDX-License-Identifier: AGPL-3.0-only */ -/* eslint-disable -@typescript-eslint/no-unsafe-call, -@typescript-eslint/no-unsafe-member-access, -@typescript-eslint/no-unsafe-return, -@typescript-eslint/require-await */ - import { Test, TestingModule } from '@nestjs/testing'; import { LoggerModule } from '../logger/logger.module'; import { HistoryService } from './history.service'; diff --git a/src/media/media.service.spec.ts b/src/media/media.service.spec.ts index 13197b078..ba9ab9d16 100644 --- a/src/media/media.service.spec.ts +++ b/src/media/media.service.spec.ts @@ -4,12 +4,6 @@ * SPDX-License-Identifier: AGPL-3.0-only */ -/* eslint-disable -@typescript-eslint/no-unsafe-call, -@typescript-eslint/no-unsafe-member-access, -@typescript-eslint/no-unsafe-return, -@typescript-eslint/require-await */ - import { ConfigModule } from '@nestjs/config'; import { Test, TestingModule } from '@nestjs/testing'; import { getRepositoryToken } from '@nestjs/typeorm'; diff --git a/src/notes/notes.service.spec.ts b/src/notes/notes.service.spec.ts index edae9cd84..31849c15c 100644 --- a/src/notes/notes.service.spec.ts +++ b/src/notes/notes.service.spec.ts @@ -4,12 +4,6 @@ * SPDX-License-Identifier: AGPL-3.0-only */ -/* eslint-disable @typescript-eslint/require-await */ -/* eslint-disable -@typescript-eslint/no-unsafe-assignment, -@typescript-eslint/no-unsafe-member-access -*/ - import { Test, TestingModule } from '@nestjs/testing'; import { getRepositoryToken } from '@nestjs/typeorm'; import { LoggerModule } from '../logger/logger.module'; diff --git a/src/permissions/permissions.service.spec.ts b/src/permissions/permissions.service.spec.ts index d6861c5ff..112b0f9b7 100644 --- a/src/permissions/permissions.service.spec.ts +++ b/src/permissions/permissions.service.spec.ts @@ -4,12 +4,6 @@ * SPDX-License-Identifier: AGPL-3.0-only */ -/* eslint-disable -@typescript-eslint/no-unsafe-call, -@typescript-eslint/no-unsafe-member-access, -@typescript-eslint/no-unsafe-return, -@typescript-eslint/require-await */ - import { Test, TestingModule } from '@nestjs/testing'; import { getRepositoryToken } from '@nestjs/typeorm'; import { AuthToken } from '../auth/auth-token.entity'; diff --git a/test/private-api/history.e2e-spec.ts b/test/private-api/history.e2e-spec.ts index c7c81f2dd..74aee331a 100644 --- a/test/private-api/history.e2e-spec.ts +++ b/test/private-api/history.e2e-spec.ts @@ -4,11 +4,6 @@ * SPDX-License-Identifier: AGPL-3.0-only */ -/* eslint-disable -@typescript-eslint/no-unsafe-assignment, -@typescript-eslint/no-unsafe-member-access -*/ - import { INestApplication } from '@nestjs/common'; import { ConfigModule } from '@nestjs/config'; import { Test } from '@nestjs/testing'; diff --git a/test/private-api/media.e2e-spec.ts b/test/private-api/media.e2e-spec.ts index c6c6e17ce..a7da82323 100644 --- a/test/private-api/media.e2e-spec.ts +++ b/test/private-api/media.e2e-spec.ts @@ -4,11 +4,6 @@ * SPDX-License-Identifier: AGPL-3.0-only */ -/* eslint-disable -@typescript-eslint/no-unsafe-assignment, -@typescript-eslint/no-unsafe-member-access -*/ - import { ConfigModule, ConfigService } from '@nestjs/config'; import { NestExpressApplication } from '@nestjs/platform-express'; import { Test } from '@nestjs/testing'; diff --git a/test/private-api/notes.e2e-spec.ts b/test/private-api/notes.e2e-spec.ts index 0b6b8f6f0..e8fc2fa3c 100644 --- a/test/private-api/notes.e2e-spec.ts +++ b/test/private-api/notes.e2e-spec.ts @@ -4,11 +4,6 @@ * SPDX-License-Identifier: AGPL-3.0-only */ -/* eslint-disable -@typescript-eslint/no-unsafe-assignment, -@typescript-eslint/no-unsafe-member-access -*/ - import { INestApplication } from '@nestjs/common'; import { ConfigModule, ConfigService } from '@nestjs/config'; import { Test } from '@nestjs/testing'; diff --git a/test/public-api/me.e2e-spec.ts b/test/public-api/me.e2e-spec.ts index 76a3161c9..62d8e6413 100644 --- a/test/public-api/me.e2e-spec.ts +++ b/test/public-api/me.e2e-spec.ts @@ -4,11 +4,6 @@ * SPDX-License-Identifier: AGPL-3.0-only */ -/* eslint-disable -@typescript-eslint/no-unsafe-assignment, -@typescript-eslint/no-unsafe-member-access -*/ - import { INestApplication } from '@nestjs/common'; import { Test } from '@nestjs/testing'; import * as request from 'supertest'; diff --git a/test/public-api/media.e2e-spec.ts b/test/public-api/media.e2e-spec.ts index 9ae4379d8..7e21216f2 100644 --- a/test/public-api/media.e2e-spec.ts +++ b/test/public-api/media.e2e-spec.ts @@ -4,11 +4,6 @@ * SPDX-License-Identifier: AGPL-3.0-only */ -/* eslint-disable -@typescript-eslint/no-unsafe-assignment, -@typescript-eslint/no-unsafe-member-access -*/ - import { ConfigModule, ConfigService } from '@nestjs/config'; import { NestExpressApplication } from '@nestjs/platform-express'; import { Test } from '@nestjs/testing'; diff --git a/test/public-api/notes.e2e-spec.ts b/test/public-api/notes.e2e-spec.ts index b7f2e2e2c..dcabd4dd5 100644 --- a/test/public-api/notes.e2e-spec.ts +++ b/test/public-api/notes.e2e-spec.ts @@ -4,11 +4,6 @@ * SPDX-License-Identifier: AGPL-3.0-only */ -/* eslint-disable -@typescript-eslint/no-unsafe-assignment, -@typescript-eslint/no-unsafe-member-access -*/ - import { INestApplication } from '@nestjs/common'; import { ConfigModule, ConfigService } from '@nestjs/config'; import { Test } from '@nestjs/testing';