cql-rb 1.2.2 → 2.0.0.pre0

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cql-rb
 version: !ruby/object:Gem::Version
-  version: 1.2.2
+  version: 2.0.0.pre0
 platform: ruby
 authors:
 - Theo Hultberg
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-05-17 00:00:00.000000000 Z
+date: 2014-02-13 00:00:00.000000000 Z
 dependencies: []
 description: A pure Ruby CQL3 driver for Cassandra
 email:
@@ -17,18 +17,22 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- .yardopts
 - README.md
 - lib/cql.rb
 - lib/cql/byte_buffer.rb
 - lib/cql/client.rb
 - lib/cql/client/asynchronous_client.rb
 - lib/cql/client/asynchronous_prepared_statement.rb
+- lib/cql/client/authenticators.rb
+- lib/cql/client/batch.rb
 - lib/cql/client/column_metadata.rb
-- lib/cql/client/connection_helper.rb
 - lib/cql/client/connection_manager.rb
+- lib/cql/client/connector.rb
 - lib/cql/client/execute_options_decoder.rb
 - lib/cql/client/keyspace_changer.rb
 - lib/cql/client/null_logger.rb
+- lib/cql/client/peer_discovery.rb
 - lib/cql/client/query_result.rb
 - lib/cql/client/query_trace.rb
 - lib/cql/client/request_runner.rb
@@ -49,6 +53,8 @@ files:
 - lib/cql/protocol/frame_decoder.rb
 - lib/cql/protocol/frame_encoder.rb
 - lib/cql/protocol/request.rb
+- lib/cql/protocol/requests/auth_response_request.rb
+- lib/cql/protocol/requests/batch_request.rb
 - lib/cql/protocol/requests/credentials_request.rb
 - lib/cql/protocol/requests/execute_request.rb
 - lib/cql/protocol/requests/options_request.rb
@@ -57,11 +63,14 @@ files:
 - lib/cql/protocol/requests/register_request.rb
 - lib/cql/protocol/requests/startup_request.rb
 - lib/cql/protocol/response.rb
+- lib/cql/protocol/responses/auth_challenge_response.rb
+- lib/cql/protocol/responses/auth_success_response.rb
 - lib/cql/protocol/responses/authenticate_response.rb
 - lib/cql/protocol/responses/detailed_error_response.rb
 - lib/cql/protocol/responses/error_response.rb
 - lib/cql/protocol/responses/event_response.rb
 - lib/cql/protocol/responses/prepared_result_response.rb
+- lib/cql/protocol/responses/raw_rows_result_response.rb
 - lib/cql/protocol/responses/ready_response.rb
 - lib/cql/protocol/responses/result_response.rb
 - lib/cql/protocol/responses/rows_result_response.rb
@@ -79,10 +88,15 @@ files:
 - spec/cql/byte_buffer_spec.rb
 - spec/cql/client/asynchronous_client_spec.rb
 - spec/cql/client/asynchronous_prepared_statement_spec.rb
+- spec/cql/client/authenticators_spec.rb
+- spec/cql/client/batch_spec.rb
 - spec/cql/client/column_metadata_spec.rb
-- spec/cql/client/connection_helper_spec.rb
 - spec/cql/client/connection_manager_spec.rb
+- spec/cql/client/connector_spec.rb
+- spec/cql/client/execute_options_decoder_spec.rb
 - spec/cql/client/keyspace_changer_spec.rb
+- spec/cql/client/peer_discovery_spec.rb
+- spec/cql/client/query_result_spec.rb
 - spec/cql/client/query_trace_spec.rb
 - spec/cql/client/request_runner_spec.rb
 - spec/cql/client/synchronous_client_spec.rb
@@ -98,6 +112,8 @@ files:
 - spec/cql/protocol/encoding_spec.rb
 - spec/cql/protocol/frame_decoder_spec.rb
 - spec/cql/protocol/frame_encoder_spec.rb
+- spec/cql/protocol/requests/auth_response_request_spec.rb
+- spec/cql/protocol/requests/batch_request_spec.rb
 - spec/cql/protocol/requests/credentials_request_spec.rb
 - spec/cql/protocol/requests/execute_request_spec.rb
 - spec/cql/protocol/requests/options_request_spec.rb
@@ -105,11 +121,14 @@ files:
 - spec/cql/protocol/requests/query_request_spec.rb
 - spec/cql/protocol/requests/register_request_spec.rb
 - spec/cql/protocol/requests/startup_request_spec.rb
