thetvdb_api 0.2.1 → 0.2.2

data/lib/thetvdb_api.rb

@@ -1,10 +1,20 @@
 module ThetvdbApi
-  module Mappers; end
+  module Mapper; end
 end
 
 require 'thetvdb_api/version'
 require 'thetvdb_api/configuration'
 require 'thetvdb_api/response'
+require 'thetvdb_api/response/collection'
+require 'thetvdb_api/response/search_multiple_episode'
+require 'thetvdb_api/response/search_multiple_series'
+require 'thetvdb_api/response/search_series'
+require 'thetvdb_api/response/update'
+require 'thetvdb_api/response/actors'
+require 'thetvdb_api/response/banners'
+require 'thetvdb_api/response/episode'
+require 'thetvdb_api/response/series'
+require 'thetvdb_api/response/full_series'
 require 'thetvdb_api/client'
 require 'thetvdb_api/base'
 require 'thetvdb_api/search'

data/spec/integrations/actor_spec.rb

@@ -6,7 +6,7 @@ describe ThetvdbApi::Actor do
   describe 'real request' do
     describe '.find' do
       it 'should return response class' do
-        model.find('70327').class.should == ThetvdbApi::Response
+        model.find('70327').class.should == ThetvdbApi::Response::Actors
       end
     end
   end

data/spec/integrations/banner_spec.rb

@@ -6,7 +6,7 @@ describe ThetvdbApi::Banner do
   describe 'real request' do
     describe '.find' do
       it 'should return response class' do
-        model.find('72449').class.should == ThetvdbApi::Response
+        model.find('72449').class.should == ThetvdbApi::Response::Banners
       end
     end
   end

data/spec/integrations/client_spec.rb

@@ -7,25 +7,25 @@ describe ThetvdbApi::Client do
     describe '.search' do
       describe '.get_series' do
         it 'should return response class' do
-          client.search.get_series('buffy').class.should == ThetvdbApi::Response
+          client.search.get_series('buffy').class.should == ThetvdbApi::Response::SearchMultipleSeries
         end
       end
 
       describe '.get_series_by_imdb_id' do
         it 'should return response class' do
-          client.search.get_series_by_imdb_id('tt0118276').class.should == ThetvdbApi::Response
+          client.search.get_series_by_imdb_id('tt0118276').class.should == ThetvdbApi::Response::SearchSeries
         end
       end
 
       describe '.get_series_by_zap2it_id' do
         it 'should return response class' do
-          client.search.get_series_by_zap2it_id('EP00213110').class.should == ThetvdbApi::Response
+          client.search.get_series_by_zap2it_id('EP00213110').class.should == ThetvdbApi::Response::SearchSeries
         end
       end
 
       describe '.get_episode' do
         it 'should return response class' do
-          client.search.get_episode('70327','1997-03-10').class.should == ThetvdbApi::Response
+          client.search.get_episode('70327','1997-03-10').class.should == ThetvdbApi::Response::SearchMultipleEpisode
         end
       end
     end
@@ -33,13 +33,13 @@ describe ThetvdbApi::Client do
     describe '.series' do
       describe '.find' do
         it 'should return response class' do
-          client.series.find('70327').class.should == ThetvdbApi::Response
+          client.series.find('70327').class.should == ThetvdbApi::Response::Series
         end
       end
 
       describe '.find_full' do
         it 'should return response class' do
-          client.series.find_full('70327').class.should == ThetvdbApi::Response
+          client.series.find_full('70327').class.should == ThetvdbApi::Response::FullSeries
         end
       end
     end
@@ -47,7 +47,7 @@ describe ThetvdbApi::Client do
     describe '.actor' do
       describe '.find' do
         it 'should return response class' do
-          client.actor.find('70327').class.should == ThetvdbApi::Response
+          client.actor.find('70327').class.should == ThetvdbApi::Response::Actors
         end
       end
     end
@@ -55,7 +55,7 @@ describe ThetvdbApi::Client do
     describe '.banner' do
       describe '.find' do
         it 'should return response class' do
-          client.banner.find('70327').class.should == ThetvdbApi::Response
+          client.banner.find('70327').class.should == ThetvdbApi::Response::Banners
         end
       end
     end
@@ -63,25 +63,25 @@ describe ThetvdbApi::Client do
     describe '.episode' do
       describe '.find_by_default_order' do
         it 'should return response class' do
