7digital 0.0.2 → 0.0.5

data/spec/digestion_tract/oauth_request_token_digestor_spec.rb

@@ -0,0 +1,35 @@
+require File.expand_path('../../spec_helper', __FILE__)
+
+describe "OAuthRequestTokenDigestor" do
+
+  before do
+    @stub_oauth_consumer = stub(OAuth::Consumer)
+    api_client = stub(Sevendigital::Client)
+    api_client.stub!(:oauth_consumer).and_return(@stub_oauth_consumer)
+    @token_digestor = Sevendigital::OAuthRequestTokenDigestor.new(api_client)
+  end
+
+
+    it "should not digest from invalid xml but throw up (exception)" do
+
+    xml_response = <<XML
+    <release id="123">
+      <name>expected artist name</name>
+    </release>
+XML
+
+    running {@token_digestor.from_xml(xml_response)}.should raise_error(Sevendigital::DigestiveProblem)
+    end
+  
+  it "should digest request token xml and populate all properties" do
+
+    xml_response = load_sample_object_xml("oauth_request_token")
+
+    token = @token_digestor.from_xml(xml_response, :oauth_request_token)
+    token.kind_of?(OAuth::RequestToken).should == true
+    token.token.should == "yu87230J29DT7tyuGslO98wrR43e39Of"
+    token.secret.should == "d2I8uj7yaoa39KKdu3upasybu98f89fln"
+    token.consumer.should == @stub_oauth_consumer
+  end
+
+end

data/spec/digestion_tract/pager_digestor_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(__FILE__), %w[../spec_helper])
+require File.expand_path('../../spec_helper', __FILE__)
 
 describe "PagerDigestor" do
 
@@ -15,7 +15,7 @@ describe "PagerDigestor" do
     </release>
 XML
 
-    running {@pager_digestor.from_xml(xml_response)}.should raise_error Sevendigital::DigestiveProblem
+    running {@pager_digestor.from_xml(xml_response)}.should raise_error(Sevendigital::DigestiveProblem)
     end
 
  it "should not digest if paging info is missing from xml but spit out nil" do

data/spec/digestion_tract/price_digestor_spec.rb

@@ -1,5 +1,5 @@
 # coding: utf-8
-require File.join(File.dirname(__FILE__), %w[../spec_helper])
+require File.expand_path('../../spec_helper', __FILE__)
 
 describe "PriceDigestor" do
 
@@ -15,7 +15,7 @@ describe "PriceDigestor" do
      </xxx>
 XML
 
-    running {@price_digestor.from_xml(xml_response)}.should raise_error Sevendigital::DigestiveProblem
+    running {@price_digestor.from_xml(xml_response)}.should raise_error(Sevendigital::DigestiveProblem)
   end
 
   it "should parse from xml and populate minimum available properties" do
@@ -34,9 +34,25 @@ XML
     price.currency_symbol.should == "£"
     price.value.should == 1.79
     price.formatted_price.should == "£1.79"
-    price.rrp.should == nil
-    price.formatted_rrp.should == nil
-    price.on_sale.should == true
+
+  end
+
+    it "should parse from xml and populate N/A price" do
+
+    xml_response = <<XML
+<price>
+  <currency code="GBP">£</currency>
+  <value />
+  <formattedPrice>N/A</formattedPrice>
+  <onSale>false</onSale>
+</price>
+XML
+
+    price = @price_digestor.from_xml(xml_response)
+    price.currency_code.should == :GBP
+    price.currency_symbol.should == "£"
+    price.value.should == nil
+    price.formatted_price.should == "N/A"
 
   end
 

data/spec/digestion_tract/release_digestor_spec.rb

@@ -1,6 +1,6 @@
 # coding: utf-8
 require 'date'
-require File.join(File.dirname(__FILE__), %w[../spec_helper])
+require File.expand_path('../../spec_helper', __FILE__)
 
 describe "ReleaseDigestor" do
 
@@ -16,7 +16,7 @@ describe "ReleaseDigestor" do
     </artist>
 XML
 
-    running {@release_digestor.from_xml(xml_response)}.should raise_error Sevendigital::DigestiveProblem
+    running {@release_digestor.from_xml(xml_response)}.should raise_error(Sevendigital::DigestiveProblem)
   end
 
   it "should digest release xml and populate minimum available properties" do