+- spec/cql/protocol/responses/auth_challenge_response_spec.rb
+- spec/cql/protocol/responses/auth_success_response_spec.rb
 - spec/cql/protocol/responses/authenticate_response_spec.rb
 - spec/cql/protocol/responses/detailed_error_response_spec.rb
 - spec/cql/protocol/responses/error_response_spec.rb
 - spec/cql/protocol/responses/event_response_spec.rb
 - spec/cql/protocol/responses/prepared_result_response_spec.rb
+- spec/cql/protocol/responses/raw_rows_result_response_spec.rb
 - spec/cql/protocol/responses/ready_response_spec.rb
 - spec/cql/protocol/responses/result_response_spec.rb
 - spec/cql/protocol/responses/rows_result_response_spec.rb
@@ -148,9 +167,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: 1.9.2
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - '>'
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubyforge_project: 
 rubygems_version: 2.2.1
@@ -161,10 +180,15 @@ test_files:
 - spec/cql/byte_buffer_spec.rb
 - spec/cql/client/asynchronous_client_spec.rb
 - spec/cql/client/asynchronous_prepared_statement_spec.rb
+- spec/cql/client/authenticators_spec.rb
+- spec/cql/client/batch_spec.rb
 - spec/cql/client/column_metadata_spec.rb
-- spec/cql/client/connection_helper_spec.rb
 - spec/cql/client/connection_manager_spec.rb
+- spec/cql/client/connector_spec.rb
+- spec/cql/client/execute_options_decoder_spec.rb
 - spec/cql/client/keyspace_changer_spec.rb
+- spec/cql/client/peer_discovery_spec.rb
+- spec/cql/client/query_result_spec.rb
 - spec/cql/client/query_trace_spec.rb
 - spec/cql/client/request_runner_spec.rb
 - spec/cql/client/synchronous_client_spec.rb
@@ -180,6 +204,8 @@ test_files:
 - spec/cql/protocol/encoding_spec.rb
 - spec/cql/protocol/frame_decoder_spec.rb
 - spec/cql/protocol/frame_encoder_spec.rb
+- spec/cql/protocol/requests/auth_response_request_spec.rb
+- spec/cql/protocol/requests/batch_request_spec.rb
 - spec/cql/protocol/requests/credentials_request_spec.rb
 - spec/cql/protocol/requests/execute_request_spec.rb
 - spec/cql/protocol/requests/options_request_spec.rb
@@ -187,11 +213,14 @@ test_files:
 - spec/cql/protocol/requests/query_request_spec.rb
 - spec/cql/protocol/requests/register_request_spec.rb
 - spec/cql/protocol/requests/startup_request_spec.rb
+- spec/cql/protocol/responses/auth_challenge_response_spec.rb
+- spec/cql/protocol/responses/auth_success_response_spec.rb
 - spec/cql/protocol/responses/authenticate_response_spec.rb
 - spec/cql/protocol/responses/detailed_error_response_spec.rb
 - spec/cql/protocol/responses/error_response_spec.rb
 - spec/cql/protocol/responses/event_response_spec.rb
 - spec/cql/protocol/responses/prepared_result_response_spec.rb
+- spec/cql/protocol/responses/raw_rows_result_response_spec.rb
 - spec/cql/protocol/responses/ready_response_spec.rb
 - spec/cql/protocol/responses/result_response_spec.rb
 - spec/cql/protocol/responses/rows_result_response_spec.rb

data/lib/cql/client/connection_helper.rb

