maciej-flickr_fu 0.3.0

data/spec/flickr/photo_spec.rb

@@ -0,0 +1,123 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+module PhotoSpecHelper
+  def valid_photo_attributes
+    {
+      :id => 2984637736,
+      :owner => "80755658@N00",
+      :secret => "9e5762e027",
+      :server => 3180,
+      :farm => 4,
+      :title => "Demolition of Hotel Rzeszów",
+      :is_public => 1,
+      :is_friend => 0,
+      :is_family => 0
+    }
+  end
+end
+
+describe Flickr::Photos::Photo do
+  
+  include PhotoSpecHelper
+  
+  before :all do
+    @info_xml = File.read(File.dirname(__FILE__) + "/../fixtures/flickr/photos/get_info-0.xml")
+    @sizes_xml = File.read(File.dirname(__FILE__) + "/../fixtures/flickr/photos/get_sizes-0.xml")
+  end
+  
+  before :each do
+    @flickr = SpecHelper.flickr
+    #@flickr.stub!(:request_over_http).and_return(info_xml)
+    @photo = Flickr::Photos::Photo.new(@flickr, valid_photo_attributes)
+  end
+  
+  describe ".description" do
+    it "should return the description" do
+      @flickr.should_receive(:request_over_http).and_return(@info_xml)
+      @photo.description.should == 
+        "The last picture from a quite old event. The demolition of the best known hotel in Rzeszów called Hotel Rzeszów."
+    end
+  end
+  
+  describe ".image_url" do
+    it "should return all standard sizes (thumbnail, square, small, medium and large) when requested" do
+	  @photo.image_url(:square).should == "http://farm4.static.flickr.com/3180/2984637736_9e5762e027_s.jpg"
+      @photo.image_url(:thumbnail).should == "http://farm4.static.flickr.com/3180/2984637736_9e5762e027_t.jpg"
+      @photo.image_url(:small).should == "http://farm4.static.flickr.com/3180/2984637736_9e5762e027_m.jpg"
+	  @photo.image_url(:medium).should == "http://farm4.static.flickr.com/3180/2984637736_9e5762e027.jpg"
+	  @photo.image_url(:large).should == "http://farm4.static.flickr.com/3180/2984637736_9e5762e027_b.jpg"
+    end
+
+	it "should return the same image even if you pass a string as an argument" do
+	  @photo.image_url("square").should == @photo.image_url(:square)
+	  @photo.image_url("large").should == @photo.image_url(:large)
+	end
+
+	it "should not call getSizes if not requested the url of the original image" do
+	  @flickr.should_not_receive(:request_over_http)
+	  @photo.image_url :square
+	  @photo.image_url :thumbnail
+	  @photo.image_url :small
+	  @photo.image_url :medium
+	  @photo.image_url :large
+	end
+
+	it "should call getSizes if requested the url of the original image" do
+	  @flickr.should_receive(:request_over_http).and_return(@sizes_xml)
+	  @photo.image_url :original
+	end
+
+	it "should return nil if original image url not available" do
+	  @flickr.should_receive(:request_over_http).and_return(@sizes_xml)
+	  @photo.image_url(:original).should == nil
+	end
+  end
+
+  describe ".video_url" do
+    it "should return nil, since it's not a video" do
+      @flickr.should_receive(:request_over_http).and_return(@sizes_xml)
+      @photo.video_url.should == nil
+    end
+  end
+
+  describe ".location" do
+    it "should return the picture location as specified in flickr.photos.geo.getLocation" do
+      location_xml = File.read(File.dirname(__FILE__) + "/../fixtures/flickr/photos/geo/get_location-0.xml")
+      @flickr.should_receive(:request_over_http).twice.and_return(location_xml)
+      expected_loc = @flickr.photos.geo.get_location(@photo.id)
+      acctual_loc = @photo.location
+      # Checking by latitude and longitude is enough for us
+      acctual_loc.latitude.should == expected_loc.latitude
+      acctual_loc.longitude.should == expected_loc.longitude
+    end
+
+    it "should return nil if picture is not geo-tagged" do
+      no_location_xml = File.read(File.dirname(__FILE__) + "/../fixtures/flickr/photos/geo/get_location-fail-2.xml")
+      @flickr.should_receive(:request_over_http).and_return(no_location_xml)
+      @photo.location.should be_nil
+    end
+  end
+
+  describe Flickr::Photos::Photo, "but it's really a video" do
+
+    include PhotoSpecHelper
+  
+    before :all do
+      @info_xml = File.read(File.dirname(__FILE__) + "/../fixtures/flickr/videos/get_info-0.xml")
+      @sizes_xml = File.read(File.dirname(__FILE__) + "/../fixtures/flickr/videos/get_sizes-0.xml")
+    end
+  
+    before :each do
+      @flickr = SpecHelper.flickr
+      #@flickr.stub!(:request_over_http).and_return(info_xml)
+      @photo = Flickr::Photos::Photo.new(@flickr, valid_photo_attributes)
+    end
+
+    describe ".video_url" do
+      it "should return something sane" do
+        @flickr.should_receive(:request_over_http).and_return(@sizes_xml)
+        @photo.video_url.should == "http://www.flickr.com/apps/video/stewart.swf?v=63881&photo_id=2729556270&photo_secret=eee23fb14a"
+      end
+    end
+  end
+end