data/spec/digestion_tract/tag_digestor_spec.rb

@@ -0,0 +1,46 @@
+# coding: utf-8
+require File.expand_path('../../spec_helper', __FILE__)
+
+describe "TagDigestor" do
+
+  before do
+     @tag_digestor = Sevendigital::TagDigestor.new(nil)
+  end
+
+   it "should not digest from invalid xml but throw up (exception)" do
+
+     xml_response = <<XML
+     <xxx>
+       <text>pop</text>
+     </xxx>
+XML
+
+    running {@tag_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
+<tag id="rock">
+  <text>ROCK</text>
+</tag>
+XML
+
+    tag = @tag_digestor.from_xml(xml_response)
+    tag.id.should == 'rock'
+    tag.text.should == "ROCK"
+    
+  end
+
+  it "should parse from xml and populate all properties" do
+
+    xml_response = load_sample_object_xml("tag")
+
+    tag = @tag_digestor.from_xml(xml_response)
+    tag.id.should == "pop"
+    tag.text.should == "pop"
+    tag.url.should == "http://www.7digital.com/tags/pop?partner=123"
+    tag.count.should == 90847
+  end
+
+end

data/spec/digestion_tract/track_digestor_spec.rb

@@ -1,6 +1,6 @@
 # coding: utf-8
 require 'date'
-require File.join(File.dirname(__FILE__), %w[../spec_helper])
+require File.expand_path('../../spec_helper', __FILE__)
 
 describe "TrackDigestor" do
 

data/spec/management/artist_manager_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(__FILE__), %w[../spec_helper])
+require File.expand_path('../../spec_helper', __FILE__)
 
 describe "ArtistManager" do
 
@@ -16,11 +16,9 @@ describe "ArtistManager" do
     mock_client_digestor(@client, :artist_digestor) \
           .should_receive(:from_xml).with(an_api_response.content.artist).and_return(an_artist)
 
-    @client.operator.should_receive(:call_api) { |api_request|
-      api_request.api_method.should == "artist/details"
-      api_request.parameters[:artistId].should  == an_artist_id
-      an_api_response
-    }
+    @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
@@ -34,17 +32,15 @@ describe "ArtistManager" do
     mock_client_digestor(@client, :release_digestor) \
          .should_receive(:list_from_xml).with(an_api_response.content.releases).and_return(a_list_of_releases)
 
-    @client.operator.should_receive(:call_api) { |api_request|
-      api_request.api_method.should == "artist/releases"
-      api_request.parameters[:artistId].should  == an_artist_id
-      an_api_response
-    }
-
+    @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
+  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")
@@ -52,11 +48,9 @@ describe "ArtistManager" do
     mock_client_digestor(@client, :track_digestor) \
         .should_receive(:list_from_xml).with(an_api_response.content.tracks).and_return(a_top_tracks_list)
 
-    @client.operator.should_receive(:call_api) { |api_request|
-      api_request.api_method.should == "artist/toptracks"
-      api_request.parameters[:artistId].should  == an_artist_id
-      an_api_response
-    }
+    @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
@@ -71,11 +65,9 @@ describe "ArtistManager" do
     mock_client_digestor(@client, :artist_digestor) \
         .should_receive(:list_from_xml).with(an_api_response.content.artists).and_return(a_similar_artists_list)
 
-    @client.operator.should_receive(:call_api) { |api_request|
-      api_request.api_method.should == "artist/similar"
-      api_request.parameters[:artistId].should  == an_artist_id
-      an_api_response
-    }
+    @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
@@ -85,45 +77,58 @@ describe "ArtistManager" do
   it "get_top_by_tag should call artist/byTag/top api method and digest the nested artist list from response" do
 
     tags = "alternative-indie"
-    api_response = fake_api_response("artist/byTag/top")
+    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(api_response.content.tagged_results, :tagged_item, :tagged_results) \
+      .with(an_api_response.content.tagged_results, :tagged_item, :tagged_results) \
       .and_return(a_release_list)
 
