koala 0.4.1 → 0.5.0

data/CHANGELOG

@@ -1,3 +1,10 @@
+v0.5.0
+-- Added several new OAuth methods for making and parsing access token requests
+-- Added test suite for the OAuth class
+-- Made second argument to url_for_access_token a hash (strings still work but trigger a deprecation warning)
+-- Added fields to facebook_data.yml
+-- Updated readme
+
 v0.4.1
 -- Encapsulated GraphAPI and OAuth classes in the Koala::Facebook module for clarity (and to avoid claiming the global Facebook class)
 -- Moved make_request method to Koala class from GraphAPI instance (for use by future OAuth class functionality)

data/Manifest

@@ -2,11 +2,11 @@ CHANGELOG
 Manifest
 Rakefile
 init.rb
-koala.gemspec
 lib/http_services.rb
 lib/koala.rb
 readme.md
 test/facebook_data.yml
 test/koala/facebook_no_access_token_tests.rb
+test/koala/facebook_oauth_tests.rb
 test/koala/facebook_with_access_token_tests.rb
 test/koala_tests.rb

data/Rakefile

@@ -4,10 +4,10 @@ require 'rake'
 require 'echoe'
 
 # gem management
-Echoe.new('koala', '0.4.1') do |p|
+Echoe.new('koala', '0.5.0') do |p|
   p.summary    = "A lightweight, flexible library for Facebook's new Graph API"
   p.description = "Koala is a lightweight, flexible Ruby SDK for Facebook's new Graph API.  It allows read/write access to the Facebook Graph and provides OAuth URLs and cookie validation for Facebook Connect sites.  Koala supports Net::HTTP and Typhoeus connections out of the box and can accept custom modules for other services."
-  p.url            = "http://github.com/arsduo/ruby-sdk"
+  p.url            = "http://github.com/arsduo/koala"
   p.author         = ["Alex Koppel", "Rafi Jacoby", "Context Optional"]
   p.email          = "alex@alexkoppel.com"
   p.ignore_pattern = ["tmp/*", "script/*", "pkg/*"]

data/init.rb

@@ -1,2 +1,2 @@
 # init.rb
-require 'facebook_graph'
+require 'koala'

data/koala.gemspec

@@ -2,16 +2,16 @@
 
 Gem::Specification.new do |s|
   s.name = %q{koala}
-  s.version = "0.4.1"
+  s.version = "0.5.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 1.2") if s.respond_to? :required_rubygems_version=
   s.authors = ["Alex Koppel, Rafi Jacoby, Context Optional"]
