test: updated test cases for file and song services

Author: tomast1337

server/src/file/file.service.spec.ts

@@ -77,7 +77,6 @@ describe('FileService', () => {
         .mockResolvedValueOnce({}); // Mock for the second bucket
 
       await fileService['verifyBucket']();
-      console.log((s3Client.send as jest.Mock).mock.calls);
 
       // Ensure the mock was called twice
       expect(s3Client.send).toHaveBeenCalledTimes(4);

server/src/song/song.service.spec.ts

@@ -644,31 +644,14 @@ describe('SongService', () => {
 
     it('should throw an error if song is not found', async () => {
       const publicId = 'test-id';
-      const user: UserDocument = { _id: 'test-user-id' } as UserDocument;
 
-      const mockFindOne = {
-        populate: jest.fn().mockResolvedValue(null),
-      };
-
-      jest.spyOn(songModel, 'findOne').mockReturnValue(mockFindOne as any);
+      const user: UserDocument = {
+        _id: 'test-user-id',
+      } as unknown as UserDocument;
 
-      await expect(service.getSong(publicId, user)).rejects.toThrow(
-        HttpException,
-      );
-    });
+      const mockFindOne = null;
 
-    it('should throw an error if song is private and user is unauthorized', async () => {
-      const publicId = 'test-id';
-      const user: UserDocument = { _id: 'test-user-id' } as UserDocument;
-
-      const songEntity = {
-        visibility: 'private',
-        uploader: 'different-user-id',
-      };
-
-      jest.spyOn(songModel, 'findOne').mockReturnValue({
-        populate: jest.fn().mockResolvedValue(songEntity),
-      } as any);
+      jest.spyOn(songModel, 'findOne').mockReturnValue(mockFindOne as any);
 
       await expect(service.getSong(publicId, user)).rejects.toThrow(
         HttpException,
@@ -680,30 +663,24 @@ describe('SongService', () => {
       const user: UserDocument = { _id: 'test-user-id' } as UserDocument;
 
       const songEntity = {
+        publicId: 'test-public-id',
         visibility: 'private',
         uploader: 'different-user-id',
       };
 
-      jest.spyOn(songModel, 'findOne').mockReturnValue({
-        populate: jest.fn().mockResolvedValue(songEntity),
-      } as any);
+      jest.spyOn(songModel, 'findOne').mockReturnValue(songEntity as any);
 
-      await expect(service.getSong(publicId, user)).rejects.toThrow(
-        HttpException,
-      );
+      expect(service.getSong(publicId, user)).rejects.toThrow(HttpException);
     });
 
     it('should throw an error if song is private and user is not logged in', async () => {
       const publicId = 'test-id';
       const user: UserDocument = null as any;
 
-      const songEntity = {
-        visibility: 'private',
-        uploader: 'different-user-id',
-      };
-
       jest.spyOn(songModel, 'findOne').mockReturnValue({
-        populate: jest.fn().mockResolvedValue(songEntity),
+        populate: jest.fn().mockImplementationOnce(() => {
+          throw new HttpException('Song not found', 404);
+        }),
       } as any);
 
       await expect(service.getSong(publicId, user)).rejects.toThrow(