discogs-wrapper 1.0.0

data/spec/wrapper_methods/get_label_spec.rb

@@ -0,0 +1,89 @@
+require File.dirname(__FILE__) + "/../spec_helper"
+
+describe Discogs::Wrapper do
+
+  before do
+    @wrapper = Discogs::Wrapper.new("some_user_agent")
+    @artist_name = "Root"
+  end
+
+  describe "when asking for label information" do
+
+    before do
+      @http_request = mock(Net::HTTP)
+      @http_response = mock(Net::HTTPResponse, :code => "200", :body => valid_label_xml)
+      @http_response_as_file = mock(StringIO, :read => valid_label_xml)
+      Zlib::GzipReader.should_receive(:new).and_return(@http_response_as_file)
+      @http_request.should_receive(:start).and_return(@http_response)
+      Net::HTTP.should_receive(:new).and_return(@http_request)
+
+      @label = @wrapper.get_label(@label_name)
+    end
+
+    describe "when calling simple label attributes" do
+
+      it "should have a name attribute" do
+        @label.name.should == "Sombre Records"
+      end
+  
+      it "should have a contact info attribute" do
+        @label.contactinfo.should == "Unknown, Germany"
+      end
+  
+      it "should have a profile attribute" do
+        @label.profile.should == "Released some OK LPs"
+      end
+  
+      it "should have a parent label attribute" do
+        @label.parentlabel.should == "SuperSombre"
+      end
+
+      it "should have one or more URLs" do
+        @label.urls.should be_instance_of(Array)
+        @label.urls[0].should == "http://www.sombrerecords.com"
+        @label.urls[1].should == "http://www.discogs.com/label/Sombre+Records"
+      end
+  
+    end
+
+    describe "when calling complex artist attributes" do
+ 
+      it "should have a traversible list of images" do
+        @label.images.should be_instance_of(Array)
+        @label.images[0].should be_instance_of(Discogs::Image)
+      end
+
+      it "should have a traversible list of sub-labels" do
+        @label.sublabels.should be_instance_of(Array)
+        @label.sublabels[0].should == "SubSombre"
+        @label.sublabels[1].should == "Sony BMG"
+      end
+ 
+      it "should have specifications for each image" do
+        specs = [ [ '400', '300', 'primary' ], [ '450', '350', 'secondary' ] ]
+
+        @label.images.each_with_index do |image, index|
+          image.width.should == specs[index][0]
+          image.height.should == specs[index][1]
+          image.type.should == specs[index][2]
+        end
+      end
+
+      it "should have a traversible list of releases" do
+        @label.releases.should be_instance_of(Array)
+        @label.releases[0].should be_instance_of(Discogs::Label::Release)
+      end
+
+      it "should have a catno for the first release" do
+        @label.releases[0].catno.should == "SMB01"
+      end
+
+      it "should have an artist name for the second release" do
+        @label.releases[1].artist.should == "Moonblood"
+      end
+
+    end
+
+  end
+
+end

data/spec/wrapper_methods/get_release_spec.rb

