ably / ably-go

Go client library SDK for Ably realtime messaging service
https://ably.com/download
Apache License 2.0
82 stars 31 forks source link

Delete unnecessary recover error test #662

Closed amnonbc closed 1 month ago

amnonbc commented 1 month ago

The test removed in this commit expects that using an invalid recovery key leads to an ERROR protocol message being returned which thus fails the connection, but this isn't part of the spec, we make no guarantee that invalid recovery keys are rejected with an ERROR, and in fact returning a CONNECTED protocol message with the error field indicating the recovery key was invalid is a perfectly good response, which is what we will start to do in the future.

The RTN15c4 test was removed from ably-dotnet for this reason. This PR removes it from ably-go.