7digital 0.0.5 → 0.0.6

data/spec/model/basket_spec.rb

@@ -1,63 +1,63 @@
-require File.expand_path('../../spec_helper', __FILE__)
-
-describe "Basket" do
-
-  before do
-    @client = stub(Sevendigital::Client)
-    @basket_manager = mock(Sevendigital::BasketManager)
-    @client.stub!(:basket).and_return @basket_manager
-    
-    @basket = Sevendigital::Basket.new(@client)
-    @basket.id = "00000000-0000-0000-0000-000000000001"
-  end
-
-  it "add_item should should call add_item on basket manager and update the basket items" do
-    expected_options = {:page => 2}
-    a_track_id = 123
-    a_release_id = 456
-
-    a_basket_id = @basket.id
-    an_updated_basket = fake_basket_with_items(@basket.id)
-
-    @basket_manager.should_receive(:add_item) { |basket_id, release_id, track_id, options|
-      basket_id.should == @basket.id
-      release_id.should == a_release_id
-      track_id.should == a_track_id
-      (options.keys & expected_options.keys).should == expected_options.keys
-      an_updated_basket
-    }
-    @basket.add_item(a_release_id, a_track_id, expected_options)
-    
-    @basket.id.should == a_basket_id
-    @basket.basket_items.should == an_updated_basket.basket_items
-   
-  end
-
-  it "remove_item should should call remove_item on basket manager and update the basket items" do
-    expected_options = {:page => 2}
-    an_item_id = 123
-
-    a_basket_id = @basket.id
-    an_updated_basket = fake_basket_with_items(@basket.id)
-
-    @basket_manager.should_receive(:remove_item) { |basket_id, item_id, options|
-      basket_id.should == @basket.id
-      item_id.should == an_item_id
-      (options.keys & expected_options.keys).should == expected_options.keys
-      an_updated_basket
-    }
-    @basket.remove_item(an_item_id, expected_options)
-
-    @basket.id.should == a_basket_id
-    @basket.basket_items.should == an_updated_basket.basket_items
-
-  end
-
-
-  def fake_basket_with_items(id)
-    basket = Sevendigital::Basket.new(@client)
-    basket.id = id
-    basket.basket_items = [Sevendigital::BasketItem.new(@client), Sevendigital::BasketItem.new(@client)]
-    basket
-end
+require File.expand_path('../../spec_helper', __FILE__)
+
+describe "Basket" do
+
+  before do
+    @client = stub(Sevendigital::Client)
+    @basket_manager = mock(Sevendigital::BasketManager)
+    @client.stub!(:basket).and_return @basket_manager
+    
+    @basket = Sevendigital::Basket.new(@client)
+    @basket.id = "00000000-0000-0000-0000-000000000001"
+  end
+
+  it "add_item should should call add_item on basket manager and update the basket items" do
+    expected_options = {:page => 2}
+    a_track_id = 123
+    a_release_id = 456
+
+    a_basket_id = @basket.id
+    an_updated_basket = fake_basket_with_items(@basket.id)
+
+    @basket_manager.should_receive(:add_item) { |basket_id, release_id, track_id, options|
+      basket_id.should == @basket.id
+      release_id.should == a_release_id
+      track_id.should == a_track_id
+      (options.keys & expected_options.keys).should == expected_options.keys
+      an_updated_basket
+    }
+    @basket.add_item(a_release_id, a_track_id, expected_options)
+    
+    @basket.id.should == a_basket_id
+    @basket.basket_items.should == an_updated_basket.basket_items
+   
+  end
+
+  it "remove_item should should call remove_item on basket manager and update the basket items" do
+    expected_options = {:page => 2}
+    an_item_id = 123
+
+    a_basket_id = @basket.id
+    an_updated_basket = fake_basket_with_items(@basket.id)
+
+    @basket_manager.should_receive(:remove_item) { |basket_id, item_id, options|
+      basket_id.should == @basket.id
+      item_id.should == an_item_id
+      (options.keys & expected_options.keys).should == expected_options.keys
+      an_updated_basket
+    }
+    @basket.remove_item(an_item_id, expected_options)
+
+    @basket.id.should == a_basket_id
+    @basket.basket_items.should == an_updated_basket.basket_items
+
+  end
+
+
+  def fake_basket_with_items(id)
+    basket = Sevendigital::Basket.new(@client)
+    basket.id = id
+    basket.basket_items = [Sevendigital::BasketItem.new(@client), Sevendigital::BasketItem.new(@client)]
+    basket
+end
 end

data/spec/model/release_spec.rb

@@ -1,111 +1,123 @@
-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
-
-  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 = "image"
-    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
+
+  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 = "image"
+    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 = stub(Sevendigital::ClientConfiguration)
-    @configuration.stub!(:lazy_load).and_return(true)
-
-    @client = stub(Sevendigital::Client)
-    @client.stub!(:configuration).and_return(@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 = stub(Sevendigital::ClientConfiguration)
+    @configuration.stub!(:lazy_load).and_return(true)
+
+    @client = stub(Sevendigital::Client)
+    @client.stub!(:configuration).and_return(@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