bassnode-discogs 0.3.2

data/spec/spec_helper.rb

@@ -0,0 +1,36 @@
+require 'rubygems'
+require 'spec'
+require File.dirname(__FILE__) + '/../lib/discogs'
+
+def valid_release_xml
+  read_sample "valid_release.xml"
+end
+
+def valid_artist_xml
+  read_sample "valid_artist.xml"
+end
+
+def valid_label_xml
+  read_sample "valid_label.xml"
+end
+
+def valid_search_xml(page=1)
+  read_sample "valid_search_results_#{page}.xml"
+end
+
+def read_sample(filename)
+  data = File.read(File.join(File.dirname(__FILE__), "samples", filename))
+  data.gsub /\n/, ''
+end
+
+def sample_valid_binary
+  "\037\213\b\000\325\3054J\002\377\265T\337O\3330\020\376W,?L\233D\353\004(\203\315\215D\3304\020T\233V\006\017\323\036\214}4\036\211\035l\207\222\377~\227_U\v\025C\232\366\344\273\363\335\227/\337\335\231;\360%\361A\204)\265w\224<\200\363\332\232)\215\307\021%\016\356+\360\301\243K\023\356 \a\341\201h\205\376\036m\253*\274;\226\022\312\000\nSt!\026\340\373\223d\240\027\031\002\037D\210\025\352\022\246\324\203\264F\tWSR9=\245Y\b\345\a\306\226\313\345Xi/\355\302\217\245-X\213\303\276\217\342\275Q\034\037\306\373\321\321\321d2\376]\302\242\255\213'\321\337K'\321\326\362\245V!\3539\261\377\302\3640\376'\246\253\362'L;QP\\\341\202\306\246\fF\302\215( \231Ykd\246s\345\300p\326\2068\353R\223\301\300\232\240C\016I\n5\266\201\\\330\a\340\254\v\361\\\334@\216\031\355I\244\b\306N\351\247\375\213x7\212\336S\322 \242\017P\356_\350[\240\004\305cC\r<\006'\266\363:\267\372F\aK\336\210\242\374HR{W\r\354\234E&_C\006\216\374\234\227\332\351P\211\\\207\372\027g\355\325\300\372\311\217\236[Wo@\\Y)r\377r\321I\346\264'_\234\330,\275\306\217\0060\243\264\336!\337\234U\225\004\267Cf\372\021\024I1\365\t\017\266\371\237\267\326\025\242iDg\364\022]iS\347\224\334\207\272[\e\005^:]\006\\+L]\363\222x\227r\266\036\330\3600\231u\300+\003C\v0\270\262\375\231|\316A\006g\215\226\234\2657\230:d\370P\347Mf{&\227 3c9\353\274>\330t\223\234\332\312\343\024\364\027\335\211e\322V&\270:\3711\347l\260\207'@%8\023\321(:\034E\021\212\324\275\v\n\a\321\006\374b/\352\016);E\025\0218kE+*\376\020\312\213\356Lx\217o\005\231U^K\3626\235\235\275\ecW[\000\216\303$\357r\034r\234\327\306Lxi\275n\024L\216c\316V\316\v\323\254*'\232\202vL{\220gX\273\317\261\25630\344\004[%5\352BR|\257\n\360\030)\220zX-\313k\340\323-TOA<4\373\331\357\334\253`\266\260<3\270Q\227\031\220y\300\031|\021\216\255\2119\264\n\247\244y\367\223?4\242\342\264\376\005\000\000"
+end
+
+def sample_invalid_data
+  <<-EOF
+    <resp requests="2" stat="fail">
+      <error>Missing or invalid API Key</error>
+    </resp> 
+  EOF
+end

data/spec/wrapper_methods/get_artist_spec.rb

@@ -0,0 +1,109 @@
+require File.dirname(__FILE__) + "/../spec_helper"
+
+describe Discogs::Wrapper do
+
+  before do
+    @wrapper = Discogs::Wrapper.new("some_key")
+    @artist_name = "Root"
+  end
+
+  describe "when asking for artist information" do
+
+    before do
+      @http_request = mock(Net::HTTP)
+      @http_response = mock(Net::HTTPResponse, :code => "200", :body => valid_artist_xml)
+      @http_response_as_file = mock(StringIO, :read => valid_artist_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)
+
+      @artist = @wrapper.get_artist(@artist_name)
+    end
+
+    describe "when calling simple artist attributes" do
+
+      it "should have a name attribute" do
+        @artist.name.should == "Root"
+      end
+  
+      it "should have a realname attribute" do
+        @artist.realname.should == "Rootan"
+      end
+
+      it "should have one or more aliases" do
+        @artist.aliases.should be_instance_of(Array)
+        @artist.aliases[0].should == "Roooot"
+      end
+
+      it "should have one or more name variations" do
+        @artist.namevariations.should be_instance_of(Array)
+        @artist.namevariations[0].should == "Rootan"
+      end
+
+      it "should have one or more members" do
+        @artist.members.should be_instance_of(Array)
+        @artist.members[0].should == "Big Boss"
+      end
+
+      it "should be able to filter non-main releases" do
+        @artist.main_releases.should be_instance_of(Array)
+        @artist.main_releases.length.should == 4
+      end
+
+      it "should be able to filter non-bootleg releases" do
+        @artist.bootlegs.should be_instance_of(Array)
+        @artist.bootlegs.length.should == 1
+      end
+
+      it "should be able to filter non-main releases" do
+        @artist.appearances.should be_instance_of(Array)
+        @artist.appearances.length.should == 1
+      end
+
+    end
+
+    describe "when calling complex artist attributes" do
+
+      it "should have a traversible list of URLs" do
+        @artist.urls.should be_instance_of(Array)
+        @artist.urls[0].should == "http://www.root.net"
+        @artist.urls[1].should == "http://www.rootan.com"
+      end
+
+      it "should have a traversible list of images" do
+        @artist.images.should be_instance_of(Array)
+        @artist.images[0].should be_instance_of(Discogs::Image)
+      end
+
+      it "should have specifications for each image" do
+        specs = [ [ '350', '240', 'secondary' ], [ '222', '226', 'secondary' ], [ '600', '600', 'primary' ] ]
+        @artist.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
+        @artist.releases.should be_instance_of(Array)
+        @artist.releases.length.should == 6
+        @artist.releases[0].should be_instance_of(Discogs::Artist::Release)
+      end
+
+      it "should have an ID for the first release" do
+        @artist.releases[0].id.should == "1805661"
+      end
+
+      it "should have a year for the first release" do
+        @artist.releases[0].year.should == "1991"
+      end
+
+      it "should have a label for the third release" do
+        @artist.releases[2].label.should == "Apollo"
+      end
+
+    end
+
+  end
+
+end

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_key")
+    @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_key")
+    @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_key")
+    @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