-          client.episode.find_by_default_order('70327', '1', '1').class.should == ThetvdbApi::Response
+          client.episode.find_by_default_order('70327', '1', '1').class.should == ThetvdbApi::Response::Episode
         end
       end
 
       describe '.find_by_dvd_order' do
         it 'should return response class' do
-          client.episode.find_by_dvd_order('70327', '1', '1').class.should == ThetvdbApi::Response
+          client.episode.find_by_dvd_order('70327', '1', '1').class.should == ThetvdbApi::Response::Episode
         end
       end
 
       describe '.find_by_absolute_order' do
         it 'should return response class' do
-          client.episode.find_by_absolute_order('70327', '1').class.should == ThetvdbApi::Response
+          client.episode.find_by_absolute_order('70327', '1').class.should == ThetvdbApi::Response::Episode
         end
       end
 
       describe '.find' do
         it 'should return response class' do
-          client.episode.find('533011').class.should == ThetvdbApi::Response
+          client.episode.find('533011').class.should == ThetvdbApi::Response::Episode
         end
       end
     end
@@ -89,19 +89,19 @@ describe ThetvdbApi::Client do
     describe '.update' do
       describe '.day' do
         it 'should return response class' do
-          client.update.day.class.should == ThetvdbApi::Response
+          client.update.day.class.should == ThetvdbApi::Response::Update
         end
       end
 
       describe '.week' do
         it 'should return response class' do
-          client.update.week.class.should == ThetvdbApi::Response
+          client.update.week.class.should == ThetvdbApi::Response::Update
         end
       end
 
       describe '.month' do
         it 'should return response class' do
-          client.update.month.class.should == ThetvdbApi::Response
+          client.update.month.class.should == ThetvdbApi::Response::Update
         end
       end
     end

data/spec/integrations/episode_spec.rb

@@ -6,25 +6,25 @@ describe ThetvdbApi::Episode do
   describe 'real request' do
     describe '.find_by_default_order' do
       it 'should return response class' do
-        model.find_by_default_order('70327', '1', '1').class.should == ThetvdbApi::Response
+        model.find_by_default_order('70327', '1', '1').class.should == ThetvdbApi::Response::Episode
       end
     end
 
     describe '.find_by_dvd_order' do
       it 'should return response class' do
-        model.find_by_dvd_order('70327', '1', '1').class.should == ThetvdbApi::Response
+        model.find_by_dvd_order('70327', '1', '1').class.should == ThetvdbApi::Response::Episode
       end
     end
 
     describe '.find_by_absolute_order' do
       it 'should return response class' do
-        model.find_by_absolute_order('70327', '1').class.should == ThetvdbApi::Response
+        model.find_by_absolute_order('70327', '1').class.should == ThetvdbApi::Response::Episode
       end
     end
 
     describe '.find' do
       it 'should return response class' do
-        model.find('533011').class.should == ThetvdbApi::Response
+        model.find('533011').class.should == ThetvdbApi::Response::Episode
       end
     end
   end

data/spec/integrations/search_spec.rb

@@ -6,25 +6,25 @@ describe ThetvdbApi::Search do
   describe 'real request' do
     describe '.get_series' do
       it 'should return response class' do
-        model.get_series('buffy').class.should == ThetvdbApi::Response
+        model.get_series('buffy').class.should == ThetvdbApi::Response::SearchMultipleSeries
       end
     end
 
     describe '.get_series_by_imdb_id' do
       it 'should return response class' do
-        model.get_series_by_imdb_id('tt0118276').class.should == ThetvdbApi::Response
+        model.get_series_by_imdb_id('tt0118276').class.should == ThetvdbApi::Response::SearchSeries
       end
     end
 
     describe '.get_series_by_zap2it_id' do
       it 'should return response class' do
-        model.get_series_by_zap2it_id('EP00213110').class.should == ThetvdbApi::Response
+        model.get_series_by_zap2it_id('EP00213110').class.should == ThetvdbApi::Response::SearchSeries
       end
     end
 
     describe '.get_episode' do
       it 'should return response class' do
-        model.get_episode('70327', '1997-03-10').class.should == ThetvdbApi::Response
+        model.get_episode('70327', '1997-03-10').class.should == ThetvdbApi::Response::SearchMultipleEpisode
       end
     end
   end

data/spec/integrations/series_spec.rb

@@ -6,13 +6,13 @@ describe ThetvdbApi::Series do
   describe 'real request' do
     describe '.find' do
       it 'should return response class' do
-        model.find('70327').class.should == ThetvdbApi::Response
+        model.find('70327').class.should == ThetvdbApi::Response::Series
       end
     end
 
     describe '.find_full' do
       it 'should return response class' do
