thetvdb_api 0.1.2 → 0.2.0

data/spec/spec_helper.rb

@@ -17,6 +17,10 @@ class SampleModel
     self
   end
 
+  def mapper(mapper)
+    self
+  end
+
   def params(options = {})
     self
   end

data/spec/thetvdb_api/base_spec.rb

@@ -7,6 +7,14 @@ describe ThetvdbApi::Base do
   let(:klass) { ExampleClass }
   let(:model) { klass.new(ThetvdbApi::Client.new) }
 
+  describe '.connection' do
+    it 'should create instance of Faraday' do
+      Faraday.should_receive(:new).with(url: 'http://thetvdb.com/api/')
+
+      model.connection
+    end
+  end
+
   describe '.get' do
     it 'should set @uri_template' do
       model.get('http://example.com')
@@ -23,7 +31,7 @@ describe ThetvdbApi::Base do
     it 'should set @params' do
       model.params(sample: 'test')
 
-      model.instance_variable_get('@params').should == { sample: 'test' }
+      model.instance_variable_get('@params').should == { language: 'en', sample: 'test' }
     end
 
     it 'should return self' do
@@ -32,9 +40,14 @@ describe ThetvdbApi::Base do
   end
 
   describe '.response' do
+    it 'should raise error when @uri_template is not filled' do
+      -> { model.response }.should raise_error
+    end
+
     it 'should call get klass method' do
       model.instance_variable_set('@uri_template', URITemplate.new('{api_key}/series/{id}'))
-      klass.should_receive(:get)
+      model.stub(:connection).and_return(double(get: true))
+      model.connection.should_receive(:get)
 
       model.response
     end
@@ -44,7 +57,16 @@ describe ThetvdbApi::Base do
     it 'should receive correct uri string' do
       model.instance_variable_set('@uri_template', URITemplate.new('{api_key}/series/{id}'))
 
-      model.prepare_uri.should == "#{ThetvdbApi::Configuration.api_key}/series/"
+      model.prepare_uri.should == "#{model.api_key}/series/"
+    end
+  end
+
+  describe '.url' do
+    it 'should receive correct string' do
+      model.stub(:base_url).and_return('BASE_URL/')
+      model.stub(:uri).and_return('URI')
+
+      model.url.should == 'BASE_URL/URI'
     end
   end
 
@@ -53,7 +75,7 @@ describe ThetvdbApi::Base do
       model.instance_variable_set('@uri_template', URITemplate.new('{api_key}/series/{id}'))
       model.instance_variable_set('@params', id: '1234')
 
-      model.uri.should == "#{ThetvdbApi::Configuration.api_key}/series/1234"
+      model.uri.should == "#{model.api_key}/series/1234"
     end
   end
 
@@ -72,13 +94,23 @@ describe ThetvdbApi::Base do
     end
   end
 
-  describe '.langauge' do
-    it 'should use default langauge' do
+  describe '.language' do
+    it 'should use default language' do
       klass.new(ThetvdbApi::Client.new).language.should == 'en'
     end
 
-    it 'should set langauge' do
+    it 'should set language' do
       klass.new(ThetvdbApi::Client.new(language: 'LANGUAGE')).language.should == 'LANGUAGE'
     end
   end
+
+  describe '.api_key' do
+    it 'should use default api_key' do
+      klass.new(ThetvdbApi::Client.new).api_key.should == ThetvdbApi::Configuration.api_key
+    end
+
+    it 'should set api_Key' do
+      klass.new(ThetvdbApi::Client.new(api_key: 'API_KEY')).api_key.should == 'API_KEY'
+    end
+  end
 end

data/spec/thetvdb_api/episode_spec.rb

@@ -13,7 +13,7 @@ describe ThetvdbApi::Episode do
     end
 
     it 'should call params with specific params' do
-      model.should_receive(:params).with(series_id: '1234', season: '1', episode: '1', language: 'en', order: 'default').and_return(mock_model)
+      model.should_receive(:params).with(series_id: '1234', season: '1', episode: '1', order: 'default').and_return(mock_model)
 
       model.find_by_default_order('1234', '1', '1')
     end
@@ -27,7 +27,7 @@ describe ThetvdbApi::Episode do
     end
 
     it 'should call params with specific params' do
-      model.should_receive(:params).with(series_id: '1234', season: '1', episode: '1', language: 'en', order: 'dvd').and_return(mock_model)
+      model.should_receive(:params).with(series_id: '1234', season: '1', episode: '1', order: 'dvd').and_return(mock_model)
 
       model.find_by_dvd_order('1234', '1', '1')
     end
@@ -41,7 +41,7 @@ describe ThetvdbApi::Episode do
     end
 
     it 'should call params with specific params' do
-      model.should_receive(:params).with(series_id: '1234', absolute: '1', language: 'en').and_return(mock_model)
+      model.should_receive(:params).with(series_id: '1234', absolute: '1').and_return(mock_model)
 
       model.find_by_absolute_order('1234', '1')
     end
@@ -55,7 +55,7 @@ describe ThetvdbApi::Episode do
     end
 
     it 'should call params with specific params' do
-      model.should_receive(:params).with(episode_id: '1234', language: 'en').and_return(mock_model)
+      model.should_receive(:params).with(episode_id: '1234').and_return(mock_model)
 
       model.find('1234')
     end

data/spec/thetvdb_api/mappers/actor_spec.rb

@@ -0,0 +1,15 @@
+require 'spec_helper'
+require 'thetvdb_api/mappers/actor'
+
+describe ThetvdbApi::Mappers::Actor do
+  let(:klass) { ThetvdbApi::Mappers::Actor }
+  let(:model) { klass.new }
+
+  describe '.image_url' do
+    it 'should return correct string' do
+      model.stub(:image_path).and_return('PATH')
+
+      model.image_url.should == 'http://thetvdb.com/banners/PATH'
+    end
+  end
+end

data/spec/thetvdb_api/mappers/banner_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+require 'thetvdb_api/mappers/banner'
+
+describe ThetvdbApi::Mappers::Banner do
+  let(:klass) { ThetvdbApi::Mappers::Banner }
+  let(:model) { klass.new }
+
+  describe '.url' do
+    it 'should return correct string' do
+      model.stub(:path).and_return('PATH')
+
+      model.url.should == 'http://thetvdb.com/banners/PATH'
+    end
+  end
+
+  describe '.thumbnail_url' do
+    it 'should return correct string' do
+      model.stub(:thumbnail_path).and_return('PATH')
+
+      model.thumbnail_url.should == 'http://thetvdb.com/banners/PATH'
+    end
+  end
+
+  describe '.vignette_url' do
+    it 'should return correct string' do
+      model.stub(:vignette_path).and_return('PATH')
+
+      model.vignette_url.should == 'http://thetvdb.com/banners/PATH'
+    end
+  end
+end

data/spec/thetvdb_api/mappers/search_series/series_spec.rb

@@ -0,0 +1,15 @@
+require 'spec_helper'
+require 'thetvdb_api/mappers/search_series/series'
+
+describe ThetvdbApi::Mappers::SearchSeries::Series do
+  let(:klass) { ThetvdbApi::Mappers::SearchSeries::Series }
+  let(:model) { klass.new }
+
+  describe '.banner_url' do
+    it 'should return correct string' do
+      model.stub(:banner_path).and_return('PATH')
+
+      model.banner_url.should == 'http://thetvdb.com/banners/PATH'
+    end
+  end
+end

data/spec/thetvdb_api/mappers/series_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+require 'thetvdb_api/mappers/series'
+
+describe ThetvdbApi::Mappers::Series do
+  let(:klass) { ThetvdbApi::Mappers::Series }
+  let(:model) { klass.new }
+
+  describe '.banner_url' do
+    it 'should return correct string' do
+      model.stub(:banner_path).and_return('PATH')
+
+      model.banner_url.should == 'http://thetvdb.com/banners/PATH'
+    end
+  end
+
+  describe '.fanart_url' do
+    it 'should return correct string' do
+      model.stub(:fanart_path).and_return('PATH')
+
+      model.fanart_url.should == 'http://thetvdb.com/banners/PATH'
+    end
+  end
+
+  describe '.poster_url' do
+    it 'should return correct string' do
+      model.stub(:poster_path).and_return('PATH')
+
+      model.poster_url.should == 'http://thetvdb.com/banners/PATH'
+    end
+  end
+end

