pokerstats 2.0.13 → 2.0.14

data/spec/hand_statistics_spec_helper.rb

@@ -10,20 +10,28 @@ module HandStatisticsFactories
       :starting_at => Time.now,
       :name => "PS0001",
       :description => "This is a description",
+      :table_name => "foobie-doobie III",
+      :max_players => 6,
+      :number_players => 6,
       :sb => "1.00".to_d,
       :bb => "2.00".to_d,
+      :ante => "0.50".to_d,
       :board => "AS KS QS JS TS",
       :total_pot => "100.00".to_d,
       :rake => "4.00".to_d,
       :played_at => Time.now,
-      :tournament => "T123"
+      :tournament => "T123",
+      :game_type => "Hold'em",
+      :limit_type => "No Limit",
+      :stakes_type => "25".to_d 
     }
   end
   
   def sample_player
     @@sample_player_result ||= {
       :screen_name => 'sample_player_screen_name', 
-      :seat => 1
+      :seat => 1, 
+      :starting_stack => 10
     }
   end
   
@@ -84,6 +92,9 @@ module HandStatisticsFactories
   
   def register_cards(player, data, hash={})
     @stats.register_action player[:screen_name], 'shows', hash.merge(:result => :cards, :data => data)
-  end  
+  end
+  def register_muck(player, hash={})
+    @stats.register_action player[:screen_name], 'mucks', hash.merge(:result => :neutral)
+  end
 end
 include HandStatisticsFactories

data/spec/pokerstars_hand_history_parser_spec.rb

