discogs-wrapper 1.1.4 → 2.0.0

data/spec/samples/valid_user_wantlist.json

@@ -0,0 +1 @@
+{"wants": [{"rating": 0, "resource_url": "http://api.discogs.com/users/abuntine/wants/2360509", "basic_information": {"labels": [{"id": 6621, "resource_url": "http://api.discogs.com/labels/6621", "catno": "4 56 577", "name": "AMIGA", "entity_type": "1"}], "formats": [{"descriptions": ["7\"", "45 RPM", "Single"], "name": "Vinyl", "qty": "1"}], "thumb": "http://api.discogs.com/image/R-150-2360509-1279466424.jpeg", "title": "18 Jahre Sein / Mach Keine Wellen", "artists": [{"join": "", "name": "Formel 1 (2)", "anv": "", "tracks": "", "role": "", "resource_url": "http://api.discogs.com/artists/961630", "id": 961630}], "resource_url": "http://api.discogs.com/releases/2360509", "year": 1985, "id": 2360509}, "id": 2360509, "notes_public": false}, {"rating": 0, "resource_url": "http://api.discogs.com/users/abuntine/wants/2621680", "basic_information": {"labels": [{"id": 223844, "resource_url": "http://api.discogs.com/labels/223844", "catno": "WW8301", "name": "Ashbury Music", "entity_type": "1"}], "formats": [{"descriptions": ["LP", "Album"], "name": "Vinyl", "qty": "1"}], "thumb": "http://api.discogs.com/image/R-150-2621680-1368988727-3696.jpeg", "title": "Endless Skies", "artists": [{"join": "", "name": "Ashbury", "anv": "", "tracks": "", "role": "", "resource_url": "http://api.discogs.com/artists/703220", "id": 703220}], "resource_url": "http://api.discogs.com/releases/2621680", "year": 1983, "id": 2621680}, "id": 2621680, "notes_public": false}, {"rating": 0, "resource_url": "http://api.discogs.com/users/abuntine/wants/1898962", "basic_information": {"labels": [{"id": 162562, "resource_url": "http://api.discogs.com/labels/162562", "catno": "none", "name": "Atom Records (6)", "entity_type": ""}], "formats": [{"descriptions": ["Single Sided"], "name": "Cassette", "qty": "1"}], "thumb": "http://api.discogs.com/image/R-150-1898962-1251019631.jpeg", "title": "Breath Of Death", "artists": [{"join": "", "name": "Krabathor", "anv": "Krabator", "tracks": "", "role": "", "resource_url": "http://api.discogs.com/artists/265899", "id": 265899}], "resource_url": "http://api.discogs.com/releases/1898962", "year": 1988, "id": 1898962}, "id": 1898962, "notes_public": false}, {"rating": 0, "resource_url": "http://api.discogs.com/users/abuntine/wants/3161516", "basic_information": {"labels": [{"id": 227019, "resource_url": "http://api.discogs.com/labels/227019", "catno": "BLP-2", "name": "Barrabas Producciones", "entity_type": "1"}], "formats": [{"descriptions": ["LP"], "name": "Vinyl", "qty": "1"}], "thumb": "http://api.discogs.com/image/R-150-3161516-1318553455.jpeg", "title": "El Mensaje Del Mago", "artists": [{"join": "", "name": "\u00d1u", "anv": "", "tracks": "", "role": "", "resource_url": "http://api.discogs.com/artists/1249321", "id": 1249321}], "resource_url": "http://api.discogs.com/releases/3161516", "year": 1987, "id": 3161516}, "id": 3161516, "notes_public": false}, {"rating": 0, "resource_url": "http://api.discogs.com/users/abuntine/wants/1737331", "basic_information": {"labels": [{"id": 232190, "resource_url": "http://api.discogs.com/labels/232190", "catno": "J1 0018-1 311", "name": "Best IA", "entity_type": "1"}], "formats": [{"descriptions": ["LP"], "name": "Vinyl", "qty": "1"}], "thumb": "http://api.discogs.com/image/R-150-1737331-1336936053-3011.jpeg", "title": "Czech Masters Of Rock Guitar / \u010ce\u0161t\u00ed Mist\u0159i Rockov\u00e9 Kytary", "artists": [{"join": "", "name": "Various", "anv": "", "tracks": "", "role": "", "resource_url": "http://api.discogs.com/artists/194", "id": 194}], "resource_url": "http://api.discogs.com/releases/1737331", "year": 1992, "id": 1737331}, "id": 1737331, "notes_public": false}], "pagination": {"per_page": 5, "items": 77, "page": 1, "urls": {"last": "http://api.discogs.com/users/abuntine/wants?per_page=5&page=16", "next": "http://api.discogs.com/users/abuntine/wants?per_page=5&page=2"}, "pages": 16}}

