feat(modules): bring back caching

src/modules/history/router/history.controller.spec.ts

@@ -4,6 +4,7 @@ import { getQueueToken } from '@nestjs/bullmq'
 import { DeepMockProxy, mockDeep } from 'vitest-mock-extended'
 import { PaginateQuery } from 'nestjs-paginate'
 import { MockInstance } from 'vitest'
+import { CacheInterceptor } from '@nestjs/cache-manager'
 
 import { HistoryQueueEvents } from '../history.queue-events'
 
@@ -71,7 +72,12 @@ describe('HistoryController', () => {
           },
         },
       ],
-    }).compile()
+    })
+      .overrideInterceptor(CacheInterceptor)
+      .useValue({
+        intercept: vi.fn(),
+      })
+      .compile()
 
     historyController = moduleRef.get(HistoryController)
     historyQueue = moduleRef.get(getQueueToken(HISTORY_QUEUE))

src/modules/history/router/history.controller.ts

@@ -1,4 +1,4 @@
-import { Controller, Get, UseGuards } from '@nestjs/common'
+import { Controller, Get, UseGuards, UseInterceptors } from '@nestjs/common'
 import { ApiOperation, ApiTags } from '@nestjs/swagger'
 import { Queue } from 'bullmq'
 import {
@@ -8,6 +8,7 @@ import {
   PaginatedSwaggerDocs,
   paginate,
 } from 'nestjs-paginate'
+import { CacheInterceptor, CacheTTL } from '@nestjs/cache-manager'
 
 import { HistoryQueueEvents } from '../history.queue-events'
 import { InjectHistoryQueue } from '../decorators'
@@ -30,6 +31,8 @@ export const historyTracksPaginateConfig: PaginateConfig<HistoryTrack> = {
 @Controller('users/:id/history')
 @ApiTags('users/{id}/history')
 @UseGuards(ValidateUserIdGuard)
+@UseInterceptors(CacheInterceptor)
+@CacheTTL(30)
 @ApiAuth()
 export class HistoryController {
   constructor(

src/modules/reports/router/reports.controller.spec.ts

@@ -1,4 +1,5 @@
 import { Test, type TestingModule } from '@nestjs/testing'
+import { CacheInterceptor } from '@nestjs/cache-manager'
 
 import { ReportsService } from '../reports.service'
 
@@ -42,7 +43,12 @@ describe('ReportsController', () => {
           },
         },
       ],
-    }).compile()
+    })
+      .overrideInterceptor(CacheInterceptor)
+      .useValue({
+        intercept: vi.fn(),
+      })
+      .compile()
 
     reportsController = moduleRef.get(ReportsController)
     reportsService = moduleRef.get(ReportsService)

src/modules/reports/router/reports.controller.ts

@@ -1,5 +1,12 @@
-import { Controller, Get, Query, UseGuards } from '@nestjs/common'
+import {
+  Controller,
+  Get,
+  Query,
+  UseGuards,
+  UseInterceptors,
+} from '@nestjs/common'
 import { ApiOkResponse, ApiOperation, ApiTags } from '@nestjs/swagger'
+import { CacheInterceptor } from '@nestjs/cache-manager'
 
 import { ReportsService } from '../reports.service'
 
@@ -28,6 +35,7 @@ import { StatsMeasurement } from '@modules/stats/enums'
 @Controller('/users/:id/reports')
 @ApiTags('users/{id}/reports')
 @UseGuards(ValidateUserIdGuard, TimeRangeGuard)
+@UseInterceptors(CacheInterceptor)
 @ApiAuth()
 @ApiUser()
 export class ReportsController {

src/modules/stats/router/stats-rigtch.controller.spec.ts

@@ -1,5 +1,6 @@
 import { Test, TestingModule } from '@nestjs/testing'
 import { MockInstance } from 'vitest'
+import { CacheInterceptor } from '@nestjs/cache-manager'
 
 import { StatsRigtchService } from '../stats-rigtch.service'
 import { StatsMeasurement } from '../enums'
@@ -38,7 +39,12 @@ describe('StatsRigtchController', () => {
           },
         },
       ],
-    }).compile()
+    })
+      .overrideInterceptor(CacheInterceptor)
+      .useValue({
+        intercept: vi.fn(),
+      })
+      .compile()
 
     statsRigtchController = moduleRef.get(StatsRigtchController)
     statsRigtchService = moduleRef.get(StatsRigtchService)

src/modules/stats/router/stats-rigtch.controller.ts

@@ -1,5 +1,12 @@
-import { Controller, Get, Query, UseGuards } from '@nestjs/common'
+import {
+  Controller,
+  Get,
+  Query,
+  UseGuards,
+  UseInterceptors,
+} from '@nestjs/common'
 import { ApiOkResponse, ApiOperation, ApiTags } from '@nestjs/swagger'
+import { CacheInterceptor } from '@nestjs/cache-manager'
 
 import { StatsMeasurement } from '../enums'
 import { StatsRigtchService } from '../stats-rigtch.service'
@@ -23,6 +30,7 @@ import { ValidateUserIdGuard } from '@modules/users/guards'
 @Controller('/users/:id/stats/rigtch')
 @ApiTags('users/{id}/stats/rigtch')
 @UseGuards(ValidateUserIdGuard, TimeRangeGuard)
