Skip to content

Commit 5e033f5

Browse files
committed
fix(test): remove test asserting removed 10-song limit on random sort
The 10-song limit was removed in cd218df but the test was not updated, causing CI failures on PR #83.
1 parent 34d834c commit 5e033f5

1 file changed

Lines changed: 1 addition & 17 deletions

File tree

apps/backend/src/song/song.controller.spec.ts

Lines changed: 1 addition & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,7 @@ import {
1010
SongSortType,
1111
FeaturedSongsDto,
1212
} from '@nbw/database';
13-
import {
14-
BadRequestException,
15-
HttpStatus,
16-
UnauthorizedException,
17-
} from '@nestjs/common';
13+
import { HttpStatus, UnauthorizedException } from '@nestjs/common';
1814
import { AuthGuard } from '@nestjs/passport';
1915
import { Test, TestingModule } from '@nestjs/testing';
2016
import { Response } from 'express';
@@ -153,18 +149,6 @@ describe('SongController', () => {
153149
expect(songService.getRandomSongs).toHaveBeenCalledWith(5, 'electronic');
154150
});
155151

156-
it('should throw error for invalid random limit', async () => {
157-
const query: SongListQueryDTO = {
158-
page: 1,
159-
limit: 15,
160-
sort: SongSortType.RANDOM,
161-
};
162-
163-
await expect(songController.getSongList(query)).rejects.toThrow(
164-
BadRequestException,
165-
);
166-
});
167-
168152
it('should handle recent sort', async () => {
169153
const query: SongListQueryDTO = {
170154
page: 1,

0 commit comments

Comments
 (0)