mtg_sdk 3.2.0 → 3.2.1

data/Rakefile

@@ -1,11 +1,11 @@
-require "bundler/gem_tasks"
-require 'rake/testtask'
-require_relative 'lib/mtg_sdk'
-
-task :default => :test
-
-Rake::TestTask.new do |t|
-  t.libs << "test"
-  t.test_files = FileList['test/*_test.rb']
-  t.verbose = true
+require "bundler/gem_tasks"
+require 'rake/testtask'
+require_relative 'lib/mtg_sdk'
+
+task :default => :test
+
+Rake::TestTask.new do |t|
+  t.libs << "test"
+  t.test_files = FileList['test/*_test.rb']
+  t.verbose = true
 end

data/lib/mtg_sdk.rb

@@ -1,33 +1,33 @@
-require "mtg_sdk/version"
-require "mtg_sdk/configuration"
-require "mtg_sdk/card"
-require "mtg_sdk/legality"
-require "mtg_sdk/foreign_name"
-require "mtg_sdk/ruling"
-require "mtg_sdk/set"
-require "mtg_sdk/type"
-require "mtg_sdk/subtype"
-require "mtg_sdk/supertype"
-require "mtg_sdk/changelog"
-require "mtg_sdk/rest_client"
-require "mtg_sdk/query_builder"
-
-module MTG
-  API_URL = 'https://api.magicthegathering.io'
-
-  class << self
-    attr_writer :configuration
-  end
-
-  def self.configuration
-    @configuration ||= Configuration.new
-  end
-
-  def self.configure
-    yield(configuration)
-  end
-
-  def self.reset
-    @configuration = Configuration.new
-  end
+require "mtg_sdk/version"
+require "mtg_sdk/configuration"
+require "mtg_sdk/card"
+require "mtg_sdk/legality"
+require "mtg_sdk/foreign_name"
+require "mtg_sdk/ruling"
+require "mtg_sdk/set"
+require "mtg_sdk/type"
+require "mtg_sdk/subtype"
+require "mtg_sdk/supertype"
+require "mtg_sdk/changelog"
+require "mtg_sdk/rest_client"
+require "mtg_sdk/query_builder"
+
+module MTG
+  API_URL = 'https://api.magicthegathering.io'
+
+  class << self
+    attr_writer :configuration
+  end
+
+  def self.configuration
+    @configuration ||= Configuration.new
+  end
+
+  def self.configure
+    yield(configuration)
+  end
+
+  def self.reset
+    @configuration = Configuration.new
+  end
 end

data/lib/mtg_sdk/card.rb

@@ -1,44 +1,44 @@
-require_relative 'representers/card_representer'
-
-module MTG
-  class Card
-    include Roar::JSON
-    include CardRepresenter
-
-    attr_accessor :name, :layout, :names, :mana_cost, :cmc, :colors, :type, :supertypes, :subtypes, :types,
-                  :rarity, :text, :flavor, :artist, :number, :power, :toughness, :loyalty, :multiverse_id, :variations,
-                  :watermark, :border, :timeshifted, :hand, :life, :reserved, :release_date, :starter,
-                  :rulings, :foreign_names, :printings, :original_text, :original_type, :legalities,
-                  :source, :image_url, :set, :id, :set_name, :color_identity
-
-    # Get the resource string
-    #
-    # @return [String] The API resource string
-    def self.Resource
-      "cards"
-    end
-
-    # Find a single card by the card multiverse id
-    #
-    # @param id [Integer] the multiverse id
-    # @return [Card] the Card object response
-    def self.find(id)
-      QueryBuilder.new(Card).find(id)
-    end
-
-    # Get all cards from a query by paging through data
-    #
-    # @return [Array<Card>] Array of Card objects
-    def self.all
-      QueryBuilder.new(Card).all
-    end
-
-    # Adds a parameter to the hash of query parameters
-    #
-    # @param args [Hash] the query parameter
-    # @return [QueryBuilder] the QueryBuilder
-    def self.where(args)
-      QueryBuilder.new(Card).where(args)
-    end
-  end
-end
+require_relative 'representers/card_representer'
+
+module MTG
+  class Card
+    include Roar::JSON
+    include CardRepresenter
+
+    attr_accessor :name, :layout, :names, :mana_cost, :cmc, :colors, :type, :supertypes, :subtypes, :types,
+                  :rarity, :text, :flavor, :artist, :number, :power, :toughness, :loyalty, :multiverse_id, :variations,
+                  :watermark, :border, :timeshifted, :hand, :life, :reserved, :release_date, :starter,
+                  :rulings, :foreign_names, :printings, :original_text, :original_type, :legalities,
+                  :source, :image_url, :set, :id, :set_name, :color_identity
+
+    # Get the resource string
+    #
+    # @return [String] The API resource string
+    def self.Resource
+      "cards"
+    end
+
+    # Find a single card by the card multiverse id
+    #
+    # @param id [Integer] the multiverse id
+    # @return [Card] the Card object response
+    def self.find(id)
+      QueryBuilder.new(Card).find(id)
+    end
+
+    # Get all cards from a query by paging through data
+    #
+    # @return [Array<Card>] Array of Card objects
+    def self.all
+      QueryBuilder.new(Card).all
+    end
+
+    # Adds a parameter to the hash of query parameters
+    #
+    # @param args [Hash] the query parameter
+    # @return [QueryBuilder] the QueryBuilder
+    def self.where(args)
+      QueryBuilder.new(Card).where(args)
+    end
+  end
+end

