7digital 0.0.2 → 0.0.5

data/spec/model/release_spec.rb

@@ -1,10 +1,14 @@
 require "date"
-require File.join(File.dirname(__FILE__), %w[../spec_helper])
+require File.expand_path('../../spec_helper', __FILE__)
 
 describe "Release" do
 
   before do
+    @configuration = stub(OpenStruct)
+    @configuration.stub!(:lazy_load?).and_return(true)
+
     @client = stub(Sevendigital::Client)
+    @client.stub!(:configuration).and_return(@configuration)
     @release_manager = mock(Sevendigital::ReleaseManager)
     @client.stub!(:release).and_return @release_manager
     
@@ -39,6 +43,13 @@ describe "Release" do
 
   end
 
+  it "should lazy load price even if already populated but price value is not available" do
+    @release.price = Sevendigital::Price.new
+    @release.should_receive(:demand_price)
+    @release.price
+
+  end
+
   it "get_tracks should get tracks from manager" do
     expected_options = {:page => 2}
    
@@ -51,18 +62,6 @@ describe "Release" do
 
   end
 
-  it "get_tracks should link all tracks back to release itself" do
-
-    @release_manager.should_receive(:get_tracks) { |release_id, options|
-      release_id.should == @release.id
-      fake_track_list
-    }
-
-    tracks = @release.get_tracks()
-
-    tracks.all?{|track| track.release == @release}.should == true
-
-  end
 
    it "get_recommendations should get recommendations from manager" do
     expected_options = {:page => 2}
@@ -104,7 +103,9 @@ describe "Release" do
     release.explicit_content = true
     release.formats = []
     release.label = Sevendigital::Label.new
-    release.price = Sevendigital::Price.new
+    price = Sevendigital::Price.new
+    price.value = 5
+    release.price = price
     release
 end
 end

data/spec/model/sevendigital_object_spec.rb

@@ -0,0 +1,74 @@
+require "date"
+require File.expand_path('../../spec_helper', __FILE__)
+
+describe "SevendigitalObject" do
+
+  before do
+
+    @configuration = stub(Sevendigital::ClientConfiguration)
+    @configuration.stub!(:lazy_load).and_return(true)
+
+    @client = stub(Sevendigital::Client)
+    @client.stub!(:configuration).and_return(@configuration)
+    @client.stub!(:verbose?).and_return(true)
+    @test_object = Sevendigital::SevendigitalObject.new(@client)
+  end
+
+  it "sevendigital extended property should define accessor that calls demand method" do
+    class TestClass < Sevendigital::SevendigitalObject
+      sevendigital_extended_property :test_property
+    end
+
+    test_object = TestClass.new(@client)
+    test_object.should_receive(:demand_test_property)
+    test_object.test_property
+
+  end
+
+  it "sevendigital extended property should define accessor that doesn't swallow SevenDigital errors" do
+    class TestClass < Sevendigital::SevendigitalObject
+      sevendigital_extended_property :test_property
+    end
+    @configuration.stub!(:ignorant_lazy_load).and_return(false)
+    test_object = TestClass.new(@client)
+    test_object.should_receive(:demand_test_property).and_raise(Sevendigital::SevendigitalError)
+    running {test_object.test_property}.should raise_error(Sevendigital::SevendigitalError)
+  end
+  
+  it "sevendigital extended property should define accessor that swallows SevenDigital errors if client is ignorant" do
+    class TestClass < Sevendigital::SevendigitalObject
+      sevendigital_extended_property :test_property
+    end
+    @configuration.stub!(:ignorant_lazy_load).and_return(true)
+    test_object = TestClass.new(@client)
+    test_object.should_receive(:demand_test_property).and_raise(Sevendigital::SevendigitalError)
+    test_object.test_property
+  end
+
+  it "sevendigital extended property should define demand method that calls get method if property not set" do
+    class TestClass < Sevendigital::SevendigitalObject
+      sevendigital_extended_property :test_property
+    end
+
+    test_object = TestClass.new(@client)
+    test_object.should_receive(:get_test_property).and_return("a value")
+    test_object.instance_variable_get("@test_property").should == nil
+    test_object.demand_test_property
+    test_object.instance_variable_get("@test_property").should == "a value"
+
+  end
+
+  it "sevendigital extended property should define demand method that does nothing if property is set" do
+    class TestClass < Sevendigital::SevendigitalObject
+      sevendigital_extended_property :test_property
+    end
+
+    test_object = TestClass.new(@client)
+    test_object.instance_variable_set("@test_property", "a value")
+    test_object.should_not_receive(:get_test_property)
+    test_object.demand_test_property
+
+  end
+
+  
+end

