7digital 0.0.5 → 0.0.6

data/spec/management/oauth_manager_spec.rb

@@ -1,73 +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
-
-
+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(:GET, "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(:GET, "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(:GET, "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

data/spec/management/release_manager_spec.rb

@@ -1,181 +1,200 @@
-# encoding: UTF-8
-require 'date'
-require File.expand_path('../../spec_helper', __FILE__)
-
-describe "ReleaseManager" do
-
-  before do
-
-    @client = stub(Sevendigital::Client)
-    @client.stub!(:operator).and_return(mock(Sevendigital::ApiOperator))
-    @release_manager = Sevendigital::ReleaseManager.new(@client)
-
-  end
-
-  it "should get release by release id" do
-    expected_release_id = 123
-    api_response = fake_api_response("release/details")
-    a_release = Sevendigital::Release.new(@client)
-
-    mock_client_digestor(@client, :release_digestor) \
-      .should_receive(:from_xml).with(api_response.content.release).and_return(a_release)
-
-    @client.should_receive(:make_api_request) \
-      .with("release/details", {:releaseId => expected_release_id}, {}) \
-      .and_return(api_response)
-
-    @release_manager.get_details(expected_release_id).should == a_release
-
-  end
-
-  it "get_tracks should call release/tracks api method and digest the track list from response" do
-    expected_options = {:page_size => 20}
-    a_release_id = 123
-    api_response = fake_api_response("release/tracks")
-    a_track_list = []
-
-    mock_client_digestor(@client, :track_digestor) \
-      .should_receive(:list_from_xml).with(api_response.content.tracks).and_return(a_track_list)
-
-    @client.should_receive(:make_api_request) \
-      .with("release/tracks", {:releaseId=>a_release_id}, expected_options) \
-      .and_return(api_response)
-
-    tracks = @release_manager.get_tracks(a_release_id, expected_options)
-    tracks.should == a_track_list
-
-  end
-
-  it "get_tracks should call use page size 100 by default" do
-
-    a_release_id = 123
-    api_response = fake_api_response("release/tracks")
-    a_track_list = []
-
-    mock_client_digestor(@client, :track_digestor) \
-      .should_receive(:list_from_xml).with(api_response.content.tracks).and_return(a_track_list)
-
-    @client.should_receive(:make_api_request) \
-      .with("release/tracks", {:releaseId => a_release_id}, {:page_size => 100}) \
-      .and_return(api_response)
-
-    tracks = @release_manager.get_tracks(a_release_id)
-    tracks.should == a_track_list
-
-  end
-
-
-  it "get_chart should call release/chart api method and digest the release list from response" do
-
-    api_response = fake_api_response("release/chart")
-    a_chart = []
-
-    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("release/chart", {}, {}) \
-      .and_return(api_response)
-
-    chart = @release_manager.get_chart
-    chart.should == a_chart
-  end
-
-  it "get_by_date should call release/byDate with supplied parameters and digest the release list from response" do
-
-    from_date = DateTime.new(2010, 01, 01)
-    to_date = DateTime.new(2010, 05, 01)
-    api_response = fake_api_response("release/bydate")
-    a_release_list = []
-
-    mock_client_digestor(@client, :release_digestor) \
-      .should_receive(:list_from_xml).with(api_response.content.releases).and_return(a_release_list)
-
-    @client.should_receive(:make_api_request) \
-         .with("release/byDate", {:fromDate => from_date.strftime("%Y%m%d"), :toDate => to_date.strftime("%Y%m%d")}, {}) \
-         .and_return(api_response)
-
-    tracks = @release_manager.get_by_date(from_date, to_date)
-    tracks.should == a_release_list
-
-  end
-
-  it "get_by_date should call release/tracks with no parameters and digest the release list from response" do
-
-    api_response = fake_api_response("release/bydate")
-    a_release_list = []
-
-    mock_client_digestor(@client, :release_digestor) \
-      .should_receive(:list_from_xml).with(api_response.content.releases).and_return(a_release_list)
-
-    @client.should_receive(:make_api_request) \
-         .with("release/byDate", {}, {}) \
-         .and_return(api_response)
-
-    releases = @release_manager.get_by_date
-    releases.should == a_release_list
-
-  end
-
-  it "get_recommendations should call release/recommend api method and digest the nested release list from response" do
-
-    a_release_id = 123
-    api_response = fake_api_response("release/recommend")
-    a_release_list = []
-
-    mock_client_digestor(@client, :release_digestor) \
-      .should_receive(:nested_list_from_xml) \
-      .with(api_response.content.recommendations, :recommended_item, :recommendations) \
-      .and_return(a_release_list)
-
-    @client.should_receive(:make_api_request) \
-         .with("release/recommend", {:releaseId => a_release_id}, {}) \
-         .and_return(api_response)
-   
-    releases = @release_manager.get_recommendations(a_release_id)
-    releases.should == a_release_list
-
-  end
-
-    it "get_top_by_tag should call release/byTag/top api method and digest the nested release list from response" do
-
-    tags = "alternative-indie"
-    api_response = fake_api_response("release/byTag/top")
-    a_release_list = []
-
-    mock_client_digestor(@client, :release_digestor) \
-      .should_receive(:nested_list_from_xml) \
-      .with(api_response.content.tagged_results, :tagged_item, :tagged_results) \
-      .and_return(a_release_list)
-
-    @client.should_receive(:make_api_request) \
-         .with("release/byTag/top", {:tags => tags}, {}) \
-         .and_return(api_response)
-    
-    releases = @release_manager.get_top_by_tag(tags)
-    releases.should == a_release_list
-
-    end
-
-    it "search should call release/search api method and digest the nested release list from response" do
-
-      query = "radiohead"
-      api_response = fake_api_response("release/search")
-      a_release_list = []
-
-      mock_client_digestor(@client, :release_digestor) \
-        .should_receive(:nested_list_from_xml) \
-        .with(api_response.content.search_results, :search_result, :search_results) \
-        .and_return(a_release_list)
-
-      @client.should_receive(:make_api_request) \
-           .with("release/search", {:q => query}, {}) \
-           .and_return(api_response)
-
-      releases = @release_manager.search(query)
-      releases.should == a_release_list
-
-    end
-
-
-end
+# encoding: UTF-8
+require 'date'
+require File.expand_path('../../spec_helper', __FILE__)
+
+describe "ReleaseManager" do
+
+  before do
+
+    @client = stub(Sevendigital::Client)
+    @client.stub!(:operator).and_return(mock(Sevendigital::ApiOperator))
+    @release_manager = Sevendigital::ReleaseManager.new(@client)
+
+  end
+
+  it "should get release by release id" do
+    expected_release_id = 123
+    api_response = fake_api_response("release/details")
+    a_release = Sevendigital::Release.new(@client)
+
+    mock_client_digestor(@client, :release_digestor) \
+      .should_receive(:from_xml).with(api_response.content.release).and_return(a_release)
+
+    @client.should_receive(:make_api_request) \
+      .with(:GET, "release/details", {:releaseId => expected_release_id}, {}) \
+      .and_return(api_response)
+
+    @release_manager.get_details(expected_release_id).should == a_release
+
+  end
+
+  it "get_tracks should call release/tracks api method and digest the track list from response" do
+    expected_options = {:page_size => 20}
+    a_release_id = 123
+    api_response = fake_api_response("release/tracks")
+    a_track_list = []
+
+    mock_client_digestor(@client, :track_digestor) \
+      .should_receive(:list_from_xml).with(api_response.content.tracks).and_return(a_track_list)
+
+    @client.should_receive(:make_api_request) \
+      .with(:GET, "release/tracks", {:releaseId=>a_release_id}, expected_options) \
+      .and_return(api_response)
+
+    tracks = @release_manager.get_tracks(a_release_id, expected_options)
+    tracks.should == a_track_list
+
+  end
+
+  it "get_tracks should call use page size 100 by default" do
+
+    a_release_id = 123
+    api_response = fake_api_response("release/tracks")
+    a_track_list = []
+
+    mock_client_digestor(@client, :track_digestor) \
+      .should_receive(:list_from_xml).with(api_response.content.tracks).and_return(a_track_list)
+
+    @client.should_receive(:make_api_request) \
+      .with(:GET, "release/tracks", {:releaseId => a_release_id}, {:page_size => 100}) \
+      .and_return(api_response)
+
+    tracks = @release_manager.get_tracks(a_release_id)
+    tracks.should == a_track_list
+
+  end
+
+
+  it "get_chart should call release/chart api method and digest the release list from response" do
+
+    api_response = fake_api_response("release/chart")
+    a_chart = []
+
+    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, "release/chart", {}, {}) \
+      .and_return(api_response)
+
+    chart = @release_manager.get_chart
+    chart.should == a_chart
+  end
+
+  it "get_by_date should call release/byDate with supplied parameters and digest the release list from response" do
+
+    from_date = DateTime.new(2010, 01, 01)
+    to_date = DateTime.new(2010, 05, 01)
+    api_response = fake_api_response("release/bydate")
+    a_release_list = []
+
+    mock_client_digestor(@client, :release_digestor) \
+      .should_receive(:list_from_xml).with(api_response.content.releases).and_return(a_release_list)
+
+    @client.should_receive(:make_api_request) \
+         .with(:GET, "release/byDate", {:fromDate => from_date.strftime("%Y%m%d"), :toDate => to_date.strftime("%Y%m%d")}, {}) \
+         .and_return(api_response)
+
+    tracks = @release_manager.get_by_date(from_date, to_date)
+    tracks.should == a_release_list
+
+  end
+
+  it "get_by_date should call release/tracks with no parameters and digest the release list from response" do
+
+    api_response = fake_api_response("release/bydate")
+    a_release_list = []
+
+    mock_client_digestor(@client, :release_digestor) \
+      .should_receive(:list_from_xml).with(api_response.content.releases).and_return(a_release_list)
+
+    @client.should_receive(:make_api_request) \
+         .with(:GET, "release/byDate", {}, {}) \
+         .and_return(api_response)
+
+    releases = @release_manager.get_by_date
+    releases.should == a_release_list
+
+  end
+
+  it "get_recommendations should call release/recommend api method and digest the nested release list from response" do
+
+    a_release_id = 123
+    api_response = fake_api_response("release/recommend")
+    a_release_list = []
+
+    mock_client_digestor(@client, :release_digestor) \
+      .should_receive(:nested_list_from_xml) \
+      .with(api_response.content.recommendations, :recommended_item, :recommendations) \
+      .and_return(a_release_list)
+
+    @client.should_receive(:make_api_request) \
+         .with(:GET, "release/recommend", {:releaseId => a_release_id}, {}) \
+         .and_return(api_response)
+   
+    releases = @release_manager.get_recommendations(a_release_id)
+    releases.should == a_release_list
+
+  end
+
+    it "get_top_by_tag should call release/byTag/top api method and digest the nested release list from response" do
+
+    tags = "alternative-indie"
+    api_response = fake_api_response("release/byTag/top")
+    a_release_list = []
+
+    mock_client_digestor(@client, :release_digestor) \
+      .should_receive(:nested_list_from_xml) \
+      .with(api_response.content.tagged_results, :tagged_item, :tagged_results) \
+      .and_return(a_release_list)
+
+    @client.should_receive(:make_api_request) \
+         .with(:GET, "release/byTag/top", {:tags => tags}, {}) \
+         .and_return(api_response)
+    
+    releases = @release_manager.get_top_by_tag(tags)
+    releases.should == a_release_list
+
+    end
+
+    it "search should call release/search api method and digest the nested release list from response" do
+
+      query = "radiohead"
+      api_response = fake_api_response("release/search")
+      a_release_list = []
+
+      mock_client_digestor(@client, :release_digestor) \
+        .should_receive(:nested_list_from_xml) \
+        .with(api_response.content.search_results, :search_result, :search_results) \
+        .and_return(a_release_list)
+
+      @client.should_receive(:make_api_request) \
+           .with(:GET, "release/search", {:q => query}, {}) \
+           .and_return(api_response)
+
+      releases = @release_manager.search(query)
+      releases.should == a_release_list
+
+    end
+
+  it "get_tags should call release/tags api method and digest the tag list from response" do
+    expected_options = {:page_size => 20}
+    a_release_id = 123
+    api_response = fake_api_response("release/tags")
+    a_tag_list = []
+
+    mock_client_digestor(@client, :tag_digestor) \
+      .should_receive(:list_from_xml).with(api_response.content.tags).and_return(a_tag_list)
+
+    @client.should_receive(:make_api_request) \
+      .with(:GET, "release/tags", {:releaseId=>a_release_id}, expected_options) \
+      .and_return(api_response)
+
+    tracks = @release_manager.get_tags(a_release_id, expected_options)
+    tracks.should == a_tag_list
+
+  end
+
+
+
+end