7digital 0.0.5 → 0.0.6

data/spec/digestion_tract/user_card_digestor_spec.rb

@@ -0,0 +1,58 @@
+# coding: utf-8
+require File.expand_path('../../spec_helper', __FILE__)
+
+describe "UserCardDigestor" do
+
+  before do
+     @card_digestor = Sevendigital::UserCardDigestor.new(nil)
+  end
+
+   it "should not digest from invalid xml but throw up (exception)" do
+
+     xml_response = <<XML
+     <xxx>
+       <type>invalid</type>
+     </xxx>
+XML
+
+    running {@card_digestor.from_xml(xml_response)}.should raise_error(Sevendigital::DigestiveProblem)
+  end
+
+  it "should parse from xml and populate minimum available properties" do
+
+    xml_response = <<XML
+<card>
+  <type>MasterCard</type>
+  <last4digits>1234</last4digits>
+</card>
+XML
+
+    card = @card_digestor.from_xml(xml_response)
+    card.type.should == "MasterCard"
+    card.last_4_digits.should == "1234"
+
+  end
+
+  it "should parse from xml and populate all properties" do
+
+    xml_response = load_sample_object_xml("user_payment_card")
+
+    card = @card_digestor.from_xml(xml_response)
+    card.id.should == 9876
+    card.type.should == "VISA"
+    card.last_4_digits.should == "2345"
+    card.expiry_date.should == "201109"
+    card.card_holder_name.should == "Mr. John Smith"
+
+  end
+
+  it "should digest xml containing empty list of cards and return an empty array" do
+
+    xml_response = load_sample_object_xml("user_payment_card_list_empty")
+
+    cards = @card_digestor.list_from_xml(xml_response, :cards)
+    cards.size.should == 0
+
+  end
+
+end

data/spec/digestion_tract/user_digestor_spec.rb

@@ -0,0 +1,44 @@
+# coding: utf-8
+require File.expand_path('../../spec_helper', __FILE__)
+
+describe "UserDigestor" do
+
+  before do
+     @user_digestor = Sevendigital::UserDigestor.new(nil)
+  end
+
+   it "should not digest from invalid xml but throw up (exception)" do
+
+     xml_response = <<XML
+     <xxx>
+       <type>invalid</type>
+     </xxx>
+XML
+
+    running {@user_digestor.from_xml(xml_response)}.should raise_error(Sevendigital::DigestiveProblem)
+  end
+
+  it "should parse from xml and populate minimum available properties" do
+
+    xml_response = <<XML
+<user>
+  <type>7digital</type>
+</user>
+XML
+
+    user = @user_digestor.from_xml(xml_response)
+    user.type.should == :"7digital"
+    
+  end
+
+  it "should parse from xml and populate all properties" do
+
+    xml_response = load_sample_object_xml("user")
+
+    user = @user_digestor.from_xml(xml_response)
+    user.id.should == "123456"
+    user.type.should == :"7digital"
+    user.email_address.should == "user@example.com"
+  end
+
+end

data/spec/management/artist_manager_spec.rb

