flickr_fu 0.1.6 → 0.3.0

data/spec/flickr/photosets_spec.rb

@@ -0,0 +1,49 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe Flickr::Photosets do
+  before :all do
+    @get_list_xml = File.read(File.dirname(__FILE__) +
+        "/../fixtures/flickr/photosets/get_list-0.xml")
+    @get_photos_xml = File.read(File.dirname(__FILE__) +
+        "/../fixtures/flickr/photosets/get_photos-0.xml")
+  end
+    
+  before :each do
+    @flickr = SpecHelper.flickr
+  end
+
+  
+  describe ".get_list" do
+    it "should call flickr.photosets.getList" do
+      @flickr.should_receive(:send_request).with("flickr.photosets.getList", {})       
+      @flickr.photosets.get_list
+    end
+    
+    it "should return an array of photoset objects" do
+      @flickr.stub!(:request_over_http).and_return(@get_list_xml)  
+      photosets = @flickr.photosets.get_list
+      
+      photosets[0].should be_an_instance_of(Flickr::Photosets::Photoset)      
+      photosets[0].title.should == 'Test'
+    end
+  end
+  
+  describe ".get_photos" do
+    before :each do
+      @photoset = Flickr::Photosets::Photoset.new(@flickr,{:id=>4})
+    end
+    
+    it "should call flickr.photosets.getPhotos" do
+      @flickr.should_receive(:send_request).with("flickr.photosets.getPhotos",{:photoset_id=>4})       
+      @photoset.get_photos
+    end
+    
+    it "should return an array of photo objects" do
+      @flickr.stub!(:request_over_http).and_return(@get_photos_xml)  
+      photos = @photoset.get_photos
+      
+      photos.should_not be_nil
+      photos[0].should be_an_instance_of(Flickr::Photos::Photo)    
+    end
+  end
+end

data/spec/flickr/test_spec.rb

@@ -0,0 +1,34 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe Flickr::Test do
+  
+  # Before all should be sufficient here. If not it should be safe to change it to
+  # before :each
+  before :all do
+    @flickr = SpecHelper.flickr
+  end
+  
+  describe ".echo" do
+    it "should return all parameters back in the response" do
+      xml = File.read(File.dirname(__FILE__) + "/../fixtures/flickr/test/echo-0.xml")
+      @flickr.should_receive(:request_over_http).and_return(xml)
+      actual = @flickr.test.echo("foo" => "bar")
+      actual.strip_api_params!
+      actual.should == {:method => "flickr.test.echo", "foo" => "bar"}
+    end
+  end
+  
+  # FIXME Hm... we should have a better way of testing if a user is authenticated or not after all
+  describe ".null" do
+    it "should fail if not authenticated with read permissions" do
+      xml = File.read(File.dirname(__FILE__) + "/../fixtures/flickr/test/null-fail-99.xml")
+      @flickr.should_receive(:request_over_http).and_return(xml)
+      lambda { @flickr.test.null }.should raise_error(Flickr::Error, /^99:/)
+    end
+    
+    it "should return true unless there is an error" do
+      @flickr.should_receive(:request_over_http).and_return("")
+      @flickr.test.null.should == true
+    end
+  end
+end

data/spec/flickr/urls_spec.rb

