blizzard_api 2.0.0 → 3.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 96579c93bc977c1e27958031f03956a7fd19a1c54507a792a8c40fad27c24aea
-  data.tar.gz: b0947d10581d4324503e561dc07059c2fc35f81f27be51b6d322d68abea87064
+  metadata.gz: ba8374332ff5b57727b9b5528294f71c723b617fc863672f2cc1673c8a1f1537
+  data.tar.gz: 29b16eee7787cfc7a5e53d3274a829b49db727ca106e9e1f53a0867a1a81b188
 SHA512:
-  metadata.gz: 1dcdbfc4de7830373ebfda93f8bca5e2f4acc2743b18a5bcd16c6141574e48e13153966041ee4e699c75c76f9ec6af976f8c59116c0dd47f5445cf74d5aded4f
-  data.tar.gz: c2ee55a8fb82065315dc7dc43ff14e6e1e511ffb67984ba57b1983da34d2c94ded0e6a49c46168e129bd991a788bfbe9a8ad4724bd3e7533ca6bd38914a82046
+  metadata.gz: e2f2dfff1fed002b4ee9ac408a59a675d175c0effbc04fd39d3aac55b29802cc5db9d9d563528b2349724111191f7d0eddd15f696c2c67399a3df05875ffa6d5
+  data.tar.gz: 335ccfcdebdcfc92c345c11a0da1330092fdc55a5cc0a7927a4099ae37f838673a1b2ad6fbd29b9aa0b05f9274049d8e38a653379054cecb29259af2b395290c

data/CHANGELOG.md

@@ -1,5 +1,22 @@
 Please view this file on the master branch, otherwise it may be outdated
 
+**Version 3.2.0**
+
+Added new AH commodities endpoint.
+
+**Version 3.1.0**
+
+Fixed the latest build to exclude non 200 responses from caching.
+
+**Version 3.0.0**
+
+Changed the way `:extended` mode is handled regarding caching. Now the extended mode will use cache and return a fake
+  response object if the content is cached (Cache is still ignored when using the `:since` option).
+A cached response can be identified by the presence of a `cached?` method on the response object.
+
+Some automated tests for SC2 endpoints are now ignoring `503` errors. The state of the API is somehow unknown since it
+  is down most of the time.
+
 **Version 2.0.0**
 
 Removed the `icon` field from PlayableClass, it was meant to mimic the old communit API behavior during the transition

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    blizzard_api (1.0.0)
+    blizzard_api (3.1.0)
       redis (~> 4.1, >= 4.1.0)
 
 GEM

data/lib/blizzard_api/api_response.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+module BlizzardApi
+  ##
+  # Simple replacement for the http response object for cached data
+  class ApiResponse
+    attr_reader :code, :body
+
+    def initialize(body)
+      @code = 200
+      @body = body
+    end
+
+    def cached?
+      true
+    end
+  end
+end
+
+module Net
+  # Workaround for fake response payloads
+  class HTTPResponse
+    def cached?
+      false
+    end
+  end
+end

data/lib/blizzard_api/diablo/community/{character.rb → character_class.rb}

@@ -9,7 +9,7 @@ module BlizzardApi
     #
     # You can get an instance of this class using the default region as follows:
     #   api_instance = BlizzardApi::Diablo.character
-    class Character < BlizzardApi::Diablo::Request
+    class CharacterClass < BlizzardApi::Diablo::Request
       ##
       # Return information about a class
       #

data/lib/blizzard_api/diablo/game_data/generic_data_endpoint.rb

@@ -35,7 +35,7 @@ module BlizzardApi
       # Fetch leaderboard data for the current endpoint
       #
       # @param [Integer] id One of the IDs returned by the {index}
-      # @param [Integer] leaderboard_id Leaderboard id
+      # @param [String] leaderboard_id Leaderboard id
       # @!macro request_options
       #
       # @!macro response

data/lib/blizzard_api/diablo.rb