data/spec/model/track_spec.rb

@@ -1,5 +1,5 @@
 require "date"
-require File.join(File.dirname(__FILE__), %w[../spec_helper])
+require File.expand_path('../../spec_helper', __FILE__)
 
 describe "Track" do
 
@@ -10,14 +10,17 @@ describe "Track" do
     
     @track = Sevendigital::Track.new(@client)
     @track.id = 1234
+    @track.release = Sevendigital::Release.new(@client)
+    @track.release.id = 5678
   end
 
-  it "get_details should get track's basic details from manager" do
+  it "get_details should get track's basic details from track manager " do
     expected_options = {:page => 2}
     fresh_track = fake_track_with_details
 
-    @track_manager.should_receive(:get_details) { |track_id, options|
+    @track_manager.should_receive(:get_details_from_release) { |track_id, release_id, options|
       track_id.should == @track.id
+      release_id.should == @track.release.id
       (options.keys & expected_options.keys).should == expected_options.keys
       fresh_track
     }
@@ -27,7 +30,7 @@ describe "Track" do
     @track.duration.should == fresh_track.duration
     @track.explicit_content.should == fresh_track.explicit_content
     @track.isrc.should == fresh_track.isrc
-    @track.release.should == fresh_track.release
+    @track.release.id.should == fresh_track.release.id
     @track.url.should == fresh_track.url
 
   end
@@ -134,6 +137,7 @@ describe "Track" do
     track.explicit_content = true
     track.isrc = "7D"
     track.release = Sevendigital::Release.new(@client)
+    track.release.id = 5678
     track.url = "http://aaa.bbb.ccc/"
     track
   end

data/spec/model/user_spec.rb

@@ -0,0 +1,98 @@
+require "date"
+require File.expand_path('../../spec_helper', __FILE__)
+
+describe "User" do
+
+  before do
+    @client = stub(Sevendigital::Client)
+    @user_manager = mock(Sevendigital::UserManager)
+    @client.stub!(:user).and_return @user_manager
+    
+    @user = Sevendigital::User.new(@client)
+  end
+
+  it "should not be authorised if does not have access_token" do
+    @user.oauth_access_token = nil
+    @user.authenticated?.should == false
+  end
+
+  it "should be authorised if does have access_token" do
+    @user.oauth_access_token = "something"
+    @user.authenticated?.should == true
+  end
+
+  it "should get user's locker from locker manager" do
+    @user.oauth_access_token = OAuth::AccessToken.new(nil, "TOKEN", "SECRET")
+    fake_locker = [Sevendigital::LockerRelease.new(@client)]
+    expected_options = {:page => 2}
+
+    @user_manager.should_receive(:get_locker) { |token, options|
+      token.should == @user.oauth_access_token
+      (options.keys & expected_options.keys).should == expected_options.keys
+      fake_locker
+    }
+    locker = @user.get_locker(expected_options)
+    locker.should == fake_locker
+  end
+
+  it "should get user manager to make a purchase" do
+    @user.oauth_access_token = OAuth::AccessToken.new(nil, "TOKEN", "SECRET")
+    a_release_id = 123
+    a_track_id = 456
+    a_price = 1.29
+    fake_locker = [Sevendigital::LockerRelease.new(@client)]
+    expected_options = {:page => 2}
+
+    @user_manager.should_receive(:purchase) { |release_id, track_id, price, token, options|
+      token.should == @user.oauth_access_token
+      release_id.should == a_release_id
+      track_id.should == a_track_id
+      price.should == a_price
+      (options.keys & expected_options.keys).should == expected_options.keys
+      fake_locker
+    }
+    purchase_locker = @user.purchase!(a_release_id, a_track_id, a_price, expected_options)
+    purchase_locker.should == fake_locker
+  end
+
+  it "should get stream track url from user manager" do
+    @user.oauth_access_token = OAuth::AccessToken.new(nil, "TOKEN", "SECRET")
+    a_track_id = 456
+    a_release_id = 123
+    a_stream_track_url = "http://whatever"
+    expected_options = {:page => 2}
+
+    @user_manager.should_receive(:get_stream_track_url) { |release_id, track_id, token, options|
+      token.should == @user.oauth_access_token
+      track_id.should == a_track_id
+      release_id.should == a_release_id
+      (options.keys & expected_options.keys).should == expected_options.keys
+      a_stream_track_url
+    }
+    @user.stream_track_url(a_release_id, a_track_id, expected_options).should == a_stream_track_url
+  end
+
+  it "should not get a stream tack url but raise Sevendigital::Error if user is not authenticated" do
+    running {@user.get_locker}.should raise_error(Sevendigital::SevendigitalError)
+  end
+
+  it "should get an add card url from user manager" do
+    @user.oauth_access_token = OAuth::AccessToken.new(nil, "TOKEN", "SECRET")
+    a_stream_track_url = "http://whatever"
+    a_return_url = "http://example.com"
+    expected_options = {:page => 2}
+
+    @user_manager.should_receive(:get_add_card_url) { |return_url, token, options|
+      return_url.should == a_return_url
+      token.should == @user.oauth_access_token
+      (options.keys & expected_options.keys).should == expected_options.keys
+      a_stream_track_url
+    }
+    @user.add_card_url(a_return_url, expected_options).should == a_stream_track_url
+  end
+
+  it "should not get a stream tack url but raise Sevendigital::Error if user is not authenticated" do
+    running {@user.get_locker}.should raise_error(Sevendigital::SevendigitalError)
+  end
+
+end

data/spec/pager_spec.rb

@@ -1,7 +1,7 @@
-require File.join(File.dirname(__FILE__), %w[spec_helper])
+require "spec_helper"
 
 describe "Pager" do
 
-  it "should paginate list" 
+  #it "should paginate list"
 
 end

data/spec/proxy_police_spec.rb

@@ -1,6 +1,6 @@
 # encoding: UTF-8
 require 'date'
-require File.join(File.dirname(__FILE__), %w[spec_helper])
+require "spec_helper"
 
 describe "Proxy Police" do
 

data/spec/spec_helper.rb

@@ -1,5 +1,13 @@
-$: << 'sevendigital'
-$: << 'spec'
+#$: << 'sevendigital'
+#$: << 'spec'
+
+require 'simplecov'
+
+SimpleCov.start do
+  add_group 'Management', 'lib/sevendigital/management'
+  add_group 'Digestion', 'lib/sevendigital/digestion_tract'
+  add_group 'Model', 'lib/sevendigital/model'
+end
 
 require 'spec'
 require File.expand_path(
@@ -10,7 +18,7 @@ require "peachy_spec_helper_patch"
 Peachy.whine
 
 Spec::Runner.configure do |config|
-  # == Mock Framework
+  # == Mock Framework 
   #
   # RSpec uses it's own mocking framework by default. If you prefer to
   # use mocha, flexmock or RR, uncomment the appropriate line:

data/spec/test-xml/methods/artist/browse.xml

@@ -0,0 +1,22 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<response status="ok" version="1.2"
+					xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+					xsi:noNamespaceSchemaLocation="http://api.7digital.com/1.2/static/7digitalAPI.xsd">
+	<artists>
+		<page>1</page>
+		<pageSize>10</pageSize>
+		<totalItems>3256</totalItems>
+		<artist id="144641">
+			<name>P J Harvey</name>
+			<url>http://www.7digital.com/artists/p-j-harvey/?partner=123</url>
+			<sortName>P J Harvey</sortName>
+			<image>http://cdn.7static.com/static/img/artistimages/00/001/446/0000144641_50.jpg</image>
+		</artist>
+		<artist id="17168">
+			<name>P J Proby</name>
+			<url>http://www.7digital.com/artists/p-j-proby/?partner=123</url>
+			<sortName>P J Proby</sortName>
+			<image>http://cdn.7static.com/static/img/artistimages/00/001/716/000017168_50.jpg</image>
+		</artist>
+	</artists>
+</response>

data/spec/test-xml/methods/artist/chart.xml

@@ -0,0 +1,31 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<response status="ok" version="1.2"
+					xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+					xsi:noNamespaceSchemaLocation="http://api.7digital.com/1.2/static/7digitalAPI.xsd">
+	<chart>
+		<type>artist</type>
+		<fromDate>2009-03-24T00:00:00Z</fromDate>
+		<toDate>2009-03-30T00:00:00+01:00</toDate>
+		<page>1</page>
+		<pageSize>10</pageSize>
+		<totalItems>20</totalItems>
+		<chartItem>
+			<position>1</position>
+			<change>Up</change>
+			<artist id="162872">
+				<name>The Last Shadow Puppets</name>
+				<url>http://www.7digital.com/artists/the-last-shadow-puppets/?partner=123</url>
+			</artist>
+		</chartItem>
+		<chartItem>
+			<position>2</position>
+			<change>Down</change>
+			<artist id="304">
+				<name>Radiohead</name>
+				<sortName>Radiohead</sortName>
+				<url>http://eu.7digital.com/artists/radiohead/?partner=827</url>
+				<image>http://cdn.7static.com/static/img/artistimages/00/000/003/0000000304_150.jpg</image>
+			</artist>
+		</chartItem>
+	</chart>
+</response>

data/spec/test-xml/methods/basket/additem.xml

@@ -0,0 +1,24 @@
+<response status="ok" version="1.2">
+  <basket id="00000000-0000-0000-0000-000000000001">
+    <itemCount>1</itemCount>
+    <price>
+      <currency code="EUR">€</currency>
+      <value>0.99</value>
+      <formattedPrice>0,99€</formattedPrice>
+    </price>
+    <basketItems>
+      <basketItem id="15284882">
+        <type>track</type>
+        <itemName>Test</itemName>
+        <artistName>Ghetto</artistName>
+        <trackId>2458384</trackId>
+        <releaseId>224820</releaseId>
+        <price>
+          <currency code="EUR">€</currency>
+          <value>0.99</value>
+          <formattedPrice>0,99€</formattedPrice>
+        </price>
+      </basketItem>
+    </basketItems>
+  </basket>
+</response>

data/spec/test-xml/methods/basket/create.xml

@@ -0,0 +1,11 @@
+<response status="ok" version="1.2">
+  <basket id="00000000-0000-0000-0000-000000000000">
+    <itemCount>0</itemCount>
+    <price>
+      <currency code="GBP">&#163;</currency>
+      <value>0</value>
+      <formattedPrice>&#163;0.00</formattedPrice>
+    </price>
+    <basketItems/>
+  </basket>
+</response>

data/spec/test-xml/methods/basket/index.xml

@@ -0,0 +1,24 @@
+<response status="ok" version="1.2">
+  <basket id="00000000-0000-0000-0000-000000000001">
+    <itemCount>1</itemCount>
+    <price>
+      <currency code="EUR">€</currency>
+      <value>0.99</value>
+      <formattedPrice>0,99€</formattedPrice>
+    </price>
+    <basketItems>
+      <basketItem id="15284882">
+        <type>track</type>
+        <itemName>Test</itemName>
+        <artistName>Ghetto</artistName>
+        <trackId>2458384</trackId>
+        <releaseId>224820</releaseId>
+        <price>
+          <currency code="EUR">€</currency>
+          <value>0.99</value>
+          <formattedPrice>0,99€</formattedPrice>
+        </price>
+      </basketItem>
+    </basketItems>
+  </basket>
+</response>

data/spec/test-xml/methods/basket/removeitem.xml

@@ -0,0 +1,11 @@
+<response status="ok" version="1.2">
+  <basket id="00000000-0000-0000-0000-000000000000">
+    <itemCount>1</itemCount>
+    <price>
+      <currency code="EUR">€</currency>
+      <value>0.00</value>
+      <formattedPrice>0,00€</formattedPrice>
+    </price>
+    <basketItems/>
+  </basket>
+</response>