Skip to content

Commit 6c54cb4

Browse files
committed
[DEBUG] Add more logs from both WS transport and cube container in tests
1 parent 2f11d20 commit 6c54cb4

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

.github/workflows/push.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -612,7 +612,7 @@ jobs:
612612
cd packages/cubejs-testing/
613613
export BIRDBOX_CUBEJS_VERSION=${{ matrix.tag }}
614614
export BIRDBOX_CUBEJS_REGISTRY_PATH=localhost:5000/
615-
export DEBUG=testcontainers
615+
export DEBUG=testcontainers*
616616
yarn run dataset:minimal
617617
yarn run birdbox:postgresql
618618
yarn run birdbox:postgresql-pre-aggregations

packages/cubejs-client-ws-transport/src/index.ts

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -90,6 +90,8 @@ class WebSocketTransport implements ITransport<WebSocketTransportResult> {
9090
}
9191

9292
public async close(): Promise<void> {
93+
// TODO sync with timeout in sendMessage: either flush or drop
94+
console.log("close call, left queue", this.messageQueue);
9395
if (this.ws) {
9496
this.ws.close();
9597
}
@@ -191,7 +193,12 @@ class WebSocketTransport implements ITransport<WebSocketTransportResult> {
191193
this.messageQueue.push(message);
192194
}
193195

196+
console.log("sendMessage call, message", message);
197+
console.log("sendMessage call, queue before timeout", this.messageQueue);
198+
194199
setTimeout(async () => {
200+
console.log("sendMessage call, message", message);
201+
console.log("sendMessage call, queue after timeout", this.messageQueue);
195202
await this.initSocket();
196203
this.ws.sendQueue();
197204
}, 100);
@@ -242,6 +249,8 @@ class WebSocketTransport implements ITransport<WebSocketTransportResult> {
242249
return callback(result, () => this.subscribe(callback));
243250
},
244251
async unsubscribe() {
252+
// TODO await unsub from server
253+
console.log("unsubscribe call: ", message.messageId);
245254
transport.sendMessage({ unsubscribe: message.messageId });
246255
delete transport.messageIdToSubscription[message.messageId];
247256
}

0 commit comments

Comments
 (0)