-        model.find_full('70327').class.should == ThetvdbApi::Response
+        model.find_full('70327').class.should == ThetvdbApi::Response::FullSeries
       end
     end
   end

data/spec/integrations/update_spec.rb

@@ -6,25 +6,25 @@ describe ThetvdbApi::Update do
   describe 'real request' do
     describe '.day' do
       it 'should return response class' do
-        model.day.class.should == ThetvdbApi::Response
+        model.day.class.should == ThetvdbApi::Response::Update
       end
     end
 
     describe '.week' do
       it 'should return response class' do
-        model.week.class.should == ThetvdbApi::Response
+        model.week.class.should == ThetvdbApi::Response::Update
       end
     end
 
     describe '.month' do
       it 'should return response class' do
-        model.month.class.should == ThetvdbApi::Response
+        model.month.class.should == ThetvdbApi::Response::Update
       end
     end
 
     describe '.all' do
       it 'should return response class' do
-        model.all.class.should == ThetvdbApi::Response
+        model.all.class.should == ThetvdbApi::Response::Update
       end
     end
   end

data/spec/spec_helper.rb

@@ -8,26 +8,5 @@ require File.join(File.dirname(__FILE__), 'support/thetvdb_api.rb')
 
 raise 'api_key missing, use ThetvdbApi::Configuration' if ThetvdbApi::Configuration.api_key.empty?
 
-class SampleModel
-  def auth
-    self
-  end
-
-  def get(uri)
-    self
-  end
-
-  def mapper(mapper)
-    self
-  end
-
-  def params(options = {})
-    self
-  end
-
-  def response
-  end
-end
-
 RSpec.configure do |config|
 end

data/spec/units/actor_spec.rb

@@ -0,0 +1,49 @@
+require 'spec_helper'
+
+describe ThetvdbApi::Actor do
+  let(:klass) { ThetvdbApi::Actor }
+  let(:model) { klass.new(ThetvdbApi::Client.new(api_key: '123456789')) }
+
+  describe '.find' do
+    it 'should call find_get_with_params with specific arguments' do
+      model.should_receive(:find_get_with_params).with('1234', {}).and_return(double(response: true))
+
+      model.find('1234')
+    end
+
+    it 'should call response with specific arguments' do
+      model.should_receive(:response).with(ThetvdbApi::Response::Actors)
+
+      model.find('1234')
+    end
+  end
+
+  describe '.find_url' do
+    it 'should call find_get_with_params with specific arguments' do
+      model.should_receive(:find_get_with_params).with('1234').and_return(double(url: true))
+
+      model.find_url('1234')
+    end
+
+    it 'should call url' do
+      model.should_receive(:url)
+
+      model.find_url('1234')
+    end
+  end
+
+  describe '.find_get_with_params' do
+    it 'should call get with specific arguments' do
+      model.stub(:find_path).and_return('PATH')
+      model.should_receive(:get).with('PATH').and_return(double(params: true))
+
+      model.find_get_with_params('1234')
+    end
+
+    it 'should call params with specific arguments' do
+      model.should_receive(:params).with(series_id: '1234')
+
+      model.find_get_with_params('1234')
+    end
+  end
+end

data/spec/units/banner_spec.rb

@@ -0,0 +1,49 @@
+require 'spec_helper'
+
+describe ThetvdbApi::Banner do
+  let(:klass) { ThetvdbApi::Banner }
+  let(:model) { klass.new(ThetvdbApi::Client.new(api_key: '123456789')) }
+
+  describe '.find' do
+    it 'should call find_get_with_params with specific arguments' do
+      model.should_receive(:find_get_with_params).with('1234', {}).and_return(double(response: true))
+
+      model.find('1234')
+    end
+
+    it 'should call response with specific arguments' do
+      model.should_receive(:response).with(ThetvdbApi::Response::Banners)
+
+      model.find('1234')
+    end
+  end
+
+  describe '.find_url' do
+    it 'should call find_get_with_params with specific arguments' do
+      model.should_receive(:find_get_with_params).with('1234').and_return(double(url: true))
+
+      model.find_url('1234')
+    end
+
+    it 'should call url' do
+      model.should_receive(:url)
+
+      model.find_url('1234')
+    end
+  end
+
+  describe '.find_get_with_params' do
+    it 'should call get with specific arguments' do
+      model.stub(:find_path).and_return('PATH')
+      model.should_receive(:get).with('PATH').and_return(double(params: true))
+
+      model.find_get_with_params('1234')
+    end
+
+    it 'should call params with specific arguments' do
+      model.should_receive(:params).with(series_id: '1234')
+
+      model.find_get_with_params('1234')
+    end
+  end
+end

