mtg_sdk 3.0.0 → 3.1.0

data/test/card_test.rb

@@ -1,67 +1,81 @@
-# encoding: UTF-8
-require_relative 'test_helper'
-
-class CardTest < Minitest::Test
-  def test_find_returns_one_card
-    VCR.use_cassette('one_card') do
-      card = MTG::Card.find(88803)
-
-      assert_equal 'Choice of Damnations', card.name
-      assert_equal '{5}{B}', card.mana_cost
-      assert_equal 6, card.cmc
-      assert_equal 'Sorcery — Arcane', card.type
-      assert card.colors.any?{|color| color == 'Black'}
-      assert card.types.any?{|type| type == 'Sorcery'}
-      assert card.subtypes.any?{|subtype| subtype == 'Arcane'}
-      assert_equal 'Rare', card.rarity
-      assert_equal 'SOK', card.set
-      assert_equal 'Saviors of Kamigawa', card.set_name
-      assert_equal "Target opponent chooses a number. You may have that player lose that much life. If you don't, that player sacrifices all but that many permanents.", card.text
-      assert_equal "\"Life is a series of choices between bad and worse.\"\n—Toshiro Umezawa", card.flavor
-      assert_equal 'Tim Hildebrandt', card.artist
-      assert_equal '62', card.number
-      assert_equal 'normal', card.layout
-      assert_equal 88803, card.multiverse_id
-      assert_equal 'http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=88803&type=card', card.image_url
-      assert card.rulings.any?{|ruling| ruling.date == Date.parse('2005-06-01')}
-      assert card.foreign_names.any?{|foreign_name| foreign_name.name == '破灭抉择' && foreign_name.image_url == 'http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=104898&type=card'}
-      assert card.printings.any?{|printing| printing == 'SOK'}
-      assert_equal "Target opponent chooses a number. You may have that player lose that much life. If you don't, that player sacrifices all but that many permanents.", card.original_text
-      assert_equal 'Sorcery — Arcane', card.original_type
-      assert card.legalities.any?{|legality| legality.format == 'Commander' && legality.legality == 'Legal'}
-      assert_equal '1c4aab072d52d283e902f2302afa255b39e0794b', card.id
-    end
-  end
-  
-  def test_find_with_invalid_id_throws_exception
-    VCR.use_cassette('invalid_id') do
-      assert_raises ArgumentError do
-        MTG::Card.find(3239482932)
-      end
-    end
-  end
-  
-  def test_where_with_page_size_and_page_returns_cards
-    VCR.use_cassette('query_cards_pageSize') do
-      cards = MTG::Card.where(pageSize: 10).where(page: 1).all
-
-      # make sure we got only 10 cards back
-      assert cards.length == 10
-      assert cards.kind_of?(Array)
-      assert cards.first.kind_of?(MTG::Card)
-    end
-  end
-
-  def test_all_returns_cards
-    VCR.use_cassette('all_filtered') do
-      cards = MTG::Card.where(supertypes: 'legendary')
-                       .where(subtypes: 'elf,warrior')
-                       .all
-
-      first_card = cards[0]
-      assert first_card.supertypes.include? 'Legendary'
-      assert first_card.subtypes.include? 'Elf'
-      assert first_card.subtypes.include? 'Warrior'
-    end
-  end
+# encoding: UTF-8
+require_relative 'test_helper'
+
+class CardTest < Minitest::Test
+  def test_find_returns_one_card
+    VCR.use_cassette('one_card') do
+      card = MTG::Card.find(88803)
+
+      assert_equal 'Choice of Damnations', card.name
+      assert_equal '{5}{B}', card.mana_cost
+      assert_equal 6, card.cmc
+      assert_equal 'Sorcery — Arcane', card.type
+      assert card.colors.any?{|color| color == 'Black'}
+      assert card.types.any?{|type| type == 'Sorcery'}
+      assert card.subtypes.any?{|subtype| subtype == 'Arcane'}
+      assert_equal 'Rare', card.rarity
+      assert_equal 'SOK', card.set
+      assert_equal 'Saviors of Kamigawa', card.set_name
+      assert_equal "Target opponent chooses a number. You may have that player lose that much life. If you don't, that player sacrifices all but that many permanents.", card.text
+      assert_equal "\"Life is a series of choices between bad and worse.\"\n—Toshiro Umezawa", card.flavor
+      assert_equal 'Tim Hildebrandt', card.artist
+      assert_equal '62', card.number
+      assert_equal 'normal', card.layout
+      assert_equal 88803, card.multiverse_id
+      assert_equal 'http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=88803&type=card', card.image_url
+      assert card.rulings.any?{|ruling| ruling.date == Date.parse('2005-06-01')}
+      assert card.foreign_names.any?{|foreign_name| foreign_name.name == '破灭抉择' && foreign_name.image_url == 'http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=104898&type=card'}
+      assert card.printings.any?{|printing| printing == 'SOK'}
+      assert_equal "Target opponent chooses a number. You may have that player lose that much life. If you don't, that player sacrifices all but that many permanents.", card.original_text
+      assert_equal 'Sorcery — Arcane', card.original_type
+      assert card.legalities.any?{|legality| legality.format == 'Commander' && legality.legality == 'Legal'}
+      assert_equal '1c4aab072d52d283e902f2302afa255b39e0794b', card.id
+    end
+  end
+  
+  def test_find_with_invalid_id_throws_exception
+    VCR.use_cassette('invalid_id') do
+      assert_raises ArgumentError do
+        MTG::Card.find(3239482932)
+      end
+    end
+  end
+  
+  def test_where_with_page_size_and_page_returns_cards
+    VCR.use_cassette('query_cards_pageSize') do
+      cards = MTG::Card.where(pageSize: 10).where(page: 1).all
+
+      # make sure we got only 10 cards back
+      assert cards.length == 10
+      assert cards.kind_of?(Array)
+      assert cards.first.kind_of?(MTG::Card)
+    end
+  end
+
+  def test_all_returns_cards
+    VCR.use_cassette('all_filtered') do
+      cards = MTG::Card.where(supertypes: 'legendary')
+                       .where(subtypes: 'elf,warrior')
+                       .all
+
+      first_card = cards[0]
+      assert first_card.supertypes.include? 'Legendary'
+      assert first_card.subtypes.include? 'Elf'
+      assert first_card.subtypes.include? 'Warrior'
+    end
+  end
+  
+  def test_all_returns_all_cards
+    VCR.use_cassette('all_cards') do
+      stub_request(:any, "https://api.magicthegathering.io/v1/cards").
+        to_return(:body => File.new('test/responses/sample_cards.json'), :status => 200, :headers => {"Content-Type"=> "application/json"})
+      
+      stub_request(:any, "https://api.magicthegathering.io/v1/cards?page=2").
+        to_return(:body => File.new('test/responses/no_cards.json'), :status => 200, :headers => {"Content-Type"=> "application/json"})
+        
+      cards = MTG::Card.all
+      
+      assert_equal 2, cards.length
+    end
+  end
 end