@@ -10,6 +10,45 @@ require File.expand_path(File.dirname(__FILE__) + '/../lib/pokerstats/pokerstars
 include Pokerstats
 include HandConstants
 
+
+describe PokerstarsTimeStringConverter do
+    context "when first created" do
+        it "should not reject valid pokerstars time strings" do
+            lambda {PokerstarsTimeStringConverter.new("2008/10/31 17:25:42 UTC")}.should_not raise_error
+            lambda {PokerstarsTimeStringConverter.new("2008/10/31 17:25:42 PT")}.should_not raise_error
+            lambda {PokerstarsTimeStringConverter.new("2008/10/31 17:25:42 MT")}.should_not raise_error
+            lambda {PokerstarsTimeStringConverter.new("2008/10/31 17:25:42 CT")}.should_not raise_error
+            lambda {PokerstarsTimeStringConverter.new("2008/10/31 17:25:42 ET")}.should_not raise_error
+            lambda {PokerstarsTimeStringConverter.new("2008/10/31 17:25:42 AT")}.should_not raise_error
+            lambda {PokerstarsTimeStringConverter.new("2008/10/31 17:25:42 BRT")}.should_not raise_error
+            lambda {PokerstarsTimeStringConverter.new("2008/10/31 17:25:42 WET")}.should_not raise_error
+            lambda {PokerstarsTimeStringConverter.new("2008/10/31 17:25:42 CET")}.should_not raise_error
+            lambda {PokerstarsTimeStringConverter.new("2008/10/31 17:25:42 EET")}.should_not raise_error
+            lambda {PokerstarsTimeStringConverter.new("2008/10/31 17:25:42 MSK")}.should_not raise_error
+            lambda {PokerstarsTimeStringConverter.new("2008/10/31 17:25:42 CCT")}.should_not raise_error
+            lambda {PokerstarsTimeStringConverter.new("2008/10/31 17:25:42 JST")}.should_not raise_error
+            lambda {PokerstarsTimeStringConverter.new("2008/10/31 17:25:42 AWST")}.should_not raise_error
+            lambda {PokerstarsTimeStringConverter.new("2008/10/31 17:25:42 ACST")}.should_not raise_error
+            lambda {PokerstarsTimeStringConverter.new("2008/10/31 17:25:42 AEST")}.should_not raise_error
+            lambda {PokerstarsTimeStringConverter.new("2008/10/31 17:25:42 NZT")}.should_not raise_error
+        end
+        
+        it "should reject invalid pokerstars time strings" do
+            lambda {PokerstarsTimeStringConverter.new("2008/10/31 17:25:42 XX")}.should raise_error(ArgumentError)
+        end
+    end
+    
+    context "when converting a pokerstars time string string" do
+        it "returns the correct utc dateime for eastern daylight time" do
+            PokerstarsTimeStringConverter.new("2008/7/30 17:25:42 ET").as_utc_datetime.should == DateTime.parse("2008/7/30 17:25:42 EDT")
+        end
+        
+        it "returns the correct utc result for eastern standard time" do
+            PokerstarsTimeStringConverter.new("2008/01/30 17:25:42 ET").as_utc_datetime.should == DateTime.parse("2008/01/30 17:25:42 EST")
+        end
+    end
+end
+
 describe PokerstarsHandHistoryParser, "recognizes valid hand history" do
   it "should recognize a valid tournament hand history" do
     first_line = "PokerStars Game #21650436825: Tournament #117620218, $10+$1 Hold'em No Limit - Level I (10/20) - 2008/10/31 17:25:42 ET"
@@ -34,12 +73,19 @@ describe PokerstarsHandHistoryParser, "when parsing structural matter" do
     @stats.should_receive(:update_hand).with(
       :name => "PS21650436825",
       :description => "117620218, $10+$1 Hold'em No Limit",
+      :ante => "0.0".to_d,
+      :table_name => "",
+      :max_players => 0,
+      :number_players => 0,
       :sb => "10".to_d,
       :bb => "20".to_d,
-      :played_at => Time.parse("2008/10/31 17:25:42 ET"),
+      :played_at => DateTime.parse("2008/10/31 17:25:42 EDT"),
       :tournament => "117620218",
       :street => :prelude,
-      :board => "" # due to pokerstars hand history bug (does not have Board record unless flop is seen)
+      :board => "",
+      :game_type => "Hold'em",
+      :stakes_type => "10".to_d,
+      :limit_type => "No Limit"
     )
     @parser.parse("PokerStars Game #21650436825: Tournament #117620218, $10+$1 Hold'em No Limit - Level I (10/20) - 2008/10/31 17:25:42 ET")
   end
@@ -52,12 +98,19 @@ describe PokerstarsHandHistoryParser, "when parsing structural matter" do
     @stats.should_receive(:update_hand).with(
       :name => 'PS21650146783',
       :description => "Hold'em No Limit ($0.25/$0.50)",
+      :ante => "0.00".to_d,
+      :table_name => "",
+      :max_players => 0,
+      :number_players => 0,
       :sb => "0.25".to_d, 
       :bb => "0.50".to_d,
-      :played_at => Time.parse("2008/10/31 17:14:44 ET"),
+      :played_at => DateTime.parse("2008/10/31 17:14:44 EDT"),
       :tournament => nil,
       :street => :prelude,
-      :board => "" # due to pokerstars hand history bug
+      :board => "", # due to pokerstars hand history bug
+      :game_type => "Hold'em",
+      :stakes_type => "0.50".to_d,
+      :limit_type => "No Limit"
     )
     @parser.parse("PokerStars Game #21650146783:  Hold'em No Limit ($0.25/$0.50) - 2008/10/31 17:14:44 ET")
   end
@@ -125,21 +178,23 @@ describe PokerstarsHandHistoryParser, "when parsing prelude matter" do
 
   it "should parse a tournament button line" do
     @stats.should_receive(:register_button).with(6)
+    @stats.should_receive(:update_hand).with(:max_players => 9, :table_name => '117620218 1')
     @parser.parse("Table '117620218 1' 9-max Seat #6 is the button")
   end
 
   it "should parse a cash game button line" do
     @stats.should_receive(:register_button).with(2)
-    @parser.parse("Table 'Charybdis IV' 9-max Seat #2 is the button")
+    @stats.should_receive(:update_hand).with(:max_players => 6, :table_name => 'Charybdis IV')
+    @parser.parse("Table 'Charybdis IV' 6-max Seat #2 is the button")
   end
 
   it "should parse a player line" do
-    @stats.should_receive(:register_player).with(:screen_name => 'BadBeat_Brat', :seat => 3)
+    @stats.should_receive(:register_player).with(:screen_name => 'BadBeat_Brat', :seat => 3, :starting_stack => "1500".to_d)
     @parser.parse("Seat 3: BadBeat_Brat (1500 in chips)")
   end
 
   it "should parse a player line with accents" do
-    @stats.should_receive(:register_player).with(:screen_name => 'Gwünni', :seat => 8)
+    @stats.should_receive(:register_player).with(:screen_name => 'Gwünni', :seat => 8, :starting_stack => "3000".to_d)
     @parser.parse("Seat 8: Gwünni (3000 in chips)")
   end
 
@@ -153,8 +208,17 @@ describe PokerstarsHandHistoryParser, "when parsing prelude matter" do
     @parser.parse("BadBeat_Brat: posts big blind 20")
   end
 
-  it "should parse an ante header and register the corresponding action" do
+  it "should parse an ante header and register the corresponding action, :updating the :ante field as well when ante is larger" do
+    @stats.update_hand(:ante=>0)
+    @stats.should_receive(:register_action).with("BadBeat_Brat", 'antes', :result => :post, :amount => "15".to_d)
+    @stats.should_receive(:update_hand).with(:ante => "15".to_d)
+    @parser.parse("BadBeat_Brat: posts the ante 15")
+  end
+
+  it "should parse an ante header and register the corresponding action, :updating the :ante field as well when ante is not larger" do
+    @stats.update_hand(:ante=>20)
     @stats.should_receive(:register_action).with("BadBeat_Brat", 'antes', :result => :post, :amount => "15".to_d)
+    @stats.should_receive(:update_hand).with(:ante => "20".to_d)
     @parser.parse("BadBeat_Brat: posts the ante 15")
   end
 end
@@ -193,5 +257,8 @@ describe PokerstarsHandHistoryParser, "when parsing poker actions" do
     @stats.should_receive(:register_action).with("billy", "wins", :result => :win, :amount => "1.90".to_d)
     @parser.parse("billy collected $1.90 from pot")
   end
-end
-
+  it "should properly parse and register a muck" do
+    @stats.should_receive(:register_action).with("billy", "mucks", :result => :neutral)
+    @parser.parse("billy mucks hand")
+  end
+end

data/spec/zpokerstars_hand_history_parser_integration.rb.txt

@@ -24,27 +24,9 @@ include HandConstants
 #   end
 # end
 # 
-describe PokerstarsHandHistoryParser, "when parsing all the PokerstarsFiles in Andy's directory" do
-  before :each do
-    @filenames = Dir["/Users/werdna/Library/Application Support/Pokerstars/HandHistory/**/*.txt"]
-  end
-
-  it "should parse every line of every entry in the file" do
-    times = 10
-    @filenames.each do |filename|
-      puts "====== #{filename} ====="
-      @psfile = PokerstarsFile.open(filename).entries.each do |handrecord|
-        printf(".")
-        lambda{handrecord.parse}.should_not raise_error
-      end
-      printf("\n")
-    end
-  end
-end
-
-# describe PokerstarsHandHistoryParser, "when parsing all the PokerstarsFiles in Judi's directory" do
+# describe PokerstarsHandHistoryParser, "when parsing all the PokerstarsFiles in Andy's directory" do
 #   before :each do
-#     @filenames = Dir["/Users/werdna/juditest/*.txt"]
+#     @filenames = Dir["/Users/werdna/Library/Application Support/Pokerstars/HandHistory/**/*.txt"]
 #   end
 # 
 #   it "should parse every line of every entry in the file" do
@@ -52,16 +34,34 @@ end
 #     @filenames.each do |filename|
 #       puts "====== #{filename} ====="
 #       @psfile = PokerstarsFile.open(filename).entries.each do |handrecord|
-#         @stats = HandStatistics.new
-#         @parser = PokerstarsHandHistoryParser.new(@stats)
-#         # puts handrecord.[/[^\n]+/]
 #         printf(".")
-#         handrecord.lines.each do |line|
-#           lambda{@parser.parse(line)}.should_not raise_error
-#         end
-#         # @stats.debug_display
+#         lambda{handrecord.parse}.should_not raise_error
 #       end
 #       printf("\n")
 #     end
 #   end
-# end
+# end
+
+describe PokerstarsHandHistoryParser, "when parsing all the PokerstarsFiles in Judi's directory" do
+  before :each do
+    @filenames = Dir["/Users/werdna/juditest/*.txt"]
+  end
+
+  it "should parse every line of every entry in the file" do
+    times = 10
+    @filenames.each do |filename|
+      puts "====== #{filename} ====="
+      @psfile = PokerstarsFile.open(filename).entries.each do |handrecord|
+        @stats = HandStatistics.new
+        @parser = PokerstarsHandHistoryParser.new(@stats)
+        # puts handrecord.[/[^\n]+/]
+        printf(".")
+        handrecord.lines.each do |line|
+          lambda{@parser.parse(line)}.should_not raise_error
+        end
+        # @stats.debug_display
+      end
+      printf("\n")
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: pokerstats
 version: !ruby/object:Gem::Version 
-  version: 2.0.13
+  version: 2.0.14
 platform: ruby
 authors: 
 - Andrew C. Greenberg
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-11-09 00:00:00 -08:00
+date: 2009-11-18 00:00:00 -08:00
 default_executable: checkps
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -75,6 +75,8 @@ files:
 - lib/pokerstats/plugins/cash_statistics.rb
 - lib/pokerstats/plugins/continuation_bet_statistics.rb
 - lib/pokerstats/plugins/preflop_raise_statistics.rb
+- lib/pokerstats/plugins/street_bet_statistics.rb
+- lib/pokerstats/plugins/street_statistics.rb
 - lib/pokerstats/poker-edge.rb
 - lib/pokerstats/pokerstars_file.rb
 - lib/pokerstats/pokerstars_hand_history_parser.rb