Added comment on skipped tests describing reason for skipping them

ably · Jul 5, 2024 · e3eb351 · e3eb351
1 parent ef7be90
commit e3eb351
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/spec/acceptance/realtime/client_spec.rb b/spec/acceptance/realtime/client_spec.rb
@@ -142,6 +142,7 @@
             context 'and an explicit client_id in ClientOptions' do
               let(:client_id) { random_str }
 
+              # Skipped because more clarification needed on RSA7e, see https://github.com/ably/ably-ruby/issues/425
               xit 'allows uses the explicit client_id in the connection' do
                 connection.__incoming_protocol_msgbus__.subscribe(:protocol_message) do |protocol_message|
                   if protocol_message.action == :connected

diff --git a/spec/acceptance/realtime/connection_spec.rb b/spec/acceptance/realtime/connection_spec.rb
@@ -383,6 +383,7 @@ def publish_and_check_disconnect(options = {})
               let(:client_id)      { random_str }
               let(:client_options) { default_options.merge(client_id: 'incompatible', token: token_string, key: nil, log_level: :none) }
 
+              # Skipped because more clarification needed on RSA7e, see https://github.com/ably/ably-ruby/issues/425
               xit 'fails the connection' do
                 expect(client.client_id).to eql('incompatible')
                 client.connection.once(:failed) do