discogs-wrapper 1.1.4 → 2.0.0

data/spec/wrapper_methods/get_label_spec.rb

@@ -4,44 +4,44 @@ describe Discogs::Wrapper do
 
   before do
     @wrapper = Discogs::Wrapper.new("some_user_agent")
-    @artist_name = "Root"
+    @label_id = 1000
   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)
+      @http_response = mock(Net::HTTPResponse, :code => "200", :body => read_sample("label"))
+      @http_response_as_file = mock(StringIO, :read => read_sample("label"))
       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)
+      @label = @wrapper.get_label(@label_id)
     end
 
     describe "when calling simple label attributes" do
 
       it "should have a name attribute" do
-        @label.name.should == "Sombre Records"
+        @label.name.should == "Warner Bros. Records"
       end
   
-      it "should have a contact info attribute" do
-        @label.contactinfo.should == "Unknown, Germany"
+      it "should have a releases_url attribute" do
+        @label.releases_url.should =~ /labels\/1000\/releases/
       end
   
       it "should have a profile attribute" do
-        @label.profile.should == "Released some OK LPs"
+        @label.profile.should =~ /Founded in 1958/
       end
   
       it "should have a parent label attribute" do
-        @label.parentlabel.should == "SuperSombre"
+        @label.parent_label.name.should == "Warner Music Group"
       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"
+        @label.urls[0].should == "http://www.warnerbrosrecords.com/"
+        @label.urls[1].should == "http://www.facebook.com/WarnerBrosRecords"
       end
   
     end
@@ -50,17 +50,17 @@ describe Discogs::Wrapper 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"
+        @label.sublabels[0].name.should == "1017 Brick Squad Records"
       end
  
       it "should have specifications for each image" do
-        specs = [ [ '400', '300', 'primary' ], [ '450', '350', 'secondary' ] ]
+        specs = [ [ 600, 780, 'primary' ], [ 533, 698, 'secondary' ], [ 600, 600, 'secondary' ],
+                  [ 532, 532, 'secondary' ], [ 175, 192, 'secondary' ], [ 200, 94, 'secondary' ],
+                  [ 500, 493, 'secondary' ] ]
 
         @label.images.each_with_index do |image, index|
           image.width.should == specs[index][0]
@@ -69,19 +69,6 @@ describe Discogs::Wrapper do
         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

data/spec/wrapper_methods/get_listing_spec.rb

@@ -0,0 +1,40 @@
+require 'spec_helper'
+
+describe Discogs::Wrapper do
+
+  before do
+    @oauth_access_token = mock(OAuth::AccessToken)
+    @wrapper = Discogs::Wrapper.new("some_user_agent", @oauth_access_token)
+    @listing_id = 41578240
+  end
+
+  describe ".get_listing" do
+
+    before do
+      @oauth_response = mock(OAuth::AccessToken, :code => "200", :body => read_sample("listing"))
+      @oauth_response_as_file = mock(StringIO, :read => read_sample("listing"))
+      Zlib::GzipReader.should_receive(:new).and_return(@oauth_response_as_file)
+      @oauth_access_token.should_receive(:get).and_return(@oauth_response)
+
+      @listing = @wrapper.get_listing(@listing_id)
+    end
+
+    describe "when calling simple listing attributes" do
+
+      it "should have a username" do
+        @listing.status.should == "For Sale"
+      end
+
+      it "should have a weight" do
+        @listing.weight.should == 239.0
+      end
+
+      it "should have a seller" do
+        @listing.seller.username.should == "EmpireDVDs"
+      end
+
+    end
+
+  end
+
+end 

data/spec/wrapper_methods/get_master_release_spec.rb

@@ -1,18 +1,19 @@
+
 require 'spec_helper'
 
 describe Discogs::Wrapper do
 
   before do
     @wrapper = Discogs::Wrapper.new("some_user_agent")
-    @master_release_id = "666666"
+    @master_release_id = "9800"
   end
 
-  describe "when asking for master_release information" do
+  describe ".get_master_release" do
 
     before do
       @http_request = mock(Net::HTTP)
-      @http_response = mock(Net::HTTPResponse, :code => "200", :body => valid_master_release_xml)
-      @http_response_as_file = mock(StringIO, :read => valid_master_release_xml)
+      @http_response = mock(Net::HTTPResponse, :code => "200", :body => read_sample("master_release"))
+      @http_response_as_file = mock(StringIO, :read => read_sample("master_release"))
       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)
