Skip to content

Commit daae8a4

Browse files
committed
Handle unexpected message types in response
1 parent ea9e5bf commit daae8a4

1 file changed

Lines changed: 18 additions & 8 deletions

File tree

packages/client/tests/functional/_utils/setupTestSuiteMatrix.ts

Lines changed: 18 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -136,10 +136,15 @@ function setupTestSuiteMatrix(
136136

137137
qpeWorker!.once('message', (response: QpeWorkerResponse) => {
138138
clearTimeout(timeoutId)
139-
if (response.type === 'ready') {
140-
resolve(response)
141-
} else if (response.type === 'error') {
142-
reject(new Error(response.message))
139+
switch (response.type) {
140+
case 'ready':
141+
resolve(response)
142+
break
143+
case 'error':
144+
reject(new Error(response.message))
145+
break
146+
default:
147+
reject(new Error(`Unexpected response type: ${response.type}`))
143148
}
144149
})
145150

@@ -255,10 +260,15 @@ function setupTestSuiteMatrix(
255260

256261
new Promise<void>((resolve, reject) => {
257262
qpeWorker!.once('message', (response: QpeWorkerResponse) => {
258-
if (response.type === 'shutdown-complete') {
259-
resolve()
260-
} else if (response.type === 'error') {
261-
reject(new Error(response.message))
263+
switch (response.type) {
264+
case 'shutdown-complete':
265+
resolve()
266+
break
267+
case 'error':
268+
reject(new Error(response.message))
269+
break
270+
default:
271+
reject(new Error(`Unexpected response type: ${response.type}`))
262272
}
263273
})
264274

0 commit comments

Comments
 (0)