@@ -0,0 +1,99 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe Flickr::Urls do
+  before :all do
+    @flickr = SpecHelper.flickr
+  end
+
+  
+  describe ".get_group" do
+    it "should return the url to a group's page if the group_id is valid" do
+      xml = File.read(File.dirname(__FILE__) + "/../fixtures/flickr/urls/get_group-0.xml")
+      @flickr.should_receive(:request_over_http).and_return(xml)
+      @flickr.urls.get_group("34427469792@N01").should == "http://www.flickr.com/groups/central/"
+    end
+
+    it "should raise an error if the group_id is not valid (Group not found error raised)" do
+      xml = File.read(File.dirname(__FILE__) + "/../fixtures/flickr/urls/get_group-fail-1.xml")
+      @flickr.should_receive(:request_over_http).and_return(xml)
+      lambda { @flickr.urls.get_group("foo") }.should raise_error(Flickr::Error, /^1:/)
+    end
+  end
+
+  describe ".get_user_photos" do
+    it "should return the url to a user's photos if the user_id is valid" do
+      xml = File.read(File.dirname(__FILE__) + "/../fixtures/flickr/urls/get_user_photos-0.xml")
+      @flickr.should_receive(:request_over_http).and_return(xml)
+      @flickr.urls.get_user_photos("80755658@N00").should == "http://www.flickr.com/photos/lvizard/"
+    end
+
+    it "should raise an error if the user_id is not valid (User not found error raised)" do
+      xml = File.read(File.dirname(__FILE__) + "/../fixtures/flickr/urls/get_user_photos-fail-1.xml")
+      @flickr.should_receive(:request_over_http).and_return(xml)
+      lambda { @flickr.urls.get_user_photos("foo") }.should raise_error(Flickr::Error, /^1:/)
+    end
+
+    it "should raise an error if the user_id is not specified (No user specified error raised)" do
+      xml = File.read(File.dirname(__FILE__) + "/../fixtures/flickr/urls/get_user_photos-fail-2.xml")
+      @flickr.should_receive(:request_over_http).and_return(xml)
+      lambda { @flickr.urls.get_user_photos(nil) }.should raise_error(Flickr::Error, /^2:/)
+    end
+  end
+
+  describe ".get_user_profile" do
+    it "should return the url to a user's profile if the user_id is valid" do
+      xml = File.read(File.dirname(__FILE__) + "/../fixtures/flickr/urls/get_user_profile-0.xml")
+      @flickr.should_receive(:request_over_http).and_return(xml)
+      @flickr.urls.get_user_profile("80755658@N00").should == "http://www.flickr.com/people/lvizard/"
+    end
+
+    it "should raise an error if the user_id is not valid (User not found error raised)" do
+      xml = File.read(File.dirname(__FILE__) + "/../fixtures/flickr/urls/get_user_profile-fail-1.xml")
+      @flickr.should_receive(:request_over_http).and_return(xml)
+      lambda { @flickr.urls.get_user_profile("foo") }.should raise_error(Flickr::Error, /^1:/)
+    end
+
+    it "should raise an error if the user_id is not specified (No user specified error raised)" do
+      xml = File.read(File.dirname(__FILE__) + "/../fixtures/flickr/urls/get_user_profile-fail-2.xml")
+      @flickr.should_receive(:request_over_http).and_return(xml)
+      lambda { @flickr.urls.get_user_profile(nil) }.should raise_error(Flickr::Error, /^2:/)
+    end
+  end
+
+  describe ".lookup_group" do
+    it "should return a group NSID, given the url to a group's page"
+
+    it "should return a group NSID, given the url to a group's photo pool"
+
+    it "should raise an error if the group_id is not valid (Group not found error raised)"
+  end
+
+  describe ".lookup_user" do
+    it "should return a user NSID, given the url to a user's photos"
+
+    it "should return a user NSID, given the url to a user's profile"
+
+    it "should raise an error if the user_id is not valid (User not found error raised)"
+
+    it "should raise an error if the user_id is not specified (No user specified error raised)"
+
+    describe "valid return value" do
+      before :all do
+        @valid_lookup_user_xml = File.read(File.dirname(__FILE__) + "/../fixtures/flickr/urls/lookup_user-0.xml")
+      end
+
+
+      it "should be kind of a String" do
+        @flickr.should_receive(:request_over_http).and_return(@valid_lookup_user_xml)
+        @flickr.urls.lookup_user("htp://www.flickr.com/photos/lvizard").should be_kind_of String
+      end
+
+      it "should have a username attribute containing the user's name" do
+        xml = File.read(File.dirname(__FILE__) + "/../fixtures/flickr/urls/lookup_user-0.xml")
+        @flickr.should_receive(:request_over_http).and_return(@valid_lookup_user_xml)
+        @flickr.urls.lookup_user("htp://www.flickr.com/photos/lvizard").username.should == "Maciej Biłas"
+      end
+    end
+  end
+
+end

data/spec/spec.opts

@@ -0,0 +1,4 @@
+--colour
+--format progress
+--loadby mtime
+--reverse

data/spec/spec_helper.rb

@@ -0,0 +1,20 @@
+require 'rubygems'
+gem 'rspec'
+require 'spec'
+ 
+$:.unshift(File.dirname(__FILE__) + '/../lib')
+require 'flickr_fu'
+
+class SpecHelper
+  def self.flickr
+    Flickr.new(:key => "2f7bf3aceba0ca2f11f9ad46e7552459", :secret => "17b59864beb29370")
+  end
+end
+
+class Hash
+  def strip_api_params!
+    delete :api_sig
+    delete :api_key
+    delete :api_token
+  end
+end

metadata

@@ -1,15 +1,16 @@
 --- !ruby/object:Gem::Specification 
 name: flickr_fu
 version: !ruby/object:Gem::Version 
-  version: 0.1.6
+  version: 0.3.0
 platform: ruby
 authors: 
 - Ben Wyrosdick
