acpc_poker_basic_proxy 3.2.0 → 3.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8d6dff95db6eef3d31825a87fc46bf7aec6090c5
-  data.tar.gz: 4a80071d3e101764f00c8e0a1812881825a09d3e
+  metadata.gz: 8830e03ad490d6a24dac3b73c647fee5988aea43
+  data.tar.gz: e476afcc89336bd58bf33d5b2c2ae88776d9525d
 SHA512:
-  metadata.gz: 04bd293fcb8daa161acd57ee2250596ea934dc604db45440f06a2e03ddfdc3b823311d61b1ffae02aeaaa5637106693792f62c659b9d2e406795955d6cf90d9d
-  data.tar.gz: 432b48c811b643e16dc13aec3fe674ff556837a833c5cd265f615e9546d71f002b8fe89dc2ecfb7b6b028dbf01773b2222b844d1d3d788e2d4c85df8a0841581
+  metadata.gz: 3c160055f942244c12365e97893236e53c74acdb94329ebeedc41f53686eb535c4023fb33ba8fc142d2d8676666d1a228027636459dd530e9b8ecf6205fa13a2
+  data.tar.gz: f98d89ceb150f1c073abbb9a0a32ce2a6f26a11a70daa68ef5bcdcb2ebcd2bc347dd5abd4787d1b60015ada37d40c97b189bf06c7315d9fc150ea705f6c5cffd

data/Rakefile

@@ -8,5 +8,6 @@ Rake::TestTask.new do |t|
   t.libs << "lib" << 'spec/support'
   t.test_files = FileList['spec/**/*_spec.rb']
   t.verbose = false
-  t.warning = false # pry-rescue has a lot of warnings
+  t.warning = true
 end
+task :default => :test

data/lib/acpc_poker_basic_proxy/basic_proxy.rb

@@ -66,6 +66,7 @@ module AcpcPokerBasicProxy
         dealer_information.port_number,
         dealer_information.host_name
       )
+      @match_state = nil
     end
 
     # @param [PokerAction] action The action to send.

data/lib/acpc_poker_basic_proxy/dealer_stream.rb

@@ -63,6 +63,7 @@ module AcpcPokerBasicProxy
     # @param [String] host_name The host on which the dealer is running. Defaults to 'localhost'
     # @raise AcpcDealerConnectionError, UnableToWriteToDealer
     def initialize(port, host_name='localhost')
+      @dealer_socket = nil
       begin
         @dealer_socket = TCPSocket.new(host_name, port)
         super @dealer_socket
@@ -94,7 +95,7 @@ module AcpcPokerBasicProxy
     # @raise UnableToWriteToDealer
     def write(string)
       begin
-        num_bytes_written = send_string_to_dealer string
+        send_string_to_dealer string
       rescue => e
         handle_error UnableToWriteToDealer, "Unable to send the string, \"#{string}\", to the dealer", e
       end

data/lib/acpc_poker_basic_proxy/version.rb

@@ -1,3 +1,3 @@
 module AcpcPokerBasicProxy
-  VERSION = "3.2.0"
+  VERSION = "3.2.1"
 end

data/spec/basic_proxy_spec.rb

@@ -163,7 +163,6 @@ describe BasicProxy 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}"

data/spec/dealer_stream_spec.rb

@@ -13,7 +13,6 @@ describe DealerStream do
   end
   describe '#new' do
     it 'works properly' do
-      start_test_connection!
       end_test_connection!
     end
     it "fails if the port doesn't correspond to a running server" do
@@ -23,56 +22,56 @@ describe DealerStream do
       -> { DealerStream.new(port) }.must_raise DealerStream::UnableToConnectToDealer
     end
   end
+  let(:port) { 0 }
+  let(:fake_dealer) { TCPServer.open(port) }
+  let(:client_connection) { fake_dealer.accept }
+  let(:patient) do
+    DealerStream.new(
+      fake_dealer.addr[1],
+      'localhost'
+    )
+  end
+  let(:match_state) { 'MATCHSTATE:0:0::5d5c' }
+
   describe "#ready_to_read?" do
     it 'lets the caller know that there is new input from the dealer' do
       connect_successfully!
-      @client_connection.puts "New input"
-      @patient.ready_to_read?.must_equal true
+      client_connection.puts "New input"
+      patient.ready_to_read?.must_equal true
     end
   end
   describe "#ready_to_write?" do
     it 'lets the caller know if the dealer is ready to receive data' do
       connect_successfully!
-      @patient.ready_to_write?.must_equal true
+      patient.ready_to_write?.must_equal true
     end
   end
   describe "#write" do
     it "properly sends actions to the dealer" do
       connect_successfully!
-      action = @match_state + ':c'
-      @patient.write action
+      action = match_state + ':c'
+      patient.write action
 
-      @client_connection.gets.chomp.must_equal(action)
+      client_connection.gets.chomp.must_equal(action)
     end
   end
   describe "#gets" do
     it "properly receives matchstate strings from the dealer" do
       connect_successfully!
-      @client_connection.puts @match_state
-      @patient.gets.must_equal(@match_state)
+      client_connection.puts match_state
+      patient.gets.must_equal(match_state)
     end
   end
 
   def end_test_connection!
-    @patient.close if @patient && !@patient.closed?
-    @client_connection.close if @client_connection && !@client_connection.closed?
-  end
-
-  def start_test_connection!(port = 0)
-    fake_dealer = TCPServer.open(port)
-    @patient = DealerStream.new(
-      fake_dealer.addr[1],
-      'localhost'
-    )
-    @client_connection = fake_dealer.accept
+    patient.close if !patient.closed?
+    client_connection.close if !client_connection.closed?
   end
 
   def connect_successfully!
-    start_test_connection!
-    @client_connection.gets.chomp.must_equal(
+    patient
+    client_connection.gets.chomp.must_equal(
       "#{DealerStream::VERSION_LABEL}:#{DealerStream::VERSION_NUMBERS[:major]}.#{DealerStream::VERSION_NUMBERS[:minor]}.#{DealerStream::VERSION_NUMBERS[:revision]}"
     )
-
-    @match_state = 'MATCHSTATE:0:0::5d5c'
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: acpc_poker_basic_proxy
 version: !ruby/object:Gem::Version
-  version: 3.2.0
+  version: 3.2.1
 platform: ruby
 authors:
 - Dustin Morrill