ruby-lol 0.12.1 → 0.12.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: de713866840ecd938942d23da2121e4e9be5de6d
-  data.tar.gz: bc47a47f5c743f9e9256b9c9320a0ea2185a6f45
+  metadata.gz: 7905f3bdcac7fda308692df41aff8d9537551896
+  data.tar.gz: e1630306ddc4bc8ece93e8453c14935fdb75700b
 SHA512:
-  metadata.gz: 3ebea6c7910240929cfd8da6c659bd47634533314e65adabd3e293eb4189667fe840dddda9400c432c37ce84487153bb54325686f32d6b3c4984020b27700658
-  data.tar.gz: 9dfdf2379335857fcf4e4205c526f9c56e5d059fe2cb941ce1e509832d5c5739a64207432d6c02bee72fc8db7723823ec010c3f1ba87aa563c975ab86fe3033c
+  metadata.gz: 69fc42943c76aa7dedab87b97d367e8f9dad3e7834b6a1694e4f9c9b0915749f3343c8e12030e5d94aa4ced8518c7f0fd26701e47002dedbb703b5e895270b34
+  data.tar.gz: 954c3a76a2e147b97146151cc05b1ea14afa8b0ed7db1751760fc664c79ddb1b16338a8db7327fb902c47daf0a80d7aaedc1f55cc401ff34b03208640ed4e455

data/.travis.yml

@@ -1,4 +1,4 @@
-rvm: 2.2.3
+rvm: 2.2.5
 services: redis-server
 addons:
     code_climate:

data/lib/lol/request.rb

@@ -38,18 +38,6 @@ module Lol
       "#{url}?#{api_query_string params}"
     end
 
-    def post_api_url path, params = {}
-      {
-        url: api_url(path, params),
-        options: {
-          headers: {
-            "X-Riot-Token" => api_key,
-            "Content-Type" => "application/json"
-          }
-        }
-      }
-    end
-
     def api_base_url
       "https://#{region}.api.pvp.net"
     end
@@ -72,23 +60,39 @@ module Lol
     # @param body [Hash] Body for POST request
     # @param options [Hash] Options passed to HTTParty
     # @return [String] raw response of the call
-    def perform_request url, verb = :get, body = {}, options = {}
+    def perform_request url, verb = :get, body = nil, options = {}
+      options_id = options.inspect
       can_cache = [:post, :put].include?(verb) ? false : cached?
-      if can_cache && result = store.get("#{clean_url(url)}#{options.inspect}")
+      if can_cache && result = store.get("#{clean_url(url)}#{options_id}")
         return JSON.parse(result)
       end
+      response = perform_uncached_request url, verb, body, options
+      store.setex "#{clean_url(url)}#{options_id}", ttl, response.to_json if can_cache
+      response
+    end
 
-      params = [:post, :put].include?(verb) ? [url, options.merge({body: body.to_json})] : url
-      response = self.class.send(verb, *params)
+    def perform_uncached_request url, verb = :get, body = nil, options = {}
+      options[:headers] ||= {}
+      options[:headers].merge!({
+        "Content-Type" => "application/json",
+        "Accept"       => "application/json"
+      })
+      if [:post, :put].include?(verb)
+        options[:body] = body.to_json if body
+        options[:headers]['X-Riot-Token'] = api_key
+      end
+      response = self.class.send(verb, url, options)
       if response.respond_to?(:code) && !(200...300).include?(response.code)
         raise NotFound.new("404 Not Found") if response.not_found?
         raise TooManyRequests.new('429 Rate limit exceeded') if response.code == 429
         raise InvalidAPIResponse.new(url, response)
       end
 
-      store.setex "#{clean_url(url)}#{options.inspect}", ttl, response.to_json if can_cache
-
-      response
+      if response.respond_to?(:parsed_response)
+        response.parsed_response
+      else
+        response
+      end
     end
 
     # @return [Redis] returns the cache store

data/lib/lol/tournament_provider_request.rb

@@ -14,12 +14,11 @@ module Lol
     # @param path [String] API path to call
     # @return [String] full fledged url
     def api_url path, params = {}
-      url = "#{api_base_url}/tournament/public/#{self.class.api_version}/#{path}"
+      "#{api_base_url}/tournament/public/#{self.class.api_version}/#{path}"
     end
 
     def tournament_request path, body
-      pau = post_api_url path
-      perform_request(pau[:url], :post, body, pau[:options]).body
+      perform_request(api_url(path), :post, body)
     end
 
     # Returns a tournament code
@@ -57,7 +56,7 @@ module Lol
       body.merge!({password: password}) if password
 
       params = URI.encode_www_form({tournamentId: tournament_id, count: count})