@@ -0,0 +1,123 @@
+require File.dirname(__FILE__) + "/../spec_helper"
+
+describe Discogs::Wrapper do
+
+  before do
+    @wrapper = Discogs::Wrapper.new("some_user_agent")
+    @release_id = "666666"
+  end
+
+  describe "when asking for release information" do
+
+    before do
+      @http_request = mock(Net::HTTP)
+      @http_response = mock(Net::HTTPResponse, :code => "200", :body => valid_release_xml)
+      @http_response_as_file = mock(StringIO, :read => valid_release_xml)
+      Zlib::GzipReader.should_receive(:new).and_return(@http_response_as_file)
+      @http_request.should_receive(:start).and_return(@http_response)
+      Net::HTTP.should_receive(:new).and_return(@http_request)
+
+      @release = @wrapper.get_release(@release_id)
+    end
+
+    describe "when calling simple release attributes" do
+
+      it "should have a title attribute" do
+        @release.title.should == "Into the Abyss"
+      end
+  
+      it "should have an ID attribute" do
+        @release.id.should == "666666"
+      end
+
+      it "should have one or more extra artists" do
+        @release.extraartists.should be_instance_of(Array)
+        @release.extraartists[0].should be_instance_of(Discogs::Release::Artist)
+      end
+
+      it "should have one or more tracks" do
+        @release.tracklist.should be_instance_of(Array)
+        @release.tracklist[0].should be_instance_of(Discogs::Release::Track)
+      end
+ 
+      it "should have one or more genres" do
+        @release.genres.should be_instance_of(Array)
+        @release.genres[0].should == "Heavy Metal"
+      end
+
+      it "should have one or more formats" do
+        @release.formats.should be_instance_of(Array)
+        @release.formats[0].should be_instance_of(Discogs::Release::Format)
+      end
+
+      it "should have one or more images" do
+        @release.images.should be_instance_of(Array)
+        @release.images[0].should be_instance_of(Discogs::Image)
+      end
+
+    end
+
+    describe "when calling complex release attributes" do
+
+      it "should have a duration for the first track" do
+        @release.tracklist[0].duration.should == "8:11"
+      end
+
+      it "should have specifications for each image" do
+        specs = [ [ '600', '595', 'primary' ], [ '600', '593', 'secondary' ], [ '600', '539', 'secondary' ], [ '600', '452', 'secondary' ], [ '600', '567', 'secondary' ] ]
+        @release.images.each_with_index do |image, index|
+          image.width.should == specs[index][0]
+          image.height.should == specs[index][1]
+          image.type.should == specs[index][2]
+        end
+      end
+
+      it "should have a traversible list of styles" do
+        @release.styles.should be_instance_of(Array)
+        @release.styles[0].should == "Black Metal"
+        @release.styles[1].should == "Thrash"
+      end
+
+      it "should have a traversible list of labels" do
+        @release.styles.should be_instance_of(Array)
+        @release.labels[0].should be_instance_of(Discogs::Release::Label)
+        @release.labels[0].catno.should == "Death9"
+        @release.labels[0].name.should == "Culted"
+      end
+
+      it "should have a name and quantity for the first format" do
+        @release.formats[0].name.should == "CD"
+        @release.formats[0].qty.should == "1"
+      end
+
+      it "should have an array of descriptions for the first format" do
+        @release.formats[0].descriptions.should be_instance_of(Array)
+        @release.formats[0].descriptions[0].should == "Album"
+      end
+
+      it "should have an artist associated to the second track" do
+        @release.tracklist[1].artists.should be_instance_of(Array)
+        @release.tracklist[1].artists[0].should be_instance_of(Discogs::Release::Track::Artist)
+        @release.tracklist[1].artists[0].name.should == "Arakain"
+      end
+
+      it "should have an extra artist associated to the second track" do
+        @release.tracklist[1].extraartists.should be_instance_of(Array)
+        @release.tracklist[1].extraartists[0].should be_instance_of(Discogs::Release::Track::Artist)
+        @release.tracklist[1].extraartists[0].name.should == "Debustrol"
+        @release.tracklist[1].extraartists[0].role.should == "Sadism"
+      end
+
+      it "should have a role associated to the first extra artist" do
+        @release.extraartists[0].role.should == "Lyrics By"
+      end
+
+      it "should have no artist associated to the third track" do
+        @release.tracklist[2].artists.should be_nil
+      end
+
+    end
+
+  end
+
+end

data/spec/wrapper_methods/search_spec.rb

