style(eslint): apply @trilon/eslint-plugin rules

.eslintrc.cjs

@@ -84,6 +84,7 @@ module.exports = {
     '@typescript-eslint/no-unsafe-argument': 'off',
     '@typescript-eslint/no-unsafe-assignment': 'off',
     '@typescript-eslint/no-unsafe-member-access': 'off',
+    '@trilon/detect-circular-reference': 'off',
     'nestjs/use-validation-pipe': 'off',
     'import/order': [
       'warn',
@@ -105,6 +106,8 @@ module.exports = {
         allowList: {
           param: true,
           params: true,
+          ref: true,
+          Ref: true,
           Param: true,
           Params: true,
           args: true,

src/common/adapters/albums.adapter.spec.ts

@@ -1,4 +1,4 @@
-import { Test } from '@nestjs/testing'
+import { Test, TestingModule } from '@nestjs/testing'
 
 import { AlbumsAdapter } from './albums.adapter'
 import { TracksAdapter } from './tracks.adapter'
@@ -19,14 +19,19 @@ import {
 } from '@common/mocks'
 
 describe('AlbumsAdapter', () => {
+  let moduleRef: TestingModule
   let albumsAdapter: AlbumsAdapter
 
   beforeEach(async () => {
-    const module = await Test.createTestingModule({
+    moduleRef = await Test.createTestingModule({
       providers: [TracksAdapter, ArtistsAdapter, AlbumsAdapter, PageAdapter],
     }).compile()
 
-    albumsAdapter = module.get(AlbumsAdapter)
+    albumsAdapter = moduleRef.get(AlbumsAdapter)
+  })
+
+  afterEach(() => {
+    moduleRef.close()
   })
 
   test('should be defined', () => {

src/common/adapters/artists.adapter.spec.ts

@@ -1,4 +1,4 @@
-import { Test } from '@nestjs/testing'
+import { Test, TestingModule } from '@nestjs/testing'
 
 import { ArtistsAdapter } from './artists.adapter'
 import { PageAdapter } from './page.adapter'
@@ -12,14 +12,19 @@ import {
 } from '@common/mocks'
 
 describe('ArtistsAdapter', () => {
+  let moduleRef: TestingModule
   let artistsAdapter: ArtistsAdapter
 
   beforeEach(async () => {
-    const module = await Test.createTestingModule({
+    moduleRef = await Test.createTestingModule({
       providers: [ArtistsAdapter, PageAdapter],
     }).compile()
 
-    artistsAdapter = module.get(ArtistsAdapter)
+    artistsAdapter = moduleRef.get(ArtistsAdapter)
+  })
+
+  afterEach(() => {
+    moduleRef.close()
   })
 
   test('should be defined', () => {

src/common/adapters/audio-features.adapter.spec.ts

@@ -1,18 +1,23 @@
-import { Test } from '@nestjs/testing'
+import { Test, TestingModule } from '@nestjs/testing'
 
 import { AudioFeaturesAdapter } from './audio-features.adapter'
 
 import { audioFeaturesMock, sdkAudioFeaturesMock } from '@common/mocks'
 
 describe('AudioFeaturesAdapter', () => {
+  let moduleRef: TestingModule
   let audioFeaturesAdapter: AudioFeaturesAdapter
 
   beforeEach(async () => {
-    const module = await Test.createTestingModule({
+    moduleRef = await Test.createTestingModule({
       providers: [AudioFeaturesAdapter],
     }).compile()
 
-    audioFeaturesAdapter = module.get(AudioFeaturesAdapter)
+    audioFeaturesAdapter = moduleRef.get(AudioFeaturesAdapter)
+  })
+
+  afterEach(() => {
+    moduleRef.close()
   })
 
   test('should be defined', () => {

src/common/adapters/devices.adapter.spec.ts

@@ -1,18 +1,23 @@
-import { Test } from '@nestjs/testing'
+import { Test, TestingModule } from '@nestjs/testing'
 
 import { DevicesAdapter } from './devices.adapter'
 
 import { deviceMock, sdkDeviceMock } from '@common/mocks'
 
 describe('DevicesAdapter', () => {
+  let moduleRef: TestingModule
   let devicesAdapter: DevicesAdapter
 
   beforeEach(async () => {
-    const module = await Test.createTestingModule({
+    moduleRef = await Test.createTestingModule({
       providers: [DevicesAdapter],
     }).compile()
 
-    devicesAdapter = module.get(DevicesAdapter)
+    devicesAdapter = moduleRef.get(DevicesAdapter)
+  })
+
+  afterEach(() => {
+    moduleRef.close()
   })
 
   test('should be defined', () => {

src/common/adapters/genres.adapter.spec.ts

@@ -1,18 +1,23 @@
-import { Test } from '@nestjs/testing'
+import { Test, TestingModule } from '@nestjs/testing'
 
 import { GenresAdapter } from './genres.adapter'
 
 import { sdkArtistMock, topGenresMock } from '@common/mocks'
 
 describe('GenresAdapter', () => {
+  let moduleRef: TestingModule
   let genresAdapter: GenresAdapter
 
   beforeEach(async () => {
-    const module = await Test.createTestingModule({
+    moduleRef = await Test.createTestingModule({
       providers: [GenresAdapter],
     }).compile()
 
-    genresAdapter = module.get(GenresAdapter)
+    genresAdapter = moduleRef.get(GenresAdapter)
+  })
+
+  afterEach(() => {
+    moduleRef.close()
   })
 
   test('should be defined', () => {

src/common/adapters/page.adapter.spec.ts

@@ -1,21 +1,26 @@
-import { Test } from '@nestjs/testing'
+import { Test, TestingModule } from '@nestjs/testing'
 
 import { PageAdapter } from './page.adapter'
 import { ArtistsAdapter } from './artists.adapter'
 
 import { artistMock, sdkArtistMock, pageMockFactory } from '@common/mocks'
 
 describe('PaginatedAdapter', () => {
+  let moduleRef: TestingModule
   let paginatedAdapter: PageAdapter
   let artistsAdapter: ArtistsAdapter
 
   beforeEach(async () => {
-    const module = await Test.createTestingModule({
+    moduleRef = await Test.createTestingModule({
       providers: [PageAdapter, ArtistsAdapter],
     }).compile()
 
-    paginatedAdapter = module.get(PageAdapter)
-    artistsAdapter = module.get(ArtistsAdapter)
+    paginatedAdapter = moduleRef.get(PageAdapter)
+    artistsAdapter = moduleRef.get(ArtistsAdapter)
+  })
+
+  afterEach(() => {
+    moduleRef.close()
   })
 
   test('should be defined', () => {

src/common/adapters/playback-state.adapter.spec.ts

@@ -1,4 +1,4 @@
-import { Test } from '@nestjs/testing'
+import { Test, TestingModule } from '@nestjs/testing'
 
 import { PlaybackStateAdapter } from './playback-state.adapter'
 import { DevicesAdapter } from './devices.adapter'
@@ -9,10 +9,11 @@ import { ArtistsAdapter } from './artists.adapter'
 import { playbackStateMock, sdkPlaybackStateMock } from '@common/mocks'
 
 describe('PlaybackStateAdapter', () => {
+  let moduleRef: TestingModule
   let playbackStateAdapter: PlaybackStateAdapter
 
   beforeEach(async () => {
-    const module = await Test.createTestingModule({
+    moduleRef = await Test.createTestingModule({
       providers: [
         PlaybackStateAdapter,
         DevicesAdapter,
@@ -22,7 +23,11 @@ describe('PlaybackStateAdapter', () => {
       ],
     }).compile()
 
-    playbackStateAdapter = module.get(PlaybackStateAdapter)
+    playbackStateAdapter = moduleRef.get(PlaybackStateAdapter)
+  })
+
+  afterEach(() => {
+    moduleRef.close()
   })
 
   test('should be defined', () => {

src/common/adapters/profile.adapter.spec.ts

@@ -1,18 +1,23 @@
-import { Test } from '@nestjs/testing'
+import { Test, TestingModule } from '@nestjs/testing'
 
 import { ProfileAdapter } from './profile.adapter'
 
 import { profileMock, sdkProfileMock } from '@common/mocks'
 
 describe('ProfileAdapter', () => {
+  let moduleRef: TestingModule
   let profileAdapter: ProfileAdapter
 
   beforeEach(async () => {
-    const module = await Test.createTestingModule({
+    moduleRef = await Test.createTestingModule({
       providers: [ProfileAdapter],
     }).compile()
 
-    profileAdapter = module.get(ProfileAdapter)
+    profileAdapter = moduleRef.get(ProfileAdapter)
+  })
+
+  afterEach(() => {
+    moduleRef.close()
   })
 
   test('should be defined', () => {

src/common/adapters/secret-data.adapter.spec.ts

@@ -1,18 +1,23 @@
-import { Test } from '@nestjs/testing'
+import { Test, TestingModule } from '@nestjs/testing'
 
 import { SecretDataAdapter } from './secret-data.adapter'
 
 import { accessTokenMock } from '@common/mocks'
 
 describe('SecretDataAdapter', () => {
+  let moduleRef: TestingModule
   let secretDataAdapter: SecretDataAdapter
 
   beforeEach(async () => {
-    const module = await Test.createTestingModule({
+    moduleRef = await Test.createTestingModule({
       providers: [SecretDataAdapter],
     }).compile()
 
-    secretDataAdapter = module.get(SecretDataAdapter)
+    secretDataAdapter = moduleRef.get(SecretDataAdapter)
+  })
+
+  afterEach(() => {
+    moduleRef.close()
   })
 
   test('should be defined', () => {

src/common/adapters/tracks.adapter.spec.ts

@@ -1,4 +1,4 @@
-import { Test } from '@nestjs/testing'
+import { Test, TestingModule } from '@nestjs/testing'
 
 import { TracksAdapter } from './tracks.adapter'
 import { PageAdapter } from './page.adapter'
@@ -18,14 +18,19 @@ import {
 } from '@common/mocks'
 
 describe('TracksAdapter', () => {
+  let moduleRef: TestingModule
   let tracksAdapter: TracksAdapter
 
   beforeEach(async () => {
-    const module = await Test.createTestingModule({
+    moduleRef = await Test.createTestingModule({
       providers: [TracksAdapter, PageAdapter, ArtistsAdapter],
     }).compile()
 
-    tracksAdapter = module.get(TracksAdapter)
+    tracksAdapter = moduleRef.get(TracksAdapter)
+  })
+
+  afterEach(() => {
+    moduleRef.close()
   })
 
   test('should be defined', () => {

src/modules/albums/albums.controller.spec.ts

@@ -1,4 +1,4 @@
-import { Test } from '@nestjs/testing'
+import { Test, TestingModule } from '@nestjs/testing'
 import { NotFoundException } from '@nestjs/common'
 import { paginate } from 'nestjs-typeorm-paginate'
 
@@ -17,11 +17,12 @@ import {
 vi.mock('nestjs-typeorm-paginate')
 
 describe('AlbumsController', () => {
+  let moduleRef: TestingModule
   let albumsController: AlbumsController
   let albumsRepository: AlbumsRepository
 
   beforeEach(async () => {
-    const module = await Test.createTestingModule({
+    moduleRef = await Test.createTestingModule({
       controllers: [AlbumsController],
       providers: [
         {
@@ -33,8 +34,12 @@ describe('AlbumsController', () => {
       ],
     }).compile()
 
-    albumsController = module.get(AlbumsController)
-    albumsRepository = module.get(AlbumsRepository)
+    albumsController = moduleRef.get(AlbumsController)
+    albumsRepository = moduleRef.get(AlbumsRepository)
+  })
+
+  afterEach(() => {
+    moduleRef.close()
   })
 
   test('should be defined', () => {

src/modules/albums/albums.repository.spec.ts

@@ -1,5 +1,5 @@
 import { DataSource, In } from 'typeorm'
-import { Test } from '@nestjs/testing'
+import { Test, TestingModule } from '@nestjs/testing'
 
 import { AlbumsRepository, albumsRelations } from './albums.repository'
 
@@ -15,10 +15,11 @@ import {
 describe('AlbumsRepository', () => {
   const externalId = 'externalId'
 
+  let moduleRef: TestingModule
   let albumsRepository: AlbumsRepository
 
   beforeEach(async () => {
-    const module = await Test.createTestingModule({
+    moduleRef = await Test.createTestingModule({
       providers: [
         AlbumsRepository,
         {
@@ -30,7 +31,11 @@ describe('AlbumsRepository', () => {
       ],
     }).compile()
 
-    albumsRepository = module.get(AlbumsRepository)
+    albumsRepository = moduleRef.get(AlbumsRepository)
+  })
+
+  afterEach(() => {
+    moduleRef.close()
   })
 
   test('should be defined', () => {