feat: implement SongBrowserService and controller with endpoints for featured, recent, categories, and random songs

Author: tomast1337

apps/backend/src/song-browser/song-browser.controller.ts

@@ -8,18 +8,17 @@ import {
 } from '@nestjs/common';
 import { ApiOperation, ApiTags } from '@nestjs/swagger';
 
-import { SongBrowserService } from './song-browser.service';
-
 @Controller('song-browser')
 @ApiTags('song-browser')
 @ApiTags('song')
 export class SongBrowserController {
-  constructor(public readonly songBrowserService: SongBrowserService) {}
+  constructor /*public readonly songBrowserService: SongBrowserService*/() {}
 
   @Get('/featured')
   @ApiOperation({ summary: 'Get a list of featured songs' })
   public async getFeaturedSongs(): Promise<FeaturedSongsDto> {
-    return await this.songBrowserService.getFeaturedSongs();
+    throw new Error('Unimplemented');
+    //return await this.songBrowserService.getFeaturedSongs();
   }
 
   @Get('/recent')
@@ -29,13 +28,15 @@ export class SongBrowserController {
   public async getSongList(
     @Query() query: PageQueryDTO,
   ): Promise<SongPreviewDto[]> {
-    return await this.songBrowserService.getRecentSongs(query);
+    throw new Error('Unimplemented');
+    //return await this.songBrowserService.getRecentSongs(query);
   }
 
   @Get('/categories')
   @ApiOperation({ summary: 'Get a list of song categories and song counts' })
   public async getCategories(): Promise<Record<string, number>> {
-    return await this.songBrowserService.getCategories();
+    throw new Error('Unimplemented');
+    //return await this.songBrowserService.getCategories();
   }
 
   @Get('/categories/:id')
@@ -44,7 +45,8 @@ export class SongBrowserController {
     @Param('id') id: string,
     @Query() query: PageQueryDTO,
   ): Promise<SongPreviewDto[]> {
-    return await this.songBrowserService.getSongsByCategory(id, query);
+    throw new Error('Unimplemented');
+    //return await this.songBrowserService.getSongsByCategory(id, query);
   }
 
   @Get('/random')
@@ -59,6 +61,7 @@ export class SongBrowserController {
       throw new BadRequestException('Invalid query parameters');
     }
 
-    return await this.songBrowserService.getRandomSongs(countInt, category);
+    throw new Error('Unimplemented');
+    //return await this.songBrowserService.getRandomSongs(countInt, category);
   }
 }

apps/backend/src/song-browser/song-browser.module.ts

@@ -3,10 +3,9 @@ import { Module } from '@nestjs/common';
 import { SongModule } from '@server/song/song.module';
 
 import { SongBrowserController } from './song-browser.controller';
-import { SongBrowserService } from './song-browser.service';
 
 @Module({
-  providers: [SongBrowserService],
+  providers: [],
   controllers: [SongBrowserController],
   imports: [SongModule],
 })

apps/backend/src/song-browser/song-browser.service.spec.ts renamed to apps/backend/src/song/song-browser/song-browser.service.spec.ts

@@ -2,7 +2,7 @@ import { PageQueryDTO, SongPreviewDto, SongWithUser } from '@nbw/database';
 import { HttpException } from '@nestjs/common';
 import { Test, TestingModule } from '@nestjs/testing';
 
-import { SongService } from '@server/song/song.service';
+import { SongService } from '../song.service';
 
 import { SongBrowserService } from './song-browser.service';
 

apps/backend/src/song-browser/song-browser.service.ts renamed to apps/backend/src/song/song-browser/song-browser.service.ts

@@ -8,7 +8,9 @@ import {
   UploadSongResponseDto,
 } from '@nbw/database';
 import type { RawBodyRequest } from '@nestjs/common';
+import { SongBrowserService } from './song-browser/song-browser.service';
 import {
+  BadRequestException,
   Body,
   Controller,
   Delete,
@@ -64,6 +66,7 @@ export class SongController {
   constructor(
     public readonly songService: SongService,
     public readonly fileService: FileService,
+    public readonly songBrowserService: SongBrowserService,
   ) {}
 
   @Get('/')
@@ -72,7 +75,37 @@ export class SongController {
   })
   public async getSongList(
     @Query() query: PageQueryDTO,
-  ): Promise<SongPreviewDto[]> {
+    @Query('q') q?: 'featured' | 'recent' | 'categories' | 'random',
+    @Param('id') id?: string,
+    @Query('count') count?: string,
+    @Query('category') category?: string,
+  ): Promise<SongPreviewDto[] | Record<string, number>> {
+    if (q) {
+      switch (q) {
+        case 'featured':
+          return await this.songBrowserService.getRecentSongs(query);
+        case 'recent':
+          return await this.songBrowserService.getRecentSongs(query);
+        case 'categories':
+          if (id) {
+            return await this.songBrowserService.getSongsByCategory(id, query);
+          }
+          return await this.songBrowserService.getCategories();
+        case 'random': {
+          const countInt = parseInt(count);
+          if (isNaN(countInt) || countInt < 1 || countInt > 10) {
+            throw new BadRequestException('Invalid query parameters');
+          }
+          return await this.songBrowserService.getRandomSongs(
+            countInt,
+            category,
+          );
+        }
+        default:
+          throw new BadRequestException('Invalid query parameters');
+      }
+    }
+
     return await this.songService.getSongByPage(query);
   }