Skip to content

Commit

Permalink
Fix integration.test.ts used deprecated callback for RealtimeChannel.…
Browse files Browse the repository at this point in the history
…whenState()

Callback API for RealtimeChannel.whenState() was removed in ably-js v2 [1]

[1] ably/ably-js@2a2ed49
  • Loading branch information
VeskeR committed Jul 17, 2024
1 parent 3d46b66 commit 8a70ad8
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions test/integration/integration.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -187,7 +187,7 @@ describe(
it('scenario 2.2: set cursor position', async () => {
// Before calling `performerSpace.cursors.set()` below, we want to be sure that `performerSpace.cursors` has found out about the presence enter operations triggered by calling `performerSpace.cursors.set()` in scenario 2.1, and by calling `observerSpace.cursors.subscribe()` below, so that it doesn’t drop the cursor updates that we pass to `set()`. So here we set up a promise which will resolve once `performerSpace.cursors.channel` sees two clients (i.e. `performerSpaces` and `observerSpaces`) as present.
const performerCursorsChannelObserverPresentPromise = new Promise<void>((resolve) => {
const presence = performerSpace.cursors.channel.presence;
const presence = performerSpace.cursors.channel!.presence;
const listener = async () => {
const members = await presence.get();
if (members.length === 2) {
Expand Down Expand Up @@ -215,9 +215,7 @@ describe(
// To be sure that the `observerSpace.cursors.subscribe()` listener will receive the cursor positions sent by the calls to `performerSpace.cursors.set()` below, we need to know that the cursors channel attach operation triggered by calling `observerSpace.cursors.subscribe()` has completed. The `cursors.subscribe()` API does not currently provide any direct way for the user to know that this attach operation has completed, so here we do so by directly observing the channel.
//
// We should consider exposing the completion of the attach operation via the `cursors.subscribe()` API, the same way as ably-js exposes it through `presence.subscribe()`. I’m not convinced of the necessity though — not sure how useful it’d be for an average user, and we can work around it in tests (as I have here).
const observerCursorsChannelAttachedPromise = new Promise((resolve) => {
observerSpace.cursors.channel.whenState('attached', resolve);
});
const observerCursorsChannelAttachedPromise = observerSpace.cursors.channel!.whenState('attached');

await Promise.all([performerCursorsChannelObserverPresentPromise, observerCursorsChannelAttachedPromise]);

Expand Down

0 comments on commit 8a70ad8

Please sign in to comment.