@@ -34,7 +34,7 @@ module BlizzardApi
     require_relative 'diablo/community/act'
     require_relative 'diablo/community/artisan'
     require_relative 'diablo/community/follower'
-    require_relative 'diablo/community/character'
+    require_relative 'diablo/community/character_class'
     require_relative 'diablo/community/item_type'
     require_relative 'diablo/community/item'
     require_relative 'diablo/community/profile'
@@ -64,7 +64,7 @@ module BlizzardApi
     # @!macro init_options
     # @return {Character}
     def self.character(**options)
-      BlizzardApi::Diablo::Character.new(**options)
+      BlizzardApi::Diablo::CharacterClass.new(**options)
     end
 
     ##

data/lib/blizzard_api/hearthstone/game_data/generic_data_endpoint.rb

@@ -20,7 +20,7 @@ module BlizzardApi
       # @!macro request_options
       #
       # @!macro response
-      def index(**options)
+      def search(**options)
         api_request "#{base_url(:community)}/#{@endpoint}/", **default_options.merge(options)
       end
 

data/lib/blizzard_api/request.rb

@@ -82,20 +82,28 @@ module BlizzardApi
     def request(url, **options)
       # Creates the whole url for request
       parsed_url = URI.parse(url)
-
       data = using_cache?(options) ? find_in_cache(parsed_url.to_s) : nil
 
       # If data was found that means cache is enabled and valid
-      return JSON.parse(data, symbolize_names: true) if data
+      return prepare_response data if data
 
       response = consume_api parsed_url, **options
 
-      save_in_cache parsed_url.to_s, response.body, options[:ttl] || CACHE_DAY if using_cache? options
+      handle_cache_on_response parsed_url, response, **options
+    end
 
-      response_data = response.code.to_i.eql?(304) ? nil : JSON.parse(response.body, symbolize_names: true)
-      return [response, response_data] if mode.eql? :extended
+    def handle_cache_on_response(parsed_url, response, **options)
+      case response.code.to_i
+      when 304
+        response_data = nil
+      when 200
+        response_data = response.body
+      else
+        return nil, response
+      end
 
-      response_data
+      save_in_cache parsed_url.to_s, response.body, options[:ttl] || CACHE_DAY if using_cache? options
+      prepare_response response_data, response
     end
 
     def api_request(uri, **query_string)
@@ -110,7 +118,7 @@ module BlizzardApi
 
       # In case uri already have query string parameters joins them with &
       if query_string.size.positive?
-        query_string = URI.encode_www_form(query_string, false)
+        query_string = URI.encode_www_form(query_string)
         uri = uri.include?('?') ? "#{uri}&#{query_string}" : "#{uri}?#{query_string}"
       end
 
@@ -119,10 +127,22 @@ module BlizzardApi
 
     private
 
+    ##
+    # Resolves the response based on the mode
+    def prepare_response(data, response = nil)
+      parsed_data = data.nil? ? data : JSON.parse(data, symbolize_names: true)
+
+      return parsed_data unless mode.eql? :extended
+
+      response ||= ApiResponse.new(data)
+
+      [response, parsed_data]
+    end
+
     ##
     # @param options [Hash] Request options
     def using_cache?(options)
-      return false if mode.eql?(:extended) || options.key?(:since)
+      return false if options.key?(:since)
 
       !options.fetch(:ignore_cache, false)
     end
@@ -130,6 +150,7 @@ module BlizzardApi
     def http_connection(url)
       Net::HTTP.new(url.host, url.port).tap do |http|
         http.use_ssl = true
+        http.keep_alive_timeout = 30
       end
     end
 

data/lib/blizzard_api/version.rb

@@ -2,5 +2,5 @@
 
 module BlizzardApi
   # Gem version
-  VERSION = '2.0.0'
+  VERSION = '3.2.0'
 end

data/lib/blizzard_api/wow/game_data/auction.rb

@@ -39,6 +39,12 @@ module BlizzardApi
 
         api_request "#{base_url(:game_data)}/connected-realm/#{connected_realm_id}/auctions", **opts
       end