data/test/changelog_test.rb

@@ -1,12 +1,12 @@
-require_relative 'test_helper'
-
-class ChangelogTest < Minitest::Test
-  def test_all_returns_all_changelogs
-    VCR.use_cassette('all_changelogs') do
-      changelogs = MTG::Changelog.all
-      
-      assert changelogs.length > 1
-      assert changelogs.any? {|changelog| changelog.version == '1.0.0'}
-    end
-  end
+require_relative 'test_helper'
+
+class ChangelogTest < Minitest::Test
+  def test_all_returns_all_changelogs
+    VCR.use_cassette('all_changelogs') do
+      changelogs = MTG::Changelog.all
+      
+      assert changelogs.length > 1
+      assert changelogs.any? {|changelog| changelog.version == '1.0.0'}
+    end
+  end
 end

data/test/configuration_test.rb

@@ -1,10 +1,20 @@
-#test/client_test.rb
-require_relative 'test_helper'
-
-class ConfigurationTest < Minitest::Test
-  def test_defaults
-    config = MTG::Configuration.new
-
-    assert_equal 1, config.api_version
-  end
+#test/client_test.rb
+require_relative 'test_helper'
+
+class ConfigurationTest < Minitest::Test
+  def test_defaults
+    config = MTG::Configuration.new
+
+    assert_equal 1, config.api_version
+  end
+  
+  def test_reset
+    MTG.configure do |config|
+      config.api_version = 2
+    end
+    
+    MTG.reset
+    
+    assert_equal 1, MTG.configuration.api_version
+  end
 end