+@UseInterceptors(CacheInterceptor)
 @ApiAuth()
 @ApiStatsRigtchQuery()
 @ApiUser()

src/modules/stats/router/stats-spotify.controller.spec.ts

@@ -1,5 +1,6 @@
 import { Test, TestingModule } from '@nestjs/testing'
 import type { MockInstance } from 'vitest'
+import { CacheInterceptor } from '@nestjs/cache-manager'
 
 import { StatsSpotifyController } from './stats-spotify.controller'
 
@@ -60,7 +61,12 @@ describe('UsersProfileController', () => {
           },
         },
       ],
-    }).compile()
+    })
+      .overrideInterceptor(CacheInterceptor)
+      .useValue({
+        intercept: vi.fn(),
+      })
+      .compile()
 
     statsSpotifyController = moduleRef.get(StatsSpotifyController)
     spotifyService = moduleRef.get(SpotifyService)

src/modules/stats/router/stats-spotify.controller.ts

@@ -1,6 +1,13 @@
-import { Controller, Get, Query, UseGuards } from '@nestjs/common'
+import {
+  Controller,
+  Get,
+  Query,
+  UseGuards,
+  UseInterceptors,
+} from '@nestjs/common'
 import { ApiOkResponse, ApiOperation, ApiTags } from '@nestjs/swagger'
 import { AccessToken } from '@spotify/web-api-ts-sdk'
+import { CacheInterceptor } from '@nestjs/cache-manager'
 
 import { ApiAuth, RequestToken } from '@common/decorators'
 import { TokenGuard } from '@common/guards'
@@ -18,6 +25,7 @@ import { TopItemQuery } from '@modules/users/dtos'
 @Controller('/users/:id/stats/spotify')
 @ApiTags('users/{id}/stats/spotify')
 @UseGuards(ValidateUserIdGuard, TokenGuard)
+@UseInterceptors(CacheInterceptor)
 @ApiAuth()
 @ApiUser()
 export class StatsSpotifyController {

src/modules/users/controllers/users-profile.controller.spec.ts

@@ -1,5 +1,6 @@
 import { Test, TestingModule } from '@nestjs/testing'
 import { MockInstance } from 'vitest'
+import { CacheInterceptor } from '@nestjs/cache-manager'
 
 import { UsersRepository } from '../users.repository'
 
@@ -71,7 +72,12 @@ describe('UsersProfileController', () => {
           },
         },
       ],
-    }).compile()
+    })
+      .overrideInterceptor(CacheInterceptor)
+      .useValue({
+        intercept: vi.fn(),
+      })
+      .compile()
 
     usersProfileController = moduleRef.get(UsersProfileController)
     spotifyService = moduleRef.get(SpotifyService)

src/modules/users/controllers/users-profile.controller.ts

@@ -1,6 +1,13 @@
-import { Controller, Get, Query, UseGuards } from '@nestjs/common'
+import {
+  Controller,
+  Get,
+  Query,
+  UseGuards,
+  UseInterceptors,
+} from '@nestjs/common'
 import { ApiOperation, ApiOkResponse, ApiTags } from '@nestjs/swagger'
 import { AccessToken } from '@spotify/web-api-ts-sdk'
+import { CacheInterceptor } from '@nestjs/cache-manager'
 
 import { ApiItemQuery, ApiUser, RequestUser } from '../decorators'
 import { LastItemQuery, TopItemQuery } from '../dtos'
@@ -59,6 +66,7 @@ export class UsersProfileController {
   }
 
   @Get('top/artists')
+  @UseInterceptors(CacheInterceptor)
   @ApiOperation({
     summary: "Getting user's top artists (cached).",
     deprecated: true,
@@ -93,6 +101,7 @@ export class UsersProfileController {
   }
 
   @Get('top/tracks')
+  @UseInterceptors(CacheInterceptor)
   @ApiOperation({
     summary: "Getting user's top tracks (cached).",
     deprecated: true,
@@ -127,6 +136,7 @@ export class UsersProfileController {
   }
 
   @Get('top/genres')
+  @UseInterceptors(CacheInterceptor)
   @ApiOperation({
     summary: "Getting user's top genres (cached).",
     deprecated: true,
@@ -149,6 +159,7 @@ export class UsersProfileController {
   }
 
   @Get('analysis')
+  @UseInterceptors(CacheInterceptor)
   @ApiOperation({
     summary: "Getting user's analysis (cached).",
     deprecated: true,

src/modules/users/controllers/users.controller.spec.ts

@@ -1,5 +1,6 @@
 import { Test, TestingModule } from '@nestjs/testing'
 import { MockInstance } from 'vitest'
+import { CacheInterceptor } from '@nestjs/cache-manager'
 
 import { MeBody } from '../dtos'
 import { UsersRepository } from '../users.repository'
@@ -66,7 +67,12 @@ describe('UsersController', () => {
           },
         },
       ],
-    }).compile()
+    })
+      .overrideInterceptor(CacheInterceptor)
+      .useValue({
+        intercept: vi.fn(),
+      })
+      .compile()
 
     usersController = moduleRef.get(UsersController)
     usersRepository = moduleRef.get(UsersRepository)