data/lib/mtg_sdk/changelog.rb

@@ -1,24 +1,24 @@
-require_relative 'representers/changelog_representer'
-
-module MTG
-  class Changelog
-    include Roar::JSON
-    include ChangelogRepresenter
-  
-    attr_accessor :version, :details, :release_date
-    
-    # Get the resource string
-    #
-    # @return [string] The API resource string
-    def self.Resource
-      "changelogs"
-    end
-    
-    # Get all changelogs
-    #
-    # @return [Array<Changelog>] Array of Changelog objects
-    def self.all
-      QueryBuilder.new(Changelog).all
-    end
-  end
-end
+require_relative 'representers/changelog_representer'
+
+module MTG
+  class Changelog
+    include Roar::JSON
+    include ChangelogRepresenter
+  
+    attr_accessor :version, :details, :release_date
+    
+    # Get the resource string
+    #
+    # @return [string] The API resource string
+    def self.Resource
+      "changelogs"
+    end
+    
+    # Get all changelogs
+    #
+    # @return [Array<Changelog>] Array of Changelog objects
+    def self.all
+      QueryBuilder.new(Changelog).all
+    end
+  end
+end

data/lib/mtg_sdk/configuration.rb

@@ -1,9 +1,9 @@
-module MTG
-  class Configuration
-    attr_accessor :api_version
-
-    def initialize
-      @api_version = 1
-    end
-  end
+module MTG
+  class Configuration
+    attr_accessor :api_version
+
+    def initialize
+      @api_version = 1
+    end
+  end
 end

data/lib/mtg_sdk/foreign_name.rb

@@ -1,10 +1,10 @@
-require_relative 'representers/foreign_name_representer'
-
-module MTG
-  class ForeignName
-    include Roar::JSON
-    include LegalityRepresenter
-  
-    attr_accessor :language, :name, :multiverse_id, :image_url
-  end
-end
+require_relative 'representers/foreign_name_representer'
+
+module MTG
+  class ForeignName
+    include Roar::JSON
+    include LegalityRepresenter
+  
+    attr_accessor :language, :name, :multiverse_id, :image_url
+  end
+end

data/lib/mtg_sdk/legality.rb

@@ -1,10 +1,10 @@
-require_relative 'representers/legality_representer'
-
-module MTG
-  class Legality
-    include Roar::JSON
-    include LegalityRepresenter
-  
-    attr_accessor :format, :legality
-  end
-end
+require_relative 'representers/legality_representer'
+
+module MTG
+  class Legality
+    include Roar::JSON
+    include LegalityRepresenter
+  
+    attr_accessor :format, :legality
+  end
+end

data/lib/mtg_sdk/query_builder.rb

