acpc_poker_basic_proxy 2.0.0 → 3.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c75b02eafb76e6ecd4895bad73bf7abdeb0377e1
-  data.tar.gz: 13699952e83f7ed0812880b7de60eed4ede2bacb
+  metadata.gz: 424876fe9b5e2498c3b29df7337b975d8f77aa9f
+  data.tar.gz: 5a49ff106ad812d598577c778c5801ab44d37f89
 SHA512:
-  metadata.gz: d093f6987211ffa930f1231edc270861ac4afd19b360f31f0b2da6da22a49a6f6726687a5165c6458a2a67d44c38e83abcfbb7d9fbc3a3c8e12ce0dc4c5b502d
-  data.tar.gz: c66b69e39fa01f95b1b5aeb7eecd6b073bf6a9cd3da7f6320455b209720c553a6f7c8d47d519448a420b9024c7e2f7111adc7ea2a3254e3b0032c0cd5d0e25a1
+  metadata.gz: 4c3e54d32b12534faf18d81d243e2d44047ee16a4c7a57388dcf4ff9df66cab6c38488ef74a4c05f3f80b63c8fe7bcb17332ee94ca4ee935fa0f3e052f33fd98
+  data.tar.gz: 333608628e63fe3cb6eb66d6a37f8ebcc6fb0633b872b1cf2880c48b1e3a2f6653dcc345df8dba8b1a759792ebb1e7f94da0ed7bfcf946a541fd0212146a9379

data/lib/acpc_poker_basic_proxy/basic_proxy.rb

@@ -1,8 +1,7 @@
 require 'acpc_dealer'
+require 'acpc_poker_types'
 
-require 'acpc_poker_basic_proxy/communication_logic/dealer_stream'
-require 'acpc_poker_basic_proxy/communication_logic/action_sender'
-require 'acpc_poker_basic_proxy/communication_logic/match_state_receiver'
+require 'acpc_poker_basic_proxy/dealer_stream'
 
 require 'contextual_exceptions'
 using ContextualExceptions::ClassRefinement
@@ -10,11 +9,39 @@ using ContextualExceptions::ClassRefinement
 # A bot that connects to a dealer as a proxy.
 module AcpcPokerBasicProxy
   class BasicProxy