@@ -0,0 +1,330 @@
+require File.dirname(__FILE__) + "/../spec_helper"
+
+describe Discogs::Wrapper do
+
+  before do
+    @wrapper = Discogs::Wrapper.new("some_user_agent")
+    @search_term = "slaughter"
+  end
+
+  def mock_search(page)
+    @http_request = mock(Net::HTTP)
+    @http_response = mock(Net::HTTPResponse, :code => "200", :body => valid_search_xml(page))
+    @http_response_as_file = mock(StringIO, :read => valid_search_xml(page))
+    Zlib::GzipReader.should_receive(:new).and_return(@http_response_as_file)
+    @http_request.should_receive(:start).and_return(@http_response)
+    Net::HTTP.should_receive(:new).and_return(@http_request)
+  end
+
+  describe "when asking for search result information" do
+
+    before do
+      mock_search(1)
+
+      @search = @wrapper.search(@search_term)
+    end
+
+    describe "when dealing with page information" do
+
+      it "should have a current_page method" do
+        @search.current_page.should == 1
+      end
+
+      it "should be able to report if this is the last page" do
+        @search.last_page?.should be_false
+      end
+
+    end
+
+    describe "when handling exact results" do
+
+      it "should have the exact results stored as an array" do
+        @search.exactresults.should be_instance_of(Array)
+      end
+
+      it "should be stored as result objects" do
+        @search.exactresults.each do |result|
+          result.should be_instance_of(Discogs::Search::Result)
+        end
+      end
+
+      it "should have a incrementing num for each exact result" do
+        @search.exactresults.each_with_index do |result, index|
+          result.num.should == (index + 1).to_s
+        end
+      end
+
+      it "should have a type for the first result" do
+        @search.exactresults[0].type.should == "artist"
+      end
+
+      it "should have an anv for the fifth result" do
+        @search.exactresults[5].anv.should == "Slaughter"
+      end
+
+      it "should be able to filter all non-artists from exact results" do
+        @search.exact(:artist).should be_instance_of(Array)
+        @search.exact(:artist).length.should == 6
+      end
+
+      it "should be able to filter all non-releases from exact results" do
+        @search.exact(:release).should be_instance_of(Array)
+        @search.exact(:release).length.should == 1
+      end
+
+      it "should be able to filter all non-labels from exact results" do
+        @search.exact(:label).should be_instance_of(Array)
+        @search.exact(:label).length.should == 1
+      end
+
+      it "should return an empty array on a junk filter" do
+        @search.exact(:cheesecake).should be_instance_of(Array)
+        @search.exact(:cheesecake).should be_empty
+      end
+
+      it "should simply return all exact results without a filter" do
+        @search.exact.should be_instance_of(Array)
+        @search.exact.length.should == 8
+      end
+
+      it "should have a shortcut for accessing the first exact artist" do
+        @search.closest(:artist).should be_instance_of(Discogs::Search::Result)
+        @search.closest(:artist).should == @search.exact(:artist)[0]
+      end
+
+      it "should have a shortcut for accessing the first exact release" do
+        @search.closest(:release).should be_instance_of(Discogs::Search::Result)
+        @search.closest(:release).should == @search.exact(:release)[0]
+      end
+
+      it "should have a shortcut for accessing the first exact label" do
+        @search.closest(:label).should be_instance_of(Discogs::Search::Result)
+        @search.closest(:label).should == @search.exact(:label)[0]
+      end
+
+      it "should return nil on junk filter for closest match" do
+        @search.closest(:alcoholic).should be_nil
+      end
+
+   end
+
+    describe "when handling search results" do
+
+      it "should have a start attribute" do
+        @search.start.should == "1"
+      end
+
+      it "should have an end attribute" do
+        @search.end.should == "20"
+      end
+
+      it "should have number of results attribute" do
+        @search.total_results.should == 1846
+      end
+
+      it "should have number of pages attribute" do
+        @search.total_pages.should == 93
+      end
+
+      it "should have the search results stored as an array" do
+        @search.searchresults.should be_instance_of(Array)
+      end
+
+      it "should be stored as result objects" do
+        @search.searchresults.each do |result|
+          result.should be_instance_of(Discogs::Search::Result)
+        end
+      end
+
+   end
+
+    describe "when handling search results" do
+
+      it "should have a start attribute" do
+        @search.start.should == "1"
+      end
+
+      it "should have an end attribute" do
+        @search.end.should == "20"
+      end
+
+      it "should have number of results attribute" do
+        @search.total_results.should == 1846
+      end
+
+      it "should have number of pages attribute" do
+        @search.total_pages.should == 93
+      end
+
+      it "should have the search results stored as an array" do
+        @search.searchresults.should be_instance_of(Array)
+      end
+
+      it "should be stored as result objects" do
+        @search.searchresults.each do |result|
+          result.should be_instance_of(Discogs::Search::Result)
+        end
+      end
+
+   end
+
+    describe "when handling search results" do
+
+      it "should have a start attribute" do
+        @search.start.should == "1"
+      end
+
+      it "should have an end attribute" do
+        @search.end.should == "20"
+      end
+
+      it "should have number of results attribute" do
+        @search.total_results.should == 1846
+      end
+
+      it "should have number of pages attribute" do
+        @search.total_pages.should == 93
+      end
+
+      it "should have the search results stored as an array" do
+        @search.searchresults.should be_instance_of(Array)
+      end
+
+      it "should be stored as result objects" do
+        @search.searchresults.each do |result|
+          result.should be_instance_of(Discogs::Search::Result)
+        end
+      end
+
+      it "should have a incrementing num for each search result" do
+        @search.searchresults.each_with_index do |result, index|
+          result.num.should == (index + 1).to_s
+        end
+      end
+
+      it "should have a type for the third result" do
+        @search.searchresults[2].type.should == "label"
+      end
+
+      it "should have a title for the fourth result" do
+        @search.searchresults[3].title.should == "Satanic Slaughter"
+      end
+
+      it "should have a summary for the sixth result" do
+        @search.searchresults[5].summary.should == "Gary Slaughter"
+      end
+
+      it "should be able to filter all non-artists from extended results" do
+        @search.results(:artist).should be_instance_of(Array)
+        @search.results(:artist).length.should == 12
+      end
+
+      it "should be able to filter all non-releases from extended results" do
+        @search.results(:release).should be_instance_of(Array)
+        @search.results(:release).length.should == 6
+      end
+
+      it "should be able to filter all non-labels from extended results" do
+        @search.results(:label).should be_instance_of(Array)
+        @search.results(:label).length.should == 2
+      end
+
+      it "should return an empty array on a junk filter" do
+        @search.results(:cheesecake).should be_instance_of(Array)
+        @search.results(:cheesecake).should be_empty
+      end
+
+      it "should simply return all extended results without a filter" do
+        @search.results.should be_instance_of(Array)
+        @search.results.length.should == 20
+      end
+
+    end
+ 
+  end
+
+  describe "when getting the next page" do
+
+    before do
+      mock_search(2)
+
+      @search = @wrapper.search(@search_term, :page => 2)
+    end
+
+    describe "when dealing with page information" do
+
+      it "should have a current_page method" do
+        @search.current_page.should == 2
+      end
+
+      it "should be able to report if this is the last page" do
+        @search.last_page?.should be_false
+      end
+
+    end
+
+    describe "when handling exact results" do
+
+      it "should not have any exact results" do
+        @search.exactresults.should be_nil
+      end
+
+      it "should return empty array for exact filtering" do
+        @search.exact.should == []
+      end
+
+      it "should still allow filtering and return an empty array" do
+        @search.exact(:artist).should == []
+      end
+
+      it "should return nil for closest matches when no exact results are available" do
+        @search.closest(:artist).should be_nil
+        @search.closest(:release).should be_nil
+        @search.closest(:label).should be_nil
+      end
+
+      it "should return nil on junk filter for closest match" do
+        @search.closest(:alcoholic).should be_nil
+      end
+
+    end
+
+    describe "when handling search results" do
+
+      it "should have a start attribute" do
+        @search.start.should == "21"
+      end
+
+      it "should have an end attribute" do
+        @search.end.should == "40"
+      end
+
+      it "should have number of results attribute" do
+        @search.total_results.should == 1846
+      end
+
+      it "should have number of pages attribute (still)" do
+        @search.total_pages.should == 93
+      end
+
+      it "should have the search results stored as an array" do
+        @search.searchresults.should be_instance_of(Array)
+      end
+
+      it "should be stored as result objects" do
+        @search.searchresults.each do |result|
+          result.should be_instance_of(Discogs::Search::Result)
+        end
+      end
+
+      it "should have a incrementing num for each search result" do
+        @search.searchresults.each_with_index do |result, index|
+          result.num.should == (index + 21).to_s
+        end
+      end
+
+    end
+
+  end
+
+end