koala 0.10.0 → 1.0.0

data/spec/{koala/test_users/test_users_tests.rb → cases/test_users_spec.rb}

@@ -1,7 +1,7 @@
-class TestUsersTests < Test::Unit::TestCase
-  include Koala
+require 'spec_helper'
 
-  describe "Koala TestUsers with access token" do
+describe "Koala::Facebook::TestUsers" do
+  context "with access token" do
     include LiveTestingDataHelper
 
     before :all do
@@ -10,40 +10,40 @@ class TestUsersTests < Test::Unit::TestCase
       @app_id = @oauth_data["app_id"]
       @secret = @oauth_data["secret"]
       @app_access_token = @oauth_data["app_access_token"]
-      
+
       # check OAuth data
       unless @app_id && @secret && @app_access_token
-        raise Exception, "Must supply OAuth app id, secret, app_access_token, and callback to run live subscription tests!" 
+        raise Exception, "Must supply OAuth app id, secret, app_access_token, and callback to run live subscription tests!"
       end
-      
+
       @is_mock = defined?(Koala::IS_MOCK) && Koala::IS_MOCK
     end
-          
+
     describe "when initializing" do
       # basic initialization
       it "should initialize properly with an app_id and an app_access_token" do
-        test_users = Facebook::TestUsers.new(:app_id => @app_id, :app_access_token => @app_access_token)
-        test_users.should be_a(Facebook::TestUsers)
+        test_users = Koala::Facebook::TestUsers.new(:app_id => @app_id, :app_access_token => @app_access_token)
+        test_users.should be_a(Koala::Facebook::TestUsers)
       end
-      
+
       # init with secret / fetching the token
-      it "should initialize properly with an app_id and a secret" do 
-        test_users = Facebook::TestUsers.new(:app_id => @app_id, :secret => @secret)
-        test_users.should be_a(Facebook::TestUsers)      
+      it "should initialize properly with an app_id and a secret" do
+        test_users = Koala::Facebook::TestUsers.new(:app_id => @app_id, :secret => @secret)
+        test_users.should be_a(Koala::Facebook::TestUsers)
       end
-      
+
       it "should use the OAuth class to fetch a token when provided an app_id and a secret" do
-        oauth = Facebook::OAuth.new(@app_id, @secret)
+        oauth = Koala::Facebook::OAuth.new(@app_id, @secret)
         token = oauth.get_app_access_token
         oauth.should_receive(:get_app_access_token).and_return(token)
-        Facebook::OAuth.should_receive(:new).with(@app_id, @secret).and_return(oauth) 
-        test_users = Facebook::TestUsers.new(:app_id => @app_id, :secret => @secret)
+        Koala::Facebook::OAuth.should_receive(:new).with(@app_id, @secret).and_return(oauth)
+        test_users = Koala::Facebook::TestUsers.new(:app_id => @app_id, :secret => @secret)
       end
     end
-    
+
     describe "when used without network" do
       before :each do
-        @test_users = Facebook::TestUsers.new({:app_access_token => @app_access_token, :app_id => @app_id})
+        @test_users = Koala::Facebook::TestUsers.new({:app_access_token => @app_access_token, :app_id => @app_id})
       end
 
       # TEST USER MANAGEMENT
@@ -53,31 +53,43 @@ class TestUsersTests < Test::Unit::TestCase
         result.should be_a(Hash)
         (result["id"] && result["access_token"] && result["login_url"]).should
       end
-      
+
       it "should create a test user when not given installed, ignoring permissions" do
         result = @test_users.create(false, "read_stream")
         @temporary_object_id = result["id"]
         result.should be_a(Hash)
         (result["id"] && result["access_token"] && result["login_url"]).should
       end
-      
+
       it "should accept permissions as a string" do
-        @test_users.graph_api.should_receive(:graph_call).with(anything, hash_including("permissions" => "read_stream,publish_stream"), anything)
+        @test_users.graph_api.should_receive(:graph_call).with(anything, hash_including("permissions" => "read_stream,publish_stream"), anything, anything)
         result = @test_users.create(true, "read_stream,publish_stream")
       end
-      
+
       it "should accept permissions as an array" do
-        @test_users.graph_api.should_receive(:graph_call).with(anything, hash_including("permissions" => "read_stream,publish_stream"), anything)
+        @test_users.graph_api.should_receive(:graph_call).with(anything, hash_including("permissions" => "read_stream,publish_stream"), anything, anything)
         result = @test_users.create(true, ["read_stream", "publish_stream"])
       end
-      
+
       it "should create a test user when given installed and a permission" do
         result = @test_users.create(true, "read_stream")
         @temporary_object_id = result["id"]
         result.should be_a(Hash)
         (result["id"] && result["access_token"] && result["login_url"]).should
       end
