7digital 0.1.2 → 0.1.5

data/spec/model/release_spec.rb

@@ -1,134 +1,134 @@
-require "date"
-require File.expand_path('../../spec_helper', __FILE__)
-
-describe "Release" do
-
-  before do
-    @configuration = stub(OpenStruct)
-    @configuration.stub!(:lazy_load).and_return(true)
-
-    @client = stub(Sevendigital::Client)
-    @client.stub!(:configuration).and_return(@configuration)
-    @release_manager = mock(Sevendigital::ReleaseManager)
-    @client.stub!(:release).and_return @release_manager
-    
-    @release = Sevendigital::Release.new(@client)
-    @release.id = 1234
-  end
-
-  it "should get its basic details from manager" do
-    expected_options = {:page => 2}
-    fresh_release = fake_release_with_details
-
-    @release_manager.should_receive(:get_details) { |release_id, options|
-      release_id.should == @release.id
-      (options.keys & expected_options.keys).should == expected_options.keys
-      fresh_release
-    }
-    @release.get_details(expected_options)
-    
-    @release.version.should == fresh_release.version
-    @release.type.should == fresh_release.type
-    @release.artist.should == fresh_release.artist
-    @release.image.should == fresh_release.image
-    @release.url.should == fresh_release.url
-    @release.release_date.should == fresh_release.release_date
-    @release.added_date.should == fresh_release.added_date
-    @release.barcode.should == fresh_release.barcode
-    @release.year.should == fresh_release.year
-    @release.explicit_content.should == fresh_release.explicit_content
-    @release.formats.should == fresh_release.formats
-    @release.label.should == fresh_release.label
-    @release.price.should == fresh_release.price
-
-  end
-
-  it "should lazy load price even if already populated but price value is not available" do
-    @release.price = Sevendigital::Price.new
-    @release.should_receive(:demand_price)
-    @release.price
-
-  end
-
-  it "get_tracks should get tracks from manager" do
-    expected_options = {:page => 2}
-   
-    @release_manager.should_receive(:get_tracks) { |release_id, options|
-      release_id.should == @release.id
-      (options.keys & expected_options.keys).should == expected_options.keys
-      fake_track_list
-    }
-    @release.get_tracks(expected_options)
-
-  end
-
-
-   it "get_recommendations should get recommendations from manager" do
-    expected_options = {:page => 2}
-
-    @release_manager.should_receive(:get_recommendations) { |release_id, options|
-      release_id.should == @release.id
-      (options.keys & expected_options.keys).should == expected_options.keys
-      fake_releases_list
-    }
-    @release.get_recommendations(expected_options)
-
-   end
-
-    it "get_tags should get tags from manager" do
-    expected_options = {:page => 2}
-
-    @release_manager.should_receive(:get_tags) { |release_id, options|
-      release_id.should == @release.id
-      (options.keys & expected_options.keys).should == expected_options.keys
-      []
-    }
-    @release.get_tags(expected_options)
-
-    end
-
-  it "image should return link to image with specified image size" do
-    release = fake_release_with_details
-    release.image(333).should == "http://cdn.7static.com/static/img/sleeveart/00/008/097/0000809794_333.jpg"
-  end
-
-  it "image should return link to image with returned image size" do
-    release = fake_release_with_details
-    release.image(333).should == "http://cdn.7static.com/static/img/sleeveart/00/008/097/0000809794_100.jpg"
-  end
-
-
-  def fake_releases_list
-    releases = Array.new
-    releases << Sevendigital::Release.new(@client)
-    releases << Sevendigital::Release.new(@client)
-    releases
-  end
-
-  def fake_track_list
-    tracks = Array.new
-    tracks << Sevendigital::Track.new(@client)
-    tracks << Sevendigital::Track.new(@client)
-    tracks
-  end
-
-  def fake_release_with_details
-    release = Sevendigital::Release.new(@client)
-    release.version = "(release version)"
-    release.type = :album
-    release.artist = Sevendigital::Artist.new(@client)
-    release.image = "http://cdn.7static.com/static/img/sleeveart/00/008/097/0000809794_100.jpg"
-    release.url = "url"
-    release.release_date = DateTime.new(2010, 1, 1)
-    release.added_date = DateTime.new(2000, 1, 1)
-    release.barcode = "AAA"
-    release.year = 1999
-    release.explicit_content = true
-    release.formats = []
-    release.label = Sevendigital::Label.new
-    price = Sevendigital::Price.new
-    price.value = 5
-    release.price = price
-    release
-end
+require "date"
+require File.expand_path('../../spec_helper', __FILE__)
+
+describe "Release" do
+
+  before do
+    @configuration = stub(OpenStruct)
+    @configuration.stub!(:lazy_load).and_return(true)
+
+    @client = stub(Sevendigital::Client)
+    @client.stub!(:configuration).and_return(@configuration)
+    @release_manager = mock(Sevendigital::ReleaseManager)
+    @client.stub!(:release).and_return @release_manager
+    
+    @release = Sevendigital::Release.new(@client)
+    @release.id = 1234
+  end
+
+  it "should get its basic details from manager" do
+    expected_options = {:page => 2}
+    fresh_release = fake_release_with_details
+
+    @release_manager.should_receive(:get_details) { |release_id, options|
+      release_id.should == @release.id
+      (options.keys & expected_options.keys).should == expected_options.keys
+      fresh_release
+    }
+    @release.get_details(expected_options)
+    
+    @release.version.should == fresh_release.version
+    @release.type.should == fresh_release.type
+    @release.artist.should == fresh_release.artist
+    @release.image.should == fresh_release.image
+    @release.url.should == fresh_release.url
+    @release.release_date.should == fresh_release.release_date
+    @release.added_date.should == fresh_release.added_date
+    @release.barcode.should == fresh_release.barcode
+    @release.year.should == fresh_release.year
+    @release.explicit_content.should == fresh_release.explicit_content
+    @release.formats.should == fresh_release.formats
+    @release.label.should == fresh_release.label
+    @release.price.should == fresh_release.price
+
+  end
+
+  it "should lazy load price even if already populated but price value is not available" do
+    @release.price = Sevendigital::Price.new
+    @release.should_receive(:demand_price)
+    @release.price
+
+  end
+
+  it "get_tracks should get tracks from manager" do
+    expected_options = {:page => 2}
+   
+    @release_manager.should_receive(:get_tracks) { |release_id, options|
+      release_id.should == @release.id
+      (options.keys & expected_options.keys).should == expected_options.keys
+      fake_track_list
+    }
+    @release.get_tracks(expected_options)
+
+  end
+
+
+   it "get_recommendations should get recommendations from manager" do
+    expected_options = {:page => 2}
+
+    @release_manager.should_receive(:get_recommendations) { |release_id, options|
+      release_id.should == @release.id
+      (options.keys & expected_options.keys).should == expected_options.keys
+      fake_releases_list
+    }
+    @release.get_recommendations(expected_options)
+
+   end
+
+    it "get_tags should get tags from manager" do
+    expected_options = {:page => 2}
+
+    @release_manager.should_receive(:get_tags) { |release_id, options|
+      release_id.should == @release.id
+      (options.keys & expected_options.keys).should == expected_options.keys
+      []
+    }
+    @release.get_tags(expected_options)
+
+    end
+
+  it "image should return link to image with specified image size" do
+    release = fake_release_with_details
+    release.image(333).should == "http://cdn.7static.com/static/img/sleeveart/00/008/097/0000809794_333.jpg"
+  end
+
+  it "image should return link to image with returned image size" do
+    release = fake_release_with_details
+    release.image.should == "http://cdn.7static.com/static/img/sleeveart/00/008/097/0000809794_100.jpg"
+  end
+
+
+  def fake_releases_list
+    releases = Array.new
+    releases << Sevendigital::Release.new(@client)
+    releases << Sevendigital::Release.new(@client)
+    releases
+  end
+
+  def fake_track_list
+    tracks = Array.new
+    tracks << Sevendigital::Track.new(@client)
+    tracks << Sevendigital::Track.new(@client)
+    tracks
+  end
+
+  def fake_release_with_details
+    release = Sevendigital::Release.new(@client)
+    release.version = "(release version)"
+    release.type = :album
+    release.artist = Sevendigital::Artist.new(@client)
+    release.image = "http://cdn.7static.com/static/img/sleeveart/00/008/097/0000809794_100.jpg"
+    release.url = "url"
+    release.release_date = DateTime.new(2010, 1, 1)
+    release.added_date = DateTime.new(2000, 1, 1)
+    release.barcode = "AAA"
+    release.year = 1999
+    release.explicit_content = true
+    release.formats = []
+    release.label = Sevendigital::Label.new
+    price = Sevendigital::Price.new
+    price.value = 5
+    release.price = price
+    release
+end
 end