data/test/responses/no_cards.json

@@ -0,0 +1 @@
+{"cards":[]}

data/test/responses/sample_cards.json

@@ -0,0 +1 @@
+{"cards":[{"name":"Decimator of the Provinces","manaCost":"{10}","cmc":10,"type":"Creature — Eldrazi Boar","types":["Creature"],"subtypes":["Eldrazi","Boar"],"rarity":"Mythic Rare","set":"EMN","setName":"Eldritch Moon","text":"Emerge {6}{G}{G}{G} (You may cast this spell by sacrificing a creature and paying the emerge cost reduced by that creature's converted mana cost.)\nWhen you cast Decimator of the Provinces, creatures you control get +2/+2 and gain trample until end of turn.\nTrample, haste","artist":"Svetlin Velinov","number":"2","power":"7","toughness":"7","layout":"normal","multiverseid":414291,"imageUrl":"http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=414291&type=card","foreignNames":[{"name":"屠省野猪","imageUrl":"http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=414514&type=card","language":"Chinese Simplified","multiverseid":414514},{"name":"屠省野豬","imageUrl":"http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=414737&type=card","language":"Chinese Traditional","multiverseid":414737},{"name":"Décimateur des provinces","imageUrl":"http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=415183&type=card","language":"French","multiverseid":415183},{"name":"Dezimierer der Provinzen","imageUrl":"http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=414960&type=card","language":"German","multiverseid":414960},{"name":"Decimatore delle Province","imageUrl":"http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=415406&type=card","language":"Italian","multiverseid":415406},{"name":"州民を滅ぼすもの","imageUrl":"http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=415629&type=card","language":"Japanese","multiverseid":415629},{"name":"민중 대량학살자","imageUrl":"http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=415852&type=card","language":"Korean","multiverseid":415852},{"name":"Dizimador das Províncias","imageUrl":"http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=416075&type=card","language":"Portuguese (Brazil)","multiverseid":416075},{"name":"Истребитель Провинций","imageUrl":"http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=416298&type=card","language":"Russian","multiverseid":416298},{"name":"Diezmador de las provincias","imageUrl":"http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=416521&type=card","language":"Spanish","multiverseid":416521}],"printings":["EMN"],"originalText":"Emerge {6}{G}{G}{G} (You may cast this spell by sacrificing a creature and paying the emerge cost reduced by that creature's converted mana cost.)\nWhen you cast Decimator of the Provinces, creatures you control get +2/+2 and gain trample until end of turn.\nTrample, haste","originalType":"Creature — Eldrazi Boar","id":"786bca6b6efb3ea1dac22e7be3f970471a17e7a6"},{"name":"Emrakul, the Promised End","manaCost":"{13}","cmc":13,"type":"Legendary Creature — Eldrazi","supertypes":["Legendary"],"types":["Creature"],"subtypes":["Eldrazi"],"rarity":"Mythic Rare","set":"EMN","setName":"Eldritch Moon","text":"Emrakul, the Promised End costs {1} less to cast for each card type among cards in your graveyard.  \nWhen you cast Emrakul, you gain control of target opponent during that player's next turn. After that turn, that player takes an extra turn.\nFlying, trample, protection from instants","flavor":"An enigma as vexing as life itself.","artist":"Jaime Jones","number":"6","power":"13","toughness":"13","layout":"normal","multiverseid":414295,"imageUrl":"http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=414295&type=card","foreignNames":[{"name":"绝望终局伊莫库","imageUrl":"http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=414518&type=card","language":"Chinese Simplified","multiverseid":414518},{"name":"絕望終局伊莫庫","imageUrl":"http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=414741&type=card","language":"Chinese Traditional","multiverseid":414741},{"name":"Emrakul, la Fin promise","imageUrl":"http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=415187&type=card","language":"French","multiverseid":415187},{"name":"Emrakul, das prophezeite Ende","imageUrl":"http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=414964&type=card","language":"German","multiverseid":414964},{"name":"Emrakul, la Fine Promessa","imageUrl":"http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=415410&type=card","language":"Italian","multiverseid":415410},{"name":"約束された終末、エムラクール","imageUrl":"http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=415633&type=card","language":"Japanese","multiverseid":415633},{"name":"약속된 종말 엠라쿨","imageUrl":"http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=415856&type=card","language":"Korean","multiverseid":415856},{"name":"Emrakul, o Fim Prometido","imageUrl":"http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=416079&type=card","language":"Portuguese (Brazil)","multiverseid":416079},{"name":"Эмракул, Обетованная Гибель","imageUrl":"http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=416302&type=card","language":"Russian","multiverseid":416302},{"name":"Emrakul, el Final Prometido","imageUrl":"http://gatherer.wizards.com/Handlers/Image.ashx?multiverseid=416525&type=card","language":"Spanish","multiverseid":416525}],"printings":["EMN"],"originalText":"Emrakul, the Promised End costs {1} less to cast for each card type among cards in your graveyard.  \nWhen you cast Emrakul, you gain control of target opponent during that player's next turn. After that turn, that player takes an extra turn.\nFlying, trample, protection from instants","originalType":"Legendary Creature — Eldrazi","id":"3961b8f27dfebff5024e0d4cea2e5dedcd8ed3cd"}]}