data/spec/flickr/photos_spec.rb

@@ -0,0 +1,50 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe Flickr::Photos do
+  
+  before :each do
+    @flickr = SpecHelper.flickr
+  end
+  
+  describe ".licenses" do
+    before :all do
+      @licenses_xml = File.read(File.dirname(__FILE__) + "/../fixtures/flickr/photos/licenses/get_info.xml")
+      @valid_licenses = {
+        0 => Flickr::Photos::License.new(:id => 0, :name => "All Rights Reserved", :url => ""),
+        4 => Flickr::Photos::License.new(:id => 4, :name => "Attribution License", :url => "http://creativecommons.org/licenses/by/2.0/"),
+        6 => Flickr::Photos::License.new(:id => 6, :name => "Attribution-NoDerivs License", :url => "http://creativecommons.org/licenses/by-nd/2.0/"),
+        3 => Flickr::Photos::License.new(:id => 3, :name => "Attribution-NonCommercial-NoDerivs License", :url => "http://creativecommons.org/licenses/by-nc-nd/2.0/"),
+        2 => Flickr::Photos::License.new(:id => 2, :name => "Attribution-NonCommercial License", :url => "http://creativecommons.org/licenses/by-nc/2.0/"),
+        1 => Flickr::Photos::License.new(:id => 1, :name => "Attribution-NonCommercial-ShareAlike License", :url => "http://creativecommons.org/licenses/by-nc-sa/2.0/"),
+        5 => Flickr::Photos::License.new(:id => 5, :name => "Attribution-ShareAlike License", :url => "http://creativecommons.org/licenses/by-sa/2.0/")
+      }
+    end
+    
+    it "should return all valid licenses" do
+      @flickr.should_receive(:request_over_http).and_return(@licenses_xml)
+      @flickr.photos.licenses.should == @valid_licenses
+    end
+    
+    it "should not not call Flickr API more than once" do
+      @flickr.should_receive(:request_over_http).once.and_return(@licenses_xml)
+      @flickr.photos.licenses
+      @flickr.photos.licenses
+    end
+  end
+  
+  describe ".find_by_id" do
+    it "should return a photo when a valid id is given" do
+      photo_id = 2984637736
+      #photo_upload_date = 1225297614
+      info_xml = File.read(File.dirname(__FILE__) + "/../fixtures/flickr/photos/get_info-0.xml")
+      @flickr.should_receive(:request_over_http).and_return(info_xml)
+      
+      photo = @flickr.photos.find_by_id(photo_id)
+      photo.id.should == photo_id
+    end
+    
+    it "should raise an error when no id is given"
+    
+    it "should raise an error when the photo does not exist"
+  end
+end

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

@@ -0,0 +1,149 @@
+--- !ruby/object:Gem::Specification 
+name: maciej-flickr_fu
+version: !ruby/object:Gem::Version 
+  version: 0.3.0
+platform: ruby
+authors: 
+- Ben Wyrosdick
+- Maciej Bilas
+autorequire: 
+bindir: bin
+cert_chain: []
+
+date: 2009-05-19 00:00:00 -07:00
+default_executable: 
+dependencies: 
+- !ruby/object:Gem::Dependency 
+  name: mime-types
+  type: :runtime
+  version_requirement: 
+  version_requirements: !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ">"
+      - !ruby/object:Gem::Version 
+        version: 0.0.0
+    version: 
+- !ruby/object:Gem::Dependency 
+  name: xml-magic
+  type: :runtime
+  version_requirement: 
+  version_requirements: !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ">"
+      - !ruby/object:Gem::Version 
+        version: 0.0.0
+    version: 
+description: Provides a ruby interface to flickr via the REST api
+email: ben@commonthread.com
+executables: []
+
+extensions: []
+
+extra_rdoc_files: 
+- README
+files: 
+- .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: false
+homepage: http://github.com/commonthread/flickr_fu
+post_install_message: 
+rdoc_options: 
+- --main
+- README
+require_paths: 
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement 
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      version: "0"
+  version: 
+required_rubygems_version: !ruby/object:Gem::Requirement 
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      version: "0"
+  version: 
+requirements: []
+
+rubyforge_project: 
+rubygems_version: 1.2.0
+signing_key: 
+specification_version: 3
+summary: Provides a ruby interface to flickr via the REST api
+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