riot_lol_api 0.3.2 → 0.3.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 75c6f27bdeed2052301f7f5f39794e78725988aa
-  data.tar.gz: 5f6888386b098d2bd10d053e39d10edaa825bbc8
+  metadata.gz: e75080a4edfe0e84ada85a331308a03ca1772bff
+  data.tar.gz: 3bcc1d81c40fc7b2d8e4d633ea47cf587290ffab
 SHA512:
-  metadata.gz: b7014966dc977243a2872d5704f83790500188225fc1cafced42d451a710b5ffd94e864ae4ce8c3c4352dc0d71508a223164d8d1f3b465385f1b9a698da7926f
-  data.tar.gz: ed201bda5e9ad41fa29682e25a38f66c263bb9111ca28073b92321d5e525e6fd202bb376a445c1765d62e8051ef8beb87bc34d1d9134e31a084eb6de856216d1
+  metadata.gz: 21a03fe45c6afdc8726e1672744922fe1adf9cbe2d0d893d8465512e70e80dd94cc3c3156e2abfd2768d2dc641e7e52a187cae7d3d44ce250ce31c26e65492bd
+  data.tar.gz: 15274f194545e8360d9a543e40ea2ffa3debf97f732b0640a1470b003ec767937712cd1c0adf123eca9cd951e399c16597c7887f49303d80a8ae19928a780110

data/.gitignore

@@ -16,3 +16,5 @@ test/tmp
 test/version_tmp
 tmp
 .DS_Store
+.ruby-gemset
+.ruby-version

data/.rubocop.yml

@@ -0,0 +1,5 @@
+Metrics/LineLength:
+  Enabled: false
+Documentation:
+  Enabled: false
+

data/.travis.yml

@@ -0,0 +1,6 @@
+language: ruby
+rvm:
+  - 2.3.0
+addons:
+  code_climate:
+    repo_token: cd03a3905d9837bf65f9747b8dfacbc31ae8d47ceda70f4530e0ef9a545c9d91

data/README.md

