thetvdb_party 0.0.11.pre → 0.0.12.pre

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: caaeef9e167cb8951dd7d04ce0ba6bfc23d56df1
-  data.tar.gz: 33c2b4c2cec606d65479e648e1816f6046b7b245
+  metadata.gz: 3fa2e06a9e0a3b23ca038ed937b7ff0a838f8361
+  data.tar.gz: fd44b41771d89e6599a234718e5352fbfac3f927
 SHA512:
-  metadata.gz: b355f88c52e93986891f815a49a8bc0e6ce0124d9c78eb4ddfebda08a44e2e58aa6ccec3257a7735c67bac79f7ac881ba69a54b56385a365e8f7e02c6843cf18
-  data.tar.gz: 454d5ef95c960f476f0c1ce577620b72e8217bfbe4a3ccc600aa2a44c153ad5d5599956a8014d64453d7ed6c32f875b9a71bb651ab78ca8d190506fc54cb6717
+  metadata.gz: 5a8a9f704ea9fca9286719ebf7a2903e364533f23fa32925160c0f4e53e2aebc9dde33da2c96c2bb13a4050c52f6f40b0700e2480362695c73a8000adb724f31
+  data.tar.gz: e239e79646c9a7627d4b16d961b7c57a2a3e0d19e74abf7ed794cb71f1b861682911c5b38c271c166becaaad8a9bdf85a16b14db386f7ff1659e4e90e51986d4

data/.gitignore

@@ -200,7 +200,8 @@ ModelManifest.xml
 
 /.idea/
 /.bundle/
-/.yardoc
+/.yardoc/
+/.report/
 /Gemfile.lock
 /_yardoc/
 /coverage/

data/Rakefile

@@ -10,5 +10,11 @@ RDoc::Task.new do |rdoc|
 end
 
 RSpec::Core::RakeTask.new do |rspec|
-  rspec.pattern = 'spec/**/*_spec.rb'
+end
+
+namespace :spec do
+  RSpec::Core::RakeTask.new :html do |rspec|
+    rspec.name = "spec::html"
+    rspec.rspec_opts = %w(--format html --out .report/rspec_test_report.html)
+  end
 end

data/lib/thetvdb_party/baseepisoderecord.rb

@@ -158,7 +158,7 @@ module TheTvDbParty
       @productioncode = @hashValues["ProductionCode"]
       @rating = @hashValues["Rating"] ? @hashValues["Rating"].to_f : 0.0
       @ratingcount = @hashValues["RatingCount"] ? @hashValues["RatingCount"].to_i : 0
-      @seasonnumber = @hashValues["SeasonNumber"] ? @hashValues["RatingCount"].to_i : -1
+      @seasonnumber = @hashValues["SeasonNumber"] ? @hashValues["SeasonNumber"].to_i : -1
       @writer = @hashValues["Writer"] ? @hashValues["Writer"].split('|').reject { |a| a.nil? || a.empty? } : []
       @absolute_number = @hashValues["absolute_number"] ? @hashValues["absolute_number"].to_i : -1
       @airsafter_season = @hashValues["airsafter_season"] ? @hashValues["airsafter_season"].to_i : -1

data/lib/thetvdb_party/baseseriesrecord.rb

@@ -128,7 +128,7 @@ module TheTvDbParty
       @airs_dayofweek = @hashValues["Airs_DayOfWeek"]
       @airs_time = @hashValues["Airs_Time"] ? Time.parse(@hashValues["Airs_Time"]) : nil
       @contentrating = @hashValues["ContentRating"]
-      @firstaired = @hashValues["FirstAired"] ? Date.parse(hashValues["FirstAired"]) : nil
+      @firstaired = @hashValues["FirstAired"] ? Date.parse(@hashValues["FirstAired"]) : nil
       @genres = @hashValues["Genre"] ? @hashValues["Genre"].split('|').reject { |a| a.nil? || a.empty? } : []
       @imdb_id = @hashValues["IMDB_ID"]
       @language = @hashValues["Language"]