@@ -1,134 +1,152 @@
-require File.expand_path('../../spec_helper', __FILE__)
-
-describe "ArtistManager" do
-
-  before do
-    @client = stub(Sevendigital::Client)
-    @client.stub!(:operator).and_return(mock(Sevendigital::ApiOperator))
-    @artist_manager = Sevendigital::ArtistManager.new(@client)
-  end
-
-  it "get_details should call artist/details api method and return digested artist" do
-    an_artist_id = 123
-    an_artist = Sevendigital::Artist.new(@client)
-    an_api_response = fake_api_response("release/details")
-
-    mock_client_digestor(@client, :artist_digestor) \
-          .should_receive(:from_xml).with(an_api_response.content.artist).and_return(an_artist)
-
-    @client.should_receive(:make_api_request) \
-            .with("artist/details", {:artistId => an_artist_id}, {}) \
-            .and_return(an_api_response)
-
-    artist = @artist_manager.get_details(an_artist_id)
-    artist.should == an_artist
-	end
-
-  it "get_releases should call artist/releases api method and return list of digested releases" do
-    an_artist_id = 123
-    a_list_of_releases = [Sevendigital::Release.new(@client), Sevendigital::Release.new(@client)]
-    an_api_response = fake_api_response("artist/releases")
-
-    mock_client_digestor(@client, :release_digestor) \
-         .should_receive(:list_from_xml).with(an_api_response.content.releases).and_return(a_list_of_releases)
-
-    @client.should_receive(:make_api_request) \
-            .with("artist/releases", {:artistId => an_artist_id}, {}) \
-            .and_return(an_api_response)
-    
-    releases = @artist_manager.get_releases(an_artist_id)
-    releases.should == a_list_of_releases
-  end
-
-  it "get_top_tracks should call artist/topTracks method and digest the returned list of tracks" do
-    an_artist_id = 123
-    a_top_tracks_list = [Sevendigital::Track.new(@client)]
-    an_api_response = fake_api_response("artist/toptracks")
-    
-    mock_client_digestor(@client, :track_digestor) \
-        .should_receive(:list_from_xml).with(an_api_response.content.tracks).and_return(a_top_tracks_list)
-
-    @client.should_receive(:make_api_request) \
-            .with("artist/topTracks", {:artistId => an_artist_id}, {}) \
-            .and_return(an_api_response)
-
-    tracks = @artist_manager.get_top_tracks(an_artist_id)
-    tracks.should == a_top_tracks_list
-
-  end
-
-  it "get_similar should call artist/similar method and digest the returned list of artists" do
-    an_artist_id = 123
-    a_similar_artists_list = [Sevendigital::Artist.new(@client), Sevendigital::Artist.new(@client)]
-    an_api_response = fake_api_response("artist/similar")
-
-    mock_client_digestor(@client, :artist_digestor) \
-        .should_receive(:list_from_xml).with(an_api_response.content.artists).and_return(a_similar_artists_list)
-
-    @client.should_receive(:make_api_request) \
-            .with("artist/similar", {:artistId => an_artist_id}, {}) \
-            .and_return(an_api_response)
-
-    artists = @artist_manager.get_similar(an_artist_id)
-    artists.should == a_similar_artists_list
-
-  end
-
-  it "get_top_by_tag should call artist/byTag/top api method and digest the nested artist list from response" do
-
-    tags = "alternative-indie"
-    an_api_response = fake_api_response("artist/byTag/top")
-    a_release_list = []
-
-    mock_client_digestor(@client, :artist_digestor) \
-      .should_receive(:nested_list_from_xml) \
-      .with(an_api_response.content.tagged_results, :tagged_item, :tagged_results) \
-      .and_return(a_release_list)
-
-    @client.should_receive(:make_api_request) \
-            .with("artist/byTag/top", {:tags => tags}, {}) \
-            .and_return(an_api_response)
-
-    releases = @artist_manager.get_top_by_tag(tags)
-    releases.should == a_release_list
-
-  end
-
-  it "browse should call artist/browse api method and digest the artist list from response" do
-
-    letter = "ra"
-    an_api_response = fake_api_response("artist/browse")
-    an_artist_list = []
-
-    mock_client_digestor(@client, :artist_digestor) \
-      .should_receive(:list_from_xml) \
-      .with(an_api_response.content.artists) \
-      .and_return(an_artist_list)
-
-    @client.should_receive(:make_api_request) \
-            .with("artist/browse", {:letter => letter}, {}) \
-            .and_return(an_api_response)
-
-    artists = @artist_manager.browse(letter)
-    artists.should == an_artist_list
-
-  end
-
-
-  it "get_chart should call artist/chart api method and digest the artist list from response" do
-
-    api_response = fake_api_response("artist/chart")
-    a_chart = [Sevendigital::ChartItem.new(@client)]
-
-    mock_client_digestor(@client, :chart_item_digestor) \
-        .should_receive(:list_from_xml).with(api_response.content.chart).and_return(a_chart)
-
-    @client.should_receive(:make_api_request) \
-      .with("artist/chart", {}, {}) \
-      .and_return(api_response)
-
-    chart = @artist_manager.get_chart
-    chart.should == a_chart
-  end
-
+require File.expand_path('../../spec_helper', __FILE__)
+
+describe "ArtistManager" do
+
+  before do
+    @client = stub(Sevendigital::Client)
+    @client.stub!(:operator).and_return(mock(Sevendigital::ApiOperator))
+    @artist_manager = Sevendigital::ArtistManager.new(@client)
+  end
+
+  it "get_details should call artist/details api method and return digested artist" do
+    an_artist_id = 123
+    an_artist = Sevendigital::Artist.new(@client)
+    an_api_response = fake_api_response("release/details")
+
+    mock_client_digestor(@client, :artist_digestor) \
+          .should_receive(:from_xml).with(an_api_response.content.artist).and_return(an_artist)
+
+    @client.should_receive(:make_api_request) \
+            .with(:GET, "artist/details", {:artistId => an_artist_id}, {}) \
+            .and_return(an_api_response)
+
+    artist = @artist_manager.get_details(an_artist_id)
+    artist.should == an_artist
+	end
+
+  it "get_releases should call artist/releases api method and return list of digested releases" do
+    an_artist_id = 123
+    a_list_of_releases = [Sevendigital::Release.new(@client), Sevendigital::Release.new(@client)]
+    an_api_response = fake_api_response("artist/releases")
+
+    mock_client_digestor(@client, :release_digestor) \
+         .should_receive(:list_from_xml).with(an_api_response.content.releases).and_return(a_list_of_releases)
+
+    @client.should_receive(:make_api_request) \
+            .with(:GET, "artist/releases", {:artistId => an_artist_id}, {}) \
+            .and_return(an_api_response)
+    
+    releases = @artist_manager.get_releases(an_artist_id)
+    releases.should == a_list_of_releases
+  end
+
+  it "get_top_tracks should call artist/topTracks method and digest the returned list of tracks" do
+    an_artist_id = 123
+    a_top_tracks_list = [Sevendigital::Track.new(@client)]
+    an_api_response = fake_api_response("artist/toptracks")
+    
+    mock_client_digestor(@client, :track_digestor) \
+        .should_receive(:list_from_xml).with(an_api_response.content.tracks).and_return(a_top_tracks_list)
+
+    @client.should_receive(:make_api_request) \
+            .with(:GET, "artist/topTracks", {:artistId => an_artist_id}, {}) \
+            .and_return(an_api_response)
+
+    tracks = @artist_manager.get_top_tracks(an_artist_id)
+    tracks.should == a_top_tracks_list
+
+  end
+
+  it "get_similar should call artist/similar method and digest the returned list of artists" do
+    an_artist_id = 123
+    a_similar_artists_list = [Sevendigital::Artist.new(@client), Sevendigital::Artist.new(@client)]
+    an_api_response = fake_api_response("artist/similar")
+
+    mock_client_digestor(@client, :artist_digestor) \
+        .should_receive(:list_from_xml).with(an_api_response.content.artists).and_return(a_similar_artists_list)
+
+    @client.should_receive(:make_api_request) \
+            .with(:GET, "artist/similar", {:artistId => an_artist_id}, {}) \
+            .and_return(an_api_response)
+
+    artists = @artist_manager.get_similar(an_artist_id)
+    artists.should == a_similar_artists_list
+
+  end
+
+  it "get_top_by_tag should call artist/byTag/top api method and digest the nested artist list from response" do
+
+    tags = "alternative-indie"
+    an_api_response = fake_api_response("artist/byTag/top")
+    a_release_list = []
+
+    mock_client_digestor(@client, :artist_digestor) \
+      .should_receive(:nested_list_from_xml) \
+      .with(an_api_response.content.tagged_results, :tagged_item, :tagged_results) \
+      .and_return(a_release_list)
+
+    @client.should_receive(:make_api_request) \
+            .with(:GET, "artist/byTag/top", {:tags => tags}, {}) \
+            .and_return(an_api_response)
+
+    releases = @artist_manager.get_top_by_tag(tags)
+    releases.should == a_release_list
+
+  end
+
+  it "browse should call artist/browse api method and digest the artist list from response" do
+
+    letter = "ra"
+    an_api_response = fake_api_response("artist/browse")
+    an_artist_list = []
+
+    mock_client_digestor(@client, :artist_digestor) \
+      .should_receive(:list_from_xml) \
+      .with(an_api_response.content.artists) \
+      .and_return(an_artist_list)
+
+    @client.should_receive(:make_api_request) \
+            .with(:GET, "artist/browse", {:letter => letter}, {}) \
+            .and_return(an_api_response)
+
+    artists = @artist_manager.browse(letter)
+    artists.should == an_artist_list
+
+  end
+
+
+  it "get_chart should call artist/chart api method and digest the artist list from response" do
+
+    api_response = fake_api_response("artist/chart")
+    a_chart = [Sevendigital::ChartItem.new(@client)]
+
+    mock_client_digestor(@client, :chart_item_digestor) \
+        .should_receive(:list_from_xml).with(api_response.content.chart).and_return(a_chart)
+
+    @client.should_receive(:make_api_request) \
+      .with(:GET, "artist/chart", {}, {}) \
+      .and_return(api_response)
+
+    chart = @artist_manager.get_chart
+    chart.should == a_chart
+  end
+
+
+  it "get_tags should call artist/tags api method and return list of digested tags" do
+    an_artist_id = 123
+    a_list_of_tags = [Sevendigital::Tag.new, Sevendigital::Tag.new]
+    an_api_response = fake_api_response("artist/tags")
+    options = {:page => 1}
+
+    mock_client_digestor(@client, :tag_digestor) \
+         .should_receive(:list_from_xml).with(an_api_response.content.tags).and_return(a_list_of_tags)
+
+    @client.should_receive(:make_api_request) \
+            .with(:GET, "artist/tags", {:artistId => an_artist_id}, options) \
+            .and_return(an_api_response)
+
+    releases = @artist_manager.get_tags(an_artist_id, options)
+    releases.should == a_list_of_tags
+  end
+
 end

