neoris/openvidu
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

tests: fix update room preferences test by ensuring timestamp is included in event payload

Browse Source
This commit is contained in:
juancarmore 2025-08-13 17:10:52 +02:00
parent 5546e92fed
commit 06b2938bf3
1 changed files with 4 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
backend/tests/integration/api/rooms/update-room.test.ts
View File

@ -52,7 +52,6 @@ describe('Room API Tests', () => {
chatPreferences: { enabled: false }, chatPreferences: { enabled: false },
virtualBackgroundPreferences: { enabled: false } virtualBackgroundPreferences: { enabled: false }
}; };
const updateResponse = await updateRoomPreferences(createdRoom.roomId, updatedPreferences); const updateResponse = await updateRoomPreferences(createdRoom.roomId, updatedPreferences);
// Verify a method of frontend event service is called // Verify a method of frontend event service is called
@ -60,7 +59,8 @@ describe('Room API Tests', () => {
createdRoom.roomId, createdRoom.roomId,
{ {
roomId: createdRoom.roomId, roomId: createdRoom.roomId,
preferences: updatedPreferences preferences: updatedPreferences,
timestamp: expect.any(Number)
}, },
{ {
topic: MeetSignalType.MEET_ROOM_PREFERENCES_UPDATED topic: MeetSignalType.MEET_ROOM_PREFERENCES_UPDATED
@ -101,7 +101,6 @@ describe('Room API Tests', () => {
chatPreferences: { enabled: true }, chatPreferences: { enabled: true },
virtualBackgroundPreferences: { enabled: true } virtualBackgroundPreferences: { enabled: true }
}; };
const updateResponse = await updateRoomPreferences(createdRoom.roomId, partialPreferences); const updateResponse = await updateRoomPreferences(createdRoom.roomId, partialPreferences);
// Verify update response // Verify update response
@ -129,7 +128,6 @@ describe('Room API Tests', () => {
// Missing chatPreferences // Missing chatPreferences
virtualBackgroundPreferences: { enabled: false } virtualBackgroundPreferences: { enabled: false }
}; };
const response = await updateRoomPreferences(roomId, invalidPreferences); const response = await updateRoomPreferences(roomId, invalidPreferences);
expect(response.status).toBe(422); expect(response.status).toBe(422);
@ -151,7 +149,6 @@ describe('Room API Tests', () => {
chatPreferences: { enabled: false }, chatPreferences: { enabled: false },
virtualBackgroundPreferences: { enabled: false } virtualBackgroundPreferences: { enabled: false }
}; };
const response = await updateRoomPreferences(createdRoom.roomId, invalidPreferences); const response = await updateRoomPreferences(createdRoom.roomId, invalidPreferences);
expect(response.status).toBe(422); expect(response.status).toBe(422);
@ -165,7 +162,6 @@ describe('Room API Tests', () => {
}); });
const emptyPreferences = {}; const emptyPreferences = {};
const response = await updateRoomPreferences(createdRoom.roomId, emptyPreferences); const response = await updateRoomPreferences(createdRoom.roomId, emptyPreferences);
expect(response.status).toBe(422); expect(response.status).toBe(422);
@ -176,6 +172,7 @@ describe('Room API Tests', () => {
const createdRoom = await createRoom({ const createdRoom = await createRoom({
roomName: 'missing-access' roomName: 'missing-access'
}); });
const invalidPreferences = { const invalidPreferences = {
recordingPreferences: { recordingPreferences: {
enabled: true // Missing allowAccessTo enabled: true // Missing allowAccessTo
@ -184,6 +181,7 @@ describe('Room API Tests', () => {
virtualBackgroundPreferences: { enabled: false } virtualBackgroundPreferences: { enabled: false }
}; };
const response = await updateRoomPreferences(createdRoom.roomId, invalidPreferences); const response = await updateRoomPreferences(createdRoom.roomId, invalidPreferences);
expect(response.status).toBe(422); expect(response.status).toBe(422);
expect(response.body.error).toContain('Unprocessable Entity'); expect(response.body.error).toContain('Unprocessable Entity');
expect(JSON.stringify(response.body.details)).toContain('recordingPreferences.allowAccessTo'); expect(JSON.stringify(response.body.details)).toContain('recordingPreferences.allowAccessTo');
@ -199,7 +197,6 @@ describe('Room API Tests', () => {
chatPreferences: { enabled: false }, chatPreferences: { enabled: false },
virtualBackgroundPreferences: { enabled: false } virtualBackgroundPreferences: { enabled: false }
}; };
const response = await updateRoomPreferences(invalidRoomId, preferences); const response = await updateRoomPreferences(invalidRoomId, preferences);
expect(response.status).toBe(422); expect(response.status).toBe(422);
@ -217,7 +214,6 @@ describe('Room API Tests', () => {
chatPreferences: { enabled: false }, chatPreferences: { enabled: false },
virtualBackgroundPreferences: { enabled: false } virtualBackgroundPreferences: { enabled: false }
}; };
const response = await updateRoomPreferences(nonExistentRoomId, preferences); const response = await updateRoomPreferences(nonExistentRoomId, preferences);
expect(response.status).toBe(404); expect(response.status).toBe(404);