+- Maciej Bilas
 autorequire: 
 bindir: bin
 cert_chain: []
 
-date: 2008-09-12 00:00:00 -05:00
+date: 2009-05-19 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -41,28 +42,77 @@ extensions: []
 extra_rdoc_files: 
 - README
 files: 
-- README
+- .gitignore
 - LICENSE
+- README
 - Rakefile
+- VERSION.yml
 - flickr_fu.gemspec
 - lib/flickr/auth.rb
 - lib/flickr/base.rb
 - lib/flickr/comment.rb
+- lib/flickr/contact.rb
+- lib/flickr/contacts.rb
+- lib/flickr/errors.rb
+- lib/flickr/geo.rb
 - lib/flickr/license.rb
+- lib/flickr/location.rb
 - lib/flickr/note.rb
 - lib/flickr/people.rb
 - lib/flickr/person.rb
 - lib/flickr/photo.rb
 - lib/flickr/photo_response.rb
 - lib/flickr/photos.rb
+- lib/flickr/photoset.rb
+- lib/flickr/photosets.rb
 - lib/flickr/size.rb
 - lib/flickr/status.rb
 - lib/flickr/test.rb
 - lib/flickr/token.rb
 - lib/flickr/uploader.rb
+- lib/flickr/urls.rb
 - lib/flickr_fu.rb
+- spec/fixtures/flickr/contacts/get_list-fail-99.xml
+- spec/fixtures/flickr/contacts/get_public_list-0.xml
+- spec/fixtures/flickr/photos/geo/get_location-0.xml
+- spec/fixtures/flickr/photos/geo/get_location-fail-2.xml
+- spec/fixtures/flickr/photos/get_info-0.xml
+- spec/fixtures/flickr/photos/get_info-1.xml
+- spec/fixtures/flickr/photos/get_sizes-0.xml
+- spec/fixtures/flickr/photos/get_sizes-1.xml
+- spec/fixtures/flickr/photos/licenses/get_info.xml
+- spec/fixtures/flickr/photosets/get_list-0.xml
+- spec/fixtures/flickr/photosets/get_photos-0.xml
+- spec/fixtures/flickr/test/echo-0.xml
+- spec/fixtures/flickr/test/null-fail-99.xml
+- spec/fixtures/flickr/urls/get_group-0.xml
+- spec/fixtures/flickr/urls/get_group-fail-1.xml
+- spec/fixtures/flickr/urls/get_user_photos-0.xml
+- spec/fixtures/flickr/urls/get_user_photos-fail-1.xml
+- spec/fixtures/flickr/urls/get_user_photos-fail-2.xml
+- spec/fixtures/flickr/urls/get_user_profile-0.xml
+- spec/fixtures/flickr/urls/get_user_profile-fail-1.xml
+- spec/fixtures/flickr/urls/get_user_profile-fail-2.xml
+- spec/fixtures/flickr/urls/lookup_group-0.xml
+- spec/fixtures/flickr/urls/lookup_group-fail-1.xml
+- spec/fixtures/flickr/urls/lookup_user-0.xml
+- spec/fixtures/flickr/videos/get_info-0.xml
+- spec/fixtures/flickr/videos/get_sizes-0.xml
+- spec/flickr/base_spec.rb
+- spec/flickr/contacts_spec.rb
+- spec/flickr/errors_spec.rb
+- spec/flickr/geo_spec.rb
+- spec/flickr/photo_spec.rb
+- spec/flickr/photos_spec.rb
+- spec/flickr/photosets_spec.rb
+- spec/flickr/test_spec.rb
+- spec/flickr/urls_spec.rb
+- spec/spec.opts
+- spec/spec_helper.rb
 has_rdoc: true
 homepage: http://github.com/commonthread/flickr_fu
+licenses: []
+
 post_install_message: 
 rdoc_options: 
 - --main
@@ -84,9 +134,18 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.2.0
+rubygems_version: 1.3.5
 signing_key: 
-specification_version: 2
+specification_version: 3
 summary: Provides a ruby interface to flickr via the REST api
-test_files: []
-
+test_files: 
+- spec/spec_helper.rb
+- spec/flickr/test_spec.rb
+- spec/flickr/geo_spec.rb
+- spec/flickr/contacts_spec.rb
+- spec/flickr/urls_spec.rb
+- spec/flickr/errors_spec.rb
+- spec/flickr/base_spec.rb
+- spec/flickr/photo_spec.rb
+- spec/flickr/photosets_spec.rb
+- spec/flickr/photos_spec.rb