data/spec/model/sevendigital_object_spec.rb

@@ -1,74 +1,74 @@
-require "date"
-require File.expand_path('../../spec_helper', __FILE__)
-
-describe "SevendigitalObject" do
-
-  before do
-
-    @configuration = test_configuration
-    @configuration.lazy_load = true
-
-    stub_api_client(@configuration)
-
-    @client.stub!(:verbose?).and_return(true)
-    @test_object = Sevendigital::SevendigitalObject.new(@client)
-  end
-
-  it "sevendigital extended property should define accessor that calls demand method" do
-    class TestClass < Sevendigital::SevendigitalObject
-      sevendigital_extended_property :test_property
-    end
-
-    test_object = TestClass.new(@client)
-    test_object.should_receive(:demand_test_property)
-    test_object.test_property
-
-  end
-
-  it "sevendigital extended property should define accessor that doesn't swallow SevenDigital errors" do
-    class TestClass < Sevendigital::SevendigitalObject
-      sevendigital_extended_property :test_property
-    end
-    @configuration.stub!(:ignorant_lazy_load).and_return(false)
-    test_object = TestClass.new(@client)
-    test_object.should_receive(:demand_test_property).and_raise(Sevendigital::SevendigitalError)
-    running {test_object.test_property}.should raise_error(Sevendigital::SevendigitalError)
-  end
-  
-  it "sevendigital extended property should define accessor that swallows SevenDigital errors if client is ignorant" do
-    class TestClass < Sevendigital::SevendigitalObject
-      sevendigital_extended_property :test_property
-    end
-    @configuration.stub!(:ignorant_lazy_load).and_return(true)
-    test_object = TestClass.new(@client)
-    test_object.should_receive(:demand_test_property).and_raise(Sevendigital::SevendigitalError)
-    test_object.test_property
-  end
-
-  it "sevendigital extended property should define demand method that calls get method if property not set" do
-    class TestClass < Sevendigital::SevendigitalObject
-      sevendigital_extended_property :test_property
-    end
-
-    test_object = TestClass.new(@client)
-    test_object.should_receive(:get_test_property).and_return("a value")
-    test_object.instance_variable_get("@test_property").should == nil
-    test_object.demand_test_property
-    test_object.instance_variable_get("@test_property").should == "a value"
-
-  end
-
-  it "sevendigital extended property should define demand method that does nothing if property is set" do
-    class TestClass < Sevendigital::SevendigitalObject
-      sevendigital_extended_property :test_property
-    end
-
-    test_object = TestClass.new(@client)
-    test_object.instance_variable_set("@test_property", "a value")
-    test_object.should_not_receive(:get_test_property)
-    test_object.demand_test_property
-
-  end
-
-  
+require "date"
+require File.expand_path('../../spec_helper', __FILE__)
+
+describe "SevendigitalObject" do
+
+  before do
+
+    @configuration = test_configuration
+    @configuration.lazy_load = true
+
+    stub_api_client(@configuration)
+
+    @client.stub!(:verbose?).and_return(true)
+    @test_object = Sevendigital::SevendigitalObject.new(@client)
+  end
+
+  it "sevendigital extended property should define accessor that calls demand method" do
+    class TestClass < Sevendigital::SevendigitalObject
+      sevendigital_extended_property :test_property
+    end
+
+    test_object = TestClass.new(@client)
+    test_object.should_receive(:demand_test_property)
+    test_object.test_property
+
+  end
+
+  it "sevendigital extended property should define accessor that doesn't swallow SevenDigital errors" do
+    class TestClass < Sevendigital::SevendigitalObject
+      sevendigital_extended_property :test_property
+    end
+    @configuration.stub!(:ignorant_lazy_load).and_return(false)
+    test_object = TestClass.new(@client)
+    test_object.should_receive(:demand_test_property).and_raise(Sevendigital::SevendigitalError)
+    running {test_object.test_property}.should raise_error(Sevendigital::SevendigitalError)
+  end
+  
+  it "sevendigital extended property should define accessor that swallows SevenDigital errors if client is ignorant" do
+    class TestClass < Sevendigital::SevendigitalObject
+      sevendigital_extended_property :test_property
+    end
+    @configuration.stub!(:ignorant_lazy_load).and_return(true)
+    test_object = TestClass.new(@client)
+    test_object.should_receive(:demand_test_property).and_raise(Sevendigital::SevendigitalError)
+    test_object.test_property
+  end
+
+  it "sevendigital extended property should define demand method that calls get method if property not set" do
+    class TestClass < Sevendigital::SevendigitalObject
+      sevendigital_extended_property :test_property
+    end
+
+    test_object = TestClass.new(@client)
+    test_object.should_receive(:get_test_property).and_return("a value")
+    test_object.instance_variable_get("@test_property").should == nil
+    test_object.demand_test_property
+    test_object.instance_variable_get("@test_property").should == "a value"
+
+  end
+
+  it "sevendigital extended property should define demand method that does nothing if property is set" do
+    class TestClass < Sevendigital::SevendigitalObject
+      sevendigital_extended_property :test_property
+    end
+
+    test_object = TestClass.new(@client)
+    test_object.instance_variable_set("@test_property", "a value")
+    test_object.should_not_receive(:get_test_property)
+    test_object.demand_test_property
+
+  end
+
+  
 end