data/spec/samples/valid_wantlist_release.json

@@ -0,0 +1,41 @@
+{
+  "id": 1,
+  "rating": 0,
+  "notes": "This is a note",
+  "resource_url": "http://api.discogs.com/users/example/wants/1",
+  "basic_information": {
+    "id": 1,
+    "resource_url": "http://api.discogs.com/releases/1",
+    "thumb": "http://api.discogs.com/image/R-150-1-1193812031.jpeg",
+    "title": "Stockholm",
+    "year": 1999,
+    "formats": [
+      {
+        "qty": "2",
+        "descriptions": [ "12\"" ],
+        "name": "Vinyl"
+      }
+    ],
+    "labels": [
+      {
+        "name": "Svek",
+        "entity_type": "1",
+        "catno": "SK032",
+        "resource_url": "http://api.discogs.com/labels/5",
+        "id": 5,
+        "entity_type_name": "Label"
+      }
+    ],
+    "artists": [
+      {
+        "join": "",
+        "name": "Persuader, The",
+        "anv": "",
+        "tracks": "",
+        "role": "",
+        "resource_url": "http://api.discogs.com/artists/1",
+        "id": 1
+      }
+    ]
+  }
+}

data/spec/spec_helper.rb

@@ -6,28 +6,14 @@ require 'rubygems'
 require 'rspec'
 require File.dirname(__FILE__) + '/../lib/discogs'
 
-def valid_release_xml
-  read_sample "valid_release.xml"
-end
-
-def valid_master_release_xml
-  read_sample "valid_master_release.xml"
-end
-
-def valid_artist_xml
-  read_sample "valid_artist.xml"
-end
-
-def valid_label_xml
-  read_sample "valid_label.xml"
-end
-
+# TODO: Remove after searhc is re-implemented.
 def valid_search_xml(page=1)
-  read_sample "valid_search_results_#{page}.xml"
+  data = File.read(File.join(File.dirname(__FILE__), "samples", "valid_search_results_#{page}.xml"))
+  data.gsub /\n/, ''
 end
 
 def read_sample(filename)
-  data = File.read(File.join(File.dirname(__FILE__), "samples", filename))
+  data = File.read(File.join(File.dirname(__FILE__), "samples", "valid_#{filename}.json"))
   data.gsub /\n/, ''
 end
 

data/spec/wrapper_methods/add_release_to_user_wantlist_spec.rb

@@ -0,0 +1,42 @@
+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)
+
+    @username = "abuntine"
+    @release_id = "1"
+  end
+
+  describe ".add_release_to_user_wantlist" do
+
+    before do
+      @oauth_response = mock(OAuth::AccessToken, :code => "200", :body => read_sample("wantlist_release"))
+      @oauth_response_as_file = mock(StringIO, :read => read_sample("wantlist_release"))
+      Zlib::GzipReader.should_receive(:new).and_return(@oauth_response_as_file)
+      @oauth_access_token.should_receive(:put).and_return(@oauth_response)
+
+      @wantlist_release = @wrapper.add_release_to_user_wantlist(@username, @release_id)
+    end
+
+    describe "when calling simple wantlist release attributes" do
+
+      it "should have an id" do
+        @wantlist_release.id.should == 1
+      end
+
+      it "should have a rating" do
+        @wantlist_release.rating.should == 0
+      end
+
+      it "should have some basic information" do
+        @wantlist_release.basic_information.title.should == "Stockholm"
+      end
+
+    end
+
+  end
+
+end 

