acpc_poker_types 6.0.2 → 6.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b4356bae466a6937d427c1addffe9b4b56a56752
-  data.tar.gz: 53d930d856434f2e99229eba50d53bb9c49c0777
+  metadata.gz: 70397088ba5bed572b116c5ce0e00da950bb8ae6
+  data.tar.gz: 08709982a9186277f5e59c4f2c7682eb42ba4905
 SHA512:
-  metadata.gz: 445658f326cc4a00cfe5dee77be60bf2b88c5d59cb341a7ea7d7d7bbc1656ae49ec5d4fa00829fa1d234391521eddd86b9c896e1bc7c7e9897f9248a2d7e8164
-  data.tar.gz: a5ac949dddab857a288f1b641b91627760d70c4c09e041c8865f5854ea0561d319c0d55a100d551e938758f9d3e681a1cb6eda94e9209d77237481f07bbbc19c
+  metadata.gz: a99cbdd394535446c908314814f56f1eee0e00f772dfcdcefafbe5ab6328f58621c4bd4f3f01ffc66164e2422027a7c59401b5c447d9cae27155279bde2a711f
+  data.tar.gz: 6a3b0269873e0d55bf9712e2869edbbdc7ef568baa4534e67a830d9c4232d61c3ff953ab73c63bf51f779c6aedadb7b451c7203c879334eb8085821a169bba41

data/lib/acpc_poker_types/hand_player.rb

@@ -63,7 +63,7 @@ class HandPlayer
   # @return [Array<PokerAction>] The list of legal actions for this player. If a wager is legal,
   # the largest possible wager will be returned in the list.
   def legal_actions(
-    round,
+    round: round,
     amount_to_call: ChipStack.new(0),
     wager_illegal: false
   )
@@ -94,11 +94,11 @@ class HandPlayer
     # @todo
   # end
 
-  def append_action!(action, round = @actions.length - 1)
+  def append_action!(action, round_num = round)
     raise Inactive if inactive?
 
-    @actions[round] ||= []
-    @actions[round] << action
+    @actions[round_num] ||= []
+    @actions[round_num] << action
 
     self
   end
@@ -114,6 +114,9 @@ class HandPlayer
   def folded?
     @actions.flatten.last == PokerAction::FOLD
   end
-end
 
+  def round
+    @actions.length - 1
+  end
+end
 end

data/lib/acpc_poker_types/version.rb

@@ -1,3 +1,3 @@
 module AcpcPokerTypes
-  VERSION = '6.0.2'
+  VERSION = '6.1.0'
 end

data/spec/hand_player_spec.rb

@@ -17,6 +17,10 @@ describe HandPlayer do
   ANTE = 100
   HAND = Hand.from_acpc('AhKs')
 
+  before do
+    @patient = nil
+  end
+
   def patient
     @patient ||= HandPlayer.new HAND, INITIAL_CHIP_STACK, ANTE
   end
@@ -200,40 +204,40 @@ describe HandPlayer do
 
   describe '#legal_actions' do
     it 'works with fold' do
-      patient.legal_actions(0, amount_to_call: 100).sort.must_equal [
+      patient.legal_actions(round: 0, amount_to_call: 100).sort.must_equal [
         PokerAction.new(PokerAction::CALL),
         PokerAction.new(PokerAction::RAISE, cost: INITIAL_CHIP_STACK - (ANTE + 100)),
         PokerAction.new(PokerAction::FOLD)
       ].sort
       patient.append_action!(PokerAction.new('c', cost: 100))
-        .legal_actions(0, amount_to_call: 200).sort.must_equal [
+        .legal_actions(round: 0, amount_to_call: 200).sort.must_equal [
           PokerAction.new(PokerAction::CALL),
           PokerAction.new(PokerAction::RAISE, cost: INITIAL_CHIP_STACK - (ANTE + 200)),
           PokerAction.new(PokerAction::FOLD)
         ].sort
       patient.append_action!(PokerAction.new('f'))
-        .legal_actions(0).empty?.must_equal true
+        .legal_actions(round: 0).empty?.must_equal true
     end
     it 'works with all in' do
-      patient.legal_actions(0).sort.must_equal [
+      patient.legal_actions(round: 0).sort.must_equal [
         PokerAction.new(PokerAction::CHECK),
         PokerAction.new(PokerAction::BET, cost: INITIAL_CHIP_STACK - ANTE),
       ].sort
       patient.append_action!(PokerAction.new('c', cost: 100))
-        .legal_actions(0).sort.must_equal [
+        .legal_actions(round: 0).sort.must_equal [
           PokerAction.new(PokerAction::CHECK),
           PokerAction.new(PokerAction::BET, cost: INITIAL_CHIP_STACK - (ANTE + 100)),
         ].sort
       patient.append_action!(PokerAction.new('c', cost: INITIAL_CHIP_STACK - (ANTE + 100)))
-        .legal_actions(0).empty?.must_equal true
+        .legal_actions(round: 0).empty?.must_equal true
     end
     it 'works when an opponent is all in' do
       patient.append_action!(PokerAction.new('c', cost: 100))
-        .legal_actions(0).sort.must_equal [
+        .legal_actions(round: 0).sort.must_equal [
           PokerAction.new(PokerAction::CHECK),
           PokerAction.new(PokerAction::RAISE, cost: INITIAL_CHIP_STACK - (ANTE + 100))
         ].sort
-      patient.legal_actions(0, amount_to_call: INITIAL_CHIP_STACK - (ANTE + 100))
+      patient.legal_actions(round: 0, amount_to_call: INITIAL_CHIP_STACK - (ANTE + 100))
         .sort.must_equal [
           PokerAction.new(PokerAction::CALL),
           PokerAction.new(PokerAction::FOLD)
@@ -241,10 +245,21 @@ describe HandPlayer do
     end
     it 'works when no more raises are allowed' do
       patient.append_action!(PokerAction.new('c', cost: 100))
-        .legal_actions(0, wager_illegal: true).sort.must_equal [
+        .legal_actions(round: 0, wager_illegal: true).must_equal [
           PokerAction.new(PokerAction::CHECK)
+        ]
+      patient.legal_actions(round: 0, amount_to_call: INITIAL_CHIP_STACK - (ANTE + 100))
+        .sort.must_equal [
+          PokerAction.new(PokerAction::CALL),
+          PokerAction.new(PokerAction::FOLD)
         ].sort
-      patient.legal_actions(0, amount_to_call: INITIAL_CHIP_STACK - (ANTE + 100))
+    end
+    it 'works without round argument' do
+      patient.append_action!(PokerAction.new('c', cost: 100))
+        .legal_actions(wager_illegal: true).must_equal [
+          PokerAction.new(PokerAction::CHECK)
+        ]
+      patient.legal_actions(amount_to_call: INITIAL_CHIP_STACK - (ANTE + 100))
         .sort.must_equal [
           PokerAction.new(PokerAction::CALL),
           PokerAction.new(PokerAction::FOLD)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: acpc_poker_types
 version: !ruby/object:Gem::Version
-  version: 6.0.2
+  version: 6.1.0
 platform: ruby
 authors:
 - Dustin Morrill