@@ -1,181 +0,0 @@
-# encoding: utf-8
-
-module Cql
-  module Client
-    # @private
-    class ConnectionHelper
-      def initialize(io_reactor, port, credentials, connections_per_node, connection_timeout, compressor, logger)
-        @io_reactor = io_reactor
-        @port = port
-        @credentials = credentials
-        @connections_per_node = connections_per_node
-        @connection_timeout = connection_timeout
-        @logger = logger
-        @request_runner = RequestRunner.new
-        @keyspace_changer = KeyspaceChanger.new
-        @compressor = compressor
-      end
-
-      def connect(hosts, initial_keyspace)
-        f = @io_reactor.start.flat_map do
-          connect_to_hosts(hosts, initial_keyspace, true)
-        end
-        f = f.map do |connections|
-          connected_connections = connections.select(&:connected?)
-          if connected_connections.empty?
-            e = connections.first.error
-            if e.is_a?(Cql::QueryError) && e.code == 0x100
-              e = AuthenticationError.new(e.message)
-            end
-            raise e
-          end
-          connected_connections
-        end
-        f
-      end
-
-      def discover_peers(seed_connections, initial_keyspace)
-        @logger.debug('Looking for additional nodes')
-        connection = seed_connections.sample
-        return Future.resolved([]) unless connection
-        request = Protocol::QueryRequest.new('SELECT peer, data_center, host_id, rpc_address FROM system.peers', :one)
-        peer_info = @request_runner.execute(connection, request)
-        peer_info.flat_map do |result|
-          seed_dcs = seed_connections.map { |c| c[:data_center] }.uniq
-          unconnected_peers = result.select do |row|
-            seed_dcs.include?(row['data_center']) && seed_connections.none? { |c| c[:host_id] == row['host_id'] }
-          end
-          if unconnected_peers.empty?
-            @logger.debug('No additional nodes found')
-          else
-            @logger.debug('%d additional nodes found' % unconnected_peers.size)
-          end
-          node_addresses = unconnected_peers.map do |row|
-            rpc_address = row['rpc_address'].to_s
-            if rpc_address == '0.0.0.0'
-              row['peer'].to_s
-            else
-              rpc_address
-            end
-          end
-          if node_addresses.any?
-            connect_to_hosts(node_addresses, initial_keyspace, false)
-          else
-            Future.resolved([])
-          end
-        end
-      end
-
-      private
-
-      def connect_to_hosts(hosts, initial_keyspace, peer_discovery)
-        connection_futures = hosts.flat_map do |host|
-          Array.new(@connections_per_node) do
-            connect_to_host(host, initial_keyspace).recover do |error|
-              FailedConnection.new(error, host, @port)
-            end
-          end
-        end
-        connection_futures.each do |cf|
-          cf.on_value do |c|
-            if c.is_a?(FailedConnection)
-              @logger.warn('Failed connecting to node at %s:%d: %s' % [c.host, c.port, c.error.message])
-            else
-              @logger.info('Connected to node %s at %s:%d in data center %s' % [c[:host_id], c.host, c.port, c[:data_center]])
-            end
-            c.on_closed do
-              @logger.warn('Connection to node %s at %s:%d in data center %s unexpectedly closed' % [c[:host_id], c.host, c.port, c[:data_center]])
-            end
-          end
-        end
-        hosts_connected_future = Future.all(*connection_futures)
-        if peer_discovery
-          hosts_connected_future.flat_map do |connections|
-            discover_peers(connections.select(&:connected?), initial_keyspace).map do |peer_connections|
-              connections + peer_connections
-            end
-          end
-        else
-          hosts_connected_future
-        end
-      end
-
-      def connect_to_host(host, keyspace)
-        if @compressor
-          @logger.debug('Connecting to node at %s:%d using "%s" compression' % [host, @port, @compressor.algorithm])
-        else
-          @logger.debug('Connecting to node at %s:%d' % [host, @port])
-        end
-        connected = @io_reactor.connect(host, @port, @connection_timeout)
-        connected.flat_map do |connection|
-          f = cache_supported_options(connection)
-          f = f.flat_map { send_startup_request(connection) }
-          f = f.flat_map { |response| maybe_authenticate(response, connection) }
-          f = f.flat_map { identify_node(connection) }
-          f = f.flat_map { change_keyspace(keyspace, connection) }
-          f
-        end
-      end
-
-      def cache_supported_options(connection)
-        f = @request_runner.execute(connection, Protocol::OptionsRequest.new)
-        f.on_value do |supported_options|
-          connection[:cql_version] = supported_options['CQL_VERSION']
-          connection[:compression] = supported_options['COMPRESSION']
-        end
-        f
-      end
-
-      def send_startup_request(connection)
-        compression = @compressor && @compressor.algorithm
-        if @compressor && !connection[:compression].include?(@compressor.algorithm)
-          @logger.warn(%[Compression algorithm "#{@compressor.algorithm}" not supported (server supports "#{connection[:compression].join('", "')}")])
-          compression = nil
-        end
-        request = Protocol::StartupRequest.new(nil, compression)
-        @request_runner.execute(connection, request)
-      end
-
-      def maybe_authenticate(response, connection)
-        return Future.resolved(connection) unless response.is_a?(AuthenticationRequired)
-        return Future.failed(AuthenticationError.new('Server requested authentication, but no credentials given')) unless @credentials
-        send_credentials(@credentials, connection)
-      end
-
-      def send_credentials(credentials, connection)
-        credentials_request = Protocol::CredentialsRequest.new(credentials)
-        @request_runner.execute(connection, credentials_request)
-      end
-
-      def identify_node(connection)
-        request = Protocol::QueryRequest.new('SELECT data_center, host_id FROM system.local', :one)
-        f = @request_runner.execute(connection, request)
-        f.on_value do |result|
-          unless result.empty?
-            connection[:host_id] = result.first['host_id']
-            connection[:data_center] = result.first['data_center']
-          end
-        end
-        f
-      end
-
-      def change_keyspace(keyspace, connection)
-        @keyspace_changer.use_keyspace(keyspace, connection)
-      end
-
-      class FailedConnection
-        attr_reader :error, :host, :port
-
-        def initialize(error, host, port)
-          @error = error
-          @host = host
-          @port = port
-        end
-
-        def connected?
-          false
-        end
-      end
-    end
-  end
-end