@@ -20,46 +21,40 @@ describe Discogs::Wrapper do
       @master_release = @wrapper.get_master_release(@master_release_id)
     end
 
-    describe "when calling simple master_release attributes" do
+    describe "when calling simple master release attributes" do
 
       it "should have an ID attribute" do
-        @master_release.id.should == "6119"
+        @master_release.id.should == 9800
       end
 
       it "should have a main_release attribute" do
-        @master_release.main_release.should == "1025"
+        @master_release.main_release.should == 5160870
       end
 
       it "should have one or more tracks" do
         @master_release.tracklist.should be_instance_of(Array)
-        @master_release.tracklist[0].should be_instance_of(Discogs::MasterRelease::Track)
+        @master_release.tracklist[0].duration.should == "2:56"
       end
  
       it "should have one or more genres" do
         @master_release.genres.should be_instance_of(Array)
-        @master_release.genres[0].should == "Electronic"
+        @master_release.genres[0].should == "Rock"
       end
 
-      it "should have one or more versions" do
-        @master_release.versions.should be_instance_of(Array)
-        @master_release.versions[0].should be_instance_of(Discogs::MasterRelease::Version)
+      it "should have a versions_url" do
+        @master_release.versions_url.should =~ /masters\/9800\/versions/
       end
 
       it "should have one or more images" do
         @master_release.images.should be_instance_of(Array)
-        @master_release.images[0].should be_instance_of(Discogs::Image)
       end
 
     end
 
     describe "when calling complex master_release attributes" do
 
-      it "should have a duration for the first track" do
-        @master_release.tracklist[0].duration.should == "4:42"
-      end
-
       it "should have specifications for each image" do
-        specs = [ [ '600', '590', 'primary' ], [ '600', '594', 'secondary' ], [ '600', '297', 'secondary' ], [ '600', '601', 'secondary' ], [ '600', '600', 'secondary' ] ]
+        specs = [ [ 600, 607, 'primary' ], [ 600, 604, 'secondary' ], [ 600, 598, 'secondary' ], [ 600, 601, 'secondary' ] ]
         @master_release.images.each_with_index do |image, index|
           image.width.should == specs[index][0]
           image.height.should == specs[index][1]
@@ -67,39 +62,18 @@ describe Discogs::Wrapper do
         end
       end
 
-      it "should have attributes for each version" do
-        specs = [ [ '1025', 'US', '2x12", Album' ], [ '4179', 'US', 'CD, Album' ], [ '416774', 'US', '12"' ] ]
-        @master_release.versions.each_with_index do |version, index|
-          version.id.should == specs[index][0]
-          version.country.should == specs[index][1]
-          version.format.should == specs[index][2]
-        end
-      end
-
       it "should have a traversible list of styles" do
         @master_release.styles.should be_instance_of(Array)
-        @master_release.styles[0].should == "Deep House"
-      end
-
-      it "should have an artist associated to the second track" do
-        @master_release.tracklist[1].artists.should be_instance_of(Array)
-        @master_release.tracklist[1].artists[0].should be_instance_of(Discogs::MasterRelease::Track::Artist)
-        @master_release.tracklist[1].artists[0].name.should == "Arakain"
+        @master_release.styles[0].should == "Prog Rock"
       end
 
       it "should have an extra artist associated to the second track" do
         @master_release.tracklist[1].extraartists.should be_instance_of(Array)
-        @master_release.tracklist[1].extraartists[0].should be_instance_of(Discogs::MasterRelease::Track::Artist)
-        @master_release.tracklist[1].extraartists[0].name.should == "Debustrol"
-        @master_release.tracklist[1].extraartists[0].role.should == "Sadism"
-      end
-
-      it "should have no artist associated to the third track" do
-        @master_release.tracklist[2].artists.should be_nil
+        @master_release.tracklist[1].extraartists[0].role.should == "Lead Vocals"
       end
 
     end
 
   end
 
-end
+end 

data/spec/wrapper_methods/get_master_release_versions_spec.rb