data/lib/thetvdb_party/version.rb

@@ -1,4 +1,4 @@
 module TheTvDbParty
   # The current version of the *thetvdb_party* gem
-  VERSION = "0.0.11.pre"
+  VERSION = "0.0.12.pre"
 end

data/spec/thetvdb_party/actor_spec.rb

@@ -1,26 +1,76 @@
 require 'rspec'
 require 'thetvdb_party'
 
-describe 'Actor' do
+describe 'TheTvDbParty::Actor' do
+  it 'should have a client associated with it' do
+    client = TheTvDbParty::Client.new(nil)
+    record = TheTvDbParty::Actor.new(client, { })
+    expect(record.client).to be_an_instance_of(TheTvDbParty::Client)
+  end
+
+  it 'should have a nil client if constructed with nil' do
+    record = TheTvDbParty::Actor.new(nil, { })
+    expect(record.client).to be_nil
+  end
+
+  it 'should have an ignored id field' do
+    record = TheTvDbParty::Actor.new(nil, { "id" => "27747" })
+    expect(record.id).to be_an_instance_of(String)
+    expect(record.id).to eq("27747")
+  end
+
+  it 'should have a nil id, if not specified' do
+    record = TheTvDbParty::Actor.new(nil, {})
+    expect(record.id).to be_nil
+  end
+
+  it 'should have a relative/full image path' do
+    record = TheTvDbParty::Actor.new(nil, { "Image" => "actors/27747.jpg" })
+    expect(record.imagepath_relative).to be_an_instance_of(String)
+    expect(record.imagepath_full).to be_an_instance_of(URI)
+    expect(record.imagepath_relative).to eq("actors/27747.jpg")
+    expect(record.imagepath_full).to eq(URI::join(TheTvDbParty::BASE_URL, 'banners/', 'actors/27747.jpg'))
+  end
+
+  it 'should have a nil relative/full image path, if not specified' do
+    record = TheTvDbParty::Actor.new(nil, {})
+    expect(record.imagepath_relative).to be_nil
+    expect(record.imagepath_full).to be_nil
+  end
+
+  it 'should have a name field' do
+    record = TheTvDbParty::Actor.new(nil, { "Name" => "Matthew Fox" })
+    expect(record.name).to be_an_instance_of(String)
+    expect(record.name).to eq("Matthew Fox")
+  end
+
+  it 'should have a nil name, if not specified' do
+    record = TheTvDbParty::Actor.new(nil, {})
+    expect(record.name).to be_nil
+  end
 
-  before(:each) do
-    @valid_actor = TheTvDbParty::Actor.new(nil,
-      {
-          "id" => "0"
-      }
-    )
-    @invalid_actor = TheTvDbParty::Actor.new(nil, { })
+  it 'should have a sorting order between 0-3' do
+    record = TheTvDbParty::Actor.new(nil, { "SortOrder" => "2" })
+    expect(record.sortOrder).to be_an_instance_of(Fixnum)
+    expect(record.sortOrder).to be_between(0, 3).inclusive
+    expect(record.sortOrder).to eq(2)
   end
 
-  it 'should have an id when valid' do
-    expect @valid_actor.id == 0
+  it 'should have a 0 sorting order, if invalid specified' do
+    record = TheTvDbParty::Actor.new(nil, { "SortOrder" => "INVALID_SORT_ORDER" })
+    expect(record.sortOrder).to be_an_instance_of(Fixnum)
+    expect(record.sortOrder).to be_between(0, 3).inclusive
+    expect(record.sortOrder).to eq(0)
   end
 
-  it 'should have a negative id when invalid' do
-    expect @invalid_actor.id < 0
+  it 'should have a 0 sorting order, if invalid specified' do
+    record = TheTvDbParty::Actor.new(nil, {  })
+    expect(record.sortOrder).to be_an_instance_of(Fixnum)
+    expect(record.sortOrder).to be_between(0, 3).inclusive
+    expect(record.sortOrder).to eq(0)
   end
 
   it 'should not instantiate when created without hash values' do
-    expect{TheTvDbParty::Actor.new(nil, nil)}.to raise_error
+    expect { TheTvDbParty::Actor.new(nil, nil) }.to raise_error
   end
 end

data/spec/thetvdb_party/banner_spec.rb

@@ -1,26 +1,233 @@
 require 'rspec'
 require 'thetvdb_party'
 
-describe 'Banner' do
+describe 'TheTvDbParty::Banner' do
+  it 'should have a client associated with it' do
+    client = TheTvDbParty::Client.new(nil)
+    record = TheTvDbParty::Banner.new(client, { })
+    expect(record.client).to be_an_instance_of(TheTvDbParty::Client)
+  end
+
+  it 'should have a nil client if constructed with nil' do
+    record = TheTvDbParty::Banner.new(nil, { })
+    expect(record.client).to be_nil
+  end
+
+  it 'should have an ignored id field' do
+    record = TheTvDbParty::Banner.new(nil, { "id" => "14820" })
+    expect(record.id).to be_an_instance_of(String)
+    expect(record.id).to eq("14820")
+  end
+
+  it 'should have a nil id, if not specified' do
+    record = TheTvDbParty::Banner.new(nil, {})
+    expect(record.id).to be_nil
+  end
+
+  it 'should have a relative/full banner path' do
+    record = TheTvDbParty::Banner.new(nil, { "BannerPath" => "text/80348.jpg" })
+    expect(record.bannerpath_relative).to be_an_instance_of(String)
+    expect(record.bannerpath_full).to be_an_instance_of(URI)
+    expect(record.bannerpath_relative).to eq("text/80348.jpg")
+    expect(record.bannerpath_full).to eq(URI::join(TheTvDbParty::BASE_URL, 'banners/', "text/80348.jpg"))
+  end
+
+  it 'should have a nil relative/full banner path, if not specified' do
+    record = TheTvDbParty::Banner.new(nil, {})
+    expect(record.bannerpath_relative).to be_nil
+    expect(record.bannerpath_full).to be_nil
+  end
+
+  it 'should have a relative/full thumbnail path' do
+    record = TheTvDbParty::Banner.new(nil, { "ThumbnailPath" => "_cache/fanart/original/80348-49.jpg" })
+    expect(record.bannerpath_relative).to be_an_instance_of(String)
+    expect(record.bannerpath_full).to be_an_instance_of(URI)
+    expect(record.bannerpath_relative).to eq("_cache/fanart/original/80348-49.jpg")
+    expect(record.bannerpath_full).to eq(URI::join(TheTvDbParty::BASE_URL, 'banners/', "_cache/fanart/original/80348-49.jpg"))
+  end
+
+  it 'should have a nil relative/full thumbnail path, if not specified' do
+    record = TheTvDbParty::Banner.new(nil, {})
+    expect(record.bannerpath_relative).to be_nil
+    expect(record.bannerpath_full).to be_nil
+  end
+
+  it 'should have a relative/full vignette path' do
+    record = TheTvDbParty::Banner.new(nil, { "VignettePath" => "fanart/vignette/80348-49.jpg" })
+    expect(record.bannerpath_relative).to be_an_instance_of(String)
+    expect(record.bannerpath_full).to be_an_instance_of(URI)
+    expect(record.bannerpath_relative).to eq("fanart/vignette/80348-49.jpg")
+    expect(record.bannerpath_full).to eq(URI::join(TheTvDbParty::BASE_URL, 'banners/', "fanart/vignette/80348-49.jpg"))
+  end
+
+  it 'should have a nil relative/full vignette path, if not specified' do
+    record = TheTvDbParty::Banner.new(nil, {})
+    expect(record.bannerpath_relative).to be_nil
+    expect(record.bannerpath_full).to be_nil
+  end
+
+  it 'should have a banner type' do
+    [ "poster", "fanart", "series", "season" ].each do |expected_valid_bannertype|
+      record = TheTvDbParty::Banner.new(nil, { "BannerType" => expected_valid_bannertype })
+      expect(record.bannertype).to be_an_instance_of(String)
+      expect(record.bannertype).to eq(expected_valid_bannertype)
+    end
+  end
+
+  it 'should have a nil banner type, if specified is invalid' do
+    record = TheTvDbParty::Banner.new(nil, { "BannerType" => "INVALID_BANNERTYPE_STRING" })
+    expect(record.bannertype).to be_nil
+  end
+
+  it 'should have a nil banner type, if not specified' do
+    record = TheTvDbParty::Banner.new(nil, {})
+    expect(record.bannertype).to be_nil
+  end
+
+  it 'should have a secondary banner type' do
+    {
+        "poster" => [ "680x1000" ],
+        "fanart" => [ "1280x720", "1920x1080" ],
+        "series" => [ "text", "graphical", "blank" ],
+        "season" => [ "season", "seasonwide" ]
+    }.each do |expected_valid_bannertype, expected_valid_bannerytype2s|
+      expected_valid_bannerytype2s.each do |expected_valid_bannerytype2|
+        record = TheTvDbParty::Banner.new(nil, { "BannerType" => expected_valid_bannertype, "BannerType2" => expected_valid_bannerytype2 })
+        expect(record.bannertype2).to be_an_instance_of(String)
+        expect(record.bannertype2).to eq(expected_valid_bannerytype2)
+      end
+    end
+  end
+
+  it 'should have a nil secondary banner type, if the primary banner type is valid, but the specified secondary banner type is invalid' do
+    record = TheTvDbParty::Banner.new(nil, { "BannerType" => "poster", "BannerType2" => "abcdef" })
+    expect(record.bannertype2).to be_nil
+  end
+
+  it 'should have a nil secondary banner type, if the primary banner type is valid, but the specified secondary banner type is invalid for that primary banner type' do
+    record = TheTvDbParty::Banner.new(nil, { "BannerType" => "poster", "BannerType2" => "seasonwide" })
+    expect(record.bannertype2).to be_nil
+  end
+
+  it 'should have a nil secondary banner type, if the primary banner type is nil' do
+    record = TheTvDbParty::Banner.new(nil, { "BannerType2" => "seasonwide" })
+    expect(record.bannertype2).to be_nil
+  end
+
+  it 'should have a language associated with it' do
+    record = TheTvDbParty::Banner.new(nil, { "Language" => "en" })
+    expect(record.language).to be_an_instance_of(String)
+    expect(record.language.length).to be_between(2, 3).inclusive
+    expect(record.language).to eq("en")
+  end
+
+  it 'should have a nil language, if the language is not specified' do
+    record = TheTvDbParty::Banner.new(nil, {  })
+    expect(record.language).to be_nil
+  end
+
+  it 'should have a season' do
+    record = TheTvDbParty::Banner.new(nil, { "Season" => "5" })
+    expect(record.season).to be_an_instance_of(Fixnum)
+    expect(record.season).to be >= 0
+  end
+
+  it 'should have a nil season, if it is not specified' do
+    record = TheTvDbParty::Banner.new(nil, {  })
+    expect(record.season).to be_nil
+  end
+
+  it 'should have a rating' do
+    record = TheTvDbParty::Banner.new(nil, { "Rating" => "6.6667" })
+    expect(record.rating).to be_an_instance_of(Float)
+    expect(record.rating).to be >= 0.0
+    expect(record.rating).to be 6.6667
+  end
+
+  it 'should have a zero rating, if it is not specified' do
+    record = TheTvDbParty::Banner.new(nil, {  })
+    expect(record.rating).to be_an_instance_of(Float)
+    expect(record.rating).to be 0.0
+  end
+
+  it 'should have a rating count' do
+    record = TheTvDbParty::Banner.new(nil, { "RatingCount" => "6" })
+    expect(record.rating).to be_an_instance_of(Fixnum)
+    expect(record.rating).to be >= 0
+    expect(record.rating).to be >= 6
+  end
+
+  it 'should have a zero rating count, if it is not specified' do
+    record = TheTvDbParty::Banner.new(nil, {  })
+    expect(record.rating).to be_an_instance_of(Fixnum)
+    expect(record.rating).to be 0
+  end
+
+  it 'should have a true series name flag' do
+    record = TheTvDbParty::Banner.new(nil, { "BannerType" => "fanart", "SeriesName" => "true" })
+    expect(record.seriesName).to be_an_instance_of(TrueClass).or be_an_instance_of(FalseClass)
+    expect(record.seriesName).to be true
+  end
+
+  it 'should have a false series name flag, if banner type is not fanart' do
+    [ "poster", "series", "season" ].each do |expected_valid_bannertype|
+      record = TheTvDbParty::Banner.new(nil, { "BannerType" => expected_valid_bannertype, "SeriesName" => "true" })
+      expect(record.seriesName).to be_an_instance_of(TrueClass).or be_an_instance_of(FalseClass)
+      expect(record.seriesName).to be false
+    end
+  end
+
+  it 'should have a false series name flag, if it is not specified' do
+    record = TheTvDbParty::Banner.new(nil, { "BannerType" => "fanart", "SeriesName" => "true" })
+    expect(record.seriesName).to be_an_instance_of(TrueClass).or be_an_instance_of(FalseClass)
+    expect(record.seriesName).to be false
+  end
+
+  it 'should have a zero rating count, if it is not specified' do
+    record = TheTvDbParty::Banner.new(nil, {  })
+    expect(record.rating).to be_an_instance_of(Fixnum)
+    expect(record.rating).to be 0
+  end
+
+  it 'should have the color fields' do
+    record = TheTvDbParty::Banner.new(nil, { "BannerType" => "fanart", "Colors" => "|81,81,81|15,15,15|201,226,246|" })
+    expect(record.light_accent_color).to be_an_instance_of(Array)
+    expect(record.light_accent_color).to match_array([81, 81, 81])
+    expect(record.dark_accent_color).to be_an_instance_of(Array)
+    expect(record.dark_accent_color).to match_array([15, 15, 15])
+    expect(record.midtone_color).to be_an_instance_of(Array)
+    expect(record.midtone_color).to match_array([201, 226, 246])
+  end
+
+  it 'should have nil color fields, if banner type is not fanart' do
+    record = TheTvDbParty::Banner.new(nil, { "Colors" => "|81,81,81|15,15,15|201,226,246|" })
+    expect(record.light_accent_color).to be_nil
+    expect(record.dark_accent_color).to be_nil
+    expect(record.midtone_color).to be_nil
+  end
 
-  before(:each) do
-    @valid_banner = TheTvDbParty::Banner.new(nil,
-      {
-          "id" => "0"
-      }
-    )
-    @invalid_banner = TheTvDbParty::Banner.new(nil, { })
+  it 'should have nil color fields, if invalid colors are specified' do
+    record = TheTvDbParty::Banner.new(nil, { "Colors" => "INVALID_COLORS" })
+    expect(record.light_accent_color).to be_nil
+    expect(record.dark_accent_color).to be_nil
+    expect(record.midtone_color).to be_nil
   end
 
-  it 'should have an id when valid' do
-    expect @valid_banner.id == 0
+  it 'should have nil color fields, if no colors are specified' do
+    record = TheTvDbParty::Banner.new(nil, {  })
+    expect(record.light_accent_color).to be_nil
+    expect(record.dark_accent_color).to be_nil
+    expect(record.midtone_color).to be_nil
   end
 
-  it 'should have a negative id when invalid' do
-    expect @invalid_banner.id < 0
+  it 'should have nil color fields, if colors are specified, but at least one of the colors does not have exactly three elements' do
+    record = TheTvDbParty::Banner.new(nil, { "Colors" => "|81,81,81,4|15,15,15|201,226,246|" })
+    expect(record.light_accent_color).to be_nil
+    expect(record.dark_accent_color).to be_nil
+    expect(record.midtone_color).to be_nil
   end
 
   it 'should not instantiate when created without hash values' do
-    expect{TheTvDbParty::Banner.new(nil, nil)}.to raise_error
+    expect { TheTvDbParty::Banner.new(nil, nil) }.to raise_error
   end
 end

data/spec/thetvdb_party/baseepisoderecord_spec.rb

@@ -1,26 +1,151 @@
 require 'rspec'
 require 'thetvdb_party'
 
-describe 'BaseEpisodeRecord' do
+describe 'TheTvDbParty::BaseEpisodeRecord' do
+  it 'should have a client associated with it' do
+    client = TheTvDbParty::Client.new(nil)
+    record = TheTvDbParty::BaseEpisodeRecord.new(client, {  })
+    expect(record.client).to be_an_instance_of(TheTvDbParty::Client)
+  end
+
+  it 'should have a nil client if constructed with nil' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, { })
+    expect(record.client).to be_nil
+  end
+
+  it 'should have an id' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, { "id" => "533011" })
+    expect(record.id).to be_an_instance_of(Fixnum)
+    expect(record.id).to eq(533011)
+  end
+
+  it 'should have a negative id, if invalid id specified' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, { "id" => "INVALID_ID" })
+    expect(record.id).to be_an_instance_of(Fixnum)
+    expect(record.id).to be < 0
+  end
+
+  it 'should have a negative id, if id not specified' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, {  })
+    expect(record.id).to be_an_instance_of(Fixnum)
+    expect(record.id).to be < 0
+  end
+
+  it 'should have a combined episode number' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, { "Combined_episodenumber" => "5.2" })
+    expect(record.combined_episodenumber).to be_an_instance_of(Float)
+    expect(record.combined_episodenumber).to eq(5.2)
+  end
+
+  it 'should have a negative combined episode number, if invalid specified' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, { "Combined_episodenumber" => "INVALID_COMBINED_EPISODE_NUMBER" })
+    expect(record.combined_episodenumber).to be_an_instance_of(Float)
+    expect(record.combined_episodenumber).to be < 0
+  end
+
+  it 'should have a negative combined episode number, if not specified' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, {  })
+    expect(record.combined_episodenumber).to be_an_instance_of(Float)
+    expect(record.combined_episodenumber).to be < 0
+  end
+
+  it 'should have a combined season number' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, { "Combined_season" => "5" })
+    expect(record.combined_season).to be_an_instance_of(Fixnum)
+    expect(record.combined_season).to eq(5)
+  end
+
+  it 'should have a negative combined episode number, if invalid specified' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, { "Combined_season" => "INVALID_COMBINED_SEASON" })
+    expect(record.combined_season).to be_an_instance_of(Fixnum)
+    expect(record.combined_season).to be < 0
+  end
+
+  it 'should have a negative combined episode number, if not specified' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, {  })
+    expect(record.combined_season).to be_an_instance_of(Fixnum)
+    expect(record.combined_season).to be < 0
+  end
+
+  it 'should have a DVD chapter field' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, { "DVD_chapter" => "5" })
+    expect(record.dvd_chapter).to be_an_instance_of(String)
+    expect(record.dvd_chapter).to eq("5")
+  end
+
+  it 'should have a nil DVD chapter field, if not specified' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, {  })
+    expect(record.dvd_chapter).to be_nil
+  end
+
+  it 'should have a DVD disc field' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, { "DVD_discid" => "5" })
+    expect(record.dvd_discid).to be_an_instance_of(String)
+    expect(record.dvd_discid).to eq("5")
+  end
+
+  it 'should have a nil DVD disc field, if not specified' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, {  })
+    expect(record.dvd_chapter).to be_nil
+  end
+
+  it 'should have a DVD episode number' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, { "DVD_episodenumber" => "1.2" })
+    expect(record.dvd_episodenumber).to be_an_instance_of(Float)
+    expect(record.dvd_episodenumber).to eq(1.2)
+  end
+
+  it 'should have a negative DVD episode number, if invalid specified' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, { "DVD_episodenumber" => "INVALID_DVD_EPISODENUMBER" })
+    expect(record.dvd_episodenumber).to be_an_instance_of(Float)
+    expect(record.dvd_episodenumber).to be < 0
+  end
+
+  it 'should have a negative DVD episode number, if not specified' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, {  })
+    expect(record.dvd_episodenumber).to be_an_instance_of(Float)
+    expect(record.dvd_episodenumber).to be < 0
+  end
+
+  it 'should have a DVD season number' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, { "DVD_season" => "1" })
+    expect(record.dvd_episodenumber).to be_an_instance_of(Fixnum)
+    expect(record.dvd_episodenumber).to eq(1.2)
+  end
+
+  it 'should have a negative DVD season number, if invalid specified' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, { "DVD_season" => "INVALID_DVD_SEASON" })
+    expect(record.dvd_episodenumber).to be_an_instance_of(Fixnum)
+    expect(record.dvd_episodenumber).to be < 0
+  end
+
+  it 'should have a negative DVD season number, if not specified' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, {  })
+    expect(record.dvd_episodenumber).to be_an_instance_of(Fixnum)
+    expect(record.dvd_episodenumber).to be < 0
+  end
+
+  it 'should have a single element array for the director, if only one name is specified' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, { "Director" => "Graeme Harper" })
+    expect(record.director).to match_array(["Graeme Harper"])
+  end
 