data/test/set_test.rb

@@ -1,42 +1,52 @@
-require_relative 'test_helper'
-
-class SetTest < Minitest::Test
-  def test_find_returns_one_set
-    VCR.use_cassette('one_set') do
-      set = MTG::Set.find('ktk')
-      
-      assert_equal 'KTK', set.code
-      assert_equal 'Khans of Tarkir', set.name
-      assert_equal 'expansion', set.type
-      assert_equal 'black', set.border
-      assert set.booster.any? {|rarity| rarity == 'common'}
-      assert_equal '2014-09-26', set.release_date
-      assert_equal 'ktk', set.magic_cards_info_code
-    end
-  end
-  
-  def test_find_with_invalid_code_throws_exception
-    VCR.use_cassette('invalid_code') do
-      assert_raises ArgumentError do
-        MTG::Set.find('invalid')
-      end
-    end
-  end
-  
-  def test_all_returns_all_sets
-    VCR.use_cassette('all_sets') do
-      sets = MTG::Set.all
-      
-      assert sets.length > 100
-    end
-  end
-  
-  def test_generate_booster_returns_cards
-    VCR.use_cassette('booster') do
-      cards = MTG::Set.generate_booster('ktk')
-      
-      assert cards.length == 15
-      assert_equal 'KTK', cards.first.set
-    end
-  end
+require_relative 'test_helper'
+
+class SetTest < Minitest::Test
+  def test_find_returns_one_set
+    VCR.use_cassette('one_set') do
+      set = MTG::Set.find('ktk')
+      
+      assert_equal 'KTK', set.code
+      assert_equal 'Khans of Tarkir', set.name
+      assert_equal 'expansion', set.type
+      assert_equal 'black', set.border
+      assert set.booster.any? {|rarity| rarity == 'common'}
+      assert_equal '2014-09-26', set.release_date
+      assert_equal 'ktk', set.magic_cards_info_code
+    end
+  end
+  
+  def test_find_with_invalid_code_throws_exception
+    VCR.use_cassette('invalid_code') do
+      assert_raises ArgumentError do
+        MTG::Set.find('invalid')
+      end
+    end
+  end
+  
+  def test_all_returns_all_sets
+    VCR.use_cassette('all_sets') do
+      sets = MTG::Set.all
+      
+      assert sets.length > 100
+    end
+  end
+
+  def test_all_with_filter_returns_sets
+    VCR.use_cassette('all_sets_filtered') do
+      sets = MTG::Set.where(name: 'khans').all
+
+      set = sets[0]
+      assert_equal 'KTK', set.code
+      assert_equal 'Khans of Tarkir', set.name
+    end
+  end
+  
+  def test_generate_booster_returns_cards
+    VCR.use_cassette('booster') do
+      cards = MTG::Set.generate_booster('ktk')
+      
+      assert cards.length == 15
+      assert_equal 'KTK', cards.first.set
+    end
+  end
 end

data/test/subtype_test.rb

@@ -1,12 +1,12 @@
-require_relative 'test_helper'
-
-class SubtypeTest < Minitest::Test
-  def test_all_returns_all_subtypes
-    VCR.use_cassette('all_subtypes') do
-      types = MTG::Subtype.all
-      
-      assert types.length > 10
-      assert types.any? {|type| type == 'Warrior'}
-    end
-  end
+require_relative 'test_helper'
+
+class SubtypeTest < Minitest::Test
+  def test_all_returns_all_subtypes
+    VCR.use_cassette('all_subtypes') do
+      types = MTG::Subtype.all
+      
+      assert types.length > 10
+      assert types.any? {|type| type == 'Warrior'}
+    end
+  end
 end

