test: use different id for song suggestions

src/modules/songs/use-cases/get-song-suggestions/get-song-suggestions.use-case.spec.ts

@@ -10,7 +10,7 @@ describe('GetSongSuggestions', () => {
   })
 
   it('should return suggestions for a song', async () => {
-    const suggestions = await getSongSuggestions.execute({ songId: '8N_6I8Gn', limit: 5 })
+    const suggestions = await getSongSuggestions.execute({ songId: 'P3qdsetQ', limit: 5 })
 
     expect(() => SongModel.parse(suggestions[0])).not.toThrow()
     expect(suggestions.length).toBe(5)

src/modules/songs/use-cases/get-song-suggestions/get-song-suggestions.use-case.ts

@@ -22,6 +22,7 @@ export class GetSongSuggestionsUseCase implements IUseCase<GetSongSuggestionsArg
   async execute({ songId, limit }: GetSongSuggestionsArgs) {
     const stationId = await this.createSongStation.execute(songId)
 
+    console.log({ stationId })
     const { data, ok } = await useFetch<z.infer<typeof SongSuggestionAPIResponseModel>>({
       endpoint: Endpoints.songs.suggestions,
       params: {
@@ -31,19 +32,19 @@ export class GetSongSuggestionsUseCase implements IUseCase<GetSongSuggestionsArg
       context: 'android'
     })
 
+    console.log({ data })
+
     if (!data || !ok) {
       throw new HTTPException(404, { message: `no suggestions found for the given song` })
     }
 
-    // eslint-disable-next-line @typescript-eslint/no-unused-vars
-    const { stationid, ...suggestions } = data // TODO: find a better way to model the response
+    const { stationid, ...suggestions } = data
 
-    const songs =
+    return (
       Object.values(suggestions)
         .map((element) => element && createSongPayload(element.song))
         .filter(Boolean)
         .slice(0, limit) || []
-
-    return songs
+    )
   }
 }