summoner masteries 1.3 and call for multiple summoners masterypages
This commit is contained in:
@@ -158,11 +158,11 @@ class SummonerModule < SightstoneBaseModule
|
|||||||
else
|
else
|
||||||
summoner
|
summoner
|
||||||
end
|
end
|
||||||
uri = "http://prod.api.pvp.net/api/lol/#{region}/v1.2/summoner/#{id}/masteries"
|
uri = "http://prod.api.pvp.net/api/lol/#{region}/v1.3/summoner/#{id}/masteries"
|
||||||
response = _get_api_response(uri)
|
response = _get_api_response(uri)
|
||||||
_parse_response(response) { |resp|
|
_parse_response(response) { |resp|
|
||||||
data = JSON.parse(resp)
|
data = JSON.parse(resp)
|
||||||
book = MasteryBook.new(data)
|
book = MasteryBook.new(data.values[0])
|
||||||
if block_given?
|
if block_given?
|
||||||
yield book
|
yield book
|
||||||
else
|
else
|
||||||
@@ -170,5 +170,36 @@ class SummonerModule < SightstoneBaseModule
|
|||||||
end
|
end
|
||||||
}
|
}
|
||||||
end
|
end
|
||||||
|
|
||||||
|
# returns the masterybooks for multiple summoners
|
||||||
|
# @param summoners [Array<(Summoner, Integer)>] list of summoner objects or ids of summoners
|
||||||
|
# @param optional [Hash<Symbol, String>] optional arguments: :region => replaces default region
|
||||||
|
# @return [Hash<Integer, Masterybook>] A hash mapping masterybooks to the ids of summoners
|
||||||
|
def masterybooks(summoners, optional={})
|
||||||
|
return {} if summoners.empty?
|
||||||
|
region = optional[:region] || @sightstone.region
|
||||||
|
ids = summoners.collect { |summoner|
|
||||||
|
if summoner.is_a? Summoner
|
||||||
|
summoner.id
|
||||||
|
else
|
||||||
|
summoner
|
||||||
|
end
|
||||||
|
}
|
||||||
|
|
||||||
|
uri = "http://prod.api.pvp.net/api/lol/#{region}/v1.3/summoner/#{ids.join(',')}/masteries"
|
||||||
|
response = _get_api_response(uri)
|
||||||
|
_parse_response(response) { |resp|
|
||||||
|
data = JSON.parse(resp)
|
||||||
|
books = {}
|
||||||
|
data.each do |key, raw_book|
|
||||||
|
books[key] = MasteryBook.new(raw_book)
|
||||||
|
end
|
||||||
|
if block_given?
|
||||||
|
yield books
|
||||||
|
else
|
||||||
|
return books
|
||||||
|
end
|
||||||
|
}
|
||||||
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
@@ -162,6 +162,40 @@ class SummonerModuleTest < BaseTest
|
|||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
def test_summoner_masterybooks
|
||||||
|
begin
|
||||||
|
masterybooks = @@sightstone.summoner.masterybooks(@@test_id_array)
|
||||||
|
rescue Sightstone::RateLimitExceededException
|
||||||
|
puts "Rate limit exeeded, waiting 1 sec"
|
||||||
|
sleep 1
|
||||||
|
retry
|
||||||
|
end
|
||||||
|
masterybooks.each do |id, masterybook|
|
||||||
|
assert_equal(@@test_id_array[0], masterybook.summonerId)
|
||||||
|
assert_instance_of(Array, masterybook.pages)
|
||||||
|
if masterybook.pages.size > 0
|
||||||
|
# test a page
|
||||||
|
page = masterybook.pages[0]
|
||||||
|
if(page.current == true || page.current == false)
|
||||||
|
assert_equal(true, true)
|
||||||
|
else
|
||||||
|
assert_equal(page.current, "asd")
|
||||||
|
end
|
||||||
|
assert_instance_of(Fixnum, page.id)
|
||||||
|
assert_instance_of(String, page.name)
|
||||||
|
assert_instance_of(Array, page.talents)
|
||||||
|
|
||||||
|
if(page.talents.size > 0)
|
||||||
|
# test talent
|
||||||
|
talent = page.talents[0]
|
||||||
|
assert_instance_of(Fixnum, talent.rank)
|
||||||
|
assert_instance_of(String, talent.name)
|
||||||
|
assert_instance_of(Fixnum, talent.id)
|
||||||
|
end
|
||||||
|
end
|
||||||
|
end
|
||||||
|
end
|
||||||
|
|
||||||
|
|
||||||
end
|
end
|
||||||
Reference in New Issue
Block a user