-  s.date = %q{2010-05-03}
+  s.date = %q{2010-05-08}
   s.description = %q{Koala is a lightweight, flexible Ruby SDK for Facebook's new Graph API.  It allows read/write access to the Facebook Graph and provides OAuth URLs and cookie validation for Facebook Connect sites.  Koala supports Net::HTTP and Typhoeus connections out of the box and can accept custom modules for other services.}
   s.email = %q{alex@alexkoppel.com}
   s.extra_rdoc_files = ["CHANGELOG", "lib/http_services.rb", "lib/koala.rb"]
-  s.files = ["CHANGELOG", "Manifest", "Rakefile", "init.rb", "koala.gemspec", "lib/http_services.rb", "lib/koala.rb", "readme.md", "test/facebook_data.yml", "test/koala/facebook_no_access_token_tests.rb", "test/koala/facebook_with_access_token_tests.rb", "test/koala_tests.rb"]
-  s.homepage = %q{http://github.com/arsduo/ruby-sdk}
+  s.files = ["CHANGELOG", "Manifest", "Rakefile", "init.rb", "lib/http_services.rb", "lib/koala.rb", "readme.md", "test/facebook_data.yml", "test/koala/facebook_no_access_token_tests.rb", "test/koala/facebook_oauth_tests.rb", "test/koala/facebook_with_access_token_tests.rb", "test/koala_tests.rb", "koala.gemspec"]
+  s.homepage = %q{http://github.com/arsduo/koala}
   s.rdoc_options = ["--line-numbers", "--inline-source", "--title", "Koala", "--main", "readme.md"]
   s.require_paths = ["lib"]
   s.rubyforge_project = %q{koala}

data/lib/http_services.rb

@@ -46,7 +46,7 @@ module Koala
       base.class_eval do
         require 'typhoeus' unless defined?(Typhoeus)
         include Typhoeus
-
+        
         def self.make_request(path, args, verb)
           # if the verb isn't get or post, send it as a post argument
           args.merge!({:method => verb}) && verb = "post" if verb != "get" && verb != "post"

data/lib/koala.rb

@@ -106,7 +106,7 @@ module Koala
         # http://developers.facebook.com/docs/authentication/ for details about
         # extended permissions.
 
-        raise GraphAPIError.new(nil, "Write operations require an access token") unless @access_token
+        raise GraphAPIError.new({"type" => "KoalaMissingAccessToken", "message" => "Write operations require an access token"}) unless @access_token
         api("#{parent_object}/#{connection_name}", args, "post")
       end
     
@@ -161,8 +161,8 @@ module Koala
         response = JSON.parse("[#{result}]")[0]
 
         # check for errors
-        if response.is_a?(Hash) && error = response["error"]
-          raise GraphAPIError.new(error["code"], error["message"])
+        if response.is_a?(Hash) && error_details = response["error"]
+          raise GraphAPIError.new(error_details)
         end
       
         response
@@ -170,14 +170,15 @@ module Koala
     end
   
     class GraphAPIError < Exception
-      attr_accessor :code
-      def initialize(code, message)
-        super(message)
-        self.code = code  
+      attr_accessor :fb_error_type
+      def initialize(details = {})
+        self.fb_error_type = details["type"]  
+        super("#{fb_error_type}: #{details["message"]}")
       end
     end
   
-    class OAuth    
+    class OAuth
+      attr_accessor :app_id, :app_secret, :oauth_callback_url
       def initialize(app_id, app_secret, oauth_callback_url = nil)
         @app_id = app_id
         @app_secret = app_secret
@@ -207,12 +208,13 @@ module Koala
           components = {}
           fb_cookie.split("&").map {|param| param = param.split("="); components[param[0]] = param[1]}
 
+          # generate the signature and make sure it matches what we expect
           auth_string = components.keys.sort.collect {|a| a == "sig" ? nil : "#{a}=#{components[a]}"}.reject {|a| a.nil?}.join("")
-          sig = Digest::MD5.hexdigest(auth_string + @app_secret)
-
-          sig == components["sig"] && (components["expires"].to_i == 0 || Time.now.to_i < components["expires"].to_i) ? components : nil
+          sig = Digest::MD5.hexdigest(auth_string + @app_secret)          
+          sig == components["sig"] && (components["expires"] == "0" || Time.now.to_i < components["expires"].to_i) ? components : nil
         end
       end
+      alias_method :get_user_from_cookies, :get_user_from_cookie
     
       def url_for_oauth_code(options = {})
         # for permissions, see http://developers.facebook.com/docs/authentication/permissions
@@ -220,15 +222,48 @@ module Koala
         scope = permissions ? "&scope=#{permissions.is_a?(Array) ? permissions.join(",") : permissions}" : ""
 
         callback = options[:callback] || @oauth_callback_url
+        raise ArgumentError, "url_for_oauth_code must get a callback either from the OAuth object or in the options!" unless callback
 
         # Creates the URL for oauth authorization for a given callback and optional set of permissions
         "https://#{GRAPH_SERVER}/oauth/authorize?client_id=#{@app_id}&redirect_uri=#{callback}#{scope}"    
       end
         
-      def url_for_access_token(code, callback = @oauth_callback_url)
+      def url_for_access_token(code, options = {})
         # Creates the URL for the token corresponding to a given code generated by Facebook
+        if options.is_a?(String) # changing the arguments
+          puts "Deprecation warning: url_for_access_token now takes an options hash as the second argument; pass the callback as :callback."
+          options = {:callback => options}
+        end
+        callback = options[:callback] || @oauth_callback_url
+        raise ArgumentError, "url_for_access_token must get a callback either from the OAuth object or in the parameters!" unless callback
         "https://#{GRAPH_SERVER}/oauth/access_token?client_id=#{@app_id}&redirect_uri=#{callback}&client_secret=#{@app_secret}&code=#{code}"
       end
+      
+      def parse_access_token(response_text)
+        components = response_text.split("&").inject({}) do |hash, bit|
+          key, value = bit.split("=")
+          hash.merge!(key => value)
+        end
+        components 
+      end
+
+      def fetch_token_string(code)
+        Koala.make_request("oauth/access_token", {
+          :client_id => @app_id, 
+          :redirect_uri => @oauth_callback_url, 
+          :client_secret => @app_secret, 
+          :code => code
+        }, "get")
+      end
+      
+      def get_access_token(code)
+        result = fetch_token_string(code)
+        
+        # if we have an error, parse the error JSON and raise an error
+        raise GraphAPIError.new((JSON.parse(result)["error"] rescue nil) || {}) if result =~ /error/
+        # otherwise, parse the access token
+        parse_access_token(result)
+      end
     end
   end
   

data/readme.md

@@ -1,6 +1,6 @@
 Koala
 ====
-Koala (<a href="http://github.com/arsduo/ruby-sdk" target="_blank">http://github.com/arsduo/ruby-sdk</a>) is a new Facebook Graph library for Ruby.  We wrote Koala with four goals: 
+Koala (<a href="http://github.com/arsduo/koala" target="_blank">http://github.com/arsduo/koala</a>) is a new Facebook Graph library for Ruby.  We wrote Koala with four goals: 
 
 * Lightweight: Koala should be as light and simple as Facebook’s own new libraries, providing API accessors and returning simple JSON.  (We clock in, with comments, just over 300 lines of code.)
 * Fast: Koala should, out of the box, be quick. In addition to supporting the vanilla Ruby networking libraries, it natively supports Typhoeus, our preferred gem for making fast HTTP requests. Of course, That brings us to our next topic:
@@ -20,7 +20,9 @@ to parse the cookies set by the JavaScript SDK for logged in users.
 
 Examples and More Details 
 -----
-There's a very detailed description and walkthrough of Koala at http://blog.twoalex.com/2010/05/03/introducing-koala-a-new-gem-for-facebooks-new-graph-api/.
+There's a very detailed description and walkthrough of Koala at <a href="http://blog.twoalex.com/2010/05/03/introducing-koala-a-new-gem-for-facebooks-new-graph-api/">http://blog.twoalex.com/2010/05/03/introducing-koala-a-new-gem-for-facebooks-new-graph-api/</a>.
+
+You can easily generate OAuth access tokens and any other data needed to play with the Graph API or OAuth at the Koala-powered <a href="http://oauth.twoalex.com" target="_blank">OAuth Playground</a>.
 
 
 Testing

data/test/facebook_data.yml

@@ -1,9 +1,28 @@
-# for testing the GraphAPI classs
-# OAuth token should have stream_publish permissions 
-oauth_token: 
+# Check out http://oauth.twoalex.com to easily generate tokens, cookies, etc.
+# Those values will work with the default settings in this yaml.
+# Of course, you can change this to work with your own app.
+# Just remember to update all fields!
 
-# for testing the OAuth class
-app_id: 
-secret: 
-callback_url: 
-cookie_hash: 
+# for testing the GraphAPI class
+# OAuth token should have publish_stream and read_stream permissions 
+oauth_token: 119908831367602|2.8VK0rWWLe3y4L8JfrpBupA__.3600.1273370400-2905623|u-QAARJz9UeZrF6ef2qu-Fk4UH0.
+
+# for testing the OAuth class    
+# baseline app
+oauth_test_data:
+  # You must supply these values yourself, since they will expire.
+  valid_cookies: # note: the tests stub the time class so these default cookies are always valid
+    fbs_119908831367602: '"access_token=119908831367602|2.LKE7ksSPOx0V_8mHPr2NHQ__.3600.1273363200-2905623|CMpi0AYbn03Oukzv94AUha2qbO4.&expires=1273363200&secret=lT_9zm5r5IbJ6Aa5O54nFw__&session_key=2.LKE7ksSPOx0V_8mHPr2NHQ__.3600.1273363200-2905623&sig=9515e93113921f9476a4efbdd4a3c746&uid=2905623"'
+  code: 2.6GneoQbnEqtSiPppZzDU4Q__.3600.1273366800-2905623|vdiZmnxKxh4WVSnxBxoEvcBfamU.
+    
+  # These values will work out of the box
+  app_id: 119908831367602
+  secret: e45e55a333eec232d4206d2703de1307
+  callback_url: http://oauth.twoalex.com/
+  raw_token_string: "access_token=119908831367602|2.6GneoQbnEqtSiPppZzDU4Q__.3600.1273366800-2905623|3OLa3w0x1K4C1S5cOgbs07TytAk.&expires=6621"
+  raw_offline_access_token_string: access_token=119908831367602|2.6GneoQbnEqtSiPppZzDU4Q__.3600.1273366800-2905623|3OLa3w0x1K4C1S5cOgbs07TytAk.
+  expired_cookies:
+    fbs_119908831367602: '"access_token=119908831367602|2.xv9mi6QSOpr474s4n2X_pw__.3600.1273287600-2905623|yVt5WH_S6J5p3gFa5_5lBzckhws.&expires=1273287600&secret=V_E79ovQnXqxGctFuC_n5A__&session_key=2.xv9mi6QSOpr474s4n2X_pw__.3600.1273287600-2905623&sig=eeef60838c0c800258d89b7e6ddddddb&uid=2905623"'
+  offline_access_cookies: 
+    # 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"'

data/test/koala/facebook_oauth_tests.rb

@@ -0,0 +1,202 @@
+# stub the Time class to always return a time for which the valid cookie is still valid
+class Time
+  def self.now
+    self
+  end
+  
+  def self.to_i
+    1273363199
+  end
+end
+
+class FacebookOAuthTests < Test::Unit::TestCase
+  describe "Koala GraphAPI without an access token" do
+    before :each do
+      # make the relevant test data easily accessible
+      @oauth_data = $testing_data["oauth_test_data"]
+      @app_id = @oauth_data["app_id"]
+      @secret = @oauth_data["secret"]
+      @code = @oauth_data["code"]
+      @callback_url = @oauth_data["callback_url"]
+      @raw_token_string = @oauth_data["raw_token_string"]
+      @raw_offline_access_token_string = @oauth_data["raw_offline_access_token_string"]
+      
+      # this should expanded to cover all variables
+      raise Exception, "Must supply app data to run FacebookOAuthTests!" unless @app_id && @secret && @callback_url && 
+                                                                                @code && @raw_token_string && 
+                                                                                @raw_offline_access_token_string
+
+      @oauth = Koala::Facebook::OAuth.new(@app_id, @secret, @callback_url)
+    end
+    
+    # initialization
+    it "should properly initialize" do
+      @oauth.should
+    end
+
+    it "should properly set 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
+      @oauth = Koala::Facebook::OAuth.new(@app_id, @secret)
+    end
+
+    it "should properly set attributes without a callback URL" do
+      @oauth = Koala::Facebook::OAuth.new(@app_id, @secret)
+      (@oauth.app_id == @app_id && 
+        @oauth.app_secret == @secret && 
+        @oauth.oauth_callback_url == nil).should be_true
+    end
+    
+    # cookie parsing
+    it "should properly parse valid cookies" do
+      result = @oauth.get_user_from_cookie(@oauth_data["valid_cookies"])
+      result["uid"].should
+    end
+    
+    it "should return all the cookie components from valid cookie string" do
+      cookie_data = @oauth_data["valid_cookies"]
+      parsing_results = @oauth.get_user_from_cookie(cookie_data)
+      number_of_components = cookie_data["fbs_#{@app_id.to_s}"].scan(/\=/).length
+      parsing_results.length.should == number_of_components
+    end
+
+    it "should properly parse valid offline access cookies (e.g. no expiration)" do 
+      result = @oauth.get_user_from_cookie(@oauth_data["offline_access_cookies"])
+      result["uid"].should      
+    end
+
+    it "should return all the cookie components from offline access cookies" do
+      cookie_data = @oauth_data["offline_access_cookies"]
+      parsing_results = @oauth.get_user_from_cookie(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 expired cookies" do
+      result = @oauth.get_user_from_cookie(@oauth_data["expired_cookies"])
+      result.should be_nil
+    end
+    
+    it "shouldn't parse invalid cookies" do
+      # make an invalid string by replacing some values
+      bad_cookie_hash = @oauth_data["valid_cookies"].inject({}) { |hash, value| hash[value[0]] = value[1].gsub(/[0-9]/, "3") }
+      result = @oauth.get_user_from_cookie(bad_cookie_hash)
+      result.should be_nil
+    end
+    
+    # OAuth URLs
+    
+    # url_for_oauth_code
+    it "should generate 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 
+      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 
+      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 
+      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 
+      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 raise an exception if no callback is given in initialization or the call" do 
+      oauth2 = Koala::Facebook::OAuth.new(@app_id, @secret)
+      begin
+        url = oauth2.url_for_oauth_code
+      rescue ArgumentError => @right_err
+      rescue
+      end
+      
+      @right_err.should
+    end
+
+    # url_for_access_token
+    it "should generate 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 
+      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}"
+    end
+    
+    it "should output a deprecation warning but generate a properly formatted OAuth token URL when provided a callback in the deprecated fashion" do 
+      callback = "foo.com"
+      url = out = nil
+      
+      begin
+        # we want to capture the deprecation warning as well as the output
+        # credit to http://thinkingdigitally.com/archive/capturing-output-from-puts-in-ruby/ for the technique
+        out = StringIO.new
+        $stdout = out
+        url = @oauth.url_for_access_token(@code, callback)
+      ensure
+        $stdout = STDOUT
+      end
+      
+      # two assertions may be bad test writing, but this is for a deprecated method
+      url.should == "https://#{Koala::Facebook::GRAPH_SERVER}/oauth/access_token?client_id=#{@app_id}&redirect_uri=#{callback}&client_secret=#{@secret}&code=#{@code}"
+      out.should_not be_nil
+    end
+    
+    # parse_access_token
+    it "should properly parse access token results" do
+      result = @oauth.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
+      result = @oauth.parse_access_token(@raw_offline_access_token_string)
+      has_both_parts = result["access_token"] && !result["expires"]
+      has_both_parts.should
+    end
+
+    # fetch_token_string
+    # note -- we just test that this looks right, since get_access_token tests the parsing too
+    it "should fetch a proper token string from Facebook when given a code" do
+      result = @oauth.fetch_token_string(@code)
+      result.should =~ /^access_token/
+    end
+    
+    # get_access_token
+    it "should properly get and parse an access token token results" do
+      result = @oauth.get_access_token(@code)
+      result["access_token"].should
+    end
+
+    it "should raise an error when get_access_token is called with a bad code" do
+      begin
+        result = @oauth.get_access_token("foo")
+      rescue Koala::Facebook::GraphAPIError => @right_err
+      rescue
+      end
+      @right_err.should
+    end
+  end # describe
+
+end #class

data/test/koala/facebook_with_access_token_tests.rb

@@ -10,7 +10,8 @@ class FacebookWithAccessTokenTests < Test::Unit::TestCase
       # clean up any temporary objects
       if @temporary_object_id
         puts "\nCleaning up temporary object #{@temporary_object_id.to_s}"
-        @graph.delete_object(@temporary_object_id)
+        result = @graph.delete_object(@temporary_object_id)
+        raise "Unable to clean up temporary Graph object #{@temporary_object_id}!" unless result
       end
     end
 
@@ -53,26 +54,47 @@ class FacebookWithAccessTokenTests < Test::Unit::TestCase
     end
     
     # PUT
-    it "should be able to put an object" do
+    it "should be able to write an object to the graph" do
       result = @graph.put_wall_post("Hello, world, from the test suite!")
       @temporary_object_id = result["id"]
       @temporary_object_id.should_not be_nil
     end
 
     # DELETE
-    it "should not be able to delete posts" do 
+    it "should be able to delete posts" do 
       result = @graph.put_wall_post("Hello, world, from the test suite delete method!")
       object_id_to_delete = result["id"]
       delete_result = @graph.delete_object(object_id_to_delete)
       delete_result.should == true
     end
 
-    # these are not strictly necessary as the other put methods resolve to put_object, but are here for completeness
-    it "should be able to post to a feed" do
+    # additional put tests
+    it "should be able to verify messages posted to a wall" do
+      message = "the cats are asleep"
+      put_result = @graph.put_wall_post(message)
+      @temporary_object_id = put_result["id"]
+      get_result = @graph.get_object(@temporary_object_id)
+      
+      # make sure the message we sent is the message that got posted
+      get_result["message"].should == message
+    end
+
+    it "should be able to post a message with an attachment to a feed" do
       result = @graph.put_wall_post("Hello, world, from the test suite again!", {:name => "Context Optional", :link => "http://www.contextoptional.com/"})
       @temporary_object_id = result["id"]
       @temporary_object_id.should_not be_nil
     end
+    
+    it "should be able to verify a message with an attachment posted to a feed" do
+      attachment = {"name" => "Context Optional", "link" => "http://www.contextoptional.com/"}
+      result = @graph.put_wall_post("Hello, world, from the test suite again!", attachment)
+      @temporary_object_id = result["id"]
+      get_result = @graph.get_object(@temporary_object_id)
+
+      # make sure the result we fetch includes all the parameters we sent
+      it_matches = attachment.inject(true) {|valid, param| valid && (get_result[param[0]] == attachment[param[0]])}
+      it_matches.should == true 
+    end
 
     it "should be able to comment on an object" do
       result = @graph.put_wall_post("Hello, world, from the test suite, testing comments!")
@@ -82,6 +104,20 @@ class FacebookWithAccessTokenTests < Test::Unit::TestCase
       comment_result = @graph.put_comment(@temporary_object_id, "it's my comment!")
       comment_result.should_not be_nil
     end
+    
+    it "should be able to verify a comment posted about an object" do
+      message_text = "Hello, world, from the test suite, testing comments!"
+      result = @graph.put_wall_post(message_text)
+      @temporary_object_id = result["id"]
+      
+      # this will be deleted when the post gets deleted 
+      comment_text = "it's my comment!"
+      comment_result = @graph.put_comment(@temporary_object_id, comment_text)
+      get_result = @graph.get_object(comment_result["id"])
+
+      # make sure the text of the comment matches what we sent
+      get_result["message"].should == comment_text
+    end
 
     it "should be able to like an object" do
       result = @graph.put_wall_post("Hello, world, from the test suite, testing comments!")

data/test/koala_tests.rb

@@ -2,16 +2,20 @@ require 'test/unit'
 require 'rubygems'
 require 'spec/test/unit'
 
+# load the libraries
 require 'koala'
+
+# load the tests
 require 'koala/facebook_no_access_token_tests'
 require 'koala/facebook_with_access_token_tests'
+require 'koala/facebook_oauth_tests'
 
 class FacebookTestSuite
   def self.suite
     suite = Test::Unit::TestSuite.new
     suite << FacebookNoAccessTokenTests.suite
     suite << FacebookWithAccessTokenTests.suite
-    #suite << FacebookCookieTest.suite
+    suite << FacebookOAuthTests.suite
     suite
   end
 end

metadata

@@ -4,9 +4,9 @@ version: !ruby/object:Gem::Version
   prerelease: false
   segments: 
   - 0
-  - 4
-  - 1
-  version: 0.4.1
+  - 5
+  - 0
+  version: 0.5.0
 platform: ruby
 authors: 
 - Alex Koppel, Rafi Jacoby, Context Optional
@@ -14,7 +14,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-05-03 00:00:00 -07:00
+date: 2010-05-08 00:00:00 -07:00
 default_executable: 
 dependencies: []
 
@@ -33,16 +33,17 @@ files:
 - Manifest
 - Rakefile
 - init.rb
-- koala.gemspec
 - lib/http_services.rb
 - lib/koala.rb
 - readme.md
 - test/facebook_data.yml
 - test/koala/facebook_no_access_token_tests.rb
+- test/koala/facebook_oauth_tests.rb
 - test/koala/facebook_with_access_token_tests.rb
 - test/koala_tests.rb
+- koala.gemspec
 has_rdoc: true
-homepage: http://github.com/arsduo/ruby-sdk
+homepage: http://github.com/arsduo/koala
 licenses: []
 
 post_install_message: