thetvdb_api 0.3.0 → 0.3.1

data/lib/thetvdb_api/search.rb

@@ -1,40 +1,78 @@
 class ThetvdbApi::Search < ThetvdbApi::Base
+  include Ov
+
   # Find the series data based on its name.
   #
   # access: FREE
-  # param (flat params):
-  #   get_series(series name)
-  #   get_series('Buffy')
-  # param (hash params):
-  #   get_series(name: 'Buffy')
+  # param:
+  #   get_series('buffy')
+  # output: Faraday::Response instance with parsed XML string
+  # example: http://thetvdb.com/wiki/index.php/API:GetSeries
+  let :get_series, String do |seriesname|
+    get_series(name: seriesname)
+  end
+
+  # Find the series data based on its name.
+  #
+  # access: FREE
+  # param:
+  #   get_series('buffy', 'de')
+  # output: Faraday::Response instance with parsed XML string
+  # example: http://thetvdb.com/wiki/index.php/API:GetSeries
+  let :get_series, String, String do |seriesname, language|
+    get_series(name: seriesname, language: language)
+  end
+
+  # Find the series data based on its name.
+  #
+  # access: FREE
+  # param:
+  #   get_series(name: 'buffy')
   # output: Faraday::Response instance with parsed XML string
   # example: http://thetvdb.com/wiki/index.php/API:GetSeries
-  def get_series(*options)
-    get_series_path_with_params(*options).get
+  let :get_series, Hash do |options|
+    get_series_path_with_params(options).get
   end
 
   # Find the series data based on its name - return only url.
   #
   # access: FREE
-  # param (flat params):
-  #   get_series_url(series name)
-  #   get_series_url('Buffy')
-  # param (hash params):
-  #   get_series_url(name: 'Buffy')
+  # param:
+  #   get_series_url('buffy')
+  # output: url string
+  let :get_series_url, String do |series|
+    get_series_url(name: series)
+  end
+
+  # Find the series data based on its name - return only url.
+  #
+  # access: FREE
+  # param:
+  #   get_series_url('buffy', 'de')
+  # output: url string
+  let :get_series_url, String, String do |series, language|
+    get_series_url(name: series, language: language)
+  end
+
+  # Find the series data based on its name - return only url.
+  #
+  # access: FREE
+  # param:
+  #   get_series_url(name: 1234)
   # output: url string
-  def get_series_url(*options)
-    get_series_path_with_params(*options).url
+  let :get_series_url, Hash do |options|
+    get_series_path_with_params(options).url
   end
 
   # Find the series data by unique ID's used on other sites.
   #
   # access: FREE
   # param: options hash
-  #   imdbid: IMDb ID (don't use with zap2itid)
-  #   zap2itid: Zap2it ID (don't use with imdbid)
+  #   imdb_id: IMDb ID (don't use with zap2it_id)
+  #   zap2_it: Zap2it ID (don't use with imdb_id)
   # output: Faraday::Response instance with parsed XML string
   # example: http://thetvdb.com/wiki/index.php/API:GetSeriesByRemoteID
-  def get_series_by_remote_id(options = {})
+  let :get_series_by_remote_id, Hash do |options|
     get_series_by_remote_id_path_with_params(options).get
   end
 
@@ -42,10 +80,10 @@ class ThetvdbApi::Search < ThetvdbApi::Base
   #
   # access: FREE
   # param: options hash
-  #   imdbid: IMDb ID (don't use with zap2itid)
-  #   zap2itid: Zap2it ID (don't use with imdbid)
+  #   imdb_id: IMDb ID (don't use with zap2it_id)
+  #   zap2it_id: Zap2it ID (don't use with imdb_id)
   # output: url string
-  def get_series_by_remote_id_url(options = {})
+  let :get_series_by_remote_id_url, Hash do |options|
     get_series_by_remote_id_path_with_params(options).url
   end
 
@@ -53,81 +91,157 @@ class ThetvdbApi::Search < ThetvdbApi::Base
   #
   # access: FREE
   # param (flat params):
-  #   get_series_by_imdb_id(imdb_id)
   #   get_series_by_imdb_id('tt01234')
   # output: Faraday::Response instance with parsed XML string
   # example: http://thetvdb.com/wiki/index.php/API:GetSeriesByRemoteID
-  def get_series_by_imdb_id(imdb_id)
-    get_series_by_remote_id_path_with_params({ imdbid: imdb_id }).get
+  let :get_series_by_imdb_id, String do |imdb_id|
+    get_series_by_remote_id_path_with_params(imdb_id: imdb_id).get
+  end
+
+  # Find the series data by unique IMDB ID
+  #
+  # access: FREE
+  # param (flat params):
+  #   get_series_by_imdb_id('tt01234', 'de')
+  # output: Faraday::Response instance with parsed XML string
+  # example: http://thetvdb.com/wiki/index.php/API:GetSeriesByRemoteID
+  let :get_series_by_imdb_id, String, String do |imdb_id, language|
+    get_series_by_remote_id_path_with_params(imdb_id: imdb_id, language: language).get
+  end
+
+  # Find the series data by unique IMDB ID - return only url.
+  #
+  # access: FREE
+  # param (flat params):
+  #   get_series_by_imdb_id_url('tt01234')
+  # output: url string
+  let :get_series_by_imdb_id_url, String do |imdb_id|
+    get_series_by_remote_id_path_with_params(imdb_id: imdb_id).url
   end
 
   # Find the series data by unique IMDB ID - return only url.
   #
   # access: FREE
   # param (flat params):
-  #   get_series_by_imdb_id_url(imdb_id)
   #   get_series_by_imdb_id_url('tt01234')
   # output: url string
-  def get_series_by_imdb_id_url(imdb_id)
-    get_series_by_remote_id_path_with_params({ imdbid: imdb_id }).url
+  let :get_series_by_imdb_id_url, String, String do |imdb_id, language|
+    get_series_by_remote_id_path_with_params(imdb_id: imdb_id, language: language).url
   end
 
   # Find the series data by unique zap2it ID
   #
   # access: FREE
   # param (flat params):
-  #   get_series_by_zap2it_id(zap2it_id)
   #   get_series_by_zap2it_id('SH01234')
   # output: Faraday::Response instance with parsed XML string
   # example: http://thetvdb.com/wiki/index.php/API:GetSeriesByRemoteID
-  def get_series_by_zap2it_id(zap2it_id)
-    get_series_by_remote_id_path_with_params({ zap2itid: zap2it_id }).get
+  let :get_series_by_zap2it_id, String do |zap2it_id|
+    get_series_by_remote_id_path_with_params(zap2it_id: zap2it_id).get
+  end
+
+  # Find the series data by unique zap2it ID
+  #
+  # access: FREE
+  # param (flat params):
+  #   get_series_by_zap2it_id('SH01234', 'de')
+  # output: Faraday::Response instance with parsed XML string
+  # example: http://thetvdb.com/wiki/index.php/API:GetSeriesByRemoteID
+  let :get_series_by_zap2it_id, String, String do |zap2it_id, language|
+    get_series_by_remote_id_path_with_params(zap2it_id: zap2it_id, language: language).get
   end
 
   # Find the series data by unique zap2it ID - return only url.
   #
   # access: FREE
   # param: options hash
-  #   get_series_by_zap2it_id_url(zap2it_id)
   #   get_series_by_zap2it_id_url('SH01234')
   # output: url string
-  def get_series_by_zap2it_id_url(zap2it_id)
-    get_series_by_remote_id_path_with_params({ zap2itid: zap2it_id }).url
+  let :get_series_by_zap2it_id_url, String do |zap2it_id|
+    get_series_by_remote_id_path_with_params(zap2it_id: zap2it_id).url
   end
 
+  # Find the series data by unique zap2it ID - return only url.
+  #
+  # access: FREE
+  # param: options hash
+  #   get_series_by_zap2it_id_url('SH01234', 'de')
+  # output: url string
+  let :get_series_by_zap2it_id_url, String, String do |zap2it_id, language|
+    get_series_by_remote_id_path_with_params(zap2it_id: zap2it_id, language: language).url
+  end
 
   # Find the episode data by episode air date.
   #
   # access: FREE
   # param (flat params):
-  #   get_episode(series_id, air date)
   #   get_episode(1234, '2000-01-01')
+  # output: Faraday::Response instance with parsed XML string
+  # example: http://thetvdb.com/wiki/index.php/API:GetEpisodeByAirDate
+  let :get_episode, Any, String do |series_id, air_date|
+    get_episode(series_id: series_id, air_date: air_date)
+  end
+
+  # Find the episode data by episode air date.
+  #
+  # access: FREE
+  # param (flat params):
+  #   get_episode(1234, '2000-01-01', 'de')
+  # output: Faraday::Response instance with parsed XML string
+  # example: http://thetvdb.com/wiki/index.php/API:GetEpisodeByAirDate
+  let :get_episode, Any, String, String do |series_id, air_date, language|
+    get_episode(series_id: series_id, air_date: air_date, language: language)
+  end
+
+  # Find the episode data by episode air date.
+  #
+  # access: FREE
   # param (hash params):
   #   get_episode(series_id: 1234, air_date: '2000-01-01')
   # output: Faraday::Response instance with parsed XML string
   # example: http://thetvdb.com/wiki/index.php/API:GetEpisodeByAirDate
-  def get_episode(*options)
-    get_episode_path_with_params(*options).get
+  let :get_episode, Hash do |options|
+    get_episode_path_with_params(options).get
   end
 
-
   # Find the episode data by episode air date - return only url.
   #
   # access: FREE
   # param (flat params):
-  #   get_episode_url(series_id, air date)
   #   get_episode_url(1234, '2000-01-01')
+  # output: url string
+  let :get_episode_url, Any, String do |series_id, air_date|
+    get_episode_url(series_id: series_id, air_date: air_date)
+  end
+
+  # Find the episode data by episode air date - return only url.
+  #
+  # access: FREE
+  # param (flat params):
+  #   get_episode_url(1234, '2000-01-01', 'de')
+  # output: url string
+  let :get_episode_url, Any, String, String do |series_id, air_date, language|
+    get_episode_url(series_id: series_id, air_date: air_date, language: language)
+  end
+
+  # Find the episode data by episode air date - return only url.
+  #
+  # access: FREE
   # param (hash params):
   #   get_episode_url(series_id: 1234, air_date: '2000-01-01')
   # output: url string
-  def get_episode_url(*options)
-    get_episode_path_with_params(*options).url
+  let :get_episode_url, Hash do |options|
+    get_episode_path_with_params(options).url
   end
 
   private
 
-  def get_series_path_with_params(*options)
-    path(get_series_path).params(language_options.merge(normalize_series_name_options(*options)))
+  def get_series_path_with_params(options)
+    path(get_series_path).params(language_options.merge(get_series_mapped_options(options)))
+  end
+
+  def get_series_mapped_options(options)
+    ThetvdbApi::AttributesMapping::Search::GetSeries.new(options).to_hash
   end
 
   def get_series_path
@@ -135,15 +249,23 @@ class ThetvdbApi::Search < ThetvdbApi::Base
   end
 
   def get_series_by_remote_id_path_with_params(options)
-    path(get_series_by_remote_id_path).params(language_options.merge(options))
+    path(get_series_by_remote_id_path).params(language_options.merge(get_serie_by_remote_id_mapped_options(options)))
+  end
+
+  def get_serie_by_remote_id_mapped_options(options)
+    ThetvdbApi::AttributesMapping::Search::GetSeriesByRemoteId.new(options).to_hash
   end
 
   def get_series_by_remote_id_path
     'GetSeriesByRemoteID.php'
   end
 
-  def get_episode_path_with_params(*options)
-    path(get_episode_path).params(api_key_with_language_options.merge(normalize_series_id_air_date_options(*options)))
+  def get_episode_path_with_params(options)
+    path(get_episode_path).params(api_key_with_language_options.merge(get_episode_mapped_options(options)))
+  end
+
+  def get_episode_mapped_options(options)
+    ThetvdbApi::AttributesMapping::Search::GetEpisode.new(options).to_hash
   end
 
   def get_episode_path

data/lib/thetvdb_api/series.rb

@@ -1,73 +1,147 @@
 class ThetvdbApi::Series < ThetvdbApi::Base
+  include Ov
+
   # Find the series data by series id.
   #
   # access: FREE
-  # param (flat params):
-  #   find(series_id)
+  # param:
   #   find(1234)
-  # param (hash params):
-  #   find(series_id: 1234)
   # output: Faraday::Response instance with parsed XML string
   # example: http://thetvdb.com/wiki/index.php/API:Base_Series_Record
-  def find(*options)
-    find_path_with_params(*options).get
+  let :find, Any do |id|
+    find(id: id)
+  end
+
+  # Find the series data by series id.
+  #
+  # access: FREE
+  # param:
+  #   find(1234, 'de')
+  # output: Faraday::Response instance with parsed XML string
+  # example: http://thetvdb.com/wiki/index.php/API:Base_Series_Record
+  let :find, Any, String do |id, language|
+    find(id: id, language: language)
+  end
+
+  # Find the series data by series id.
+  #
+  # access: FREE
+  # param:
+  #   find(id: 1234)
+  # output: Faraday::Response instance with parsed XML string
+  # example: http://thetvdb.com/wiki/index.php/API:Base_Series_Record
+  let :find, Hash do |options|
+    find_path_with_params(options).get
   end
 
   # Find the series data by series id - return only url.
   #
   # access: FREE
-  # param (flat params):
-  #   find_url(series_id)
+  # param:
   #   find_url(1234)
-  # param (hash params):
-  #   find_url(series_id: 1234)
   # output: url string
-  def find_url(*options)
-    find_path_with_params(*options).url
+  let :find_url, Any do |id|
+    find_url(id: id)
+  end
+
+  # Find the series data by series id - return only url.
+  #
+  # access: FREE
+  # param:
+  #   find_url(1234, 'de')
+  # output: url string
+  let :find_url, Any do |id, language|
+    find_url(id: id, language: language)
+  end
+
+  # Find the series data by series id - return only url.
+  #
+  # access: FREE
+  # param:
+  #   find_url(id: 1234)
+  # output: url string
+  let :find_url, Hash do |options|
+    find_path_with_params(options).url
   end
 
   # Find the full series data by series id.
   #
   # access: FREE
-  # param (flat params):
-  #   find_full(series_id)
+  # param:
   #   find_full(1234)
-  # param (hash params):
-  #   find_full(series_id: 1234)
   # output: Faraday::Response instance with parsed XML string
   # example: http://thetvdb.com/wiki/index.php/API:Full_Series_Record
-  def find_full(*options)
-    find_full_path_with_params(*options).get
+  let :find_full, Any do |id|
+    find_full(id: id)
   end
 
-  # Find the full series data by series id - return only url.
+  # Find the full series data by series id.
+  #
+  # access: FREE
+  # param:
+  #   find_full(1234, 'de')
+  # output: Faraday::Response instance with parsed XML string
+  # example: http://thetvdb.com/wiki/index.php/API:Full_Series_Record
+  let :find_full, Any, String do |id, language|
+    find_full(id: id, language: language)
+  end
+
+  # Find the full series data by series id.
   #
   # access: FREE
-  # param (flat params):
-  #   find_full_url(series_id)
+  # param:
+  #   find_full(id: 1234)
+  # output: Faraday::Response instance with parsed XML string
+  # example: http://thetvdb.com/wiki/index.php/API:Full_Series_Record
+  let :find_full, Hash do |options|
+    find_full_path_with_params(options).get
+  end
+
+  # Find the series data by series id - return only url.
+  #
+  # access: FREE
+  # param:
   #   find_full_url(1234)
-  # param (hash params):
-  #   find_full_url(series_id: 1234)
   # output: url string
-  def find_full_url(*options)
-    find_full_path_with_params(*options).url
+  let :find_full_url, Any do |id|
+    find_full_url(id: id)
+  end
+
+  # Find the series data by series id - return only url.
+  #
+  # access: FREE
+  # param:
+  #   find_full_url(1234, 'de')
+  # output: url string
+  let :find_full_url, Any do |id, language|
+    find_full_url(id: id, language: language)
+  end
+
+  # Find the series data by series id - return only url.
+  #
+  # access: FREE
+  # param:
+  #   find_full_url(id: 1234)
+  # output: url string
+  let :find_full_url, Hash do |options|
+    find_full_path_with_params(options).url
   end
 
   private
 
-  def find_path_with_params(*options)
-    path(find_path).params(api_key_with_language_options.merge(normalize_series_id_options(*options)))
+  def find_path_with_params(options)
+    path(find_path).params(api_key_with_language_options.merge(options))
   end
 
   def find_path
-    ':apikey/series/:series_id/:language.xml'
+    ':apikey/series/:id/:language.xml'
   end
 
-  def find_full_path_with_params(*options)
-    path(find_full_path).params(api_key_with_language_options.merge(normalize_series_id_options(*options)))
+  def find_full_path_with_params(options)
+    path(find_full_path).params(api_key_with_language_options.merge(options))
   end
 
   def find_full_path
-    ':apikey/series/:series_id/all/:language.xml'
+    ':apikey/series/:id/all/:language.xml'
   end
 end

data/lib/thetvdb_api/server.rb

@@ -0,0 +1,15 @@
+class ThetvdbApi::Server < ThetvdbApi::Base
+  # Returns server time.
+  #
+  # access: FREE
+  # output: Faraday::Response instance with parsed XML string
+  def time
+    path(find_path).get
+  end
+
+  private
+  
+  def find_path
+    'Updates.php?type=none'
+  end
+end

data/lib/thetvdb_api/version.rb

@@ -1,3 +1,3 @@
 module ThetvdbApi
-  VERSION = '0.3.0'
+  VERSION = '0.3.1'
 end

data/lib/thetvdb_api.rb

@@ -1,7 +1,14 @@
+require 'ov'
+
 module ThetvdbApi
-  module Mapper; end
+  module AttributesMapping
+    module Search; end
+  end
 end
 
+require 'thetvdb_api/attributes_mapping/search/get_episode'
+require 'thetvdb_api/attributes_mapping/search/get_series'
+require 'thetvdb_api/attributes_mapping/search/get_series_by_remote_id'
 require 'thetvdb_api/version'
 require 'thetvdb_api/client'
 require 'thetvdb_api/base'
@@ -11,3 +18,4 @@ require 'thetvdb_api/series'
 require 'thetvdb_api/episode'
 require 'thetvdb_api/actor'
 require 'thetvdb_api/banner'
+require 'thetvdb_api/server'

data/spec/fixtures/server_time.xml

@@ -0,0 +1,4 @@
+<?xml version="1.0" encoding="UTF-8" ?>
+<Items>
+<Time>1410781200</Time>
+</Items>

data/spec/functionals/episode_spec.rb

@@ -138,11 +138,11 @@ describe ThetvdbApi::Episode do
   describe '.find' do
     context 'hash attributes' do
       it 'should return Faraday::Response class' do
-        expect(model.find(episode_id: 1234)).to be_a(Faraday::Response)
+        expect(model.find(id: 1234)).to be_a(Faraday::Response)
       end
 
       it 'should return Hash class for body reponse' do
-        expect(model.find(episode_id: 1234).body).to be_a(Hash)
+        expect(model.find(id: 1234).body).to be_a(Hash)
       end
     end
 
@@ -161,7 +161,7 @@ describe ThetvdbApi::Episode do
     context 'hash attributes' do
       it 'should return correct url' do
         expect(
-          model.find_url(episode_id: 1234)
+          model.find_url(id: 1234)
         ).to eq('http://thetvdb.com/api/123456789/episodes/1234/en.xml')
       end
     end

data/spec/functionals/search_spec.rb

@@ -12,7 +12,7 @@ describe ThetvdbApi::Search do
     Faraday::Adapter::Test::Stubs.new do |stub|
       stub.get('/api/GetSeries.php?language=en&seriesname=Supernatural') { [200, { content_type: 'xml' }, get_series_data] }
       stub.get('/api/GetSeriesByRemoteID.php?language=en&imdbid=tt0290978') { [200, { content_type: 'xml' }, get_series_by_remote_data] }
-      stub.get('/api/GetSeriesByRemoteID.php?language=en&zap2itid=SH01234') { [200, { content_type: 'xml' }, get_series_by_remote_data] }
+      stub.get('/api/GetSeriesByRemoteID.php?language=en&zap2it=SH01234') { [200, { content_type: 'xml' }, get_series_by_remote_data] }
       stub.get('/api/GetEpisodeByAirDate.php?language=en&airdate=2007-09-24&apikey=123456789&language=en&seriesid=80348') do
         [200, { content_type: 'xml' }, get_episode_data]
       end
@@ -62,11 +62,11 @@ describe ThetvdbApi::Search do
   describe '.get_series_by_remote_id' do
     context 'hash attributes' do
       it 'should return Faraday::Response class' do
-        expect(model.get_series_by_remote_id(imdbid: 'tt0290978')).to be_a(Faraday::Response)
+        expect(model.get_series_by_remote_id(imdb_id: 'tt0290978')).to be_a(Faraday::Response)
       end
 
       it 'should return Hash class for body reponse' do
-        expect(model.get_series_by_remote_id(imdbid: 'tt0290978').body).to be_a(Hash)
+        expect(model.get_series_by_remote_id(imdb_id: 'tt0290978').body).to be_a(Hash)
       end
     end
   end
@@ -75,7 +75,7 @@ describe ThetvdbApi::Search do
     context 'hash attributes' do
       it 'should return correct url' do
         expect(
-          model.get_series_by_remote_id_url(imdbid: 'tt0290978')
+          model.get_series_by_remote_id_url(imdb_id: 'tt0290978')
         ).to eq('http://thetvdb.com/api/GetSeriesByRemoteID.php?language=en&imdbid=tt0290978')
       end
     end
@@ -84,11 +84,11 @@ describe ThetvdbApi::Search do
   describe '.get_series_by_remote_id' do
     context 'hash attributes' do
       it 'should return Faraday::Response class' do
-        expect(model.get_series_by_remote_id(zap2itid: 'SH01234')).to be_a(Faraday::Response)
+        expect(model.get_series_by_remote_id(zap2it_id: 'SH01234')).to be_a(Faraday::Response)
       end
 
       it 'should return Hash class for body reponse' do
-        expect(model.get_series_by_remote_id(zap2itid: 'SH01234').body).to be_a(Hash)
+        expect(model.get_series_by_remote_id(zap2it_id: 'SH01234').body).to be_a(Hash)
       end
     end
   end
@@ -97,8 +97,8 @@ describe ThetvdbApi::Search do
     context 'hash attributes' do
       it 'should return correct url' do
         expect(
-          model.get_series_by_remote_id_url(zap2itid: 'SH01234')
-        ).to eq('http://thetvdb.com/api/GetSeriesByRemoteID.php?language=en&zap2itid=SH01234')
+          model.get_series_by_remote_id_url(zap2it_id: 'SH01234')
+        ).to eq('http://thetvdb.com/api/GetSeriesByRemoteID.php?language=en&zap2it=SH01234')
       end
     end
   end
@@ -142,7 +142,7 @@ describe ThetvdbApi::Search do
       it 'should return correct url' do
         expect(
           model.get_series_by_zap2it_id_url('SH01234')
-        ).to eq('http://thetvdb.com/api/GetSeriesByRemoteID.php?language=en&zap2itid=SH01234')
+        ).to eq('http://thetvdb.com/api/GetSeriesByRemoteID.php?language=en&zap2it=SH01234')
       end
     end
   end

data/spec/functionals/series_spec.rb

@@ -17,11 +17,11 @@ describe ThetvdbApi::Series do
   describe '.find' do
     context 'hash attributes' do
       it 'should return Faraday::Response class' do
-        expect(model.find(series_id: 1234)).to be_a(Faraday::Response)
+        expect(model.find(id: 1234)).to be_a(Faraday::Response)
       end
 
       it 'should return Hash class for body reponse' do
-        expect(model.find(series_id: 1234).body).to be_a(Hash)
+        expect(model.find(id: 1234).body).to be_a(Hash)
       end
     end
 
@@ -39,7 +39,7 @@ describe ThetvdbApi::Series do
   describe '.find_url' do
     context 'hash attributes' do
       it 'should return correct url' do
-        expect(model.find_url(series_id: 1234)).to eq('http://thetvdb.com/api/123456789/series/1234/en.xml')
+        expect(model.find_url(id: 1234)).to eq('http://thetvdb.com/api/123456789/series/1234/en.xml')
       end
     end
 
@@ -53,11 +53,11 @@ describe ThetvdbApi::Series do
   describe '.find_full' do
     context 'hash attributes' do
       it 'should return Faraday::Response class' do
-        expect(model.find_full(series_id: 1234)).to be_a(Faraday::Response)
+        expect(model.find_full(id: 1234)).to be_a(Faraday::Response)
       end
 
       it 'should return Hash class for body reponse' do
-        expect(model.find_full(series_id: 1234).body).to be_a(Hash)
+        expect(model.find_full(id: 1234).body).to be_a(Hash)
       end
     end
 
@@ -75,7 +75,7 @@ describe ThetvdbApi::Series do
   describe '.find_full_url' do
     context 'hash attributes' do
       it 'should return correct url' do
-        expect(model.find_full_url(series_id: 1234)).to eq('http://thetvdb.com/api/123456789/series/1234/all/en.xml')
+        expect(model.find_full_url(id: 1234)).to eq('http://thetvdb.com/api/123456789/series/1234/all/en.xml')
       end
     end