Merge pull request #43 from fedemoretti/current_game_region_fix

- Fixed platform id for some regions
mikamai · May 4, 2015 · e3a0dc8 · e3a0dc8
2 parents bacdb32 + 9754d9b
commit e3a0dc8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/lib/lol/current_game_request.rb b/lib/lol/current_game_request.rb
@@ -9,7 +9,7 @@ def api_url path, params = {}
     end
 
     def spectator_game_info platform_id, summoner_id
-      url = api_url "getSpectatorGameInfo/#{region.upcase}#{platform_id}/#{summoner_id}"
+      url = api_url "getSpectatorGameInfo/#{platform_id}/#{summoner_id}"
       DynamicModel.new perform_request(url)
     end
   end

diff --git a/spec/lol/current_game_request_spec.rb b/spec/lol/current_game_request_spec.rb
@@ -35,13 +35,13 @@
 
     it 'returns a DynamicModel' do
       stub_request subject, 'current-game', expected_url
-      expect(subject.spectator_game_info 1, '1').to be_a DynamicModel
+      expect(subject.spectator_game_info 'EUW1', '1').to be_a DynamicModel
     end
 
     it 'gives the response to DynamicModel' do
       allow(subject).to receive(:perform_request).with(instance_of(String)).and_return 'a'
       expect(Lol::DynamicModel).to receive(:new).with('a').and_return 'foo'
-      subject.spectator_game_info 1, '1'
+      subject.spectator_game_info 'EUW1', '1'
     end
   end
 end