data/spec/units/episode_spec.rb

@@ -0,0 +1,167 @@
+require 'spec_helper'
+
+describe ThetvdbApi::Episode do
+  let(:klass) { ThetvdbApi::Episode }
+  let(:model) { klass.new(ThetvdbApi::Client.new(api_key: '123456789')) }
+
+  describe '.find_by_default_order' do
+    it 'should call find_by_order with specific params' do
+      model.should_receive(:find_by_order).with('default', '1234', '1', '1', {})
+
+      model.find_by_default_order('1234', '1', '1')
+    end
+  end
+
+  describe '.find_by_default_order_url' do
+    it 'should call find_by_order with specific params' do
+      model.should_receive(:find_by_order_url).with('default', '1234', '1', '1', {})
+
+      model.find_by_default_order_url('1234', '1', '1')
+    end
+  end
+
+  describe '.find_by_dvd_order' do
+    it 'should call find_by_order with specific params' do
+      model.should_receive(:find_by_order).with('dvd', '1234', '1', '1', {})
+
+      model.find_by_dvd_order('1234', '1', '1')
+    end
+  end
+
+  describe '.find_by_dvd_order_url' do
+    it 'should call find_by_order with specific params' do
+      model.should_receive(:find_by_order_url).with('dvd', '1234', '1', '1', {})
+
+      model.find_by_dvd_order_url('1234', '1', '1')
+    end
+  end
+
+  describe '.find_by_absolute_order' do
+    it 'should call find_get_with_params with specific arguments' do
+      model.should_receive(:find_by_absolute_order_get_with_params).with('1234', '1', {}).and_return(double(response: true))
+
+      model.find_by_absolute_order('1234', '1')
+    end
+
+    it 'should call response with specific arguments' do
+      model.should_receive(:response).with(ThetvdbApi::Response::Episode)
+
+      model.find_by_absolute_order('1234', '1')
+    end
+  end
+
+  describe '.find_by_absolute_order_url' do
+    it 'should call find_get_with_params with specific arguments' do
+      model.should_receive(:find_by_absolute_order_get_with_params).with('1234', '1', {}).and_return(double(url: true))
+
+      model.find_by_absolute_order_url('1234', '1')
+    end
+
+    it 'should call url' do
+      model.should_receive(:url)
+
+      model.find_by_absolute_order_url('1234', '1')
+    end
+  end
+
+  describe '.find_by_absolute_order_get_with_params' do
+    it 'should call get with specific arguments' do
+      model.stub(:find_by_absolute_order_path).and_return('PATH')
+      model.should_receive(:get).with('PATH').and_return(double(params: true))
+
+      model.find_by_absolute_order_get_with_params('1234', '1', {})
+    end
+
+    it 'should call params with specific arguments' do
+      model.should_receive(:params).with(series_id: '1234', absolute: '1')
+
+      model.find_by_absolute_order_get_with_params('1234', '1', {})
+    end
+  end
+
+  describe '.find' do
+    it 'should call find_get_with_params with specific arguments' do
+      model.should_receive(:find_get_with_params).with('1234', {}).and_return(double(response: true))
+
+      model.find('1234')
+    end
+
+    it 'should call response with specific arguments' do
+      model.should_receive(:response).with(ThetvdbApi::Response::Episode)
+
+      model.find('1234')
+    end
+  end
+
+  describe '.find_url' do
+    it 'should call find_get_with_params with specific arguments' do
+      model.should_receive(:find_get_with_params).with('1234', {}).and_return(double(url: true))
+
+      model.find_url('1234')
+    end
+
+    it 'should call url' do
+      model.should_receive(:url)
+
+      model.find_url('1234')
+    end
+  end
+
+  describe '.find_get_with_params' do
+    it 'should call get with specific arguments' do
+      model.stub(:find_path).and_return('PATH')
+      model.should_receive(:get).with('PATH').and_return(double(params: true))
+
+      model.find_get_with_params('1234', {})
+    end
+
+    it 'should call params with specific arguments' do
+      model.should_receive(:params).with(episode_id: '1234')
+
+      model.find_get_with_params('1234', {})
+    end
+  end
+
+  describe '.find_by_order' do
+    it 'should call find_get_with_params with specific arguments' do
+      model.should_receive(:find_by_order_get_with_params).with('default', '1234', '1', '1', {}).and_return(double(response: true))
+
+      model.find_by_order('default', '1234', '1', '1', {})
+    end
+
+    it 'should call response with specific arguments' do
+      model.should_receive(:response).with(ThetvdbApi::Response::Episode)
+
+      model.find_by_order('default', '1234', '1', '1', {})
+    end
+  end
+
+  describe '.find_by_order_url' do
+    it 'should call find_get_with_params with specific arguments' do
+      model.should_receive(:find_by_order_get_with_params).with('default', '1234', '1', '1', {}).and_return(double(url: true))
+
+      model.find_by_order_url('default', '1234', '1', '1', {})
+    end
+
+    it 'should call url' do
+      model.should_receive(:url)
+
+      model.find_by_order_url('default', '1234', '1', '1', {})
+    end
+  end
+
+  describe '.find_by_order_get_with_params' do
+    it 'should call get with specific arguments' do
+      model.stub(:find_by_order_path).and_return('PATH')
+      model.should_receive(:get).with('PATH').and_return(double(params: true))
+
+      model.find_by_order_get_with_params('default', '1234', '1', '1', {})
+    end
+
+    it 'should call params with specific arguments' do
+      model.should_receive(:params).with(series_id: '1234', season: '1', episode: '1', order: 'default')
+
+      model.find_by_order_get_with_params('default', '1234', '1', '1', {})
+    end
+  end
+end

