improve client tests to avoid wait patten

[sibelius]

Story

I'd like to add some tests in our GraphQL subscriptions server to make sure subscriptions are working well, my current problem is that subscribe happens after our pubsub.publish call.

example:

client.subscribe()

pubsub.publish() // this happens before `subscribe` completes

similar to the current test that uses wait(10)
https://github.com/enisdenjo/graphql-transport-ws/blob/f76ac73e9d21c80abe0118007e168e4f5d525036/src/tests/client.ts#L113

does it make sense to emit an event when the subscribe is "ready"

[sibelius]

I've got working like this

const onConnectedPromise = createDeferred();

  client.on('connected', d => {
    onConnectedPromise.resolve();
  });

const disposeSubscription = client.subscribe({
    query,
  }, sink);

  await onConnectedPromise.promise;

I wait the connected event after subscribe call

[enisdenjo]

🎉 This issue has been resolved in version 1.9.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