-  before(:each) do
-    @valid_record = TheTvDbParty::BaseEpisodeRecord.new(nil,
-      {
-          "id" => "0"
-      }
-    )
-    @invalid_record = TheTvDbParty::BaseEpisodeRecord.new(nil, { })
+  it 'should have a multi element array for the director, if a pipe-delimited list is specified' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, { "Director" => "Bruno Heller|Tom Szentgyorgyi|Jordan Harper" })
+    expect(record.director).to match_array(["Bruno Heller", "Tom Szentgyorgyi", "Jordan Harper"])
   end
 
-  it 'should have an id when valid' do
-    expect @valid_record.id == 0
+  it 'should have a multi element array for the director, if a pipe-surrounded list is specified' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, { "Director" => "|Bruno Heller|Tom Szentgyorgyi|Jordan Harper|" })
+    expect(record.director).to match_array(["Bruno Heller", "Tom Szentgyorgyi", "Jordan Harper"])
   end
 
-  it 'should have a negative id when invalid' do
-    expect @invalid_record.id < 0
+  it 'should have an empty array for director, if none is specified' do
+    record = TheTvDbParty::BaseEpisodeRecord.new(nil, {  })
+    expect(record.director).to be_empty
   end
 
   it 'should not instantiate when created without hash values' do
-    expect{TheTvDbParty::BaseEpisodeRecord.new(nil, nil)}.to raise_error
+    expect { TheTvDbParty::BaseEpisodeRecord.new(nil, nil) }.to raise_error
   end
 end

data/thetvdb_party.gemspec

@@ -27,6 +27,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "bundler", "~> 1.7"
   spec.add_development_dependency "rake", "~> 10.0"
   spec.add_development_dependency "rspec", "~> 3.2"
+  spec.add_development_dependency "coderay", "~> 1.1"
 
   spec.add_dependency "httparty", "~> 0.6"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: thetvdb_party
 version: !ruby/object:Gem::Version
-  version: 0.0.11.pre
+  version: 0.0.12.pre
 platform: ruby
 authors:
 - Fredrik Høisæther Rasch
@@ -12,7 +12,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-02-23 00:00:00.000000000 Z
+date: 2015-02-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -56,6 +56,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.2'
+- !ruby/object:Gem::Dependency
+  name: coderay
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.1'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.1'
 - !ruby/object:Gem::Dependency
   name: httparty
   requirement: !ruby/object:Gem::Requirement