@@ -0,0 +1,49 @@
+require 'spec_helper'
+
+describe Discogs::Wrapper do
+
+  before do
+    @wrapper = Discogs::Wrapper.new("some_user_agent")
+    @master_id = "9800"
+  end
+
+  describe ".get_master_release_versions" do
+
+    before do
+      @http_request = mock(Net::HTTP)
+      @http_response = mock(Net::HTTPResponse, :code => "200", :body => read_sample("master_release_versions"))
+      @http_response_as_file = mock(StringIO, :read => read_sample("master_release_versions"))
+      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)
+
+      @master_versions = @wrapper.get_master_release_versions(@master_id)
+    end
+
+    describe "when calling simple master release versions attributes" do
+
+      it "should have 3 versions per page" do
+        @master_versions.versions.length.should == 3
+      end
+
+      it "should have 3 versions total" do
+        @master_versions.pagination.items.should == 3
+      end
+
+      it "should have a first version with a label" do
+        @master_versions.versions.first.label.should == "Panton"
+      end
+
+      it "should have a first release with a released field" do
+        @master_versions.versions.first.released.should == "1982"
+      end
+
+      it "should not have a bogus attribute" do
+      	@master_versions.bogus_attribute.should be_nil
+      end
+
+    end
+
+  end
+
+end 

data/spec/wrapper_methods/get_order_messages_spec.rb

@@ -0,0 +1,36 @@
+require 'spec_helper'
+
+describe Discogs::Wrapper do
+
+  before do
+    @oauth_access_token = mock(OAuth::AccessToken)
+    @wrapper = Discogs::Wrapper.new("some_user_agent", @oauth_access_token)
+    @order_id = "1-1"
+  end
+
+  describe ".get_order_messages" do
+
+    before do
+      @oauth_response = mock(OAuth::AccessToken, :code => "200", :body => read_sample("order_messages"))
+      @oauth_response_as_file = mock(StringIO, :read => read_sample("order_messages"))
+      Zlib::GzipReader.should_receive(:new).and_return(@oauth_response_as_file)
+      @oauth_access_token.should_receive(:get).and_return(@oauth_response)
+
+      @order_messages = @wrapper.get_order_messages(@order_id)
+    end
+
+    describe "when calling simple order messages attributes" do
+
+      it "should have an array of messages" do
+        @order_messages.messages.should be_instance_of(Array)
+      end
+
+      it "should have a username for the first message" do
+        @order_messages.messages[0].from.username.should == "example_seller"
+      end
+
+    end
+
+  end
+
+end 

data/spec/wrapper_methods/get_order_spec.rb

@@ -0,0 +1,36 @@
+require 'spec_helper'
+
+describe Discogs::Wrapper do
+
+  before do
+    @oauth_access_token = mock(OAuth::AccessToken)
+    @wrapper = Discogs::Wrapper.new("some_user_agent", @oauth_access_token)
+    @order_id = "1-1"
+  end
+
+  describe ".get_order" do
+
+    before do
+      @oauth_response = mock(OAuth::AccessToken, :code => "200", :body => read_sample("order"))
+      @oauth_response_as_file = mock(StringIO, :read => read_sample("order"))
+      Zlib::GzipReader.should_receive(:new).and_return(@oauth_response_as_file)
+      @oauth_access_token.should_receive(:get).and_return(@oauth_response)
+
+      @order = @wrapper.get_order(@order_id)
+    end
+
+    describe "when calling simple order attributes" do
+
+      it "should have a status" do
+        @order.status.should == "New Order"
+      end
+
+      it "should have a fee" do
+        @order.fee.value.should == 2.52
+      end
+
+    end
+
+  end
+
+end 

data/spec/wrapper_methods/get_price_suggestions_spec.rb

@@ -0,0 +1,36 @@
+require 'spec_helper'
+
+describe Discogs::Wrapper do
+
+  before do
+    @oauth_access_token = mock(OAuth::AccessToken)
+    @wrapper = Discogs::Wrapper.new("some_user_agent", @oauth_access_token)
+    @release_id = "1"
+  end
+
+  describe ".get_price_suggestions" do
+
+    before do
+      @oauth_response = mock(OAuth::AccessToken, :code => "200", :body => read_sample("price_suggestions"))
+      @oauth_response_as_file = mock(StringIO, :read => read_sample("price_suggestions"))
+      Zlib::GzipReader.should_receive(:new).and_return(@oauth_response_as_file)
+      @oauth_access_token.should_receive(:get).and_return(@oauth_response)
+
+      @price_suggestions = @wrapper.get_price_suggestions(@release_id)
+    end
+
+    describe "when calling simple price suggestion attributes" do
+
+      it "should have a block for Near Mint" do
+        @price_suggestions["Near Mint (NM or M-)"].should_not be_nil
+      end
+
+      it "should have a price for Mint" do
+        @price_suggestions["Mint (M)"].value.should == 14.319139100000001
+      end
+
+    end
+
+  end
+
+end 