-            
+
+      it "lets you specify other graph arguments, like uid and access token" do
+        args = {:uid => "some test user ID", :owner_access_token => "some owner access token"}
+        @test_users.graph_api.should_receive(:graph_call).with(anything, hash_including(args), anything, anything)
+        @test_users.create(true, nil, args)
+      end
+      
+      it "lets you specify http options that get passed through to the graph call" do
+        options = {:some_http_option => true}
+        @test_users.graph_api.should_receive(:graph_call).with(anything, anything, anything, options)
+        @test_users.create(true, nil, {}, options)
+      end
+
       describe "with a user to delete" do
         before :each do
           @user1 = @test_users.create(true, "read_stream")
@@ -85,27 +97,25 @@ class TestUsersTests < Test::Unit::TestCase
         end
 
         after :each do
-          print "\nCleaning up test users..."
           @test_users.delete(@user1) if @user1
-          @test_users.delete(@user2) if @user2 
-          puts "done."
+          @test_users.delete(@user2) if @user2
         end
-        
+
         it "should delete a user by id" do
           @test_users.delete(@user1['id']).should be_true
           @user1 = nil
         end
-        
+
         it "should delete a user by hash" do
           @test_users.delete(@user2).should be_true
           @user2 = nil
         end
-        
+
         it "should not delete users when provided a false ID" do
           lambda { @test_users.delete("#{@user1['id']}1") }.should raise_exception(Koala::Facebook::APIError)
         end
       end
-      
+
       describe "with delete_all" do
         it "should delete all users found by the list commnand" do
           array = [1, 2, 3]
@@ -114,18 +124,18 @@ class TestUsersTests < Test::Unit::TestCase
           @test_users.delete_all
         end
       end
-      
+
       describe "with existing users" do
         before :each do
           @user1 = @test_users.create(true, "read_stream")
           @user2 = @test_users.create(true, "read_stream,user_interests")
         end
-        
+
         after :each do
           @test_users.delete(@user1)
           @test_users.delete(@user2)
         end
-      
+
         it "should list test users" do
           result = @test_users.list
           result.should be_an(Array)
@@ -133,67 +143,64 @@ class TestUsersTests < Test::Unit::TestCase
           (first_user["id"] && first_user["access_token"] && first_user["login_url"]).should
           (second_user["id"] && second_user["access_token"] && second_user["login_url"]).should
         end
-        
-        it "should make two users into friends by id" do
-          result = @test_users.befriend(@user1['id'], @user2['id'])
+      
+        it "should make two users into friends with string hashes" do
+          result = @test_users.befriend(@user1, @user2)
           result.should be_true
         end
+      
+        it "should make two users into friends with symbol hashes" do
+          new_user_1 = {}
+          @user1.each_pair {|k, v| new_user_1[k.to_sym] = v}
+          new_user_2 = {}
+          @user2.each_pair {|k, v| new_user_2[k.to_sym] = v}
         
-        it "should make two users into friends by hash" do
-          result = @test_users.befriend(@user1, @user2)
+          result = @test_users.befriend(new_user_1, new_user_2)
           result.should be_true
+        end        
+      
+        it "should not accept user IDs anymore" do
+          lambda { @test_users.befriend(@user1["id"], @user2["id"]) }.should raise_exception
         end
-        
       end # with existing users
-      
+
     end # when used without network
-    
-    describe "when creating a network of friends" do    
+
+    describe "when creating a network of friends" do
       before :each do
-        @test_users = Facebook::TestUsers.new({:app_access_token => @app_access_token, :app_id => @app_id})
+        @test_users = Koala::Facebook::TestUsers.new({:app_access_token => @app_access_token, :app_id => @app_id})
         @network = []
-        
+
         if @is_mock
           id_counter = 999999900
           @test_users.stub!(:create).and_return do
             id_counter += 1
-            {"id" => id_counter, "access_token" => "119908831367602|o3wswWQ88LYjEC9-ukR_gjRIOMw.", "login_url" => "https://www.facebook.com/platform/test_account.."}
+            {"id" => id_counter, "access_token" => @token, "login_url" => "https://www.facebook.com/platform/test_account.."}
           end
           @test_users.stub!(:befriend).and_return(true)
           @test_users.stub!(:delete).and_return(true)
         end
       end
-      
+
       describe "tests that create users" do
         before :each do
-          print "\nCleaning up test user network..."
-          test_users = Facebook::TestUsers.new({:app_access_token => @app_access_token, :app_id => @app_id})
+          test_users = Koala::Facebook::TestUsers.new({:app_access_token => @app_access_token, :app_id => @app_id})
           test_users.delete_all
-          puts "done!"
         end
 
         after :each do
-          print "\nCleaning up test user network..."
-          test_users = Facebook::TestUsers.new({:app_access_token => @app_access_token, :app_id => @app_id})
+          test_users = Koala::Facebook::TestUsers.new({:app_access_token => @app_access_token, :app_id => @app_id})
           test_users.delete_all
