yahoo_nba 0.0.2.alpha → 0.0.2

data/README.md

@@ -1,6 +1,6 @@
 # YahooNba
 
-This gem uses the yahoo sports api to retrieve stats for nba players
+This gem uses the YAHOO Sports API to retrieve stats for NBA players
 
 ## Installation
 
@@ -18,7 +18,25 @@ Or install it yourself as:
 
 ## Usage
 
-TODO: Write usage instructions here
+This gem assists in retrieving player keys and player stats using the Yahoo Sports API.
+
+require 'yahoo_nba'
+query = YahooNba::Query.new 'your_consumer_key', 'your_consumer_secret'
+(to obtain a key, go to http://developer.yahoo.com/)
+
+To get a hash with all the player keys:
+player_keys_hash = query.get_all_player_keys_hash
+
+The hash will be in the format:
+{"Player Name"=>"key123", "Player2 Name"=> "key234"}
+
+To get nba player stats:
+query.get_players_stats_hash_using(players_keys_hash)
+
+The hash will be in the format returned by the Yahoo Sports API.  
+
+For more information see:
+http://developer.yahoo.com/fantasysports/guide/
 
 ## Contributing
 

data/lib/yahoo_nba/version.rb

@@ -1,3 +1,3 @@
 module YahooNba
-  VERSION = "0.0.2.alpha"
+  VERSION = "0.0.2"
 end

data/lib/yahoo_nba/yahoo_nba.rb

@@ -16,7 +16,9 @@ module YahooNba
 
     def get_players_key_hash_from(players_info_array) 
       players_key_hash = {}
-      players_info_array.each{|item| players_key_hash[item["name"]["full"]] = item['player_key']}
+      players_info_array.each do |item| 
+        players_key_hash[item["name"]["full"]] = item['player_key']
+      end
       players_key_hash
     end
 
@@ -29,5 +31,13 @@ module YahooNba
       end
       players_stats_hash
     end
+
+    def get_all_player_keys_hash
+      combined_array = []
+      0.step(500, 25) do |n|
+        combined_array = combined_array | get_players_info_array_starting_at(n)
+      end
+      get_players_key_hash_from(combined_array)
+    end
   end
 end

data/spec/yahoo_nba/yahoo_nba_spec.rb

@@ -12,51 +12,102 @@ describe YahooNba do
       @consumer_secret = "a7d3c791b23f35774c608b7863b2d47508560927"  
     end
 
-    it "should have initialize creates a new Query" do
-      query = YahooNba::Query.new(@consumer_key, @consumer_secret)
-      query.should_not equal(nil)
-      query.class.should equal(YahooNba::Query)
-    end
+    describe "initialize" do
 
-    it "should have initialize creates new OAuth::Consumer" do
-      OAuth::Consumer.should_receive(:new).with(@consumer_key, @consumer_secret, anything)
-      query = YahooNba::Query.new(@consumer_key, @consumer_secret)
-    end
+      it "should create a new Query" do
+        query = YahooNba::Query.new(@consumer_key, @consumer_secret)
+        query.should_not equal(nil)
+        query.class.should equal(YahooNba::Query)
+      end
+
+      it "should create new OAuth::Consumer" do
+        OAuth::Consumer.should_receive(:new).with(@consumer_key, @consumer_secret, anything)
+        query = YahooNba::Query.new(@consumer_key, @consumer_secret)
+      end
+
+      it "should sets access_token" do
+        query = YahooNba::Query.new(@consumer_key, @consumer_secret)
+        query.instance_variable_get(:@access_token).should_not equal(nil)
+      end
 
-    it "should have initialize sets access_token" do
-      query = YahooNba::Query.new(@consumer_key, @consumer_secret)
-      query.instance_variable_get(:@access_token).should_not equal(nil)
     end
+    
+    describe "get_players_starting_at" do
+
+      it "should call get on the access token and parse the response" do
+        query = YahooNba::Query.new(@consumer_key, @consumer_secret)
+        access_token = query.instance_variable_get(:@access_token) 
+        num = 20
+        mock_xml = double
+        mock_response = double(:body => mock_xml)
+        access_token.should_receive(:get).with("/fantasy/v2/game/249/players;start=#{num};count=25").and_return(mock_response)
+        Crack::XML.should_receive(:parse).with(mock_xml).and_return({'fantasy_content' => {'game' => {'players' => {'player' => 'right!'}}}})
+        query.get_players_info_array_starting_at(num).should == 'right!'
+      end
+
+      it "should call get request for players up to 500 when requested" do
+        query = YahooNba::Query.new(@consumer_key, @consumer_secret)
+        access_token = query.instance_variable_get(:@access_token)
+        mock_xml = double
+        mock_get_response = double(:body => mock_xml)
+        0.step(500, 25) do |num|
+          Crack::XML.should_receive(:parse).with(mock_xml).and_return({'fantasy_content' => {'game' => {'players' => {'player' => 'right!'}}}})
+          access_token.should_receive(:get).with("/fantasy/v2/game/249/players;start=#{num};count=25").and_return(mock_get_response)
+        end
+        0.step(500, 25) do |n|
+          query.get_players_info_array_starting_at(n)
+        end
+      end
 
-    it "should have get_players_starting_at call get on the access token and parse the response" do
-      query = YahooNba::Query.new(@consumer_key, @consumer_secret)
-      access_token = query.instance_variable_get(:@access_token) 
-      num = 20
-      mock_xml = double
-      mock_response = double(:body => mock_xml)
-      access_token.should_receive(:get).with("/fantasy/v2/game/249/players;start=#{num};count=25").and_return(mock_response)
-      Crack::XML.should_receive(:parse).with(mock_xml).and_return({'fantasy_content' => {'game' => {'players' => {'player' => 'right!'}}}})
-      query.get_players_info_array_starting_at(num).should == 'right!'
     end
 
-    it "should get players key has from players_info array" do
-      players_info_array = [{"player_key"=>"249.p.4244", "player_id"=>"4244", "name"=>{"full"=>"Kevin Durant", "first"=>"Kevin", "last"=>"Durant", "ascii_first"=>"Kevin", "ascii_last"=>"Durant"}, "editorial_player_key"=>"nba.p.4244", "editorial_team_key"=>"nba.t.25", "editorial_team_full_name"=>"Oklahoma City Thunder", "editorial_team_abbr"=>"OKC", "uniform_number"=>"35", "display_position"=>"SF", "image_url"=>"http://l.yimg.com/a/i/us/sp/v/nba/players_l/20101116/4244.jpg?x=46&y=60&xc=1&yc=1&wc=164&hc=215&q=100&sig=OTXf3hs63PFp0fv8wxtCIg--", "is_undroppable"=>"0", "position_type"=>"P", "eligible_positions"=>{"position"=>"SF"}, "has_player_notes"=>"1", "has_recent_player_notes"=>"1"}]
-      players_key_hash = {"Kevin Durant"=>"249.p.4244"} 
-      query = YahooNba::Query.new(@consumer_key, @consumer_secret)
-      hash_results = query.get_players_key_hash_from(players_info_array)
-      hash_results.should == players_key_hash
+    describe "should_get_player_key" do
+
+      it "should be able to get players keys from players_info array" do
+        players_info_array = [{"player_key"=>"249.p.4244", "player_id"=>"4244", "name"=>{"full"=>"Kevin Durant", "first"=>"Kevin", "last"=>"Durant", "ascii_first"=>"Kevin", "ascii_last"=>"Durant"}, "editorial_player_key"=>"nba.p.4244", "editorial_team_key"=>"nba.t.25", "editorial_team_full_name"=>"Oklahoma City Thunder", "editorial_team_abbr"=>"OKC", "uniform_number"=>"35", "display_position"=>"SF", "image_url"=>"http://l.yimg.com/a/i/us/sp/v/nba/players_l/20101116/4244.jpg?x=46&y=60&xc=1&yc=1&wc=164&hc=215&q=100&sig=OTXf3hs63PFp0fv8wxtCIg--", "is_undroppable"=>"0", "position_type"=>"P", "eligible_positions"=>{"position"=>"SF"}, "has_player_notes"=>"1", "has_recent_player_notes"=>"1"}]
+        players_key_hash = {"Kevin Durant"=>"249.p.4244"} 
+        query = YahooNba::Query.new(@consumer_key, @consumer_secret)
+        hash_results = query.get_players_key_hash_from(players_info_array)
+        hash_results.should == players_key_hash
+      end
+
     end
+    
+    describe "should_get_players_stats" do
 
-    it "should get players stats using players_key_hash" do
-      query = YahooNba::Query.new(@consumer_key, @consumer_secret)
-      players_key_hash = {"Kevin Durant"=>"249.p.4244"} 
-      access_token = query.instance_variable_get(:@access_token) 
-      mock_xml = double
-      mock_response = double(:body => mock_xml)
-      access_token.should_receive(:get).with("/fantasy/v2/player/#{players_key_hash['Kevin Durant']}/stats").and_return(mock_response)
-      Crack::XML.should_receive(:parse).with(mock_xml).and_return({"fantasy_content" => {"player" => "the right stats!"}})
-      player_stats_hash = query.get_players_stats_hash_using(players_key_hash)
-      player_stats_hash.should == {"Kevin Durant" => "the right stats!"}
+      it "should be able to get player stats using players_key_hash" do
+        query = YahooNba::Query.new(@consumer_key, @consumer_secret)
+        players_key_hash = {"Kevin Durant"=>"249.p.4244"} 
+        access_token = query.instance_variable_get(:@access_token) 
+        mock_xml = double
+        mock_response = double(:body => mock_xml)
+        access_token.should_receive(:get).with("/fantasy/v2/player/#{players_key_hash['Kevin Durant']}/stats").and_return(mock_response)
+        Crack::XML.should_receive(:parse).with(mock_xml).and_return({"fantasy_content" => {"player" => "the right stats!"}})
+        player_stats_hash = query.get_players_stats_hash_using(players_key_hash)
+        player_stats_hash.should == {"Kevin Durant" => "the right stats!"}
+      end
+
+    end
+   
+    describe "get_all_player_keys_hash" do
+      it "should get a hash with all player keys for 500 players" do
+        query = YahooNba::Query.new(@consumer_key, @consumer_secret)
+        access_token = query.instance_variable_get(:@access_token)
+        mock_xml = double
+        mock_get_response = double(:body => mock_xml)
+        0.step(500, 25) do |num|
+          Crack::XML.should_receive(:parse).with(mock_xml).and_return(
+                                            {'fantasy_content' => 
+                                            {'game' => 
+                                            {'players' => 
+                                            {'player' => 
+                                            [{"name" => 
+                                            {"full" => "Kevin Durant"},
+                                            "player_key" => "key123"}]}}}})
+          access_token.should_receive(:get).with("/fantasy/v2/game/249/players;start=#{num};count=25").and_return(mock_get_response)
+        end
+        p query.get_all_player_keys_hash
+      end
     end
 
   end

data/yahoo_nba.gemspec

@@ -6,7 +6,7 @@ Gem::Specification.new do |gem|
   gem.email         = ["jc582@optonline.net	"]
   gem.description   = %q{Get nba stats using yahoo api}
   gem.summary       = %q{Gem to get nba player stats using yahoo api}
-  gem.homepage      = ""
+  gem.homepage      = "https://github.com/jamesjn/yahoo_nba"
 
   gem.files         = `git ls-files`.split($\)
   gem.executables   = gem.files.grep(%r{^bin/}).map{ |f| File.basename(f) }
@@ -15,6 +15,9 @@ Gem::Specification.new do |gem|
   gem.require_paths = ["lib"]
   gem.version       = YahooNba::VERSION
 
+  gem.add_runtime_dependency 'oauth'
+  gem.add_runtime_dependency 'crack'
+
 	gem.add_development_dependency "rake"
 	gem.add_development_dependency "rspec"
 end

metadata

@@ -1,19 +1,41 @@
 --- !ruby/object:Gem::Specification
 name: yahoo_nba
 version: !ruby/object:Gem::Version
-  version: 0.0.2.alpha
-  prerelease: 6
+  version: 0.0.2
+  prerelease: 
 platform: ruby
 authors:
 - jamesjn
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-04-01 00:00:00.000000000 Z
+date: 2012-04-07 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: oauth
+  requirement: &70253070039920 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: *70253070039920
+- !ruby/object:Gem::Dependency
+  name: crack
+  requirement: &70253070039420 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: *70253070039420
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70167040790720 !ruby/object:Gem::Requirement
+  requirement: &70253070038980 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +43,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70167040790720
+  version_requirements: *70253070038980
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70167040790300 !ruby/object:Gem::Requirement
+  requirement: &70253070038480 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,7 +54,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70167040790300
+  version_requirements: *70253070038480
 description: Get nba stats using yahoo api
 email:
 - ! "jc582@optonline.net\t"
@@ -53,7 +75,7 @@ files:
 - spec/spec_helper.rb
 - spec/yahoo_nba/yahoo_nba_spec.rb
 - yahoo_nba.gemspec
-homepage: ''
+homepage: https://github.com/jamesjn/yahoo_nba
 licenses: []
 post_install_message: 
 rdoc_options: []
@@ -68,9 +90,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
-  - - ! '>'
+  - - ! '>='
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.6