data/spec/wrapper_methods/get_release_spec.rb

@@ -1,18 +1,19 @@
+
 require 'spec_helper'
 
 describe Discogs::Wrapper do
 
   before do
     @wrapper = Discogs::Wrapper.new("some_user_agent")
-    @release_id = "666666"
+    @release_id = "1"
   end
 
-  describe "when asking for release information" do
+  describe ".get_release" 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)
+      @http_response = mock(Net::HTTPResponse, :code => "200", :body => read_sample("release"))
+      @http_response_as_file = mock(StringIO, :read => read_sample("release"))
       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)
@@ -23,58 +24,48 @@ describe Discogs::Wrapper do
     describe "when calling simple release attributes" do
 
       it "should have a title attribute" do
-        @release.title.should == "Into the Abyss"
+        @release.title.should == "Stockholm"
       end
   
       it "should have an ID attribute" do
-        @release.id.should == "666666"
+        @release.id.should == 1
       end
 
       it "should have a master_id attribute" do
-        @release.master_id.should == "6111"
+        @release.master_id.should == 5427
       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)
+        @release.extraartists[0].id.should == 239
       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)
+        @release.tracklist[0].position.should == "A"
       end
  
       it "should have one or more genres" do
         @release.genres.should be_instance_of(Array)
-        @release.genres[0].should == "Heavy Metal"
+        @release.genres[0].should == "Electronic"
       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)
+        @release.formats[0].name.should == "Vinyl"
       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
-
-      it "should have one or more videos" do
-        @release.videos.should be_instance_of(Array)
-        @release.videos[0].should be_instance_of(Discogs::Video)
+        @release.images[0].type.should == "primary"
       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' ] ]
+        specs = [ [ 600, 600, 'primary' ], [ 600, 600, 'secondary' ], [ 600, 600, 'secondary' ], [ 600, 600, 'secondary' ] ]
         @release.images.each_with_index do |image, index|
           image.width.should == specs[index][0]
           image.height.should == specs[index][1]
@@ -83,52 +74,35 @@ describe Discogs::Wrapper do
       end
 
       it "should have specifications for each video" do
-        specs = [ [ '334', 'true', 'http://www.youtube.com/watch?v=QVdDhOnoR8k' ], [ '350', 'true', 'http://www.youtube.com/watch?v=QVdDhOnoR7k' ] ]
+        specs = [ [ 380, true, 'http://www.youtube.com/watch?v=5rA8CTKKEP4' ], [ 335, true, 'http://www.youtube.com/watch?v=QVdDhOnoR8k' ],
+                  [ 290, true, 'http://www.youtube.com/watch?v=AHuQWcylaU4' ], [ 175, true, 'http://www.youtube.com/watch?v=sLZvvJVir5g' ],
+                  [ 324, true, 'http://www.youtube.com/watch?v=js_g1qtPmL0' ], [ 289, true, 'http://www.youtube.com/watch?v=hy47qgyJeG0' ] ]
         @release.videos.each_with_index do |video, index|
           video.duration.should == specs[index][0]
           video.embed.should == specs[index][1]
-          video.src.should == specs[index][2]
+          video.uri.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"
+        @release.styles[0].should == "Deep House"
       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"
+        @release.labels[0].catno.should == "SK032"
+        @release.labels[0].name.should == "Svek"
       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"
+        @release.formats.should be_instance_of(Array)
+        @release.formats[0].name.should == "Vinyl"
+        @release.formats[0].qty.should == "2"
       end
 
       it "should have a role associated to the first extra artist" do
-        @release.extraartists[0].role.should == "Lyrics By"
+        @release.extraartists[0].role.should == "Music By [All Tracks By]"
       end
 
       it "should have no artist associated to the third track" do
@@ -139,4 +113,4 @@ describe Discogs::Wrapper do
 
   end
 
-end
+end