@@ -1,66 +1,75 @@
-module MTG
-  class QueryBuilder
-    include RestClient
-    attr_accessor :type, :query
-    
-    def initialize(type)
-      @type = type
-      @query = {}
-    end
-
-    # Adds a parameter to the hash of query parameters
-    #
-    # @param args [Hash] the query parameter
-    # @return [QueryBuilder] the QueryBuilder
-    def where(args)
-      @query.merge!(args)
-      self
-    end
-    
-    # Find a single resource by the resource id
-    #
-    # @param id [String] the resource id
-    # @return [Object] the Type object response
-    def find(id)
-      response = RestClient.get("#{@type.Resource}/#{id}")
-      singular_resource = @type.Resource[0...-1]
-      if response.body[singular_resource].nil?
-        raise ArgumentError, 'Resource not found'
-      end
-      
-      type.new.from_json(response.body[singular_resource].to_json)
-    end
-    
-    # Get all resources from a query by paging through data
-    #
-    # @return [Array<Object>] Array of resources
-    def all
-      list = []
-      page = 1
-      fetch_all = true
-
-      if @query.has_key?(:page)
-        page = @query[:page]
-        fetch_all = false
-      end
-      
-      while true
-        response = RestClient.get(@type.Resource, @query)
-        data = response.body[@type.Resource]      
-        if !data.empty?
-          data.each {|item| list << @type.new.from_json(item.to_json)}
-          
-          if !fetch_all
-            break
-          else
-            where(page: page += 1)
-          end
-        else
-          break
-        end
-      end
-
-      return list
-    end
-  end
-end
+module MTG
+  class QueryBuilder
+    include RestClient
+    attr_accessor :type, :query
+
+    def initialize(type)
+      @type = type
+      @query = {}
+    end
+
+    # Adds a parameter to the hash of query parameters
+    #
+    # @param args [Hash] the query parameter
+    # @return [QueryBuilder] the QueryBuilder
+    def where(args)
+      @query.merge!(args)
+      self
+    end
+
+    # Find a single resource by the resource id
+    #
+    # @param id [String] the resource id
+    # @return [Object] the Type object response
+    def find(id)
+      response = RestClient.get("#{@type.Resource}/#{id}")
+      singular_resource = @type.Resource[0...-1]
+
+      raise ArgumentError, 'Resource not found' if response.body[singular_resource].nil?
+      raise ArgumentError, 'Unavailable Service' if unavailable_service?(response)
+
+      type.new.from_json(response.body[singular_resource].to_json)
+    end
+
+    # Get all resources from a query by paging through data
+    #
+    # @return [Array<Object>] Array of resources
+    def all
+      list = []
+      page = 1
+      fetch_all = true
+
+      if @query.has_key?(:page)
+        page = @query[:page]
+        fetch_all = false
+      end
+
+      while true
+        response = RestClient.get(@type.Resource, @query)
+        data = response.body[@type.Resource]
+
+        raise ArgumentError, 'Unavailable Service' if unavailable_service?(response)
+
+        if !data.empty?
+          data.each {|item| list << @type.new.from_json(item.to_json)}
+
+          if !fetch_all
+            break
+          else
+            where(page: page += 1)
+          end
+        else
+          break
+        end
+      end
+
+      return list
+    end
+
+    private
+
+    def unavailable_service?(response)
+      (500..599).include?(response.status)
+    end
+  end
+end

data/lib/mtg_sdk/representers/card_representer.rb

@@ -1,57 +1,57 @@
-require 'roar/json'
-require 'roar/coercion'
-require_relative 'legality_representer'
-require_relative 'foreign_name_representer'
-require_relative 'ruling_representer'
-require_relative '../legality'
-require_relative '../foreign_name'
-require_relative '../ruling'
-
-module MTG
-  module CardRepresenter
-    include Roar::JSON
-    include Roar::Coercion
-
-    property :name
-    property :layout
-    property :mana_cost, as: :manaCost
-    property :cmc
-    property :type
-    property :rarity
-    property :text
-    property :flavor
-    property :artist
-    property :number
-    property :power
-    property :toughness
-    property :loyalty
-    property :multiverse_id, as: :multiverseid
-    property :watermark
-    property :border
-    property :timeshifted
-    property :hand
-    property :life
-    property :reserved
-    property :release_date, as: :releaseDate, type: Date
-    property :starter
-    property :original_text, as: :originalText
-    property :original_type, as: :originalType
-    property :source
-    property :set
-    property :set_name, as: :setName
-    property :id
-    property :image_url, as: :imageUrl
-
-    collection :names
-    collection :supertypes
-    collection :subtypes
-    collection :types
-    collection :colors
-    collection :color_identity, as: :colorIdentity
-    collection :variations
-    collection :printings
-    collection :legalities, extend: LegalityRepresenter, class: Legality
-    collection :rulings, extend: RulingRepresenter, class: Ruling
-    collection :foreign_names, as: :foreignNames, extend: ForeignNameRepresenter, class: ForeignName
-  end
-end
+require 'roar/json'
+require 'roar/coercion'
+require_relative 'legality_representer'
+require_relative 'foreign_name_representer'
+require_relative 'ruling_representer'
+require_relative '../legality'
+require_relative '../foreign_name'
+require_relative '../ruling'
+
+module MTG
+  module CardRepresenter
+    include Roar::JSON
+    include Roar::Coercion
+
+    property :name
+    property :layout
+    property :mana_cost, as: :manaCost
+    property :cmc
+    property :type
+    property :rarity
+    property :text
+    property :flavor
+    property :artist
+    property :number
+    property :power
+    property :toughness
+    property :loyalty
+    property :multiverse_id, as: :multiverseid
+    property :watermark
+    property :border
+    property :timeshifted
+    property :hand
+    property :life
+    property :reserved
+    property :release_date, as: :releaseDate, type: Date
+    property :starter
+    property :original_text, as: :originalText
+    property :original_type, as: :originalType
+    property :source
+    property :set
+    property :set_name, as: :setName
+    property :id
+    property :image_url, as: :imageUrl
+
+    collection :names
+    collection :supertypes
+    collection :subtypes
+    collection :types
+    collection :colors
+    collection :color_identity, as: :colorIdentity
+    collection :variations
+    collection :printings
+    collection :legalities, extend: LegalityRepresenter, class: Legality
+    collection :rulings, extend: RulingRepresenter, class: Ruling
+    collection :foreign_names, as: :foreignNames, extend: ForeignNameRepresenter, class: ForeignName
+  end
+end