@@ -1,6 +1,10 @@
 # Gem Riot games API wrapper for Ruby - League of Legends
 
 [![Gem Version](https://badge.fury.io/rb/riot_lol_api.svg)](http://badge.fury.io/rb/riot_lol_api)
+[![Build Status](https://travis-ci.org/francois-blanchard/riot_lol_api.svg)](https://travis-ci.org/francois-blanchard/riot_lol_api)
+[![Code Climate](https://codeclimate.com/github/francois-blanchard/riot_lol_api/badges/gpa.svg)](https://codeclimate.com/github/francois-blanchard/riot_lol_api)
+[![Issue Count](https://codeclimate.com/github/francois-blanchard/riot_lol_api/badges/issue_count.svg)](https://codeclimate.com/github/francois-blanchard/riot_lol_api)
+[![Test Coverage](https://codeclimate.com/github/francois-blanchard/riot_lol_api/badges/coverage.svg)](https://codeclimate.com/github/francois-blanchard/riot_lol_api/coverage)
 
 ```
  __________.__        __                                      _____ __________.___
@@ -17,6 +21,7 @@
 
 ```
 - champion-v1.2         NO IMPLEMENT
+- championmastery       OK
 - current-game-v1.0     OK
 - featured-games-v1.0   OK
 - game-v1.3             OK
@@ -24,7 +29,7 @@
 - lol-static-data-v1.2  OK
 - lol-status-v1.0       NO IMPLEMENT
 - match-v2.2            OK
-- matchhistory-v2.2     PROGRESS
+- matchlist-v2.2     PROGRESS
 - stats-v1.3            OK
 - summoner-v1.4         OK
 - team-v2.4             NO IMPLEMENT
@@ -54,11 +59,11 @@ Get token api on [http://developer.riotgames.com/](http://developer.riotgames.co
 ###2) Scripts
 Sample scripts
 ```ruby
-# Set token
-RiotLolApi::TOKEN = XXXXXXXXXXXXXXXXXXXXXXXXXXXX
-
 # Create client
-client = RiotLolApi::Client.new(:region => 'your_region')
+client = RiotLolApi::Client.new do |config|
+  config.region = 'your_region'
+  config.api_key = 'xxxxxxxxxxxx'
+end
 
 # Start get data
 summoner = client.get_summoner_by_name 'your_summoner_name'
@@ -85,7 +90,7 @@ client.get_rune_by_id id, {:version => num_version, :runeListData => 'all'}, 'fr
 client.get_all_runes {:version => num_version, :runeListData => 'all'}, 'fr_FR'
 client.get_summoner_spell_by_id id, {:version => num_version, :spellData => 'all'}, 'fr_FR'
 client.get_all_summoner_spells {:version => num_version, :spellData => 'all'}, 'false', 'fr_FR'
-client.get_versions
+client.versions
 
 ############
 # SUMMONER #

data/Rakefile

@@ -1 +1,6 @@
-require "bundler/gem_tasks"
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
+
+RSpec::Core::RakeTask.new(:spec)
+
+task default: :spec

data/curl_import.rb

@@ -3,193 +3,213 @@ API_KEY = ARGV[0]
 
 # Tab files
 TAB_CALLS = [
+  # championmastery
+  {
+    url: 'euw.api.pvp.net/championmastery/location/EUW1/player/20639710/champion/89',
+    filename: 'championmastery_by_summoner_by_champion',
+    params: nil
+  },
+  {
+    url: 'euw.api.pvp.net/championmastery/location/EUW1/player/20639710/champions',
+    filename: 'championmastery_by_summoner',
+    params: nil
+  },
+  {
+    url: 'euw.api.pvp.net/championmastery/location/EUW1/player/20639710/score',
+    filename: 'championmastery_score',
+    params: nil
+  },
+  {
+    url: 'euw.api.pvp.net/championmastery/location/EUW1/player/20639710/topchampions',
+    filename: 'championmastery_top',
+    params: nil
+  },
   # game-v1.3
   {
-    :url => "euw.api.pvp.net/api/lol/euw/v1.3/game/by-summoner/20639710/recent",
-    :filename => "get_summoner_games_by_id",
-    :params => nil
+    url: 'euw.api.pvp.net/api/lol/euw/v1.3/game/by-summoner/20639710/recent',
+    filename: 'get_summoner_games_by_id',
+    params: nil
   },
   # league-v2.5
   {
-    :url => "euw.api.pvp.net/api/lol/euw/v2.5/league/by-summoner/20639710/entry",
-    :filename => "get_player_league",
-    :params => nil
+    url: 'euw.api.pvp.net/api/lol/euw/v2.5/league/by-summoner/20639710/entry',
+    filename: 'get_player_league',
+    params: nil
   },
   # lol-static-data-v1.2
   {
-    :url => "global.api.pvp.net/api/lol/static-data/euw/v1.2/champion",
-    :filename => "get_all_champions_by_ids",
-    :params => {
-        :locale => 'fr_FR'
-      }
+    url: 'global.api.pvp.net/api/lol/static-data/euw/v1.2/champion',
+    filename: 'get_all_champions_by_ids',
+    params: {
+      locale: 'fr_FR'
+    }
   },
   {
-    :url => "global.api.pvp.net/api/lol/static-data/euw/v1.2/champion",
-    :filename => "get_all_champions_by_ids_all_data",
-    :params => {
-        :locale => 'fr_FR',
-        :champData => 'all'
-      }
+    url: 'global.api.pvp.net/api/lol/static-data/euw/v1.2/champion',
+    filename: 'get_all_champions_by_ids_all_data',
+    params: {
+      locale: 'fr_FR',
+      champData: 'all'
+    }
   },
   {
-    :url => "global.api.pvp.net/api/lol/static-data/euw/v1.2/champion/412",
-    :filename => "get_champion_by_id",
-    :params => {
-      :locale => 'fr_FR'
+    url: 'global.api.pvp.net/api/lol/static-data/euw/v1.2/champion/412',
+    filename: 'get_champion_by_id',
+    params: {
+      locale: 'fr_FR'
     }
   },
   {
-    :url => "global.api.pvp.net/api/lol/static-data/euw/v1.2/champion/412",
-    :filename => "get_champion_by_id_all_data",
-    :params => {
-      :locale => 'fr_FR',
-      :champData => 'all'
+    url: 'global.api.pvp.net/api/lol/static-data/euw/v1.2/champion/412',
+    filename: 'get_champion_by_id_all_data',
+    params: {
+      locale: 'fr_FR',
+      champData: 'all'
     }
   },
   {
-    :url => "global.api.pvp.net/api/lol/static-data/euw/v1.2/item",
-    :filename => "get_all_item",
-    :params => {
-        :locale => 'fr_FR',
-        :itemListData => 'all'
-      }
+    url: 'global.api.pvp.net/api/lol/static-data/euw/v1.2/item',
+    filename: 'get_all_item',
+    params: {
+      locale: 'fr_FR',
+      itemListData: 'all'
+    }
   },
   {
-    :url => "global.api.pvp.net/api/lol/static-data/euw/v1.2/item/2009",
-    :filename => "get_item_by_id",
-    :params => {
-        :locale => 'fr_FR',
-        :itemData => 'all'
-      }
+    url: 'global.api.pvp.net/api/lol/static-data/euw/v1.2/item/2009',
+    filename: 'get_item_by_id',
+    params: {
+      locale: 'fr_FR',
+      itemData: 'all'
+    }
   },
   {
-    :url => "global.api.pvp.net/api/lol/static-data/euw/v1.2/mastery",
-    :filename => "get_all_masteries",
-    :params => {
-      :locale => 'fr_FR',
-      :masteryListData => 'all'
+    url: 'global.api.pvp.net/api/lol/static-data/euw/v1.2/mastery',
+    filename: 'get_all_masteries',
+    params: {
+      locale: 'fr_FR',
+      masteryListData: 'all'
     }
   },
   {
-    :url => "global.api.pvp.net/api/lol/static-data/euw/v1.2/mastery/4353",
-    :filename => "get_mastery_by_id",
-    :params => {
-      :locale => 'fr_FR',
-      :masteryData => 'all'
+    url: 'global.api.pvp.net/api/lol/static-data/euw/v1.2/mastery/6363',
+    filename: 'get_mastery_by_id',
+    params: {
+      locale: 'fr_FR',
+      masteryData: 'all'
     }
   },
   {
-    :url => "global.api.pvp.net/api/lol/static-data/euw/v1.2/realm",
-    :filename => "get_realm",
-    :params => nil
+    url: 'global.api.pvp.net/api/lol/static-data/euw/v1.2/realm',
+    filename: 'realm',
+    params: nil
   },
   {
-    :url => "global.api.pvp.net/api/lol/static-data/euw/v1.2/rune",
-    :filename => "get_all_runes",
-    :params => {
-      :locale => 'fr_FR',
-      :runeListData => 'all'
+    url: 'global.api.pvp.net/api/lol/static-data/euw/v1.2/rune',
+    filename: 'get_all_runes',
+    params: {
+      locale: 'fr_FR',
+      runeListData: 'all'
     }
   },
   {
-    :url => "global.api.pvp.net/api/lol/static-data/euw/v1.2/rune/5235",
-    :filename => "get_rune_by_id",
-    :params => {
-      :locale => 'fr_FR',
-      :runeData => 'all'
+    url: 'global.api.pvp.net/api/lol/static-data/euw/v1.2/rune/5235',
+    filename: 'get_rune_by_id',
+    params: {
+      locale: 'fr_FR',
+      runeData: 'all'
     }
   },
   {
-    :url => "global.api.pvp.net/api/lol/static-data/euw/v1.2/summoner-spell",
-    :filename => "get_all_summoner_spells",
-    :params => {
-      :locale => 'fr_FR',
-      :dataById => 'false',
-      :spellData => 'all'
+    url: 'global.api.pvp.net/api/lol/static-data/euw/v1.2/summoner-spell',
+    filename: 'get_all_summoner_spells',
+    params: {
+      locale: 'fr_FR',
+      dataById: 'false',
+      spellData: 'all'
     }
   },
   {
-    :url => "global.api.pvp.net/api/lol/static-data/euw/v1.2/summoner-spell",
-    :filename => "get_all_summoner_spells_by_ids",
-    :params => {
-      :locale => 'fr_FR',
-      :dataById => 'true',
-      :spellData => 'all'
+    url: 'global.api.pvp.net/api/lol/static-data/euw/v1.2/summoner-spell',
+    filename: 'get_all_summoner_spells_by_ids',
+    params: {
+      locale: 'fr_FR',
+      dataById: 'true',
+      spellData: 'all'
     }
   },
   {
-    :url => "global.api.pvp.net/api/lol/static-data/euw/v1.2/summoner-spell/17",
-    :filename => "get_summoner_spell_by_id",
-    :params => {
-      :locale => 'fr_FR',
-      :spellData => 'all'
+    url: 'global.api.pvp.net/api/lol/static-data/euw/v1.2/summoner-spell/1',
+    filename: 'get_summoner_spell_by_id',
+    params: {
+      locale: 'fr_FR',
+      spellData: 'all'
     }
   },
   {
-    :url => "global.api.pvp.net/api/lol/static-data/euw/v1.2/versions",
-    :filename => "get_version",
-    :params => nil
+    url: 'global.api.pvp.net/api/lol/static-data/euw/v1.2/versions',
+    filename: 'get_version',
+    params: nil
   },
   # match-v2.2
   {
     # A faire
-    :url => "euw.api.pvp.net/api/lol/euw/v2.2/match/1617870200",
-    :filename => "get_match_by_id",
-    :params => nil
+    url: 'euw.api.pvp.net/api/lol/euw/v2.2/match/1617870200',
+    filename: 'get_match_by_id',
+    params: nil
   },
-  # matchhistory-v2.2
+  # matchlist-v2.2
   {
-    # A faire
-    :url => "euw.api.pvp.net/api/lol/euw/v2.2/matchhistory/20639710",
-    :filename => "get_match_history",
-    :params => nil
+    url: 'euw.api.pvp.net/api/lol/euw/v2.2/matchlist/by-summoner/20639710',
+    filename: 'match_list',
+    params: nil
   },
   # stats-v1.3
   {
-    :url => "euw.api.pvp.net/api/lol/euw/v1.3/stats/by-summoner/20639710/ranked",
-    :filename => "get_player_stat_ranked",
-    :params => {
-      :season => 'SEASON2015'
+    url: 'euw.api.pvp.net/api/lol/euw/v1.3/stats/by-summoner/20639710/ranked',
+    filename: 'get_player_stat_ranked',
+    params: {
+      season: 'SEASON2015'
     }
   },
   {
-    :url => "euw.api.pvp.net/api/lol/euw/v1.3/stats/by-summoner/20639710/summary",
-    :filename => "get_player_stat_summaries",
-    :params => {
-      :season => 'SEASON2015'
+    url: 'euw.api.pvp.net/api/lol/euw/v1.3/stats/by-summoner/20639710/summary',
+    filename: 'get_player_stat_summaries',
+    params: {
+      season: 'SEASON2015'
     }
   },
   # summoner-v1.4
   {
-    :url => "euw.api.pvp.net/api/lol/euw/v1.4/summoner/by-name/pacoloco",
-    :filename => "summoner",
-    :params => nil
+    url: 'euw.api.pvp.net/api/lol/euw/v1.4/summoner/by-name/pacoloco',
+    filename: 'summoner',
+    params: nil
   },
   {
-    :url => "euw.api.pvp.net/api/lol/euw/v1.4/summoner/by-name/pacoloco",
-    :filename => "get_summoner_by_name",
-    :params => nil
+    url: 'euw.api.pvp.net/api/lol/euw/v1.4/summoner/by-name/pacoloco',
+    filename: 'get_summoner_by_name',
+    params: nil
   },
   {
-    :url => "euw.api.pvp.net/api/lol/euw/v1.4/summoner/20639710",
-    :filename => "get_summoner_by_id",
-    :params => nil
+    url: 'euw.api.pvp.net/api/lol/euw/v1.4/summoner/20639710',
+    filename: 'get_summoner_by_id',
+    params: nil
   },
   {
-    :url => "euw.api.pvp.net/api/lol/euw/v1.4/summoner/20639710/masteries",
-    :filename => "get_summoner_masteries_by_id",
-    :params => nil
+    url: 'euw.api.pvp.net/api/lol/euw/v1.4/summoner/20639710/masteries',
+    filename: 'get_summoner_masteries_by_id',
+    params: nil
   },
   {
-    :url => "euw.api.pvp.net/api/lol/euw/v1.4/summoner/20639710/runes",
-    :filename => "get_summoner_runes_by_id",
-    :params => nil
+    url: 'euw.api.pvp.net/api/lol/euw/v1.4/summoner/20639710/runes',
+    filename: 'get_summoner_runes_by_id',
+    params: nil
   },
   # featured-games-v1.0
   {
-    :url => "euw.api.pvp.net/observer-mode/rest/featured",
-    :filename => "get_featured_games",
-    :params => nil
+    url: 'euw.api.pvp.net/observer-mode/rest/featured',
+    filename: 'get_featured_games',
+    params: nil
   }
   # current-game-v1.0
   # {
@@ -203,24 +223,23 @@ class Mock
   attr_accessor :data, :url, :file
 
   def format_url
-    full_url = "https://#{self.data[:url]}"
+    full_url = "https://#{data[:url]}"
     full_url += "\?"
-    unless self.data[:params].nil?
-      self.data[:params].each do |key,val|
-        full_url+= "#{key}\=#{val}"
-        full_url+= "\&"
+    unless data[:params].nil?
+      data[:params].each do |key, val|
+        full_url += "#{key}\=#{val}"
+        full_url += "\&"
       end
     end
     full_url += "api_key\=#{API_KEY}"
 
     self.url = full_url
-    self.file = self.data[:filename]
+    self.file = data[:filename]
   end
 
   def call_curl
-    Kernel.system "curl -is '#{self.url}' > spec/mock_response/#{self.file}.json"
+    Kernel.system "curl -is '#{url}' > spec/mock_response/#{file}.json"
   end
-
 end
 
 # List CURL commands