-      JSON.parse(tournament_request "code?#{params}", body)
+      tournament_request "code?#{params}", body
     end
 
     # Returns the details of the tournament code
@@ -87,8 +86,7 @@ module Lol
         mapType: map_type
       }.reject{ |k,v| v.nil? }
 
-      pau = post_api_url "code/#{tournament_code}"
-      perform_request(pau[:url], :put, body, pau[:options])
+      perform_request(api_url("code/#{tournament_code}"), :put, body)
       get_code tournament_code
     end
 

data/lib/lol/version.rb

@@ -1,3 +1,3 @@
 module Lol
-  VERSION = "0.12.1"
+  VERSION = "0.12.2"
 end

data/ruby-lol.gemspec

@@ -26,8 +26,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "guard-rspec"
   spec.add_development_dependency "ZenTest"
   spec.add_development_dependency "autotest-growl"
-  spec.add_development_dependency "autotest-fsevent"
-  spec.add_development_dependency "codeclimate-test-reporter"
+  spec.add_development_dependency "codeclimate-test-reporter", '~> 0.5.0'
   spec.add_development_dependency "coveralls"
   spec.add_development_dependency "vcr"
   spec.add_development_dependency "webmock", ">= 1.8.0", "< 1.16"

data/spec/lol/request_spec.rb

@@ -37,25 +37,60 @@ describe Request do
 
     it "calls HTTParty get" do
       expect(subject.class).to receive(:get).and_return(error401)
-      expect { subject.perform_request "foo?api_key=asd"}.to raise_error(InvalidAPIResponse)
+      expect { subject.perform_request subject.api_url("foo")}.to raise_error(InvalidAPIResponse)
+    end
+
+    it "sets content type in get requests" do
+      expect(subject.class).to receive(:get) do |url, options|
+        expect(options[:headers]["Content-Type"]).to eq "application/json"
+      end
+      subject.perform_request subject.api_url("foo")
+    end
+
+    it "sets the accept header in get requests" do
+      expect(subject.class).to receive(:get) do |url, options|
+        expect(options[:headers]["Accept"]).to eq "application/json"
+      end
+      subject.perform_request subject.api_url("foo")
+    end
+
+    it "sets content type in post requests" do
+      expect(subject.class).to receive(:post) do |url, options|
+        expect(options[:headers]["Content-Type"]).to eq "application/json"
+      end
+      subject.perform_request subject.api_url("foo"), :post
+    end
+
+    it "sets the accept header in post requests" do
+      expect(subject.class).to receive(:post) do |url, options|
+        expect(options[:headers]["Accept"]).to eq "application/json"
+      end
+      subject.perform_request subject.api_url("foo"), :post
+    end
+
+    it "sets the api key header in post requests" do
+      expect(subject.class).to receive(:post) do |url, options|
+        expect(options[:headers]["X-Riot-Token"]).to eq(subject.api_key)
+      end
+      subject.perform_request subject.api_url("foo"), :post
     end
 
     it "handles 401" do
       expect(subject.class).to receive(:get).and_return(error401)
-      expect { subject.perform_request "foo?api_key=asd" }.to raise_error(InvalidAPIResponse)
+      expect { subject.perform_request subject.api_url("foo") }.to raise_error(InvalidAPIResponse)
     end
 
     it "handles 404" do
       expect(error401).to receive(:respond_to?).at_least(:once).and_return(true)
       expect(error401).to receive(:not_found?).and_return(true)
       expect(subject.class).to receive(:get).and_return(error401)
-      expect { subject.perform_request "foo?api_key=asd"}.to raise_error(NotFound)
+      expect { subject.perform_request subject.api_url("foo")}.to raise_error(NotFound)
     end
 
     it 'handles 429' do
       expect(error429).to receive(:respond_to?).and_return(true)
       expect(subject.class).to receive(:get).and_return(error429)
-      expect { subject.perform_request "foo?api_key=asd"}.to raise_error(TooManyRequests)
+      expect { subject.perform_request subject.api_url("foo")}.to raise_error(TooManyRequests)
     end
 
     context "post requests" do
@@ -88,7 +123,7 @@ describe Request do
       let(:fake_redis) { FakeRedis.new }
       let(:request) { Request.new "api_key", "euw", {redis: fake_redis, ttl: 60, cached: true }}
       before :each do
-        expect(request.class).to receive(:get).with("/foo?api_key=asd").and_return({foo: "bar"}).at_least(:once)
+        expect(request.class).to receive(:get).with("/foo?api_key=asd", instance_of(Hash)).and_return({foo: "bar"}).at_least(:once)
         first_result = request.perform_request "/foo?api_key=asd"
       end
 
@@ -157,39 +192,6 @@ describe Request do
     end
   end
 
