koala 1.2.0beta1 → 1.2.1

data/spec/cases/oauth_spec.rb

@@ -30,21 +30,21 @@ describe "Koala::Facebook::OAuth" do
   end
 
   # initialization
-  it "should properly initialize" do
+  it "properly initializes" do
     @oauth.should
   end
 
-  it "should properly set attributes" do
+  it "properly sets attributes" do
     (@oauth.app_id == @app_id &&
       @oauth.app_secret == @secret &&
       @oauth.oauth_callback_url == @callback_url).should be_true
   end
 
-  it "should properly initialize without a callback_url" do
+  it "properly initializes without a callback_url" do
     @oauth = Koala::Facebook::OAuth.new(@app_id, @secret)
   end
 
-  it "should properly set attributes without a callback URL" do
+  it "properly sets attributes without a callback URL" do
     @oauth = Koala::Facebook::OAuth.new(@app_id, @secret)
     (@oauth.app_id == @app_id &&
       @oauth.app_secret == @secret &&
@@ -53,60 +53,158 @@ describe "Koala::Facebook::OAuth" do
 
   describe "for cookie parsing" do
     describe "get_user_info_from_cookies" do
-      it "should properly parse valid cookies" do
-        result = @oauth.get_user_info_from_cookies(KoalaTest.oauth_test_data["valid_cookies"])
-        result.should be_a(Hash)
-      end
+      context "for signed cookies" do
+        before :each do
+          # we don't actually want to make requests to Facebook to redeem the code
+          @cookie = KoalaTest.oauth_test_data["valid_signed_cookies"]
+          @token = "my token"
+          @oauth.stub(:get_access_token_info).and_return("access_token" => @token)
+        end
+        
+        it "parses valid cookies" do
+          result = @oauth.get_user_info_from_cookies(@cookie)
+          result.should be_a(Hash)
+        end
 
-      it "should return all the cookie components from valid cookie string" do
-        cookie_data = KoalaTest.oauth_test_data["valid_cookies"]
-        parsing_results = @oauth.get_user_info_from_cookies(cookie_data)
-        number_of_components = cookie_data["fbs_#{@app_id.to_s}"].scan(/\=/).length
-        parsing_results.length.should == number_of_components
-      end
+        it "returns all the components in the signed request" do
+          result = @oauth.get_user_info_from_cookies(@cookie)
+          @oauth.parse_signed_request(@cookie.values.first).each_pair do |k, v|
+            result[k].should == v
+          end          
+        end
+        
+        it "makes a request to Facebook to redeem the code if present" do
+          code = "foo"
+          @oauth.stub(:parse_signed_request).and_return({"code" => code})
+          @oauth.should_receive(:get_access_token_info).with(code, anything)
+          @oauth.get_user_info_from_cookies(@cookie)
+        end
 
-      it "should properly parse valid offline access cookies (e.g. no expiration)" do
-        result = @oauth.get_user_info_from_cookies(KoalaTest.oauth_test_data["offline_access_cookies"])
-        result["uid"].should
-      end
+        it "sets the code redemption redirect_uri to ''" do
+          @oauth.should_receive(:get_access_token_info).with(anything, :redirect_uri => '')
+          @oauth.get_user_info_from_cookies(@cookie)
+        end
 
-      it "should return all the cookie components from offline access cookies" do
-        cookie_data = KoalaTest.oauth_test_data["offline_access_cookies"]
-        parsing_results = @oauth.get_user_info_from_cookies(cookie_data)
-        number_of_components = cookie_data["fbs_#{@app_id.to_s}"].scan(/\=/).length
-        parsing_results.length.should == number_of_components
+        context "if the code is missing" do
+          it "doesn't make a request to Facebook" do
+            @oauth.stub(:parse_signed_request).and_return({})
+            @oauth.should_receive(:get_access_token_info).never
+            @oauth.get_user_info_from_cookies(@cookie)
+          end
+
+          it "returns nil" do
+            @oauth.stub(:parse_signed_request).and_return({})
+            @oauth.get_user_info_from_cookies(@cookie).should be_nil
+          end
+        end
+        
+        context "if the code is present" do
+          it "adds the access_token into the hash" do
+            @oauth.get_user_info_from_cookies(@cookie)["access_token"].should == @token
+          end
+          
+          it "returns nil if the call to FB fails" do
+            @oauth.stub(:get_access_token_info).and_return(nil)
+            @oauth.get_user_info_from_cookies(@cookie).should be_nil
+          end
+        end
+        
+        it "doesn't parse invalid cookies" do
+          # make an invalid string by replacing some values
+          bad_cookie_hash = @cookie.inject({}) { |hash, value| hash[value[0]] = value[1].gsub(/[0-9]/, "3") }
+          result = @oauth.get_user_info_from_cookies(bad_cookie_hash)
+          result.should be_nil
+        end
       end
+      
+      context "for unsigned cookies" do
+        it "properly parses valid cookies" do
+          result = @oauth.get_user_info_from_cookies(KoalaTest.oauth_test_data["valid_cookies"])
+          result.should be_a(Hash)
+        end
 
-      it "shouldn't parse expired cookies" do
-        result = @oauth.get_user_info_from_cookies(KoalaTest.oauth_test_data["expired_cookies"])
-        result.should be_nil
-      end
+        it "returns all the cookie components from valid cookie string" do
+          cookie_data = KoalaTest.oauth_test_data["valid_cookies"]
+          parsing_results = @oauth.get_user_info_from_cookies(cookie_data)
+          number_of_components = cookie_data["fbs_#{@app_id.to_s}"].scan(/\=/).length
+          parsing_results.length.should == number_of_components
+        end
+
+        it "properly parses valid offline access cookies (e.g. no expiration)" do
+          result = @oauth.get_user_info_from_cookies(KoalaTest.oauth_test_data["offline_access_cookies"])
+          result["uid"].should
+        end
+
+        it "returns all the cookie components from offline access cookies" do
+          cookie_data = KoalaTest.oauth_test_data["offline_access_cookies"]
+          parsing_results = @oauth.get_user_info_from_cookies(cookie_data)
+          number_of_components = cookie_data["fbs_#{@app_id.to_s}"].scan(/\=/).length
+          parsing_results.length.should == number_of_components
+        end
 
-      it "shouldn't parse invalid cookies" do
-        # make an invalid string by replacing some values
-        bad_cookie_hash = KoalaTest.oauth_test_data["valid_cookies"].inject({}) { |hash, value| hash[value[0]] = value[1].gsub(/[0-9]/, "3") }
-        result = @oauth.get_user_info_from_cookies(bad_cookie_hash)
-        result.should be_nil
+        it "doesn't parse expired cookies" do
+          new_time = @time.to_i * 2
+          @time.stub(:to_i).and_return(new_time)          
+          @oauth.get_user_info_from_cookies(KoalaTest.oauth_test_data["valid_cookies"]).should be_nil
+        end
+
+        it "doesn't parse invalid cookies" do
+          # make an invalid string by replacing some values
+          bad_cookie_hash = KoalaTest.oauth_test_data["valid_cookies"].inject({}) { |hash, value| hash[value[0]] = value[1].gsub(/[0-9]/, "3") }
+          result = @oauth.get_user_info_from_cookies(bad_cookie_hash)
+          result.should be_nil
+        end
       end
     end
 
     describe "get_user_from_cookies" do
-      it "should use get_user_info_from_cookies to parse the cookies" do
-        data = KoalaTest.oauth_test_data["valid_cookies"]
-        @oauth.should_receive(:get_user_info_from_cookies).with(data).and_return({})
-        @oauth.get_user_from_cookies(data)
-      end
+      describe "for signed cookies" do
+        before :each do
+          # we don't actually want to make requests to Facebook to redeem the code
+          @cookie = KoalaTest.oauth_test_data["valid_signed_cookies"]
+          @oauth.stub(:get_access_token_info).and_return("access_token" => "my token")          
+        end
 
-      it "should use return a string if the cookies are valid" do
-        result = @oauth.get_user_from_cookies(KoalaTest.oauth_test_data["valid_cookies"])
-        result.should be_a(String)
+        it "uses get_user_info_from_cookies to parse the cookies" do
+          @oauth.should_receive(:get_user_info_from_cookies).with(@cookie).and_return({})
+          @oauth.get_user_from_cookies(@cookie)
+        end
+
+        it "uses return the token string if the cookies are valid" do
+          result = @oauth.get_user_from_cookies(@cookie)
+          result.should == "2905623" # the user who generated the original test cookie
+        end
+
+        it "returns nil if the cookies are invalid" do
+          # make an invalid string by replacing some values
+          bad_cookie_hash = @cookie.inject({}) { |hash, value| hash[value[0]] = value[1].gsub(/[0-9]/, "3") }
+          result = @oauth.get_user_from_cookies(bad_cookie_hash)
+          result.should be_nil
+        end
       end
+      
+      describe "for unsigned cookies" do
+        before :each do
+          # we don't actually want to make requests to Facebook to redeem the code
+          @cookie = KoalaTest.oauth_test_data["valid_cookies"]
+        end
+
+        it "uses get_user_info_from_cookies to parse the cookies" do
+          @oauth.should_receive(:get_user_info_from_cookies).with(@cookie).and_return({})
+          @oauth.get_user_from_cookies(@cookie)
+        end
 
-      it "should return nil if the cookies are invalid" do
-        # make an invalid string by replacing some values
-        bad_cookie_hash = KoalaTest.oauth_test_data["valid_cookies"].inject({}) { |hash, value| hash[value[0]] = value[1].gsub(/[0-9]/, "3") }
-        result = @oauth.get_user_from_cookies(bad_cookie_hash)
-        result.should be_nil
+        it "uses return a string if the cookies are valid" do
+          result = @oauth.get_user_from_cookies(@cookie)
+          result.should == "2905623" # the user who generated the original test cookie
+        end
+
+        it "returns nil if the cookies are invalid" do
+          # make an invalid string by replacing some values
+          bad_cookie_hash = @cookie.inject({}) { |hash, value| hash[value[0]] = value[1].gsub(/[0-9]/, "3") }
+          result = @oauth.get_user_from_cookies(bad_cookie_hash)
+          result.should be_nil
+        end
       end
     end
   end
@@ -117,42 +215,42 @@ describe "Koala::Facebook::OAuth" do
 
     describe "for OAuth codes" do
       # url_for_oauth_code
-      it "should generate a properly formatted OAuth code URL with the default values" do
+      it "generates a properly formatted OAuth code URL with the default values" do
         url = @oauth.url_for_oauth_code
         url.should == "https://#{Koala::Facebook::GRAPH_SERVER}/oauth/authorize?client_id=#{@app_id}&redirect_uri=#{@callback_url}"
       end
 
-      it "should generate a properly formatted OAuth code URL when a callback is given" do
+      it "generates a properly formatted OAuth code URL when a callback is given" do
         callback = "foo.com"
         url = @oauth.url_for_oauth_code(:callback => callback)
         url.should == "https://#{Koala::Facebook::GRAPH_SERVER}/oauth/authorize?client_id=#{@app_id}&redirect_uri=#{callback}"
       end
 
-      it "should generate a properly formatted OAuth code URL when permissions are requested as a string" do
+      it "generates a properly formatted OAuth code URL when permissions are requested as a string" do
         permissions = "publish_stream,read_stream"
         url = @oauth.url_for_oauth_code(:permissions => permissions)
         url.should == "https://#{Koala::Facebook::GRAPH_SERVER}/oauth/authorize?client_id=#{@app_id}&redirect_uri=#{@callback_url}&scope=#{permissions}"
       end
 
-      it "should generate a properly formatted OAuth code URL when permissions are requested as a string" do
+      it "generates a properly formatted OAuth code URL when permissions are requested as a string" do
         permissions = ["publish_stream", "read_stream"]
         url = @oauth.url_for_oauth_code(:permissions => permissions)
         url.should == "https://#{Koala::Facebook::GRAPH_SERVER}/oauth/authorize?client_id=#{@app_id}&redirect_uri=#{@callback_url}&scope=#{permissions.join(",")}"
       end
 
-      it "should generate a properly formatted OAuth code URL when both permissions and callback are provided" do
+      it "generates a properly formatted OAuth code URL when both permissions and callback are provided" do
         permissions = "publish_stream,read_stream"
         callback = "foo.com"
         url = @oauth.url_for_oauth_code(:callback => callback, :permissions => permissions)
         url.should == "https://#{Koala::Facebook::GRAPH_SERVER}/oauth/authorize?client_id=#{@app_id}&redirect_uri=#{callback}&scope=#{permissions}"
       end
 
-      it "should generate a properly formatted OAuth code URL when a display is given as a string" do
+      it "generates a properly formatted OAuth code URL when a display is given as a string" do
         url = @oauth.url_for_oauth_code(:display => "page")
         url.should == "https://#{Koala::Facebook::GRAPH_SERVER}/oauth/authorize?client_id=#{@app_id}&redirect_uri=#{@callback_url}&display=page"
       end
 
-      it "should raise an exception if no callback is given in initialization or the call" do
+      it "raises an exception if no callback is given in initialization or the call" do
         oauth2 = Koala::Facebook::OAuth.new(@app_id, @secret)
         lambda { oauth2.url_for_oauth_code }.should raise_error(ArgumentError)
       end
@@ -165,12 +263,12 @@ describe "Koala::Facebook::OAuth" do
       end
 
       # url_for_access_token
-      it "should generate a properly formatted OAuth token URL when provided a code" do
+      it "generates a properly formatted OAuth token URL when provided a code" do
         url = @oauth.url_for_access_token(@code)
         url.should == "https://#{Koala::Facebook::GRAPH_SERVER}/oauth/access_token?client_id=#{@app_id}&redirect_uri=#{@callback_url}&client_secret=#{@secret}&code=#{@code}"
       end
 
-      it "should generate a properly formatted OAuth token URL when provided a callback" do
+      it "generates a properly formatted OAuth token URL when provided a callback" do
         callback = "foo.com"
         url = @oauth.url_for_access_token(@code, :callback => callback)
         url.should == "https://#{Koala::Facebook::GRAPH_SERVER}/oauth/access_token?client_id=#{@app_id}&redirect_uri=#{callback}&client_secret=#{@secret}&code=#{@code}"
@@ -179,61 +277,82 @@ describe "Koala::Facebook::OAuth" do
   end
 
   describe "for fetching access tokens" do
-    if KoalaTest.code
-      describe "get_access_token_info" do
-        it "should properly get and parse an access token token results into a hash" do
+    describe ".get_access_token_info" do
+      it "uses options[:redirect_uri] if provided" do
+        uri = "foo"
+        Koala.should_receive(:make_request).with(anything, hash_including(:redirect_uri => uri), anything, anything).and_return(Koala::Response.new(200, "", {}))
+        @oauth.get_access_token_info(@code, :redirect_uri => uri)
+      end
+      
+      it "uses the redirect_uri used to create the @oauth if no :redirect_uri option is provided" do
+        Koala.should_receive(:make_request).with(anything, hash_including(:redirect_uri => @callback_url), anything, anything).and_return(Koala::Response.new(200, "", {}))
+        @oauth.get_access_token_info(@code)        
+      end
+      
+      it "makes a GET request" do
+        Koala.should_receive(:make_request).with(anything, anything, "get", anything).and_return(Koala::Response.new(200, "", {}))
+        @oauth.get_access_token_info(@code)                
+      end
+
+      if KoalaTest.code
+        it "properly gets and parses an access token token results into a hash" do
           result = @oauth.get_access_token_info(@code)
           result.should be_a(Hash)
         end
 
-        it "should properly include the access token results" do
+        it "properly includes the access token results" do
           result = @oauth.get_access_token_info(@code)
           result["access_token"].should
         end
-
-        it "should raise an error when get_access_token is called with a bad code" do
+        
+        it "raises an error when get_access_token is called with a bad code" do
           lambda { @oauth.get_access_token_info("foo") }.should raise_error(Koala::Facebook::APIError)
         end
       end
+    end
+    
+    describe ".get_access_token" do
+      # TODO refactor these to be proper tests with stubs and tests against real data
+      it "passes on any options provided to make_request" do
+        options = {:a => 2}
+        Koala.should_receive(:make_request).with(anything, anything, anything, hash_including(options)).and_return(Koala::Response.new(200, "", {}))
+        @oauth.get_access_token(@code, options)
+      end
 
-      describe "get_access_token" do
-        it "should use get_access_token_info to get and parse an access token token results" do
+      if KoalaTest.code
+        it "uses get_access_token_info to get and parse an access token token results" do
           result = @oauth.get_access_token(@code)
           result.should be_a(String)
         end
 
-        it "should return the access token as a string" do
+        it "returns the access token as a string" do
           result = @oauth.get_access_token(@code)
           original = @oauth.get_access_token_info(@code)
           result.should == original["access_token"]
         end
 
-        it "should raise an error when get_access_token is called with a bad code" do
+        it "raises an error when get_access_token is called with a bad code" do
           lambda { @oauth.get_access_token("foo") }.should raise_error(Koala::Facebook::APIError)
         end
-
-        it "should pass on any options provided to make_request" do
-          options = {:a => 2}
-          Koala.should_receive(:make_request).with(anything, anything, anything, hash_including(options)).and_return(Koala::Response.new(200, "", {}))
-          @oauth.get_access_token(@code, options)
-        end
       end
-    else
-      it "OAuth code tests will not be run since the code field in facebook_data.yml is blank."
+    end
+    
+    unless KoalaTest.code
+      it "Some OAuth code tests will not be run since the code field in facebook_data.yml is blank."
     end
 
     describe "get_app_access_token_info" do
-      it "should properly get and parse an app's access token as a hash" do
+      it "properly gets and parses an app's access token as a hash" do
         result = @oauth.get_app_access_token_info
         result.should be_a(Hash)
       end
 
-      it "should include the access token" do
+      it "includes the access token" do
         result = @oauth.get_app_access_token_info
         result["access_token"].should
       end
 
-      it "should pass on any options provided to make_request" do
+      it "passes on any options provided to make_request" do
         options = {:a => 2}
         Koala.should_receive(:make_request).with(anything, anything, anything, hash_including(options)).and_return(Koala::Response.new(200, "", {}))
         @oauth.get_app_access_token_info(options)
@@ -241,18 +360,18 @@ describe "Koala::Facebook::OAuth" do
     end
 
     describe "get_app_acess_token" do
-      it "should use get_access_token_info to get and parse an access token token results" do
+      it "uses get_access_token_info to get and parse an access token token results" do
         result = @oauth.get_app_access_token
         result.should be_a(String)
       end
 
-      it "should return the access token as a string" do
+      it "returns the access token as a string" do
         result = @oauth.get_app_access_token
         original = @oauth.get_app_access_token_info
         result.should == original["access_token"]
       end
 
-      it "should pass on any options provided to make_request" do
+      it "passes on any options provided to make_request" do
         options = {:a => 2}
         Koala.should_receive(:make_request).with(anything, anything, anything, hash_including(options)).and_return(Koala::Response.new(200, "", {}))
         @oauth.get_app_access_token(options)
@@ -265,13 +384,13 @@ describe "Koala::Facebook::OAuth" do
       # since these are pretty fundamental and pretty testable, we want to test them
 
       # parse_access_token
-      it "should properly parse access token results" do
+      it "properly parses access token results" do
         result = @oauth.send(:parse_access_token, @raw_token_string)
         has_both_parts = result["access_token"] && result["expires"]
         has_both_parts.should
       end
 
-      it "should properly parse offline access token results" do
+      it "properly parses offline access token results" do
         result = @oauth.send(:parse_access_token, @raw_offline_access_token_string)
         has_both_parts = result["access_token"] && !result["expires"]
         has_both_parts.should
@@ -281,7 +400,7 @@ describe "Koala::Facebook::OAuth" do
       # somewhat duplicative with the tests for get_access_token and get_app_access_token
       # but no harm in thoroughness
       if KoalaTest.code
-        it "should fetch a proper token string from Facebook when given a code" do
+        it "fetches a proper token string from Facebook when given a code" do
           result = @oauth.send(:fetch_token_string, :code => @code, :redirect_uri => @callback_url)
           result.should =~ /^access_token/
         end
@@ -289,7 +408,7 @@ describe "Koala::Facebook::OAuth" do
         it "fetch_token_string code test will not be run since the code field in facebook_data.yml is blank."
       end
 
-      it "should fetch a proper token string from Facebook when asked for the app token" do
+      it "fetches a proper token string from Facebook when asked for the app token" do
         result = @oauth.send(:fetch_token_string, {:type => 'client_cred'}, true)
         result.should =~ /^access_token/
       end
@@ -299,41 +418,41 @@ describe "Koala::Facebook::OAuth" do
   describe "for exchanging session keys" do
     if KoalaTest.session_key
       describe "with get_token_info_from_session_keys" do
-        it "should get an array of session keys from Facebook when passed a single key" do
+        it "gets an array of session keys from Facebook when passed a single key" do
           result = @oauth.get_tokens_from_session_keys([KoalaTest.session_key])
           result.should be_an(Array)
           result.length.should == 1
         end
 
-        it "should get an array of session keys from Facebook when passed multiple keys" do
+        it "gets an array of session keys from Facebook when passed multiple keys" do
           result = @oauth.get_tokens_from_session_keys(@multiple_session_keys)
           result.should be_an(Array)
           result.length.should == 2
         end
 
-        it "should return the original hashes" do
+        it "returns the original hashes" do
           result = @oauth.get_token_info_from_session_keys(@multiple_session_keys)
           result[0].should be_a(Hash)
         end
 
-        it "should properly handle invalid session keys" do
+        it "properly handles invalid session keys" do
           result = @oauth.get_token_info_from_session_keys(["foo", "bar"])
           #it should return nil for each of the invalid ones
           result.each {|r| r.should be_nil}
         end
 
-        it "should properly handle a mix of valid and invalid session keys" do
+        it "properly handles a mix of valid and invalid session keys" do
           result = @oauth.get_token_info_from_session_keys(["foo"].concat(@multiple_session_keys))
           # it should return nil for each of the invalid ones
           result.each_with_index {|r, index| index > 0 ? r.should(be_a(Hash)) : r.should(be_nil)}
         end
 
-        it "should throw an APIError if Facebook returns an empty body (as happens for instance when the API breaks)" do
+        it "throws an APIError if Facebook returns an empty body (as happens for instance when the API breaks)" do
           @oauth.should_receive(:fetch_token_string).and_return("")
           lambda { @oauth.get_token_info_from_session_keys(@multiple_session_keys) }.should raise_error(Koala::Facebook::APIError)
         end
 
-        it "should pass on any options provided to make_request" do
+        it "passes on any options provided to make_request" do
           options = {:a => 2}
           Koala.should_receive(:make_request).with(anything, anything, anything, hash_including(options)).and_return(Koala::Response.new(200, "[{}]", {}))
           @oauth.get_token_info_from_session_keys([], options)
@@ -341,31 +460,31 @@ describe "Koala::Facebook::OAuth" do
       end
 
       describe "with get_tokens_from_session_keys" do
-        it "should call get_token_info_from_session_keys" do
+        it "calls get_token_info_from_session_keys" do
           args = @multiple_session_keys
           @oauth.should_receive(:get_token_info_from_session_keys).with(args, anything).and_return([])
           @oauth.get_tokens_from_session_keys(args)
         end
 
-        it "should return an array of strings" do
+        it "returns an array of strings" do
           args = @multiple_session_keys
           result = @oauth.get_tokens_from_session_keys(args)
           result.each {|r| r.should be_a(String) }
         end
 
-        it "should properly handle invalid session keys" do
+        it "properly handles invalid session keys" do
           result = @oauth.get_tokens_from_session_keys(["foo", "bar"])
           # it should return nil for each of the invalid ones
           result.each {|r| r.should be_nil}
         end
 
-        it "should properly handle a mix of valid and invalid session keys" do
+        it "properly handles a mix of valid and invalid session keys" do
           result = @oauth.get_tokens_from_session_keys(["foo"].concat(@multiple_session_keys))
           # it should return nil for each of the invalid ones
           result.each_with_index {|r, index| index > 0 ? r.should(be_a(String)) : r.should(be_nil)}
         end
 
-        it "should pass on any options provided to make_request" do
+        it "passes on any options provided to make_request" do
           options = {:a => 2}
           Koala.should_receive(:make_request).with(anything, anything, anything, hash_including(options)).and_return(Koala::Response.new(200, "[{}]", {}))
           @oauth.get_tokens_from_session_keys([], options)
@@ -373,29 +492,29 @@ describe "Koala::Facebook::OAuth" do
       end
 
       describe "get_token_from_session_key" do
-        it "should call get_tokens_from_session_keys when the get_token_from_session_key is called" do
+        it "calls get_tokens_from_session_keys when the get_token_from_session_key is called" do
           key = KoalaTest.session_key
           @oauth.should_receive(:get_tokens_from_session_keys).with([key], anything).and_return([])
           @oauth.get_token_from_session_key(key)
         end
 
-        it "should get back the access token string from get_token_from_session_key" do
+        it "gets back the access token string from get_token_from_session_key" do
           result = @oauth.get_token_from_session_key(KoalaTest.session_key)
           result.should be_a(String)
         end
 
-        it "should be the first value in the array" do
+        it "returns the first value in the array" do
           result = @oauth.get_token_from_session_key(KoalaTest.session_key)
           array = @oauth.get_tokens_from_session_keys([KoalaTest.session_key])
           result.should == array[0]
         end
 
-        it "should properly handle an invalid session key" do
+        it "properly handles an invalid session key" do
           result = @oauth.get_token_from_session_key("foo")
           result.should be_nil
         end
 
-        it "should pass on any options provided to make_request" do
+        it "passes on any options provided to make_request" do
           options = {:a => 2}
           Koala.should_receive(:make_request).with(anything, anything, anything, hash_including(options)).and_return(Koala::Response.new(200, "[{}]", {}))
           @oauth.get_token_from_session_key("", options)
@@ -409,15 +528,21 @@ describe "Koala::Facebook::OAuth" do
   describe "for parsing signed requests" do
     # the signed request code is ported directly from Facebook
     # so we only need to test at a high level that it works
-    it "should throw an error if the algorithm is unsupported" do
+    it "throws an error if the algorithm is unsupported" do
       MultiJson.stub(:decode).and_return("algorithm" => "my fun algorithm")
       lambda { @oauth.parse_signed_request(@signed_request) }.should raise_error
     end
 
-    it "should throw an error if the signature is invalid" do
+    it "throws an error if the signature is invalid" do
       OpenSSL::HMAC.stub!(:hexdigest).and_return("i'm an invalid signature")
       lambda { @oauth.parse_signed_request(@signed_request) }.should raise_error
     end
+    
+    it "throws an error if the signature string is empty" do
+      # this occasionally happens due to Facebook error
+      lambda { @oauth.parse_signed_request("") }.should raise_error
+      lambda { @oauth.parse_signed_request("abc-def") }.should raise_error
+    end
 
     it "properly parses requests" do
       @oauth = Koala::Facebook::OAuth.new(@app_id, @secret || @app_secret)
@@ -425,4 +550,4 @@ describe "Koala::Facebook::OAuth" do
     end
   end
 
-end # describe
+end # describe

data/spec/cases/rest_api_spec.rb

@@ -19,23 +19,4 @@ describe "Koala::Facebook::RestAPI" do
       api = Koala::Facebook::RestAPI.new("token")
     end
   end
-
-  context "without an access token" do
-    before :each do
-      @api = Koala::Facebook::API.new
-    end
-
-    it_should_behave_like "Koala RestAPI"
-    it_should_behave_like "Koala RestAPI without an access token"
-  end
-
-  context "with an access token" do
-    before :each do
-      @api = Koala::Facebook::API.new(@token)
-    end
-
-    it_should_behave_like "Koala RestAPI"
-    it_should_behave_like "Koala RestAPI with an access token"
-  end
-
 end