-          puts "done!"
-        end
-        
-        it "should create a 2 person network" do
-          @network = @test_users.create_network(2)
-          @network.should be_a(Array)
-          @network.size.should == 2
         end
-      
-        it "should create a 50 person network" do
-          puts "\nStarting 50-person network test (this may take several minutes)..."
-          @network = @test_users.create_network(50)
+
+        it "should create a 5 person network" do
+          size = 5
+          @network = @test_users.create_network(size)
           @network.should be_a(Array)
-          @network.size.should == 50
-          puts "done!"
+          @network.size.should == size
         end
       end
-      
+
       it "should limit to a 50 person network" do
         @test_users.should_receive(:create).exactly(50).times
         @test_users.stub!(:befriend)
@@ -208,8 +215,7 @@ class TestUsersTests < Test::Unit::TestCase
         @test_users.stub!(:befriend)
         @network = @test_users.create_network(count, installed, perms)
       end
-      
-    end # when creating network
 
-  end  # describe Koala TestUsers
-end # class
+    end # when creating network
+  end
+end  # describe Koala TestUsers

data/spec/cases/uploadable_io_spec.rb

@@ -0,0 +1,151 @@
+# fake MIME::Types
+module Koala::MIME
+  module Types
+    def self.type_for(type)
+      # this should be faked out in tests
+      nil
+    end
+  end
+end
+
+describe "Koala::UploadableIO" do
+  describe "the constructor" do
+    describe "when given a file path" do
+      before(:each) do
+        @koala_io_params = [File.open(BEACH_BALL_PATH)]
+      end
+
+      describe "and a content type" do
+        before :each do
+          @koala_io_params.concat([stub("image/jpg")])
+        end
+
+        it "should return an UploadIO with the same file path" do
+          stub_path = @koala_io_params[0] = "/stub/path/to/file"
+          Koala::UploadableIO.new(*@koala_io_params).io_or_path.should == stub_path
+        end
+
+        it "should return an UploadIO with the same content type" do
+          stub_type = @koala_io_params[1] = stub('Content Type')
+          Koala::UploadableIO.new(*@koala_io_params).content_type.should == stub_type
+        end
+      end
+
+      describe "and no content type" do
+        it_should_behave_like "determining a mime type"
+      end
+    end
+
+    describe "when given a File object" do
+      before(:each) do
+        @koala_io_params = [File.open(BEACH_BALL_PATH)]
+      end
+
+      describe "and a content type" do
+        before :each do
+          @koala_io_params.concat(["image/jpg"])
+        end
+
+        it "should return an UploadIO with the same io" do
+          Koala::UploadableIO.new(*@koala_io_params).io_or_path.should == @koala_io_params[0]
+        end
+
+        it "should return an UplaodIO with the same content_type" do
+          content_stub = @koala_io_params[1] = stub('Content Type')
+          Koala::UploadableIO.new(*@koala_io_params).content_type.should == content_stub
+        end
+      end
+
+      describe "and no content type" do
+        it_should_behave_like "determining a mime type"
+      end
+    end
+
+    describe "when given a Rails 3 ActionDispatch::Http::UploadedFile" do
+      before(:each) do
+        @tempfile = stub('Tempfile', :path => "foo")
+        @uploaded_file = stub('ActionDispatch::Http::UploadedFile',
+          :content_type => true,
+          :tempfile => @tempfile
+        )
+
+        @tempfile.stub!(:respond_to?).with(:path).and_return(true)
+      end
+
+      it "should get the content type via the content_type method" do
+        expected_content_type = stub('Content Type')
+        @uploaded_file.should_receive(:content_type).and_return(expected_content_type)
+        Koala::UploadableIO.new(@uploaded_file).content_type.should == expected_content_type
+      end
+
+      it "should get the path from the tempfile associated with the UploadedFile" do
+        expected_path = stub('Tempfile')
+        @tempfile.should_receive(:path).and_return(expected_path)
+        Koala::UploadableIO.new(@uploaded_file).io_or_path.should == expected_path
+      end
+    end
+
+    describe "when given a Sinatra file parameter hash" do
+      before(:each) do
+        @file_hash = {
+          :type => "type",
+          :tempfile => "Tempfile"
+        }
+      end
+
+      it "should get the content type from the :type key" do
+        expected_content_type = stub('Content Type')
+        @file_hash[:type] = expected_content_type
+
+        uploadable = Koala::UploadableIO.new(@file_hash)
+        uploadable.content_type.should == expected_content_type
+      end
+
+      it "should get the io_or_path from the :tempfile key" do
+        expected_file = stub('File')
+        @file_hash[:tempfile] = expected_file
+
+        uploadable = Koala::UploadableIO.new(@file_hash)
+        uploadable.io_or_path.should == expected_file
+      end
+    end
+
+    describe "for files with with recognizable MIME types" do
+      # what that means is tested below
+      it "should accept a file object alone" do
+        params = [BEACH_BALL_PATH]
+        lambda { Koala::UploadableIO.new(*params) }.should_not raise_exception(Koala::KoalaError)
+      end
+
+      it "should accept a file path alone" do
+        params = [BEACH_BALL_PATH]
+        lambda { Koala::UploadableIO.new(*params) }.should_not raise_exception(Koala::KoalaError)
+      end
+    end
+  end
+
+  describe "getting an UploadableIO" do
+    before(:each) do
+      @upload_io = stub("UploadIO")
+      UploadIO.stub!(:new).with(anything, anything, anything).and_return(@upload_io)
+    end
+
+    it "should call the constructor with the content type, file name, and a dummy file name" do
+      UploadIO.should_receive(:new).with(BEACH_BALL_PATH, "content/type", anything).and_return(@upload_io)
+      Koala::UploadableIO.new(BEACH_BALL_PATH, "content/type").to_upload_io.should == @upload_io
+    end
+  end
+
+  describe "getting a file" do
+    it "should return the File if initialized with a file" do
+      f = File.new(BEACH_BALL_PATH)
+      Koala::UploadableIO.new(f).to_file.should == f
+    end
+
+    it "should open up and return a file corresponding to the path if io_or_path is a path" do
+      result = stub("File")
+      File.should_receive(:open).with(BEACH_BALL_PATH).and_return(result)
+      Koala::UploadableIO.new(BEACH_BALL_PATH).to_file.should == result
+    end
+  end
+end  # describe UploadableIO