+
+      def commodities(**options)
+        opts = { ttl: CACHE_HOUR, namespace: :dynamic }.merge(options)
+
+        api_request "#{base_url(:game_data)}/auctions/commodities", **opts
+      end
     end
   end
 end

data/lib/blizzard_api/wow/game_data/guild_crest.rb

@@ -13,7 +13,7 @@ module BlizzardApi
       setup 'guild-crest', :static, CACHE_TRIMESTER
 
       def get
-        raise BlizzardApi::ApiException, 'This endpoint doens\'t have a get method'
+        raise BlizzardApi::ApiException, 'This endpoint does not have a get method'
       end
 
       ##

data/lib/blizzard_api/wow/game_data/reputation.rb

@@ -17,7 +17,7 @@ module BlizzardApi
       #
       # @!macro response
       def index
-        raise BlizzardApi::ApiException, 'This endpoint does not have a index method'
+        raise BlizzardApi::ApiException, 'This endpoint does not have an index method'
       end
 
       ##

data/lib/blizzard_api/wow/game_data/spell.rb

@@ -19,7 +19,7 @@ module BlizzardApi
       #
       # @!macro response
       def index
-        raise BlizzardApi::ApiException, 'This endpoint does not have a index method'
+        raise BlizzardApi::ApiException, 'This endpoint does not have an index method'
       end
 
       ##

data/lib/blizzard_api/wow/game_data/wow_token.rb

@@ -9,16 +9,11 @@ module BlizzardApi
     #
     # You can get an instance of this class using the default region as follows:
     #   api_instance = BlizzardApi::Wow.wow_token
-    class WowToken < Wow::Request
-      ##
-      # Returns wow token data
-      #
-      # @!macro request_options
-      #
-      # @!macro response
-      def get(**options)
-        opts = { namespace: :dynamic, ttl: CACHE_HOUR }.merge(options)
-        api_request "#{base_url(:game_data)}/token/index", **opts
+    class WowToken < GenericDataEndpoint
+      setup 'token', :dynamic, CACHE_HOUR
+
+      def get
+        raise BlizzardApi::ApiException, 'This endpoint does not have a index method'
       end
     end
   end

data/lib/blizzard_api/wow/profile/character_profile.rb

@@ -13,7 +13,7 @@ module BlizzardApi
       include BlizzardApi::Wow::Slug
 
       ##
-      # Return character achievements
+      # Return character basic data
       #
       # @see https://develop.battle.net/documentation/api-reference/world-of-warcraft-profile-api
       #

data/lib/blizzard_api.rb

@@ -3,6 +3,7 @@
 require_relative 'blizzard_api/configuration'
 require_relative 'blizzard_api/token_manager'
 require_relative 'blizzard_api/api_standards'
+require_relative 'blizzard_api/api_response'
 require_relative 'blizzard_api/request'
 require_relative 'blizzard_api/exception'
 require_relative 'blizzard_api/version'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: blizzard_api
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 3.2.0
 platform: ruby
 authors:
 - Francis Schiavo
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-01-12 00:00:00.000000000 Z
+date: 2022-08-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redis
@@ -121,12 +121,13 @@ files:
 - bin/setup
 - blizzard_api.gemspec
 - lib/blizzard_api.rb
+- lib/blizzard_api/api_response.rb
 - lib/blizzard_api/api_standards.rb
 - lib/blizzard_api/configuration.rb
 - lib/blizzard_api/diablo.rb
 - lib/blizzard_api/diablo/community/act.rb
 - lib/blizzard_api/diablo/community/artisan.rb
-- lib/blizzard_api/diablo/community/character.rb
+- lib/blizzard_api/diablo/community/character_class.rb
 - lib/blizzard_api/diablo/community/follower.rb
 - lib/blizzard_api/diablo/community/item.rb
 - lib/blizzard_api/diablo/community/item_type.rb