data/spec/wrapper_methods/edit_release_in_user_wantlist_spec.rb

@@ -0,0 +1,38 @@
+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)
+
+    @username = "abuntine"
+    @release_id = "1"
+  end
+
+  describe ".edit_release_in_user_wantlist" do
+
+    before do
+      @oauth_response = mock(OAuth::AccessToken, :code => "200", :body => read_sample("wantlist_release"))
+      @oauth_response_as_file = mock(StringIO, :read => read_sample("wantlist_release"))
+      Zlib::GzipReader.should_receive(:new).and_return(@oauth_response_as_file)
+      @oauth_access_token.should_receive(:post).and_return(@oauth_response)
+
+      @wantlist_release = @wrapper.edit_release_in_user_wantlist(@username, @release_id, {:notes => "This is a note"})
+    end
+
+    describe "when calling simple wantlist release attributes" do
+
+      it "should have an id" do
+        @wantlist_release.id.should == 1
+      end
+
+      it "should have a some notes" do
+        @wantlist_release.notes.should == "This is a note"
+      end
+
+    end
+
+  end
+
+end 

data/spec/wrapper_methods/edit_user_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)
+    @username = "abuntine"
+  end
+
+  describe ".edit_user" do
+
+    before do
+      @oauth_response = mock(OAuth::AccessToken, :code => "200", :body => read_sample("user_profile"))
+      @oauth_response_as_file = mock(StringIO, :read => read_sample("user_profile"))
+      Zlib::GzipReader.should_receive(:new).and_return(@oauth_response_as_file)
+      @oauth_access_token.should_receive(:post).and_return(@oauth_response)
+
+      @user_profile = @wrapper.edit_user(@username, {:profile => "This is a new profile"})
+    end
+
+    describe "when calling simple user profile attributes" do
+
+      it "should have a username" do
+        @user_profile.username.should == "abuntine"
+      end
+
+      it "should have a new profile" do
+        @user_profile.profile.should == "This is a new profile"
+      end
+
+    end
+
+  end
+
+end 

data/spec/wrapper_methods/get_artist_releases_spec.rb

@@ -0,0 +1,49 @@
+require 'spec_helper'
+
+describe Discogs::Wrapper do
+
+  before do
+    @wrapper = Discogs::Wrapper.new("some_user_agent")
+    @artist_id = "313929"
+  end
+
+  describe ".get_artists_releases" do
+
+    before do
+      @http_request = mock(Net::HTTP)
+      @http_response = mock(Net::HTTPResponse, :code => "200", :body => read_sample("artist_releases"))
+      @http_response_as_file = mock(StringIO, :read => read_sample("artist_releases"))
+      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_releases = @wrapper.get_artist_releases(@artist_id)
+    end
+
+    describe "when calling simple releases attributes" do
+
+      it "should have 37 releases per page" do
+        @artist_releases.releases.length.should == 37
+      end
+
+      it "should have 37 releases total" do
+        @artist_releases.pagination.items.should == 37
+      end
+
+      it "should have a first release with a title" do
+        @artist_releases.releases.first.title.should == "Frost And Fire"
+      end
+
+      it "should have a first release with a type" do
+        @artist_releases.releases.first.type.should == "master"
+      end
+
+      it "should not have a bogus attribute" do
+        @artist_releases.bogus_attr.should be_nil
+      end
+
+    end
+
+  end
+
+end 

data/spec/wrapper_methods/get_artist_spec.rb

@@ -4,72 +4,47 @@ describe Discogs::Wrapper do
 
   before do
     @wrapper = Discogs::Wrapper.new("some_user_agent")
-    @artist_name = "Root"
+    @artist_id = 313929 # Cirith Ungol
   end
 
-  describe "when asking for artist information" do
+  describe ".get_artist" 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)
+      @http_response = mock(Net::HTTPResponse, :code => "200", :body => read_sample("artist"))
+      @http_response_as_file = mock(StringIO, :read => read_sample("artist"))
       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)
+      @artist = @wrapper.get_artist(@artist_id)
     end
 
     describe "when calling simple artist attributes" do
 
       it "should have a name attribute" do