data/test/supertype_test.rb

@@ -1,12 +1,12 @@
-require_relative 'test_helper'
-
-class SupertypeTest < Minitest::Test
-  def test_all_returns_all_types
-    VCR.use_cassette('all_supertypes') do
-      types = MTG::Supertype.all
-      
-      assert types.length == 5
-      assert types.any? {|type| type == 'Legendary'}
-    end
-  end
+require_relative 'test_helper'
+
+class SupertypeTest < Minitest::Test
+  def test_all_returns_all_types
+    VCR.use_cassette('all_supertypes') do
+      types = MTG::Supertype.all
+      
+      assert types.length == 5
+      assert types.any? {|type| type == 'Legendary'}
+    end
+  end
 end

data/test/test_helper.rb

@@ -1,22 +1,22 @@
-#test/test_helper.rb
-require 'simplecov'
-require 'coveralls'
-require 'codeclimate-test-reporter'
-
-SimpleCov.formatter = SimpleCov::Formatter::MultiFormatter.new([
-  SimpleCov::Formatter::HTMLFormatter,
-  Coveralls::SimpleCov::Formatter,
-  CodeClimate::TestReporter::Formatter
-])
-SimpleCov.start
-
-require_relative '../lib/mtg_sdk'
-require 'minitest/autorun'
-require 'webmock/minitest'
-require 'vcr'
-
-VCR.configure do |c|
-  c.cassette_library_dir = "test/fixtures"
-  c.hook_into :webmock
-  c.ignore_hosts 'codeclimate.com'
+#test/test_helper.rb
+require 'simplecov'
+require 'coveralls'
+require 'codeclimate-test-reporter'
+
+SimpleCov.formatter = SimpleCov::Formatter::MultiFormatter.new([
+  SimpleCov::Formatter::HTMLFormatter,
+  Coveralls::SimpleCov::Formatter,
+  CodeClimate::TestReporter::Formatter
+])
+SimpleCov.start
+
+require_relative '../lib/mtg_sdk'
+require 'minitest/autorun'
+require 'webmock/minitest'
+require 'vcr'
+
+VCR.configure do |c|
+  c.cassette_library_dir = "test/fixtures"
+  c.hook_into :webmock
+  c.ignore_hosts 'codeclimate.com'
 end

data/test/type_test.rb

@@ -1,12 +1,12 @@
-require_relative 'test_helper'
-
-class TypeTest < Minitest::Test
-  def test_all_returns_all_types
-    VCR.use_cassette('all_types') do
-      types = MTG::Type.all
-      
-      assert types.length > 10
-      assert types.any? {|type| type == 'Creature'}
-    end
-  end
+require_relative 'test_helper'
+
+class TypeTest < Minitest::Test
+  def test_all_returns_all_types
+    VCR.use_cassette('all_types') do
+      types = MTG::Type.all
+      
+      assert types.length > 10
+      assert types.any? {|type| type == 'Creature'}
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mtg_sdk
 version: !ruby/object:Gem::Version
-  version: 3.0.0
+  version: 3.1.0
 platform: ruby
 authors:
 - Andrew Backes
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-07-17 00:00:00.000000000 Z
+date: 2016-10-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -244,6 +244,8 @@ files:
 - test/card_test.rb
 - test/changelog_test.rb
 - test/configuration_test.rb
+- test/responses/no_cards.json
+- test/responses/sample_cards.json
 - test/set_test.rb
 - test/subtype_test.rb
 - test/supertype_test.rb
@@ -269,7 +271,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.4.5.1
 signing_key: 
 specification_version: 4
 summary: 'Magic: The Gathering SDK for magicthegathering.io'
@@ -277,6 +279,8 @@ test_files:
 - test/card_test.rb
 - test/changelog_test.rb
 - test/configuration_test.rb
+- test/responses/no_cards.json
+- test/responses/sample_cards.json
 - test/set_test.rb
 - test/subtype_test.rb
 - test/supertype_test.rb