data/spec/units/response/actors_spec.rb

@@ -0,0 +1,15 @@
+require 'spec_helper'
+
+describe ThetvdbApi::Response::Actors do
+  let(:klass) { ThetvdbApi::Response::Actors }
+  let(:model) { klass.new(faraday_response) }
+  let(:faraday_response) { double(env: true, status: true, headers: true, body: 'TEST') }
+
+  describe '.xml_parse' do
+    it 'should return correct data' do
+      model.stub(:multi_xml_parse).and_return({ 'Actors' => { 'Actor' => 'ACTOR DATA' } })
+
+      model.xml_parse.should == 'ACTOR DATA'
+    end
+  end
+end

data/spec/units/response/banners_spec.rb

@@ -0,0 +1,15 @@
+require 'spec_helper'
+
+describe ThetvdbApi::Response::Banners do
+  let(:klass) { ThetvdbApi::Response::Banners }
+  let(:model) { klass.new(faraday_response) }
+  let(:faraday_response) { double(env: true, status: true, headers: true, body: 'TEST') }
+
+  describe '.xml_parse' do
+    it 'should return correct data' do
+      model.stub(:multi_xml_parse).and_return({ 'Banners' => { 'Banner' => 'BANNER DATA' } })
+
+      model.xml_parse.should == 'BANNER DATA'
+    end
+  end
+end

data/spec/units/response/collection_spec.rb

@@ -0,0 +1,38 @@
+require 'spec_helper'
+
+class ExampleResponseClass
+  include ThetvdbApi::Response::Collection
+end
+
+describe ThetvdbApi::Response::Collection do
+  let(:klass) { ExampleResponseClass }
+  let(:model) { klass.new }
+  let(:faraday_response) { double(env: true, status: true, headers: true, body: 'TEST') }
+
+  describe '.body' do
+    describe 'with mapping' do
+      it 'should return array when parse return array' do
+        model.instance_variable_set('@mapping', true)
+        model.stub(:parse).and_return([])
+
+        model.body.class.should == Array
+      end
+
+      it 'should return array when parse return hash' do
+        model.instance_variable_set('@mapping', true)
+        model.stub(:parse).and_return({})
+
+        model.body.class.should == Array
+      end
+    end
+
+    describe 'without mapping' do
+      it 'should return faraday response' do
+        model.instance_variable_set('@mapping', false)
+        model.instance_variable_set('@faraday_response', faraday_response)
+
+        model.body.should == 'TEST'
+      end
+    end
+  end
+end

data/spec/units/response/episode_spec.rb

@@ -0,0 +1,15 @@
+require 'spec_helper'
+
+describe ThetvdbApi::Response::Episode do
+  let(:klass) { ThetvdbApi::Response::Episode }
+  let(:model) { klass.new(faraday_response) }
+  let(:faraday_response) { double(env: true, status: true, headers: true, body: 'TEST') }
+
+  describe '.xml_parse' do
+    it 'should return correct data' do
+      model.stub(:multi_xml_parse).and_return({ 'Episode' => 'EPISODE DATA' })
+
+      model.xml_parse.should == 'EPISODE DATA'
+    end
+  end
+end