-        @artist.name.should == "Root"
+        @artist.name.should == "Cirith Ungol"
       end
 
       it "should have an id attribute" do
-        @artist.id.should == "123"
+        @artist.id.should == @artist_id
       end
   
-      it "should have a realname attribute" do
-        @artist.realname.should == "Rootan"
-      end
-
       it "should have a profile attribute" do
-        @artist.profile.should == "Richard David James was born to Welsh parents"
+        @artist.profile[0..15].should == "Heavy metal band"
       end
 
       it "should have a data_quality attribute" do
-        @artist.data_quality.should == "Needs Vote"
-      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"
+        @artist.data_quality.should == "Correct"
       end
 
       it "should have one or more members" do
         @artist.members.should be_instance_of(Array)
-        @artist.members[0].should == "Big Boss"
+        @artist.members[0].name.should == "Greg Lindstrom"
       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
+      it "should have a releases_url attribute" do
+        @artist.releases_url.should =~ /artists\/313929\/releases/
       end
 
     end
@@ -78,17 +53,16 @@ describe Discogs::Wrapper 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"
+        @artist.urls[0].should == "http://www.truemetal.org/cirithungol"
+        @artist.urls[1].should == "http://www.myspace.com/cirithungol"
       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' ] ]
+        specs = [ [ 360, 288, 'secondary' ], [ 170, 176, 'secondary' ] ]
         @artist.images.each_with_index do |image, index|
           image.width.should == specs[index][0]
           image.height.should == specs[index][1]
@@ -96,40 +70,6 @@ describe Discogs::Wrapper do
         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 type for the first release" do
-        @artist.releases[0].release_type.should == "release"
-      end
-
-      it "should have a type for the second release" do
-        @artist.releases[1].release_type.should == "master"
-      end
-
-      it "should have a main release for the second release" do
-        @artist.releases[1].main_release.should == "12345"
-      end
-
-      it "should have a thumb for the second release" do
-        @artist.releases[1].thumb.should == "http://images"
-      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

data/spec/wrapper_methods/get_identity_spec.rb

@@ -0,0 +1,35 @@
+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)
+  end
+
+  describe ".get_identity" do
+
+    before do
+      @oauth_response = mock(OAuth::AccessToken, :code => "200", :body => read_sample("identity"))
+      @oauth_response_as_file = mock(StringIO, :read => read_sample("identity"))
+      Zlib::GzipReader.should_receive(:new).and_return(@oauth_response_as_file)
+      @oauth_access_token.should_receive(:get).and_return(@oauth_response)
+
+      @identity = @wrapper.get_identity
+    end
+
+    describe "when calling simple identity attributes" do
+
+      it "should have a username" do
+        @identity.username.should == "example"
+      end
+
+      it "should have a consumer_name" do
+        @identity.consumer_name.should == "Your Application Name"
+      end
+
+    end
+
+  end
+
+end 

data/spec/wrapper_methods/get_label_releases_spec.rb

@@ -0,0 +1,49 @@
+require 'spec_helper'
+
+describe Discogs::Wrapper do
+
+  before do
+    @wrapper = Discogs::Wrapper.new("some_user_agent")
+    @label_id = "9800"
+  end
+
+  describe ".get_labels_releases" do
+
+    before do
+      @http_request = mock(Net::HTTP)
+      @http_response = mock(Net::HTTPResponse, :code => "200", :body => read_sample("label_releases"))
+      @http_response_as_file = mock(StringIO, :read => read_sample("label_releases"))
+      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_releases = @wrapper.get_label_releases(@label_id)
+    end
+
+    describe "when calling simple releases attributes" do
+
+      it "should have 8 releases per page" do
+        @label_releases.releases.length.should == 8
+      end
+
+      it "should have 8 releases total" do
+        @label_releases.pagination.items.should == 8
+      end
+
+      it "should have a first release with a Cat No" do
+        @label_releases.releases.first.catno.should == "SSS 001"
+      end
+
+      it "should have a first release with a status" do
+        @label_releases.releases.first.status.should == "Accepted"
+      end
+
+      it "should not have a bogus attribute" do
+      	@label_releases.bogus_attribute.should be_nil
+      end
+
+    end
+
+  end
+
+end