data/spec/management/basket_manager_spec.rb

@@ -1,78 +1,78 @@
-require File.expand_path('../../spec_helper', __FILE__)
-
-describe "BasketManager" do
-
-  before do
-    @client = stub(Sevendigital::Client)
-    @client.stub!(:operator).and_return(mock(Sevendigital::ApiOperator))
-    @basket_manager = Sevendigital::BasketManager.new(@client)
-  end
-
-  it "get should call basket api method and return digested basket" do
-    a_basket_id = "00000000-0000-0000-0000-000000000001"
-    a_basket = Sevendigital::Basket.new(@client)
-    an_api_response = fake_api_response("basket/index")
-
-    mock_client_digestor(@client, :basket_digestor) \
-          .should_receive(:from_xml).with(an_api_response.content.basket).and_return(a_basket)
-
-    @client.should_receive(:make_api_request) \
-              .with("basket", {:basketId => a_basket_id}, {}) \
-              .and_return(an_api_response)
-
-    basket = @basket_manager.get(a_basket_id)
-    basket.should == a_basket
-  end
-
-  it "create should call basket/create api method and return digested created basket" do
-    a_basket = Sevendigital::Basket.new(@client)
-    an_api_response = fake_api_response("basket/create")
-
-    mock_client_digestor(@client, :basket_digestor) \
-          .should_receive(:from_xml).with(an_api_response.content.basket).and_return(a_basket)
-
-    @client.should_receive(:make_api_request) \
-              .with("basket/create", {}, {}) \
-              .and_return(an_api_response)
-
-    basket = @basket_manager.create
-    basket.should == a_basket
-  end
-
-  it "add_item should call basket/addItem api method and return digested basket" do
-    a_basket_id = "00000000-0000-0000-0000-000000000001"
-    a_release_id = 123
-    a_track_id = 456
-    a_basket = Sevendigital::Basket.new(@client)
-    an_api_response = fake_api_response("basket/additem")
-
-    mock_client_digestor(@client, :basket_digestor) \
-          .should_receive(:from_xml).with(an_api_response.content.basket).and_return(a_basket)
-
-    @client.should_receive(:make_api_request) \
-      .with("basket/addItem", {:basketId => a_basket_id, :releaseId => a_release_id, :trackId => a_track_id}, {}) \
-      .and_return(an_api_response)
-
-    basket = @basket_manager.add_item(a_basket_id, a_release_id, a_track_id)
-    basket.should == a_basket
-  end
-
-  it "remove_item should call basket/removeItem api method and return digested basket" do
-    a_basket_id = "00000000-0000-0000-0000-000000000001"
-    an_item_id = 123456
-    a_basket = Sevendigital::Basket.new(@client)
-    an_api_response = fake_api_response("basket/removeitem")
-
-    mock_client_digestor(@client, :basket_digestor) \
-          .should_receive(:from_xml).with(an_api_response.content.basket).and_return(a_basket)
-
-    @client.should_receive(:make_api_request) \
-              .with("basket/removeItem", {:basketId => a_basket_id, :itemId => an_item_id}, {}) \
-              .and_return(an_api_response)
-
-    basket = @basket_manager.remove_item(a_basket_id, an_item_id)
-    basket.should == a_basket
-  end
-
-
+require File.expand_path('../../spec_helper', __FILE__)
+
+describe "BasketManager" do
+
+  before do
+    @client = stub(Sevendigital::Client)
+    @client.stub!(:operator).and_return(mock(Sevendigital::ApiOperator))
+    @basket_manager = Sevendigital::BasketManager.new(@client)
+  end
+
+  it "get should call basket api method and return digested basket" do
+    a_basket_id = "00000000-0000-0000-0000-000000000001"
+    a_basket = Sevendigital::Basket.new(@client)
+    an_api_response = fake_api_response("basket/index")
+
+    mock_client_digestor(@client, :basket_digestor) \
+          .should_receive(:from_xml).with(an_api_response.content.basket).and_return(a_basket)
+
+    @client.should_receive(:make_api_request) \
+              .with(:GET, "basket", {:basketId => a_basket_id}, {}) \
+              .and_return(an_api_response)
+
+    basket = @basket_manager.get(a_basket_id)
+    basket.should == a_basket
+  end
+
+  it "create should call basket/create api method and return digested created basket" do
+    a_basket = Sevendigital::Basket.new(@client)
+    an_api_response = fake_api_response("basket/create")
+
+    mock_client_digestor(@client, :basket_digestor) \
+          .should_receive(:from_xml).with(an_api_response.content.basket).and_return(a_basket)
+
+    @client.should_receive(:make_api_request) \
+              .with(:GET, "basket/create", {}, {}) \
+              .and_return(an_api_response)
+
+    basket = @basket_manager.create
+    basket.should == a_basket
+  end
+
+  it "add_item should call basket/addItem api method and return digested basket" do
+    a_basket_id = "00000000-0000-0000-0000-000000000001"
+    a_release_id = 123
+    a_track_id = 456
+    a_basket = Sevendigital::Basket.new(@client)
+    an_api_response = fake_api_response("basket/additem")
+
+    mock_client_digestor(@client, :basket_digestor) \
+          .should_receive(:from_xml).with(an_api_response.content.basket).and_return(a_basket)
+
+    @client.should_receive(:make_api_request) \
+      .with(:GET, "basket/addItem", {:basketId => a_basket_id, :releaseId => a_release_id, :trackId => a_track_id}, {}) \
+      .and_return(an_api_response)
+
+    basket = @basket_manager.add_item(a_basket_id, a_release_id, a_track_id)
+    basket.should == a_basket
+  end
+
+  it "remove_item should call basket/removeItem api method and return digested basket" do
+    a_basket_id = "00000000-0000-0000-0000-000000000001"
+    an_item_id = 123456
+    a_basket = Sevendigital::Basket.new(@client)
+    an_api_response = fake_api_response("basket/removeitem")
+
+    mock_client_digestor(@client, :basket_digestor) \
+          .should_receive(:from_xml).with(an_api_response.content.basket).and_return(a_basket)
+
+    @client.should_receive(:make_api_request) \
+              .with(:GET, "basket/removeItem", {:basketId => a_basket_id, :itemId => an_item_id}, {}) \
+              .and_return(an_api_response)
+
+    basket = @basket_manager.remove_item(a_basket_id, an_item_id)
+    basket.should == a_basket
+  end
+
+
 end