data/spec/thetvdb_api/mappers/update/banner_spec.rb

@@ -0,0 +1,15 @@
+require 'spec_helper'
+require 'thetvdb_api/mappers/update/banner'
+
+describe ThetvdbApi::Mappers::Update::Banner do
+  let(:klass) { ThetvdbApi::Mappers::Update::Banner }
+  let(:model) { klass.new }
+
+  describe '.url' do
+    it 'should return correct string' do
+      model.stub(:path).and_return('PATH')
+
+      model.url.should == 'http://thetvdb.com/banners/PATH'
+    end
+  end
+end

data/spec/thetvdb_api/response_spec.rb

@@ -0,0 +1,60 @@
+require 'spec_helper'
+
+describe ThetvdbApi::Search do
+  let(:klass) { ThetvdbApi::Response }
+  let(:model) { klass.new(faraday_response) }
+  let(:faraday_response) { double(env: true, status: true, headers: true, body: 'TEST') }
+
+  describe '.mapper' do
+    it 'should set mapper' do
+      model = klass.new(faraday_response, 'MAPPER')
+      model.mapper.should == 'MAPPER'
+    end
+  end
+
+  describe '.env' do
+    it 'should call env in faraday_response' do
+      faraday_response.should_receive(:env)
+
+      model.env
+    end
+  end
+
+  describe '.status' do
+    it 'should call status in faraday_response' do
+      faraday_response.should_receive(:status)
+
+      model.status
+    end
+  end
+
+  describe '.headers' do
+    it 'should call headers in faraday_response' do
+      faraday_response.should_receive(:headers)
+
+      model.headers
+    end
+  end
+
+  describe '.body' do
+    it 'should map response when give mapper' do
+      mapper = double(parse: true)
+      model = klass.new(faraday_response, mapper)
+      mapper.should_receive(:parse)
+
+      model.body
+    end
+
+    it 'should return string' do
+      model.body.class.should == String
+    end
+  end
+
+  describe '.inspect' do
+    it 'should return correct string' do
+      model.stub(:body).and_return('BODY')
+      model.stub(:mapper).and_return('MAPPER')
+      model.inspect.should == '<ThetvdbApi::Response body="BODY", mapper="MAPPER">'
+    end
+  end
+end

data/spec/thetvdb_api/search_spec.rb

@@ -9,13 +9,13 @@ describe ThetvdbApi::Search do
     it 'should call get with specific params' do
       model.should_receive(:get).with('GetSeries.php').and_return(mock_model)
 
-      model.get_series('buffy', 'en')
+      model.get_series('buffy')
     end
 
     it 'should call params with specific params' do
-      model.should_receive(:params).with(seriesname: 'buffy', language: 'en').and_return(mock_model)
+      model.should_receive(:params).with(seriesname: 'buffy').and_return(mock_model)
 
-      model.get_series('buffy', 'en')
+      model.get_series('buffy')
     end
   end
 
@@ -23,13 +23,13 @@ describe ThetvdbApi::Search do
     it 'should call get with specific params' do
       model.should_receive(:get).with('GetSeriesByRemoteID.php').and_return(mock_model)
 
-      model.get_series_by_imdb_id('1234', 'en')
+      model.get_series_by_imdb_id('1234')
     end
 
     it 'should call params with specific params' do
-      model.should_receive(:params).with(imdbid: '1234', language: 'en').and_return(mock_model)
+      model.should_receive(:params).with(imdbid: '1234').and_return(mock_model)
 
-      model.get_series_by_imdb_id('1234', 'en')
+      model.get_series_by_imdb_id('1234')
     end
   end
 
@@ -37,27 +37,27 @@ describe ThetvdbApi::Search do
     it 'should call get with specific params' do
       model.should_receive(:get).with('GetSeriesByRemoteID.php').and_return(mock_model)
 
-      model.get_series_by_zap2it_id('1234', 'en')
+      model.get_series_by_zap2it_id('1234')
     end
 
     it 'should call params with specific params' do