-    exceptions :initial_match_state_not_yet_received
+    exceptions :initial_match_state_not_yet_received, :illegal_action_format
+
+    CONCATENATED_MODIFIABLE_ACTIONS = AcpcPokerTypes::PokerAction::MODIFIABLE_ACTIONS.to_a.join
+
+    # Sends the given +action+ to through the given +connection+ in the ACPC
+    # format.
+    # @param [#write, #ready_to_write?] connection The connection through which the +action+
+    #  should be sent.
+    # @param [#to_s] match_state The current match state.
+    # @param [#to_s] action The action to be sent through the +connection+.
+    # @return [Integer] The number of bytes written.
+    # @raise (see #validate_match_state)
+    # @raise (see #validate_action)
+    def self.send_action(connection, match_state, action)
+      validate_action action
+
+      full_action = "#{AcpcPokerTypes::MatchState.parse(match_state.to_s)}:#{action.to_s}"
+      connection.write full_action
+    end
+
+    # @raise IllegalActionFormat
+    def self.validate_action(action)
+      raise IllegalActionFormat unless self.valid_action?(action)
+    end
+
+    def self.valid_action?(action)
+      action.to_s.match(/^[#{AcpcPokerTypes::PokerAction::CONCATONATED_ACTIONS}]$/) ||
+      action.to_s.match(/^[#{CONCATENATED_MODIFIABLE_ACTIONS}]\d+$/)
+    end
 
     # @param [AcpcDealer::ConnectionInformation] dealer_information Information about the dealer to which this bot should connect.
     def initialize(dealer_information)
-      @dealer_communicator = CommunicationLogic::DealerStream.new dealer_information.port_number, dealer_information.host_name, dealer_information.millisecond_response_timeout
+      @dealer_communicator = DealerStream.new dealer_information.port_number, dealer_information.host_name, dealer_information.millisecond_response_timeout
     end
 
     # @param [PokerAction] action The action to be sent.
@@ -23,12 +50,12 @@ module AcpcPokerBasicProxy
     # @raise (see ActionSender#send_action)
     def send_action(action)
       raise InitialMatchStateNotYetReceived unless @match_state
-      CommunicationLogic::ActionSender.send_action @dealer_communicator, @match_state, action
+      BasicProxy.send_action @dealer_communicator, @match_state, action
     end
 
     # @see MatchStateReceiver#receive_match_state
     def receive_match_state!
-      @match_state = CommunicationLogic::MatchStateReceiver.receive_match_state @dealer_communicator
+      @match_state = AcpcPokerTypes::MatchState.receive @dealer_communicator
     end
   end
 end

data/lib/acpc_poker_basic_proxy/dealer_stream.rb

@@ -0,0 +1,146 @@
+require 'socket'
+require 'delegate'
+
+module AcpcPokerBasicProxy
+  module IoRefinement
+    refine IO do
+      # Checks if the socket is ready to be read from.
+      # @param [Integer] timeout_in_seconds Amount of time to wait for the sever to respond, in seconds. Must be positive or +nil+.
+      # @return [Boolean] +true+ if the socket is ready to be read from, +false+ otherwise.
+      def ready_to_read?(timeout_in_seconds=nil)
+        read_array = [self]
+        write_array = nil
+        error_array = nil
+
+        select?(read_array, write_array, error_array, timeout_in_seconds)
+      end
+
+      # Checks if the socket is ready to be written to.
+      # @param [Integer] timeout_in_seconds Amount of time to wait for the sever to respond, in seconds. Must be positive or +nil+.
+      # @return [Boolean] +true+ if the socket is ready to be written to, +false+ otherwise.
+      def ready_to_write?(timeout_in_seconds=nil)
+        read_array = nil
+        write_array = [self]
+        error_array = nil
+
+        select?(read_array, write_array, error_array, timeout_in_seconds)
+      end
+
+      private
+
+      # @see IO#select
+      def select?(read_array, write_array=[], error_array=[], timeout_in_seconds=nil)
+        IO.select(read_array, write_array, error_array, timeout_in_seconds) != nil
+      end
+    end
+  end
+end
+using AcpcPokerBasicProxy::IoRefinement
+
+require 'contextual_exceptions'
+using ContextualExceptions::ClassRefinement
+
+# Communication service to the ACPC Dealer.
+# It acts solely as an abstraction of the communication protocol and
+# implements the main Ruby communication interface through 'gets' and 'puts'
+# methods.
+module AcpcPokerBasicProxy
+  class DealerStream < DelegateClass(TCPSocket)
+    exceptions :unable_to_connect_to_dealer, :unable_to_write_to_dealer, :unable_to_get_from_dealer
+
+    # @return [String] The ACPC dealer version label.
+    VERSION_LABEL = 'VERSION'
+
+    # @return [Hash] The ACPC dealer version numbers.
+    VERSION_NUMBERS = {:major => 2, :minor => 0, :revision => 0}
+
+    # @return [String] Dealer specified string terminator.
+    TERMINATION_STRING = "\r\n"
+
+    # @param [Integer] port The port on which to connect to the dealer.
+    # @param [String] host_name The host on which the dealer is running. Defaults to 'localhost'
+    # @param [Integer] millisecond_response_timeout The dealer's response timeout, in milliseconds.
+    # @raise AcpcDealerConnectionError, UnableToWriteToDealer
+    def initialize(port, host_name='localhost', millisecond_response_timeout=nil)
+      begin
+        @dealer_socket = TCPSocket.new(host_name, port)
+        super @dealer_socket
+
+        @response_timeout_in_seconds = if millisecond_response_timeout
+          millisecond_response_timeout/(10**3).to_r
+        else
+          nil
+        end
+
+        send_version_string_to_dealer
+      rescue UnableToWriteToDealer
+        raise
+      rescue Errno::ECONNREFUSED => e
+        handle_error UnableToConnectToDealer, "Unable to connect to the dealer on #{host_name} through port #{port}", e
+      end
+    end
+
+    # Retrieves a string from the dealer.
+    #
+    # @return [String] A string from the dealer.
+    # @raise UnableToGetFromDealer
+    def gets
+      begin
+        string_from_dealer
+      rescue => e
+        handle_error UnableToGetFromDealer, "Unable to get a string from the dealer", e
+      end
+    end
+
+    # Sends a given +string+ to the dealer.
+    #
+    # @param [String] string The string to send.
+    # @return (see #send_string_to_dealer)
+    # @raise UnableToWriteToDealer
+    def write(string)
+      begin
+        num_bytes_written = send_string_to_dealer string
+      rescue => e
+        handle_error UnableToWriteToDealer, "Unable to send the string, \"#{string}\", to the dealer", e
+      end
+    end
+
+    # @see TCPSocket#ready_to_write?
+    def ready_to_write?
+      @dealer_socket.ready_to_write? @response_timeout_in_seconds
+    end
+
+    # @see TCPSocket#ready_to_read?
+    def ready_to_read?
+      @dealer_socket.ready_to_read? @response_timeout_in_seconds
+    end
+
+    private
+
+    def handle_error(exception, message, context_exception)
+      close if @dealer_socket && !closed?
+      raise exception.with_context(message, context_exception)
+    end
+
+    # @return (see #send_string_to_dealer)
+    def send_version_string_to_dealer
+      version_string = "#{VERSION_LABEL}:#{VERSION_NUMBERS[:major]}.#{VERSION_NUMBERS[:minor]}.#{VERSION_NUMBERS[:revision]}"
+      begin
+        send_string_to_dealer version_string
+      rescue => e
+        handle_error UnableToWriteToDealer, "Unable to send version string, \"#{version_string}\", to the dealer", e
+      end
+    end
+
+    # @return [Integer] The number of bytes written to the dealer.
+    def send_string_to_dealer(string)
+      raise unless ready_to_write?
+      @dealer_socket.write(string + TERMINATION_STRING)
+    end
+
+    def string_from_dealer
+      raise unless ready_to_read?
+      @dealer_socket.gets.chomp
+    end
+  end
+end

data/lib/acpc_poker_basic_proxy/version.rb

@@ -1,3 +1,3 @@
 module AcpcPokerBasicProxy
-  VERSION = "2.0.0"
+  VERSION = "3.0.0"
 end

data/lib/acpc_poker_basic_proxy.rb

@@ -1,7 +1,7 @@
 require 'acpc_poker_basic_proxy/version'
 
 require 'acpc_poker_basic_proxy/basic_proxy'
-require 'acpc_poker_basic_proxy/communication_logic'
+require 'acpc_poker_basic_proxy/dealer_stream'
 
 module AcpcPokerBasicProxy
 end

data/spec/basic_proxy_spec.rb

@@ -7,11 +7,9 @@ require 'acpc_poker_types/acpc_dealer_data/poker_match_data'
 require 'acpc_dealer'
 
 require 'acpc_poker_basic_proxy/basic_proxy'
-require 'acpc_poker_basic_proxy/communication_logic/action_sender'
 
 include AcpcPokerBasicProxy
 include AcpcPokerTypes
-include CommunicationLogic
 include AcpcDealerData
 
 describe BasicProxy do
@@ -25,10 +23,19 @@ describe BasicProxy do
     DealerStream.expects(:new).once.with(port_number, host_name, millisecond_response_timeout).returns(@dealer_communicator)
 
     @patient = BasicProxy.new delaer_info
+
+    @connection = MiniTest::Mock.new
+    @match_state = AcpcDealerData::PokerMatchData.parse_files(
+      MatchLog.all[0].actions_file_path,
+      MatchLog.all[0].results_file_path,
+      MatchLog.all[0].player_names,
+      AcpcDealer::DEALER_DIRECTORY,
+      1
+    ).data[0].data[0].action_message.state
   end
 
   it 'given a sequence of match states and actions, it properly sends and receives them' do
-    match_logs.each do |log_description|
+    MatchLog.all.each do |log_description|
       match = PokerMatchData.parse_files(
         log_description.actions_file_path,
         log_description.results_file_path,
@@ -52,7 +59,7 @@ describe BasicProxy do
 
             if action && match_state == match.current_hand.next_action.state && match.current_hand.next_action.seat == seat
 
-              ActionSender.expects(:send_action).once.with(@dealer_communicator, match_state.to_s, action)
+              BasicProxy.expects(:send_action).once.with(@dealer_communicator, match_state.to_s, action)
 
               @patient.send_action(action)
             end
@@ -61,10 +68,74 @@ describe BasicProxy do
       end
     end
   end
-
   describe '#send_action' do
     it 'raises an exception if a match state was not received before an action was sent' do
       -> {@patient.send_action(mock('PokerAction'))}.must_raise BasicProxy::InitialMatchStateNotYetReceived
     end
   end
+  describe "#send_action" do
+    it 'does not send an illegal action and raises an exception' do
+      -> do
+        BasicProxy.send_action(@connection, @match_state, 'illegal action format')
+      end.must_raise BasicProxy::IllegalActionFormat
+    end
+    it 'raises an exception if the given match state does not have the proper format' do
+      -> do
+        BasicProxy.send_action(@connection, 'illegal match state format', PokerAction::CALL)
+      end.must_raise MatchState::IncompleteMatchState
+    end
+    it 'can send all legal actions through the provided connection without a modifier' do
+      PokerAction::ACTIONS.each do |action|
+        action_that_should_be_sent = @match_state.to_s + ":#{action}"
+        @connection.expect :write, nil, [action_that_should_be_sent]
+
+        BasicProxy.send_action @connection, @match_state, action
+      end
+    end
+    it 'does not send legal unmodifiable actions that have a modifier and raises an exception' do
+      (PokerAction::ACTIONS - PokerAction::MODIFIABLE_ACTIONS).each do |unmodifiable_action|
+        -> do
+          BasicProxy.send_action(@connection, @match_state, unmodifiable_action + 9001.to_s)
+        end.must_raise BasicProxy::IllegalActionFormat
+      end
+    end
+    it 'can send all legal modifiable actions through the provided connection with a modifier' do
+      PokerAction::MODIFIABLE_ACTIONS.each do |action|
+        arbitrary_modifier = 9001
+        action_string = action + arbitrary_modifier.to_s
+        action_that_should_be_sent = @match_state.to_s + ":#{action_string}"
+        @connection.expect :write, nil, [action_that_should_be_sent]
+
+        BasicProxy.send_action @connection, @match_state, action_string
+      end
+    end
+    it 'works for all test data examples' do
+      MatchLog.all.each do |log_description|
+        match = AcpcDealerData::PokerMatchData.parse_files(
+          log_description.actions_file_path,
+          log_description.results_file_path,
+          log_description.player_names,
+          AcpcDealer::DEALER_DIRECTORY,
+          20
+        )
+        match.for_every_seat! do |seat|
+          match.for_every_hand! do
+            match.for_every_turn! do
+              next unless match.current_hand.next_action
+
+              from_player_message = match.current_hand.next_action.state
+              seat_taking_action = match.current_hand.next_action.seat
+              action = match.current_hand.next_action.action
+
+              action_that_should_be_sent = "#{from_player_message.to_s}:#{action.to_acpc}"
+
+              @connection.expect :write, nil, [action_that_should_be_sent]
+
+              BasicProxy.send_action @connection, from_player_message, action
+            end
+          end
+        end
+      end
+    end
+  end
 end

data/spec/coverage/index.html

@@ -14,7 +14,7 @@
       <img src="./assets/0.7.1/loading.gif" alt="loading"/>
     </div>
     <div id="wrapper" style="display:none;">
-      <div class="timestamp">Generated <abbr class="timeago" title="2013-05-08T18:52:12-06:00">2013-05-08T18:52:12-06:00</abbr></div>
+      <div class="timestamp">Generated <abbr class="timeago" title="2013-05-16T14:30:23-06:00">2013-05-16T14:30:23-06:00</abbr></div>
       <ul class="group_tabs"></ul>
 
       <div id="content">

data/spec/dealer_stream_spec.rb

@@ -3,10 +3,9 @@ require_relative 'support/spec_helper'
 
 require 'socket'
 
-require 'acpc_poker_basic_proxy/communication_logic/dealer_stream'
+require 'acpc_poker_basic_proxy/dealer_stream'
 
 include AcpcPokerBasicProxy
-include CommunicationLogic
 
 describe DealerStream do
   after do

data/spec/support/spec_helper.rb

@@ -39,6 +39,31 @@ class MatchLog
 
   attr_reader :results_file_name, :actions_file_name, :player_names, :dealer_log_directory
 
+  def self.all
+    [
+      MatchLog.new(
+        '2p.limit.h1000.r0.log',
+        '2p.limit.h1000.r0.actions.log',
+        ['p1', 'p2']
+      ),
+      MatchLog.new(
+        '2p.nolimit.h1000.r0.log',
+        '2p.nolimit.h1000.r0.actions.log',
+        ['p1', 'p2']
+      ),
+      MatchLog.new(
+        '3p.limit.h1000.r0.log',
+        '3p.limit.h1000.r0.actions.log',
+        ['p1', 'p2', 'p3']
+      ),
+      MatchLog.new(
+        '3p.nolimit.h1000.r0.log',
+        '3p.nolimit.h1000.r0.actions.log',
+        ['p1', 'p2', 'p3']
+      )
+    ]
+  end
+
   def initialize(results_file_name, actions_file_name, player_names)
     @results_file_name = results_file_name
     @actions_file_name = actions_file_name
@@ -52,29 +77,4 @@ class MatchLog
   def results_file_path
     "#{DEALER_LOG_DIRECTORY}/#{@results_file_name}"
   end
-end
-
-def match_logs
-  [
-    MatchLog.new(
-      '2p.limit.h1000.r0.log',
-      '2p.limit.h1000.r0.actions.log',
-      ['p1', 'p2']
-    ),
-    MatchLog.new(
-      '2p.nolimit.h1000.r0.log',
-      '2p.nolimit.h1000.r0.actions.log',
-      ['p1', 'p2']
-    ),
-    MatchLog.new(
-      '3p.limit.h1000.r0.log',
-      '3p.limit.h1000.r0.actions.log',
-      ['p1', 'p2', 'p3']
-    ),
-    MatchLog.new(
-      '3p.nolimit.h1000.r0.log',
-      '3p.nolimit.h1000.r0.actions.log',
-      ['p1', 'p2', 'p3']
-    )
-  ]
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: acpc_poker_basic_proxy
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 3.0.0
 platform: ruby
 authors:
 - Dustin Morrill
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-05-09 00:00:00.000000000 Z
+date: 2013-05-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: acpc_poker_types
@@ -145,15 +145,11 @@ extra_rdoc_files: []
 files:
 - lib/acpc_poker_basic_proxy.rb
 - lib/acpc_poker_basic_proxy/basic_proxy.rb
-- lib/acpc_poker_basic_proxy/communication_logic.rb
-- lib/acpc_poker_basic_proxy/communication_logic/action_sender.rb
-- lib/acpc_poker_basic_proxy/communication_logic/match_state_receiver.rb
-- lib/acpc_poker_basic_proxy/communication_logic/dealer_stream.rb
 - lib/acpc_poker_basic_proxy/version.rb
+- lib/acpc_poker_basic_proxy/dealer_stream.rb
 - Rakefile
 - acpc_poker_basic_proxy.gemspec
 - README.md
-- spec/match_state_receiver_spec.rb
 - spec/support/dealer_logs/3p.limit.h1000.r0.actions.log
 - spec/support/dealer_logs/3p.limit.h1000.r0.log
 - spec/support/dealer_logs/3p.nolimit.h1000.r0.actions.log
@@ -206,7 +202,6 @@ files:
 - spec/coverage/assets/0.7.1/favicon_red.png
 - spec/coverage/assets/0.7.1/loading.gif
 - spec/dealer_stream_spec.rb
-- spec/action_sender_spec.rb
 homepage: https://github.com/dmorrill10/acpc_poker_basic_proxy
 licenses: []
 metadata: {}
@@ -231,7 +226,6 @@ signing_key:
 specification_version: 4
 summary: ACPC Poker Basic Proxy
 test_files:
-- spec/match_state_receiver_spec.rb
 - spec/support/dealer_logs/3p.limit.h1000.r0.actions.log
 - spec/support/dealer_logs/3p.limit.h1000.r0.log
 - spec/support/dealer_logs/3p.nolimit.h1000.r0.actions.log
@@ -284,4 +278,3 @@ test_files:
 - spec/coverage/assets/0.7.1/favicon_red.png
 - spec/coverage/assets/0.7.1/loading.gif
 - spec/dealer_stream_spec.rb
-- spec/action_sender_spec.rb

data/lib/acpc_poker_basic_proxy/communication_logic/action_sender.rb

@@ -1,45 +0,0 @@
-
-require 'acpc_poker_types/poker_action'
-require 'acpc_poker_types/match_state'
-
-require 'contextual_exceptions'
-using ContextualExceptions::ClassRefinement
-
-# Sends poker actions according to the ACPC protocol.
-module AcpcPokerBasicProxy
-  module CommunicationLogic
-    class ActionSender
-      exceptions :illegal_action_format
-
-      CONCATENATED_MODIFIABLE_ACTIONS = AcpcPokerTypes::PokerAction::MODIFIABLE_ACTIONS.to_a.join
-
-      # Sends the given +action+ to through the given +connection+ in the ACPC
-      # format.
-      # @param [#write, #ready_to_write?] connection The connection through which the +action+
-      #  should be sent.
-      # @param [#to_s] match_state The current match state.
-      # @param [#to_s] action The action to be sent through the +connection+.
-      # @return [Integer] The number of bytes written.
-      # @raise (see #validate_match_state)
-      # @raise (see #validate_action)
-      def self.send_action(connection, match_state, action)
-        validate_action action
-
-        full_action = "#{AcpcPokerTypes::MatchState.parse(match_state.to_s)}:#{action.to_s}"
-        connection.write full_action
-      end
-
-      private
-
-      # @raise IllegalActionFormat
-      def self.validate_action(action)
-        raise IllegalActionFormat unless self.valid_action?(action)
-      end
-
-      def self.valid_action?(action)
-        action.to_s.match(/^[#{AcpcPokerTypes::PokerAction::CONCATONATED_ACTIONS}]$/) ||
-        action.to_s.match(/^[#{CONCATENATED_MODIFIABLE_ACTIONS}]\d+$/)
-      end
-    end
-  end
-end

data/lib/acpc_poker_basic_proxy/communication_logic/dealer_stream.rb

@@ -1,150 +0,0 @@
-require 'socket'
-require 'delegate'
-
-module AcpcPokerBasicProxy
-  module CommunicationLogic
-    module IoRefinement
-      refine IO do
-        # Checks if the socket is ready to be read from.
-        # @param [Integer] timeout_in_seconds Amount of time to wait for the sever to respond, in seconds. Must be positive or +nil+.
-        # @return [Boolean] +true+ if the socket is ready to be read from, +false+ otherwise.
-        def ready_to_read?(timeout_in_seconds=nil)
-          read_array = [self]
-          write_array = nil
-          error_array = nil
-
-          select?(read_array, write_array, error_array, timeout_in_seconds)
-        end
-
-        # Checks if the socket is ready to be written to.
-        # @param [Integer] timeout_in_seconds Amount of time to wait for the sever to respond, in seconds. Must be positive or +nil+.
-        # @return [Boolean] +true+ if the socket is ready to be written to, +false+ otherwise.
-        def ready_to_write?(timeout_in_seconds=nil)
-          read_array = nil
-          write_array = [self]
-          error_array = nil
-
-          select?(read_array, write_array, error_array, timeout_in_seconds)
-        end
-
-        private
-
-        # @see IO#select
-        def select?(read_array, write_array=[], error_array=[], timeout_in_seconds=nil)
-          IO.select(read_array, write_array, error_array, timeout_in_seconds) != nil
-        end
-      end
-    end
-  end
-end
-using AcpcPokerBasicProxy::CommunicationLogic::IoRefinement
-
-require 'contextual_exceptions'
-using ContextualExceptions::ClassRefinement
-
-# Communication service to the ACPC Dealer.
-# It acts solely as an abstraction of the communication protocol and
-# implements the main Ruby communication interface through 'gets' and 'puts'
-# methods.
-module AcpcPokerBasicProxy
-  module CommunicationLogic
-    class DealerStream < DelegateClass(TCPSocket)
-      exceptions :unable_to_connect_to_dealer, :unable_to_write_to_dealer, :unable_to_get_from_dealer
-
-      # @return [String] The ACPC dealer version label.
-      VERSION_LABEL = 'VERSION'
-
-      # @return [Hash] The ACPC dealer version numbers.
-      VERSION_NUMBERS = {:major => 2, :minor => 0, :revision => 0}
-
-      # @return [String] Dealer specified string terminator.
-      TERMINATION_STRING = "\r\n"
-
-      # @param [Integer] port The port on which to connect to the dealer.
-      # @param [String] host_name The host on which the dealer is running. Defaults to 'localhost'
-      # @param [Integer] millisecond_response_timeout The dealer's response timeout, in milliseconds.
-      # @raise AcpcDealerConnectionError, UnableToWriteToDealer
-      def initialize(port, host_name='localhost', millisecond_response_timeout=nil)
-        begin
-          @dealer_socket = TCPSocket.new(host_name, port)
-          super @dealer_socket
-
-          @response_timeout_in_seconds = if millisecond_response_timeout
-            millisecond_response_timeout/(10**3).to_r
-          else
-            nil
-          end
-
-          send_version_string_to_dealer
-        rescue UnableToWriteToDealer
-          raise
-        rescue Errno::ECONNREFUSED => e
-          handle_error UnableToConnectToDealer, "Unable to connect to the dealer on #{host_name} through port #{port}", e
-        end
-      end
-
-      # Retrieves a string from the dealer.
-      #
-      # @return [String] A string from the dealer.
-      # @raise UnableToGetFromDealer
-      def gets
-        begin
-          string_from_dealer
-        rescue => e
-          handle_error UnableToGetFromDealer, "Unable to get a string from the dealer", e
-        end
-      end
-
-      # Sends a given +string+ to the dealer.
-      #
-      # @param [String] string The string to send.
-      # @return (see #send_string_to_dealer)
-      # @raise UnableToWriteToDealer
-      def write(string)
-        begin
-          num_bytes_written = send_string_to_dealer string
-        rescue => e
-          handle_error UnableToWriteToDealer, "Unable to send the string, \"#{string}\", to the dealer", e
-        end
-      end
-
-      # @see TCPSocket#ready_to_write?
-      def ready_to_write?
-        @dealer_socket.ready_to_write? @response_timeout_in_seconds
-      end
-
-      # @see TCPSocket#ready_to_read?
-      def ready_to_read?
-        @dealer_socket.ready_to_read? @response_timeout_in_seconds
-      end
-
-      private
-
-      def handle_error(exception, message, context_exception)
-        close if @dealer_socket && !closed?
-        raise exception.with_context(message, context_exception)
-      end
-
-      # @return (see #send_string_to_dealer)
-      def send_version_string_to_dealer
-        version_string = "#{VERSION_LABEL}:#{VERSION_NUMBERS[:major]}.#{VERSION_NUMBERS[:minor]}.#{VERSION_NUMBERS[:revision]}"
-        begin
-          send_string_to_dealer version_string
-        rescue => e
-          handle_error UnableToWriteToDealer, "Unable to send version string, \"#{version_string}\", to the dealer", e
-        end
-      end
-
-      # @return [Integer] The number of bytes written to the dealer.
-      def send_string_to_dealer(string)
-        raise unless ready_to_write?
-        @dealer_socket.write(string + TERMINATION_STRING)
-      end
-
-      def string_from_dealer
-        raise unless ready_to_read?
-        @dealer_socket.gets.chomp
-      end
-    end
-  end
-end

data/lib/acpc_poker_basic_proxy/communication_logic/match_state_receiver.rb

@@ -1,18 +0,0 @@
-
-require 'acpc_poker_types/match_state'
-
-# Receives match state strings.
-module AcpcPokerBasicProxy
-  module CommunicationLogic
-    class MatchStateReceiver
-
-      # Receives a match state string from the given +connection+.
-      # @param [#gets] connection The connection from which a match state string should be received.
-      # @return [MatchState] The match state string that was received from the +connection+ or +nil+ if none could be received.
-      def self.receive_match_state(connection)
-        raw_match_state = connection.gets
-        AcpcPokerTypes::MatchState.parse raw_match_state
-      end
-    end
-  end
-end

data/lib/acpc_poker_basic_proxy/communication_logic.rb

@@ -1,8 +0,0 @@
-require 'acpc_poker_basic_proxy/communication_logic/dealer_stream'
-require 'acpc_poker_basic_proxy/communication_logic/match_state_receiver'
-require 'acpc_poker_basic_proxy/communication_logic/action_sender'
-
-module AcpcPokerBasicProxy
-  module CommunicationLogic
-  end
-end

data/spec/action_sender_spec.rb

@@ -1,91 +0,0 @@
-
-require_relative 'support/spec_helper'
-
-require 'acpc_poker_types/acpc_dealer_data/poker_match_data'
-require 'acpc_poker_types/poker_action'
-require 'acpc_poker_types/match_state'
-require 'acpc_dealer'
-
-require 'acpc_poker_basic_proxy/communication_logic/action_sender'
-
-include AcpcPokerBasicProxy::CommunicationLogic
-include AcpcPokerTypes
-
-describe ActionSender do
-  before(:each) do
-    @connection = MiniTest::Mock.new
-    @match_state = AcpcDealerData::PokerMatchData.parse_files(
-      match_logs[0].actions_file_path,
-      match_logs[0].results_file_path,
-      match_logs[0].player_names,
-      AcpcDealer::DEALER_DIRECTORY,
-      1
-    ).data[0].data[0].action_message.state
-  end
-
-  describe "#send_action" do
-    it 'does not send an illegal action and raises an exception' do
-      -> do
-        ActionSender.send_action(@connection, @match_state, 'illegal action format')
-      end.must_raise ActionSender::IllegalActionFormat
-    end
-    it 'raises an exception if the given match state does not have the proper format' do
-      -> do
-        ActionSender.send_action(@connection, 'illegal match state format', PokerAction::CALL)
-      end.must_raise MatchState::IncompleteMatchState
-    end
-    it 'can send all legal actions through the provided connection without a modifier' do
-      PokerAction::ACTIONS.each do |action|
-        action_that_should_be_sent = @match_state.to_s + ":#{action}"
-        @connection.expect :write, nil, [action_that_should_be_sent]
-
-        ActionSender.send_action @connection, @match_state, action
-      end
-    end
-    it 'does not send legal unmodifiable actions that have a modifier and raises an exception' do
-      (PokerAction::ACTIONS - PokerAction::MODIFIABLE_ACTIONS).each do |unmodifiable_action|
-        -> do
-          ActionSender.send_action(@connection, @match_state, unmodifiable_action + 9001.to_s)
-        end.must_raise ActionSender::IllegalActionFormat
-      end
-    end
-    it 'can send all legal modifiable actions through the provided connection with a modifier' do
-      PokerAction::MODIFIABLE_ACTIONS.each do |action|
-        arbitrary_modifier = 9001
-        action_string = action + arbitrary_modifier.to_s
-        action_that_should_be_sent = @match_state.to_s + ":#{action_string}"
-        @connection.expect :write, nil, [action_that_should_be_sent]
-
-        ActionSender.send_action @connection, @match_state, action_string
-      end
-    end
-    it 'works for all test data examples' do
-      match_logs.each do |log_description|
-        match = AcpcDealerData::PokerMatchData.parse_files(
-          log_description.actions_file_path,
-          log_description.results_file_path,
-          log_description.player_names,
-          AcpcDealer::DEALER_DIRECTORY,
-          60
-        )
-        match.for_every_seat! do |seat|
-          match.for_every_hand! do
-            match.for_every_turn! do
-              next unless match.current_hand.next_action
-
-              from_player_message = match.current_hand.next_action.state
-              seat_taking_action = match.current_hand.next_action.seat
-              action = match.current_hand.next_action.action
-
-              action_that_should_be_sent = "#{from_player_message.to_s}:#{action.to_acpc}"
-
-              @connection.expect :write, nil, [action_that_should_be_sent]
-
-              ActionSender.send_action @connection, from_player_message, action
-            end
-          end
-        end
-      end
-    end
-  end
-end

data/spec/match_state_receiver_spec.rb

@@ -1,38 +0,0 @@
-
-require_relative 'support/spec_helper'
-
-require 'acpc_dealer'
-require 'acpc_poker_types'
-
-require 'acpc_poker_basic_proxy/communication_logic/match_state_receiver'
-require 'acpc_poker_basic_proxy/communication_logic/dealer_stream'
-
-describe AcpcPokerBasicProxy::CommunicationLogic::MatchStateReceiver do
-  before(:each) do
-    @connection = MiniTest::Mock.new
-  end
-
-  describe "#receive_matchstate_string" do
-    it 'receives matchstate strings properly' do
-      match_logs.each do |log_description|
-        match = AcpcPokerTypes::AcpcDealerData::PokerMatchData.parse_files(
-          log_description.actions_file_path,
-          log_description.results_file_path,
-          log_description.player_names,
-          AcpcDealer::DEALER_DIRECTORY,
-          60
-        )
-        match.for_every_seat! do |seat|
-          match.for_every_hand! do
-            match.for_every_turn! do
-              @connection.expect(:gets, match.current_hand.current_match_state.to_s)
-
-              AcpcPokerBasicProxy::CommunicationLogic::MatchStateReceiver
-                .receive_match_state(@connection).must_equal match.current_hand.current_match_state
-            end
-          end
-        end
-      end
-    end
-  end
-end