data/spec/{facebook_data.yml → fixtures/facebook_data.yml}

@@ -4,7 +4,7 @@
 # Just remember to update all fields!
  
 # You must supply this value yourself to test the GraphAPI class.
-# Your OAuth token should have publish_stream and read_stream permissions.
+# Your OAuth token should have publish_stream, read_stream, and user_photos permissions.
 oauth_token: 
 
 # for testing the OAuth class    
@@ -35,17 +35,21 @@ oauth_test_data:
     # note: I've revoked the offline access for security reasons, so you can't make calls against this :)
     fbs_119908831367602: '"access_token=119908831367602|08170230801eb225068e7a70-2905623|Q3LDCYYF8CX9cstxnZLsxiR0nwg.&expires=0&secret=78abaee300b392e275072a9f2727d436&session_key=08170230801eb225068e7a70-2905623&sig=423b8aa4b6fa1f9a571955f8e929d567&uid=2905623"'
 
-  # These values will work out of the box
-  # They're from Facebook's example at http://developers.facebook.com/docs/authentication/canvas
+  # These values from the OAuth Playground (see above) will work out of the box
   # You can update this to live data if desired
   # request_secret is optional and will fall back to the secret above if absent
-  request_secret: "secret"
-  signed_request: "vlXgu64BQGFSQrY0ZcJBZASMvYvTHu9GQ0YM9rjPSso.eyJhbGdvcml0aG0iOiJITUFDLVNIQTI1NiIsIjAiOiJwYXlsb2FkIn0"
-  signed_request_result:
-      "0": payload
+  signed_params: "zWRm0gd5oHW_jzXP_WA9CirO7c5CLHotn-SKRqH2NmU.eyJhbGdvcml0aG0iOiJITUFDLVNIQTI1NiIsImV4cGlyZXMiOjEzMDE5MjIwMDAsImlzc3VlZF9hdCI6MTMwMTkxNzI5OSwib2F1dGhfdG9rZW4iOiIxMTk5MDg4MzEzNjc2MDJ8Mi56VkZfNk5yTUVMSHVKYTRnSVU5dEt3X18uMzYwMC4xMzAxOTIyMDAwLTI5MDU2MjN8emdxUHNyMkJHOUxvT0s5a2VrR2dSVVJaeDBrIiwidXNlciI6eyJjb3VudHJ5IjoiZGUiLCJsb2NhbGUiOiJkZV9ERSIsImFnZSI6eyJtaW4iOjIxfX0sInVzZXJfaWQiOiIyOTA1NjIzIn0"
+  signed_params_result:
+      expires: 1301922000
       algorithm: HMAC-SHA256
-    
-  
+      user_id: "2905623"
+      oauth_token: 119908831367602|2.zVF_6NrMELHuJa4gIU9tKw__.3600.1301922000-2905623|zgqPsr2BG9LoOK9kekGgRURZx0k
+      user: 
+        country: de
+        locale: de_DE
+        age: 
+          min: 21
+      issued_at: 1301917299
 
 subscription_test_data:
   subscription_path: http://oauth.twoalex.com/subscriptions