data/spec/model/track_spec.rb

@@ -1,145 +1,147 @@
-require "date"
-require File.expand_path('../../spec_helper', __FILE__)
-
-describe "Track" do
-
-  before do
-    @client = stub(Sevendigital::Client)
-    @track_manager = mock(Sevendigital::TrackManager)
-    @client.stub!(:track).and_return @track_manager
-    @client.stub!(:configuration).and_return OpenStruct.new
-    
-    @track = Sevendigital::Track.new(@client)
-    @track.id = 1234
-    @track.release = Sevendigital::Release.new(@client)
-    @track.release.id = 5678
-  end
-
-  it "get_details should get track's basic details from track manager " do
-    expected_options = {:page => 2}
-    fresh_track = fake_track_with_details
-
-    @track_manager.should_receive(:get_details_from_release) { |track_id, release_id, options|
-      track_id.should == @track.id
-      release_id.should == @track.release.id
-      (options.keys & expected_options.keys).should == expected_options.keys
-      fresh_track
-    }
-    @track.get_details(expected_options)
-    
-    @track.track_number.should == fresh_track.track_number
-    @track.duration.should == fresh_track.duration
-    @track.explicit_content.should == fresh_track.explicit_content
-    @track.isrc.should == fresh_track.isrc
-    @track.release.id.should == fresh_track.release.id
-    @track.url.should == fresh_track.url
-
-  end
-
-  it "preview_url should get preview URL from manager" do
-    expected_options = {:country => "mx"}
-    fake_preview_url = "http://preview.com/trackid/123456"
-
-    @track_manager.should_receive(:build_preview_url) { |track_id, options|
-      track_id.should == @track.id
-      (options.keys & expected_options.keys).should == expected_options.keys
-      fake_preview_url
-    }
-    
-    @track.preview_url(expected_options).should == fake_preview_url
-
-  end
-
-  it "should have a short title without any version in brackets" do
-
-    @track.title = "Girl With One Eye (Bayou Percussion Version)"
-    @track.short_title.should == "Girl With One Eye"
-
-    @track.title = "The Thing I Like Best About Him Is His Girlfriend (Gr's Man-Made Island Edition)"
-    @track.short_title.should == "The Thing I Like Best About Him Is His Girlfriend"
-
-    @track.title = "Your Head to Your Toes (Live at the BBC)"
-    @track.short_title.should == "Your Head to Your Toes"
-
-    @track.title = "Welcome To The World of The Plastic Beach (Feat. Snoop Dogg and Hypnotic Brass Ensemble)"
-    @track.short_title.should == "Welcome To The World of The Plastic Beach"
-
-    @track.title = "(lp version)"
-    @track.short_title.should == "(lp version)"
-
-    @track.title = "What's the Story (Morning Glory)"
-    @track.short_title.should == "What's the Story (Morning Glory)"
-
-    @track.title = "track 1 (something else)"
-    @track.short_title.should == "track 1 (something else)"
-
-    @track.title = "track 1 (dj remix)"
-    @track.short_title.should == "track 1"
-
-    @track.title = "track  1  (  dj mix  )"
-    @track.short_title.should == "track  1"
- end
-
-  describe "alternate_version?" do
-
-    before do
-      @artist1 = Sevendigital::Artist.new(@client)
-      @artist2 = Sevendigital::Artist.new(@client)
-      @track1 = Sevendigital::Track.new(@client)
-      @track1.artist = @artist1
-      @track2 = Sevendigital::Track.new(@client)
-      @track2.artist = @artist2
-
-      @artist1.name = "artist 1"
-      @track1.title = "track 1"
-
-    end
-
-    it "should be considered alternate version if artist names and titles match"  do
-      @artist2.name = "Artist 1"
-      @track2.title = "Track 1"
-
-      @track1.alternate_version_of?(@track2).should == true
-    end
-
-    it "should be considered alternate version if artist names and short titles match"  do
-      @track1.title = "track 1 (uk version)"
-     
-      @artist2.name = "Artist 1"
-      @track2.title = "TRACK 1 (us remix)"
-
-      @track1.alternate_version_of?(@track2).should == true
-    end
-
-    it "should not be considered alternate version if artist names don't match" do
-      @artist2.name = "artist 2"
-      @track2.title = "track 1"
-
-      @track1.alternate_version_of?(@track2).should == false
-    end
-
-    it "should not be considered alternate version if short titles don't match" do
-      @artist2.name = "artist 1"
-      @track2.title = "track 2"
-
-      @track1.alternate_version_of?(@track2).should == false
-    end
-
-
-  end
-
-  def fake_track_with_details
-    track = Sevendigital::Track.new(@client)
-    track.title = "Song 1"
-    track.version = "Version 2"
-    track.artist = Sevendigital::Artist.new(@client)
-    track.track_number = 7
-    track.duration = 77
-    track.explicit_content = true
-    track.isrc = "7D"
-    track.release = Sevendigital::Release.new(@client)
-    track.release.id = 5678
-    track.url = "http://aaa.bbb.ccc/"
-    track
-  end
+require "date"
+require File.expand_path('../../spec_helper', __FILE__)
+
+describe "Track" do
+
+  before do
+    @client = stub(Sevendigital::Client)
+    @track_manager = mock(Sevendigital::TrackManager)
+    @client.stub!(:track).and_return @track_manager
+    @client.stub!(:configuration).and_return OpenStruct.new
+    
+    @track = Sevendigital::Track.new(@client)
+    @track.id = 1234
+    @track.release = Sevendigital::Release.new(@client)
+    @track.release.id = 5678
+  end
+
+  it "get_details should get track's basic details from track manager " do
+    expected_options = {:page => 2}
+    fresh_track = fake_track_with_details
+
+    @track_manager.should_receive(:get_details_from_release) { |track_id, release_id, options|
+      track_id.should == @track.id
+      release_id.should == @track.release.id
+      (options.keys & expected_options.keys).should == expected_options.keys
+      fresh_track
+    }
+    @track.get_details(expected_options)
+    
+    @track.track_number.should == fresh_track.track_number
+    @track.duration.should == fresh_track.duration
+    @track.explicit_content.should == fresh_track.explicit_content
+    @track.isrc.should == fresh_track.isrc
+    @track.release.id.should == fresh_track.release.id
+    @track.url.should == fresh_track.url
+    @track.type.should == fresh_track.type
+
+  end
+
+  it "preview_url should get preview URL from manager" do
+    expected_options = {:country => "mx"}
+    fake_preview_url = "http://preview.com/trackid/123456"
+
+    @track_manager.should_receive(:build_preview_url) { |track_id, options|
+      track_id.should == @track.id
+      (options.keys & expected_options.keys).should == expected_options.keys
+      fake_preview_url
+    }
+    
+    @track.preview_url(expected_options).should == fake_preview_url
+
+  end
+
+  it "should have a short title without any version in brackets" do
+
+    @track.title = "Girl With One Eye (Bayou Percussion Version)"
+    @track.short_title.should == "Girl With One Eye"
+
+    @track.title = "The Thing I Like Best About Him Is His Girlfriend (Gr's Man-Made Island Edition)"
+    @track.short_title.should == "The Thing I Like Best About Him Is His Girlfriend"
+
+    @track.title = "Your Head to Your Toes (Live at the BBC)"
+    @track.short_title.should == "Your Head to Your Toes"
+
+    @track.title = "Welcome To The World of The Plastic Beach (Feat. Snoop Dogg and Hypnotic Brass Ensemble)"
+    @track.short_title.should == "Welcome To The World of The Plastic Beach"
+
+    @track.title = "(lp version)"
+    @track.short_title.should == "(lp version)"
+
+    @track.title = "What's the Story (Morning Glory)"
+    @track.short_title.should == "What's the Story (Morning Glory)"
+
+    @track.title = "track 1 (something else)"
+    @track.short_title.should == "track 1 (something else)"
+
+    @track.title = "track 1 (dj remix)"
+    @track.short_title.should == "track 1"
+
+    @track.title = "track  1  (  dj mix  )"
+    @track.short_title.should == "track  1"
+ end
+
+  describe "alternate_version?" do
+
+    before do
+      @artist1 = Sevendigital::Artist.new(@client)
+      @artist2 = Sevendigital::Artist.new(@client)
+      @track1 = Sevendigital::Track.new(@client)
+      @track1.artist = @artist1
+      @track2 = Sevendigital::Track.new(@client)
+      @track2.artist = @artist2
+
+      @artist1.name = "artist 1"
+      @track1.title = "track 1"
+
+    end
+
+    it "should be considered alternate version if artist names and titles match"  do
+      @artist2.name = "Artist 1"
+      @track2.title = "Track 1"
+
+      @track1.alternate_version_of?(@track2).should == true
+    end
+
+    it "should be considered alternate version if artist names and short titles match"  do
+      @track1.title = "track 1 (uk version)"
+     
+      @artist2.name = "Artist 1"
+      @track2.title = "TRACK 1 (us remix)"
+
+      @track1.alternate_version_of?(@track2).should == true
+    end
+
+    it "should not be considered alternate version if artist names don't match" do
+      @artist2.name = "artist 2"
+      @track2.title = "track 1"
+
+      @track1.alternate_version_of?(@track2).should == false
+    end
+
+    it "should not be considered alternate version if short titles don't match" do
+      @artist2.name = "artist 1"
+      @track2.title = "track 2"
+
+      @track1.alternate_version_of?(@track2).should == false
+    end
+
+
+  end
+
+  def fake_track_with_details
+    track = Sevendigital::Track.new(@client)
+    track.title = "Song 1"
+    track.version = "Version 2"
+    track.artist = Sevendigital::Artist.new(@client)
+    track.track_number = 7
+    track.duration = 77
+    track.explicit_content = true
+    track.isrc = "7D"
+    track.release = Sevendigital::Release.new(@client)
+    track.release.id = 5678
+    track.url = "http://aaa.bbb.ccc/"
+    track.type = :audio
+    track
+  end
 end