Skip to content

Commit 58202ec

Browse files
christian-byrneviva-jinyi
authored andcommitted
[tests] Update useServerLogs test after log subscription change
The test was expecting subscribeLogs(false) to be called, but this was commented out in commit 33d6447 to fix logs stopping after the first of multiple queue tasks. Updated test to reflect this temporary change.
1 parent 47c4bd5 commit 58202ec

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

tests-ui/tests/composables/useServerLogs.test.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,10 @@ describe('useServerLogs', () => {
5454
await startListening()
5555
await stopListening()
5656

57-
expect(api.subscribeLogs).toHaveBeenCalledWith(false)
57+
// TODO: Update this test when subscribeLogs(false) is re-enabled
58+
// Currently commented out in useServerLogs to prevent logs from stopping
59+
// after 1st of multiple queue tasks
60+
expect(api.subscribeLogs).toHaveBeenCalledWith(true)
5861
})
5962

6063
it('should register event listener when starting', async () => {

0 commit comments

Comments
 (0)