-  describe "post_api_url" do
-    let(:pau) { subject.post_api_url "/foo" }
-
-    it "returns an hash" do
-      expect(pau).to be_a(Hash)
-    end
-
-    it "contains an url key" do
-      expect(pau).to have_key(:url)
-    end
-
-    it "contains an options key" do
-      expect(pau).to have_key(:options)
-    end
-
-    it "contains a header key in options" do
-      expect(pau[:options]).to have_key(:headers)
-    end
-
-    it "returns the api key in an header" do
-      expect(pau[:options][:headers]).to have_key("X-Riot-Token")
-      expect(pau[:options][:headers]["X-Riot-Token"]).to eq(subject.api_key)
-    end
-
-    it "includes a content-type header" do
-      expect(pau[:options][:headers]["Content-Type"]).to eq("application/json")
-    end
-
-    it "uses api_url for the url part" do
-      expect(pau[:url]).to eq(subject.api_url "/foo")
-    end
-  end
-
   describe "#api_base_url" do
     it "returns the base domain" do
       expect(subject.api_base_url).to eq "https://euw.api.pvp.net"

data/spec/lol/tournament_provider_request_spec.rb

@@ -21,20 +21,25 @@ describe TournamentProviderRequest do
   describe "#provider" do
     subject { request.provider("EUW", "https://foo.com") }
 
-    # before(:each) { stub_request(request, 'tournament', "tournament/public/#/recent") }
-
-    # it 'returns an array' do
-    #   expect(subject).to be_a(Array)
-    # end
-    #
-    # it 'returns an array of Games' do
-    #   expect(subject.map(&:class).uniq).to eq([Game])
-    # end
-    #
-    # it 'fetches games from the API' do
-    #   fixture = load_fixture('game', GameRequest.api_version)
-    #   expect(subject.size).to eq(fixture['games'].size)
-    # end
+    before(:each) do
+      expect(TournamentProviderRequest).to receive(:perform_request).and_return 10
+    end
+
+    it 'return the provider id' do
+      expect(subject).to eq 10
+    end
+  end
+
+  describe "#tournament" do
+    subject { request.provider("test", 10) }
+
+    before(:each) do
+      expect(TournamentProviderRequest).to receive(:perform_request).and_return 20
+    end
+
+    it 'return the tournament provider id' do
+      expect(subject).to eq 20
+    end
   end
 
   describe "#get_code" do
@@ -43,7 +48,7 @@ describe TournamentProviderRequest do
       full_url = request.api_url "code/CODE-FOR-TEST?#{request.api_query_string}"
       fixture_json = load_fixture('get-code', TournamentProviderRequest.api_version)
 
-      expect(TournamentProviderRequest).to receive(:get).with(full_url).and_return(fixture_json)
+      expect(TournamentProviderRequest).to receive(:get).with(full_url, instance_of(Hash)).and_return(fixture_json)
     end
     subject { request.get_code "CODE-FOR-TEST" }
 
@@ -59,8 +64,7 @@ describe TournamentProviderRequest do
       expect(request).to receive(:perform_request).once.ordered.with(
         instance_of(String),
         :put,
-        { allowedParticipants: "1,2,3,4,5,6,7,8,9,10" },
-        instance_of(Hash)
+        { allowedParticipants: "1,2,3,4,5,6,7,8,9,10" }
       )
       expect(request).to receive(:perform_request).once.ordered.with(instance_of(String)).and_return(fixture)
       request.update_code "CODE-FOR-TEST", { allowed_participants: [1,2,3,4,5,6,7,8,9,10] }

data/spec/support/helpers.rb

@@ -51,13 +51,13 @@ module Helpers
     full_url = request_object.api_url(url, params)
     fixture_json = load_fixture(fixture_name, request_class.api_version, :get)
 
-    expect(request_class).to receive(:get).with(full_url).and_return(fixture_json)
+    expect(request_class).to receive(:get).with(full_url, instance_of(Hash)).and_return(fixture_json)
   end
 
   def stub_request_raw(request_object, raw_response, url, params={})
     request_class = request_object.class
     full_url = request_object.api_url(url, params)
 
-    expect(request_class).to receive(:get).with(full_url).and_return(raw_response)
+    expect(request_class).to receive(:get).with(full_url, instance_of(Hash)).and_return(raw_response)
   end
-end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ruby-lol
 version: !ruby/object:Gem::Version
-  version: 0.12.1
+  version: 0.12.2
 platform: ruby
 authors:
 - Giovanni Intini
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-08-29 00:00:00.000000000 Z
+date: 2016-10-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -122,34 +122,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: autotest-fsevent
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: codeclimate-test-reporter
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.5.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.5.0
 - !ruby/object:Gem::Dependency
   name: coveralls
   requirement: !ruby/object:Gem::Requirement