-    @client.operator.should_receive(:call_api) { |api_request|
-       api_request.api_method.should == "artist/byTag/top"
-       api_request.parameters[:tags].should == tags
-       api_response
-    }
+    @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 "search should call artist/search api method and digest the nested artist list from response" do
+  it "browse should call artist/browse api method and digest the artist list from response" do
 
-    query = "radiohead"
-    api_response = fake_api_response("artist/search")
+    letter = "ra"
+    an_api_response = fake_api_response("artist/browse")
     an_artist_list = []
 
     mock_client_digestor(@client, :artist_digestor) \
-      .should_receive(:nested_list_from_xml) \
-      .with(api_response.content.search_results, :search_result, :search_results) \
+      .should_receive(:list_from_xml) \
+      .with(an_api_response.content.artists) \
       .and_return(an_artist_list)
 
-    @client.operator.should_receive(:call_api) { |api_request|
-       api_request.api_method.should == "artist/search"
-       api_request.parameters[:q].should == query
-       api_response
-    }
+    @client.should_receive(:make_api_request) \
+            .with("artist/browse", {:letter => letter}, {}) \
+            .and_return(an_api_response)
 
-    artists = @artist_manager.search(query)
+    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
+
 end

data/spec/management/basket_manager_spec.rb

@@ -0,0 +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
+
+
+end

data/spec/management/oauth_manager_spec.rb

@@ -0,0 +1,73 @@
+require File.expand_path('../../spec_helper', __FILE__)
+
+describe "OAuthManager" do
+
+  before do
+    @client = stub(Sevendigital::Client)
+
+    @client.stub!(:operator).and_return(mock(Sevendigital::ApiOperator))
+    @oauth_manager = Sevendigital::OAuthManager.new(@client)
+  end
+
+  it "get_oauth_request_token should call oauth/requestToken api method and digest the request_token from response" do
+
+    fake_token = OAuth::RequestToken.new("aaa", "bbb", "ccc")
+    an_api_response = fake_api_response("oauth/requesttoken")
+
+    digestor = mock(Sevendigital::OAuthRequestTokenDigestor)
+    @client.stub!(:oauth_request_token_digestor).and_return(digestor)
+    
+    digestor.should_receive(:from_xml) \
+      .with(an_api_response.content.oauth_request_token, :oauth_request_token) \
+      .and_return(fake_token)
+
+    @client.should_receive(:make_signed_api_request) \
+       .with("oauth/requestToken", {}) \
+       .and_return(an_api_response)
+
+    token = @oauth_manager.get_request_token
+    token.should == fake_token
+
+  end
+
+  it "get_oauth_access_token should call oauth/accessToken api method and digest the access_token from response" do
+
+    a_request_token = OAuth::RequestToken.new("aaa", "bbb", "ccc")
+    fake_token = OAuth::AccessToken.new("aaa", "bbb", "ccc")
+    an_api_response = fake_api_response("oauth/accessToken")
+
+    digestor = mock(Sevendigital::OAuthAccessTokenDigestor)
+    @client.stub!(:oauth_access_token_digestor).and_return(digestor)
+
+    digestor.should_receive(:from_xml) \
+      .with(an_api_response.content.oauth_access_token, :oauth_access_token) \
+      .and_return(fake_token)
+
+    @client.should_receive(:make_signed_api_request) \
+       .with("oauth/accessToken", {}, {}, a_request_token) \
+       .and_return(an_api_response)
+
+    token = @oauth_manager.get_access_token(a_request_token)
+    token.should == fake_token
+
+  end
+
+  it "authorise_request_token should call oauth/requestToken/authorise api method" do
+
+    a_request_token = OAuth::RequestToken.new("aaa", "bbb", "ccc")
+    an_email_address = "email"
+    a_password = "password"
+    an_api_response = fake_api_response("oauth/requesttoken")
+
+    @client.should_receive(:make_signed_api_request) \
+       .with("oauth/requestToken/authorise", \
+              {:username => an_email_address, :password => a_password, :token => a_request_token.token}) \
+       .and_return(an_api_response)
+    
+    authorised = @oauth_manager.authorise_request_token(an_email_address, a_password, a_request_token)
+    authorised.should == true
+
+  end
+
+
+end