-      model.should_receive(:params).with(zap2it: '1234', language: 'en').and_return(mock_model)
+      model.should_receive(:params).with(zap2it: '1234').and_return(mock_model)
 
-      model.get_series_by_zap2it_id('1234', 'en')
+      model.get_series_by_zap2it_id('1234')
     end
   end
 
-  describe '#get_episode_by_air_date' do
+  describe '.get_episode' do
     it 'should call get with specific params' do
       model.should_receive(:get).with('GetEpisodeByAirDate.php').and_return(mock_model)
 
-      model.get_episode_by_air_date('1234', '2000-01-01', 'en')
+      model.get_episode('1234', '2000-01-01')
     end
 
     it 'should call params with specific params' do
-      model.should_receive(:params).with(seriesid: '1234', airdate: '2000-01-01', language: 'en').and_return(mock_model)
+      model.should_receive(:params).with(seriesid: '1234', airdate: '2000-01-01').and_return(mock_model)
 
-      model.get_episode_by_air_date('1234', '2000-01-01', 'en')
+      model.get_episode('1234', '2000-01-01')
     end
   end
 end

data/spec/thetvdb_api/series_spec.rb

@@ -9,13 +9,13 @@ describe ThetvdbApi::Series do
     it 'should call get with specific params' do
       model.should_receive(:get).with('{api_key}/series/{series_id}/{language}.xml').and_return(mock_model)
 
-      model.find('1234', 'en')
+      model.find('1234')
     end
 
     it 'should call params with specific params' do
-      model.should_receive(:params).with(series_id: '1234', language: 'en').and_return(mock_model)
+      model.should_receive(:params).with(series_id: '1234').and_return(mock_model)
 
-      model.find('1234', 'en')
+      model.find('1234')
     end
   end
 
@@ -23,13 +23,13 @@ describe ThetvdbApi::Series do
     it 'should call get with specific params' do
       model.should_receive(:get).with('{api_key}/series/{series_id}/all/{language}.xml').and_return(mock_model)
 
-      model.find_full('1234', 'en')
+      model.find_full('1234')
     end
 
     it 'should call params with specific params' do
-      model.should_receive(:params).with(series_id: '1234', language: 'en').and_return(mock_model)
+      model.should_receive(:params).with(series_id: '1234').and_return(mock_model)
 
-      model.find_full('1234', 'en')
+      model.find_full('1234')
     end
   end
 end

data/spec/thetvdb_api/update_spec.rb

@@ -13,6 +13,12 @@ describe ThetvdbApi::Update do
     end
   end
 
+  describe '.day_url' do
+    it 'should return correct string' do
+      model.day_url.should == "http://thetvdb.com/api/#{model.api_key}/updates/updates_day.xml"
+    end
+  end
+
   describe '.week' do
     it 'should call get with specific params' do
       model.should_receive(:get).with('{api_key}/updates/updates_week.xml').and_return(mock_model)
@@ -21,6 +27,12 @@ describe ThetvdbApi::Update do
     end
   end
 
+  describe '.week_url' do
+    it 'should return correct string' do
+      model.week_url.should == "http://thetvdb.com/api/#{model.api_key}/updates/updates_week.xml"
+    end
+  end
+
   describe '.month' do
     it 'should call get with specific params' do
       model.should_receive(:get).with('{api_key}/updates/updates_month.xml').and_return(mock_model)
@@ -29,6 +41,12 @@ describe ThetvdbApi::Update do
     end
   end
 
+  describe '.month_url' do
+    it 'should return correct string' do
+      model.month_url.should == "http://thetvdb.com/api/#{model.api_key}/updates/updates_month.xml"
+    end
+  end
+
   describe '.all' do
     it 'should call get with specific params' do
       model.should_receive(:get).with('{api_key}/updates/updates_all.xml').and_return(mock_model)
@@ -36,4 +54,10 @@ describe ThetvdbApi::Update do
       model.all
     end
   end
+
+  describe '.all_url' do
+    it 'should return correct string' do
+      model.all_url.should == "http://thetvdb.com/api/#{model.api_key}/updates/updates_all.xml"
+    end
+  end
 end