data/spec/cql/client/connection_helper_spec.rb

@@ -1,429 +0,0 @@
-# encoding: utf-8
-
-require 'spec_helper'
-
-
-module Cql
-  module Client
-    describe ConnectionHelper do
-      let :connection_helper do
-        described_class.new(io_reactor, 9876, nil, 1, 7, compressor, logger)
-      end
-
-      let :io_reactor do
-        double(:io_reactor)
-      end
-
-      let :logger do
-        NullLogger.new
-      end
-
-      let :compressor do
-        nil
-      end
-
-      describe '#connect' do
-        let :hosts do
-          %w[host0 host1]
-        end
-
-        let :local_metadata do
-          [
-            ['system', 'local', 'data_center', :text],
-            ['system', 'local', 'host_id', :uuid],
-          ]
-        end
-
-        before do
-          io_reactor.stub(:start).and_return(Future.resolved)
-          io_reactor.stub(:connect).and_return(Future.resolved)
-        end
-
-        it 'starts the IO reactor' do
-          connection_helper.connect(hosts, nil)
-          io_reactor.should have_received(:start)
-        end
-
-        it 'fails when the IO reactor fails to start' do
-          io_reactor.stub(:start).and_return(Future.failed(StandardError.new('bork')))
-          f = connection_helper.connect(hosts, nil)
-          expect { f.value }.to raise_error('bork')
-        end
-
-        it 'connects to the specified hosts' do
-          connection_helper.connect(hosts, nil)
-          io_reactor.should have_received(:connect).with('host0', 9876, 7)
-          io_reactor.should have_received(:connect).with('host1', 9876, 7)
-        end
-
-        it 'logs a message when a node connects' do
-          logger.stub(:info)
-          io_reactor.stub(:connect).and_return(Future.resolved(FakeConnection.new('host', 9876, 7)))
-          connection_helper.connect(hosts, nil)
-          logger.should have_received(:info).with(/Connected to node/).exactly(hosts.size).times
-        end
-
-        it 'logs a message when connecting to a node' do
-          logger.stub(:debug)
-          io_reactor.stub(:connect).and_return(Future.resolved(FakeConnection.new('host', 9876, 7)))
-          connection_helper.connect(hosts, nil)
-          logger.should have_received(:debug).with(/Connecting to node/).exactly(hosts.size).times
-        end
-
-        it 'fails when all hosts fail to connect' do
-          io_reactor.stub(:connect).and_return(Future.failed(StandardError.new('bork')))
-          f = connection_helper.connect(hosts, nil)
-          expect { f.value }.to raise_error('bork')
-        end
-
-        it 'logs a message when a node fails to connect' do
-          logger.stub(:warn)
-          io_reactor.stub(:connect).and_return(Future.failed(StandardError.new('bork')))
-          connection_helper.connect(hosts, nil)
-          logger.should have_received(:warn).with(/Failed connecting to node/).exactly(hosts.size).times
-        end
-
-        it 'fails with an AuthenticationError when the connections fail to connect because of authentication issues' do
-          io_reactor.stub(:connect).and_return(Future.failed(QueryError.new(0x100, 'bork')))
-          f = connection_helper.connect(hosts, nil)
-          expect { f.value }.to raise_error(AuthenticationError)
-        end
-
-        it 'initializes the connections' do
-          connection0 = FakeConnection.new('host0', 9876, 7)
-          connection1 = FakeConnection.new('host1', 9876, 7)
-          io_reactor.stub(:connect).with('host0', 9876, 7).and_return(Future.resolved(connection0))
-          io_reactor.stub(:connect).with('host1', 9876, 7).and_return(Future.resolved(connection1))
-          connection_helper.connect(hosts, 'some_keyspace')
-          [connection0, connection1].each do |c|
-            c.requests[0].should be_a(Protocol::OptionsRequest)
-            c.requests[1].should be_a(Protocol::StartupRequest)
-            c.requests[2].cql.should match(/SELECT .* FROM system.local/)
-            c.requests[3].cql.should == 'USE some_keyspace'
-          end
-        end
-
-        it 'saves the supported CQL version and compression algorithms on the connection' do
-          connection = FakeConnection.new('host0', 9876, 7)
-          connection.handle_request do |request, timeout|
-            if request.is_a?(Protocol::OptionsRequest)
-              Protocol::SupportedResponse.new('CQL_VERSION' => %w[3.1.1], 'COMPRESSION' => %w[lz4 snappy])
-            else
-              connection.default_request_handler(request, timeout)
-            end
-          end
-          io_reactor.stub(:connect).with('host0', 9876, 7).and_return(Future.resolved(connection))
-          connection_helper.connect(hosts.take(1), 'some_keyspace')
-          connection[:cql_version].should == %w[3.1.1]
-          connection[:compression].should == %w[lz4 snappy]
-        end
-
-        it 'fails if authentication is required and no credentials were specified' do
-          connection = FakeConnection.new('host0', 9876, 7)
-          connection.handle_request do |request|
-            if request.is_a?(Protocol::StartupRequest)
-              Protocol::AuthenticateResponse.new('xyz')
-            else
-              connection.default_request_handler(request)
-            end
-          end
-          io_reactor.stub(:connect).with('host0', 9876, 7).and_return(Future.resolved(connection))
-          f = connection_helper.connect(hosts, nil)
-          expect { f.value }.to raise_error(AuthenticationError)
-        end
-
-        it 'authenticates when authentication is required and credentials were specified' do
-          credentials = {'username' => 'foo', 'password' => 'bar'}
-          connection_helper = described_class.new(io_reactor, 9876, credentials, 1, 7, nil, logger)
-          connection = FakeConnection.new('host0', 9876, 7)
-          authentication_sent = false
-          connection.handle_request do |request|
-            if request.is_a?(Protocol::StartupRequest)
-              Protocol::AuthenticateResponse.new('xyz')
-            elsif request == Protocol::CredentialsRequest.new(credentials)
-              authentication_sent = true
-              Protocol::ReadyResponse.new
-            else
-              connection.default_request_handler(request)
-            end
-          end
-          io_reactor.stub(:connect).with('host0', 9876, 7).and_return(Future.resolved(connection))
-          f = connection_helper.connect(hosts, nil)
-          f.value
-          authentication_sent.should be_true
-        end
-
-        it 'decorates the connections with :host_id and :data_center' do
-          connection = FakeConnection.new('host0', 9876, 7)
-          connection.handle_request do |request|
-            if request.is_a?(Protocol::QueryRequest) && request.cql =~ /SELECT .* FROM system\.local/
-              row = {'data_center' => 'dc1', 'host_id' => Uuid.new('eac69196-1e28-11e3-8e2b-191b6d153d0c')}
-              Protocol::RowsResultResponse.new([row], local_metadata, nil)
-            else
-              connection.default_request_handler(request)
-            end
-          end
-          io_reactor.stub(:connect).with('host0', 9876, 7).and_return(Future.resolved(connection))
-          connection_helper.connect(hosts, nil)
-          connection[:host_id].should == Uuid.new('eac69196-1e28-11e3-8e2b-191b6d153d0c')
-          connection[:data_center].should == 'dc1'
-        end
-
-        it 'registers a close handler that logs when connections close' do
-          logger.stub(:warn)
-          connection = FakeConnection.new('host', 9876, 7)
-          io_reactor.stub(:connect).and_return(Future.resolved(connection))
-          connection_helper.connect(hosts.take(1), nil)
-          connection.close
-          logger.should have_received(:warn).with(/Connection to node .* closed/)
-        end
-
-        it 'initializes a peer discovery when connected to the specified hosts' do
-          connection_helper.stub(:discover_peers)
-          connection_helper.connect(hosts, nil)
-          connection0 = FakeConnection.new('host0', 9876, 7)
-          connection1 = FakeConnection.new('host1', 9876, 7)
-          io_reactor.stub(:connect).with('host0', 9876, 7).and_return(Future.resolved(connection0))
-          io_reactor.stub(:connect).with('host1', 9876, 7).and_return(Future.resolved(connection1))
-          connection_helper.connect(hosts, 'some_keyspace')
-          connection_helper.should have_received(:discover_peers).with([connection0, connection1], 'some_keyspace')
-        end
-
-        it 'initializes a peer discovery with the successfull connections as seeds' do
-          connection_helper.stub(:discover_peers)
-          connection_helper.connect(hosts, nil)
-          connection = FakeConnection.new('host0', 9876, 7)
-          io_reactor.stub(:connect).with('host0', 9876, 7).and_return(Future.resolved(connection))
-          io_reactor.stub(:connect).with('host1', 9876, 7).and_return(Future.failed(StandardError.new('bork')))
-          connection_helper.connect(hosts, 'some_keyspace')
-          connection_helper.should have_received(:discover_peers).with([connection], 'some_keyspace')
-        end
-
-        it 'connects to each node a configurable number of times' do
-          connection_helper = described_class.new(io_reactor, 9876, nil, connections_per_node = 3, 7, nil, logger)
-          connection_helper.connect(hosts, nil)
-          io_reactor.should have_received(:connect).with('host0', 9876, 7).exactly(3).times
-          io_reactor.should have_received(:connect).with('host1', 9876, 7).exactly(3).times
-        end
-
-        context 'when a compressor is specified' do
-          let :compressor do
-            double(:compressor, algorithm: 'snappy')
-          end
-
-          let :connection do
-            FakeConnection.new('host0', 9876, 7)
-          end
-
-          it 'enables compression by sending the algorithm with the STARTUP request' do
-            connection.handle_request do |request, timeout|
-              if request.is_a?(Protocol::OptionsRequest)
-                Protocol::SupportedResponse.new('CQL_VERSION' => %w[3.1.1], 'COMPRESSION' => %w[lz4 snappy])
-              else
-                connection.default_request_handler(request, timeout)
-              end
-            end
-            io_reactor.stub(:connect).with('host0', 9876, 7).and_return(Future.resolved(connection))
-            connection_helper.connect(hosts.take(1), 'some_keyspace')
-            connection.requests[1].options['COMPRESSION'].should == 'snappy'
-          end
-
-          it 'does not enable compression when the algorithm is not supported' do
-            connection.handle_request do |request, timeout|
-              if request.is_a?(Protocol::OptionsRequest)
-                Protocol::SupportedResponse.new('CQL_VERSION' => %w[3.1.1], 'COMPRESSION' => %w[lz4])
-              else
-                connection.default_request_handler(request, timeout)
-              end
-            end
-            io_reactor.stub(:connect).with('host0', 9876, 7).and_return(Future.resolved(connection))
-            connection_helper.connect(hosts.take(1), 'some_keyspace')
-            connection.requests[1].options.should_not have_key('COMPRESSION')
-          end
-
-          it 'logs a warning when compression was disabled because the algorithm was not supported' do
-            logger.stub(:warn)
-            connection.handle_request do |request, timeout|
-              if request.is_a?(Protocol::OptionsRequest)
-                Protocol::SupportedResponse.new('CQL_VERSION' => %w[3.1.1], 'COMPRESSION' => %w[lz4])
-              else
-                connection.default_request_handler(request, timeout)
-              end
-            end
-            io_reactor.stub(:connect).with('host0', 9876, 7).and_return(Future.resolved(connection))
-            connection_helper.connect(hosts.take(1), 'some_keyspace')
-            logger.should have_received(:warn).with(/not supported/)
-          end
-
-          it 'logs the name of the compression algorithm when connecting' do
-            logger.stub(:debug)
-            io_reactor.stub(:connect).with('host0', 9876, 7).and_return(Future.resolved(connection))
-            connection_helper.connect(hosts.take(1), 'some_keyspace')
-            logger.should have_received(:debug).with(/using "snappy" compression/)
-          end
-        end
-      end
-
-      describe '#discover_peers' do
-        let :seed_connections do
-          [
-            FakeConnection.new('host0', 9042, 5),
-            FakeConnection.new('host1', 9042, 5),
-            FakeConnection.new('host2', 9042, 5),
-          ]
-        end
-
-        let :seed_connection_rows do
-          [
-            {'peer' => IPAddr.new('2.0.0.0'), 'rpc_address' => IPAddr.new('1.0.0.0'), 'data_center' => 'dc1', 'host_id' => Uuid.new('eac69196-1e28-11e3-8e2b-191b6d153d0c')},
-            {'peer' => IPAddr.new('2.0.0.1'), 'rpc_address' => IPAddr.new('1.0.0.1'), 'data_center' => 'dc1', 'host_id' => Uuid.new('fa5f9562-1e28-11e3-bf05-3d3a155d0608')},
-            {'peer' => IPAddr.new('2.0.0.2'), 'rpc_address' => IPAddr.new('1.0.0.2'), 'data_center' => 'dc1', 'host_id' => Uuid.new('018b8f1c-1e29-11e3-b14f-532d016437ce')},
-          ]
-        end
-
-        let :extra_connection_rows do
-          [
-            {'peer' => IPAddr.new('2.0.0.3'), 'rpc_address' => IPAddr.new('1.0.0.3'), 'data_center' => 'dc1', 'host_id' => Uuid.new('7a3ccace-1e2a-11e3-a447-43312b1c66e4')},
-            {'peer' => IPAddr.new('2.0.0.4'), 'rpc_address' => IPAddr.new('1.0.0.4'), 'data_center' => 'dc1', 'host_id' => Uuid.new('7bbd4e32-1e2a-11e3-b21d-69d7c02cece8')},
-            {'peer' => IPAddr.new('2.0.0.5'), 'rpc_address' => IPAddr.new('1.0.0.5'), 'data_center' => 'dc1', 'host_id' => Uuid.new('7d7e76f6-1e2a-11e3-bfa0-4fb416ef4064')},
-          ]
-        end
-
-        let :peer_metadata do
-          [
-            ['system', 'peers', 'peer', :inet],
-            ['system', 'peers', 'data_center', :varchar],
-            ['system', 'peers', 'host_id', :uuid],
-            ['system', 'peers', 'rpc_address', :inet],
-          ]
-        end
-
-        before do
-          seed_connections.each_with_index do |c, i|
-            c[:host_id] = seed_connection_rows[i]['host_id']
-            c[:data_center] = seed_connection_rows[i]['data_center']
-          end
-        end
-
-        def peer_request_response
-          seed_connections.each do |c|
-            c.handle_request do |request|
-              if request.cql =~ /SELECT .* FROM system\.peers/
-                Protocol::RowsResultResponse.new(yield, peer_metadata, nil)
-              end
-            end
-          end
-        end
-
-        it 'returns immediately if there are no seed connections' do
-          f = connection_helper.discover_peers([], nil)
-          f.value
-        end
-
-        it 'logs a message when it begins' do
-          logger.stub(:debug)
-          connection_helper.discover_peers([], nil)
-          logger.should have_received(:debug).with(/Looking for additional nodes/)
-        end
-
-        it 'asks a random connection for its peers' do
-          connection_helper.discover_peers(seed_connections, nil)
-          connection = seed_connections.find { |c| c.requests.any? }
-          connection.requests.first.cql.should match(/SELECT .* FROM system\.peers/)
-        end
-
-        it 'returns an empty list when it only finds nodes it\'s already connected to' do
-          peer_request_response { seed_connection_rows }
-          f = connection_helper.discover_peers(seed_connections, nil)
-          f.value.should be_empty
-        end
-
-        it 'logs a message when it finds no new nodes' do
-          logger.stub(:debug)
-          peer_request_response { seed_connection_rows }
-          connection_helper.discover_peers(seed_connections, nil)
-          logger.should have_received(:debug).with(/No additional nodes found/)
-        end
-
-        it 'returns an empty list when it only finds nodes data centers other than those of the seed connections' do
-          seed_connections[1][:data_center] = 'dc2'
-          seed_connection_rows[1]['data_center'] = 'dc2'
-          extra_connection_rows[0]['data_center'] = 'dc3'
-          peer_request_response { seed_connection_rows + extra_connection_rows.take(1) }
-          f = connection_helper.discover_peers(seed_connections, nil)
-          f.value.should be_empty
-        end
-
-        it 'connects to the nodes it finds that it is not already connected to' do
-          connection = FakeConnection.new('host3', 9876, 7)
-          io_reactor.stub(:connect).with('1.0.0.3', 9876, 7).and_return(Future.resolved(connection))
-          peer_request_response { seed_connection_rows + extra_connection_rows.take(1) }
-          f = connection_helper.discover_peers(seed_connections, nil)
-          f.value
-        end
-
-        it 'logs the number of new nodes found' do
-          logger.stub(:debug)
-          connection = FakeConnection.new('host3', 9876, 7)
-          io_reactor.stub(:connect).with('1.0.0.3', 9876, 7).and_return(Future.resolved(connection))
-          peer_request_response { seed_connection_rows + extra_connection_rows.take(1) }
-          connection_helper.discover_peers(seed_connections, nil)
-          logger.should have_received(:debug).with(/1 additional nodes found/)
-        end
-
-        it 'returns the new connections' do
-          connection = FakeConnection.new('host3', 9876, 7)
-          io_reactor.stub(:connect).with('1.0.0.3', 9876, 7).and_return(Future.resolved(connection))
-          peer_request_response { seed_connection_rows + extra_connection_rows.take(1) }
-          f = connection_helper.discover_peers(seed_connections, nil)
-          f.value.should == [connection]
-        end
-
-        it 'initializes the new connections' do
-          connection = FakeConnection.new('host3', 9876, 7)
-          io_reactor.stub(:connect).with('1.0.0.3', 9876, 7).and_return(Future.resolved(connection))
-          peer_request_response { seed_connection_rows + extra_connection_rows.take(1) }
-          f = connection_helper.discover_peers(seed_connections, 'some_keyspace')
-          f.value
-          connection.requests[0].should be_a(Protocol::OptionsRequest)
-          connection.requests[1].should be_a(Protocol::StartupRequest)
-          connection.requests[2].cql.should match(/SELECT .* FROM system.local/)
-          connection.requests[3].cql.should == 'USE some_keyspace'
-        end
-
-        it 'connects only to node in the same data centers as the seed nodes' do
-          seed_connections[1][:data_center] = 'dc2'
-          seed_connection_rows[1]['data_center'] = 'dc2'
-          extra_connection_rows[0]['data_center'] = 'dc3'
-          extra_connection_rows[1]['data_center'] = 'dc2'
-          extra_connection_rows[2]['data_center'] = 'dc1'
-          connection4 = FakeConnection.new('host4', 9876, 7)
-          connection5 = FakeConnection.new('host5', 9876, 7)
-          io_reactor.stub(:connect).with('1.0.0.4', 9876, 7).and_return(Future.resolved(connection4))
-          io_reactor.stub(:connect).with('1.0.0.5', 9876, 7).and_return(Future.resolved(connection5))
-          peer_request_response { seed_connection_rows + extra_connection_rows.take(3) }
-          f = connection_helper.discover_peers(seed_connections, nil)
-          f.value.should == [connection4, connection5]
-        end
-
-        it 'uses the peer address instead of the RPC address when latter is 0.0.0.0' do
-          extra_connection_rows[0]['rpc_address'] = IPAddr.new('0.0.0.0')
-          connection = FakeConnection.new('host3', 9876, 7)
-          io_reactor.stub(:connect).with(extra_connection_rows[0]['peer'].to_s, 9876, 7).and_return(Future.resolved(connection))
-          peer_request_response { seed_connection_rows + extra_connection_rows.take(1) }
-          f = connection_helper.discover_peers(seed_connections, nil)
-          f.value
-        end
-
-        it 'connects to each node a configurable number of times' do
-          connection_helper = described_class.new(io_reactor, 9876, nil, connections_per_node = 3, 7, nil, logger)
-          connection = FakeConnection.new('host3', 9876, 7)
-          io_reactor.stub(:connect).with('1.0.0.3', 9876, 7).and_return(Future.resolved(connection))
-          peer_request_response { seed_connection_rows + extra_connection_rows.take(1) }
-          connection_helper.discover_peers(seed_connections, nil).value
-          io_reactor.should have_received(:connect).with('1.0.0.3', 9876, 7).exactly(3).times
-        end
-      end
-    end
-  end
-end