facebook_test_users 0.0.2 → 0.1.4

data/lib/facebook_test_users/user.rb

@@ -3,20 +3,32 @@ require 'uri'
 module FacebookTestUsers
   class User
 
-    attr_reader :id, :access_token, :login_url
+    attr_accessor :id, :access_token, :login_url, :email, :password
 
     def initialize(attrs)
-      # Hacky, but it allows for use of symbol keys, which are nice in
-      # a console.
-      @id, @access_token, @login_url = %w[id access_token login_url].map do |field|
-        attrs[field.to_s] || attrs[field.to_sym]
+      attrs.each do |field, value|
+        instance_variable_set("@#{field}", value) if respond_to?(field)
       end
     end
 
+    def change(options = {})
+      RestClient.post(change_url, {:access_token => access_token}.merge(options))
+    end
+
+    def owner_apps(app)
+      RestClient.get(owner_apps_url(app.access_token))
+    end
+
     def destroy
       RestClient.delete(destroy_url)
     end
 
+    # Facebook test users all share the same birthday. Perhaps it's the
+    # developer's!
+    def birthday
+      Date.new(1980, 8, 8)
+    end
+
     def send_friend_request_to(other)
       RestClient.post(friend_request_url_for(other),
         'access_token' => access_token.to_s)
@@ -24,10 +36,18 @@ module FacebookTestUsers
 
     private
 
+    def change_url
+      GRAPH_API_BASE + "/#{id}"
+    end
+
     def destroy_url
       GRAPH_API_BASE + "/#{id}?access_token=#{URI.escape(access_token.to_s)}"
     end
 
+    def owner_apps_url(token)
+      GRAPH_API_BASE + "/#{id}/ownerapps?access_token=#{URI.escape(token.to_s)}"
+    end
+
     def friend_request_url_for(other)
       GRAPH_API_BASE + "/#{id}/friends/#{other.id}"
     end

data/lib/facebook_test_users/version.rb

@@ -1,3 +1,3 @@
 module FacebookTestUsers
-  VERSION = "0.0.2"
+  VERSION = "0.1.4"
 end

data/spec/fbtu/apps/add_user_spec.rb

@@ -0,0 +1,61 @@
+require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', 'spec_helper'))
+
+describe "fbtu apps add-user" do
+  context "with both apps installed and a user created" do
+    before(:each) do
+      @alpha = register_app('alpha')
+      beta  = register_app('beta')
+      @alice = create_user_for(@alpha)
+    end
+
+    it "adds the user to another app" do
+      fbtu %w[apps add-user --from-app alpha --to-app beta --user] + [@alice.id]
+      @out.should include("User #{@alice.id} added to app 'beta'")
+    end
+
+    it "adds the user to another app without read_stream permissions" do
+      fbtu %w[apps add-user --from-app alpha --to-app beta
+              --permissions none --user] + [@alice.id]
+      @out.should include("User #{@alice.id} added to app 'beta'")
+    end
+
+    it "adds the user to another app without installing the app" do
+      fbtu %w[apps add-user --from-app alpha --to-app beta
+              --installed false --user] + [@alice.id]
+      @out.should include("User #{@alice.id} added to app 'beta'")
+    end
+
+    it "doesn't add the user from an app it's not associated to" do
+      error = "Facebook refused for some other reason."
+      fakeweb_register_bad_request(:post,
+        "https://graph.facebook.com/#{@alpha.id}/accounts/test-users",
+        9999, error)
+      lambda do
+        fbtu %w[apps add-user --from-app beta --to-app alpha --user] + [@alice.id], \
+          :quiet => true
+      end.should raise_error
+      @err.should include(error)
+    end
+  end
+
+  context "with only one app installed and a user created" do
+    before(:each) do
+      alpha = register_app('alpha')
+      @alice = create_user_for(alpha)
+    end
+
+    it "tells you if there was no such app to add the user to" do
+      lambda do
+        fbtu %w[apps add-user --from-app alpha --to-app beta --user] + [@alice.id], :quiet => true
+      end.should raise_error
+      @err.should include("Unknown app beta")
+    end
+
+    it "tells you if there was no such app to add the user from" do
+      lambda do
+        fbtu %w[apps add-user --from-app beta --to-app alpha --user] + [@alice.id], :quiet => true
+      end.should raise_error
+      @err.should include("Unknown app beta")
+    end
+  end
+end

data/spec/fbtu/apps/register_spec.rb

@@ -0,0 +1,36 @@
+require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', 'spec_helper'))
+
+describe "fbtu apps register" do
+  it "lets you create an app" do
+    fbtu %w[apps register --app-id 123456 --app-secret 7890 --name hydrogen]
+
+    fbtu %w[apps list]
+    @out.should include("hydrogen")
+    @out.should_not include("squirrel")
+  end
+
+  it "won't let you create an app with a bogus ID" do
+    lambda do
+      fbtu %w[apps register --app-id xyzzy --app-secret 7890 --name hydrogen], :quiet => true
+    end.should raise_error
+  end
+
+  it "won't let you create an app with a bogus secret" do
+    lambda do
+      fbtu %w[apps register --app-id 123456 --app-secret xyzzy --name hydrogen], :quiet => true
+    end.should raise_error
+  end
+
+  it "won't let you create an app without a name" do
+    lambda do
+      fbtu %w[apps register --app-id 123456 --app-secret 123456], :quiet => true
+    end.should raise_error
+  end
+
+  it "won't let you create an app with a duplicate name" do
+    fbtu %w[apps register --app-id 123456 --app-secret 7890 --name hydrogen]
+    lambda do
+      fbtu %w[apps register --app-id 123456 --app-secret 7890 --name hydrogen], :quiet => true
+    end.should raise_error
+  end
+end

data/spec/fbtu/apps/rm_user_spec.rb

@@ -0,0 +1,42 @@
+require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', 'spec_helper'))
+
+describe "fbtu apps rm-user" do
+  context "with both apps installed and a user created" do
+    before(:each) do
+      @alpha = register_app('alpha')
+      @beta  = register_app('beta')
+      @alice = create_user_for(@alpha)
+    end
+
+    it "removes the user from another app" do
+      delete_url = "https://graph.facebook.com/#{@alpha.id}/accounts/test-users?uid=#{@alice.id}&access_token=#{@alpha.access_token}"
+      FakeWeb.register_uri(:delete, delete_url, :body => "true")
+      fbtu %w[apps rm-user --app alpha --user] + [@alice.id]
+      @out.should include("User #{@alice.id} removed from app 'alpha'")
+    end
+
+    it "does not removes the user from an app it's not associated with" do
+      delete_url = "https://graph.facebook.com/#{@beta.id}/accounts/test-users?uid=#{@alice.id}&access_token=#{@beta.access_token}"
+      error = "This is not a valid test user for this app"
+      fakeweb_register_bad_request(:delete, delete_url, 2901, error)
+      lambda do
+        fbtu %w[apps rm-user --app beta --user] + [@alice.id], :quiet => true
+      end.should raise_error
+      @err.should include(error)
+    end
+  end
+
+  context "with only one app installed and a user created" do
+    before(:each) do
+      alpha = register_app('alpha')
+      @alice = create_user_for(alpha)
+    end
+
+    it "tells you if there was no such app to remove the user from" do
+      lambda do
+        fbtu %w[apps rm-user --app beta --user] + [@alice.id], :quiet => true
+      end.should raise_error
+      @err.should include("Unknown app beta")
+    end
+  end
+end

data/spec/fbtu/apps_spec.rb

@@ -8,7 +8,9 @@ describe "fbtu apps" do
   end
 
   it "defaults to listing apps" do
-    fbtu %w[apps add --name shlomo --app-id 12345 --app-secret abcdef]
+    pending "thor issue with default_task in subcommands " \
+            "(https://github.com/wycats/thor/issues/306)"
+    fbtu %w[apps register --name shlomo --app-id 12345 --app-secret abcdef]
     fbtu %w[apps]
     @out.should include("shlomo")
     @out.should include("12345")

data/spec/fbtu/users/change_spec.rb

@@ -0,0 +1,46 @@
+require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', 'spec_helper'))
+
+describe "fbtu users change" do
+  before(:each) do
+    alpha = register_app('alpha')
+
+    @alice = create_user_for(alpha)
+    @bob   = create_user_for(alpha)
+
+    FakeWeb.register_uri(:post,
+      "https://graph.facebook.com/#{@alice.id}",
+      :body => 'true')
+
+    FakeWeb.register_uri(:post,
+      "https://graph.facebook.com/#{@bob.id}",
+      :body => 'false')
+  end
+
+  it "changes an existing user's name" do
+    fbtu %w[users change --app alpha --user] + [@alice.id] + %w[--name Alice]
+    @out.should include("Successfully changed user")
+  end
+
+  it "changes an existing user's password" do
+    fbtu %w[users change --app alpha --user] + [@alice.id] + %w[--password topsecret]
+    @out.should include("Successfully changed user")
+  end
+
+  it "changes an existing user's user and password" do
+    fbtu %w[users change --app alpha --user] + [@alice.id] +
+      %w[--name Alice --password topsecret]
+    @out.should include("Successfully changed user")
+  end
+
+  it "fails to change an existing user's name" do
+    fbtu %w[users change --app alpha --user] + [@bob.id] + %w[--name Bob]
+    @out.should include("Failed to change user")
+  end
+
+  it "tells you if there was no such user" do
+    lambda do
+      fbtu %w[users change --app alpha --user bogus], :quiet => true
+    end.should raise_error
+    @err.should include("Unknown user")
+  end
+end

data/spec/fbtu/users/create_spec.rb

@@ -0,0 +1,50 @@
+require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', 'spec_helper'))
+
+describe "fbtu users create" do
+  context "with the app installed" do
+    before(:each) do
+      alpha = register_app('alpha')
+
+      @new_user_id = 60189
+      new_user_response = {
+        "id" => @new_user_id,
+        "access_token" => 5795927166794,
+        "login_url" => "https://facebook.example.com/login/#@new_user_id",
+      }
+
+      FakeWeb.register_uri(:post,
+                            "https://graph.facebook.com/#{alpha.id}/accounts/test-users",
+                            :body => new_user_response.to_json)
+    end
+
+    it "creates a user" do
+      fbtu %w[users create --app alpha]
+      @out.should include(@new_user_id.to_s)
+    end
+
+    it "creates a user with the app not installed" do
+      # The API doesn't return the installed status
+      fbtu %w[users create --app alpha --installed false]
+      @out.should include(@new_user_id.to_s)
+    end
+
+    it "creates a named user" do
+      fbtu %w[users create --app alpha --name Joe]
+      # The API doesn't return the name
+      @out.should include(@new_user_id.to_s)
+    end
+
+    it "creates a user with a locale" do
+      fbtu %w[users create --app alpha --locale en_GB]
+      # The API doesn't return the locale
+      @out.should include(@new_user_id.to_s)
+    end
+  end
+
+  it "tells you if there was no such app" do
+    lambda do
+      fbtu %w[users create --app beta], :quiet => true
+    end.should raise_error
+    @err.should include("Unknown app beta")
+  end
+end

data/spec/fbtu/users/friend_spec.rb

@@ -2,10 +2,10 @@ require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', 'spec_hel
 
 describe "fbtu users friend" do
   before(:each) do
-    alpha = add_app('alpha')
+    alpha = register_app('alpha')
 
-    @alice = add_user_to(alpha)
-    @bob = add_user_to(alpha)
+    @alice = create_user_for(alpha)
+    @bob = create_user_for(alpha)
 
     FakeWeb.register_uri(:post,
       "https://graph.facebook.com/#{@alice.id}/friends/#{@bob.id}",

data/spec/fbtu/users/list_apps_spec.rb

@@ -0,0 +1,56 @@
+require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', 'spec_helper'))
+
+describe "fbtu users list-apps" do
+  context "with two apps installed and a user created" do
+    before(:each) do
+      @alpha = register_app('alpha')
+      @beta  = register_app('beta')
+      @alice = create_user_for(@alpha)
+    end
+
+    it "lists the apps" do
+      list_url = "https://graph.facebook.com/#{@alice.id}/ownerapps?access_token=#{@alpha.access_token}"
+      response = {
+        "data" => [
+          {"name" => @alpha.name, "id" => @alpha.id },
+          {"name" => @beta.name,  "id" => @beta.id  },
+        ],
+        "paging" => {
+          "next" => "https => \\/\\/graph.facebook.com\\/#{@alice.id}\\/ownerapps?access_token=#{@alpha.access_token}&limit=5000&offset=5000&__after_id=137431703386"
+        }
+      }
+      FakeWeb.register_uri(:get, list_url, :body => response.to_json)
+
+      fbtu %w[users list-apps --app alpha --user] + [@alice.id]
+      @out.should include(@alpha.name)
+      @out.should include(@beta.name)
+    end
+  end
+
+  context "with only one app installed and no user" do
+    before(:each) do
+      alpha = register_app('alpha')
+    end
+
+    it "tells you if there was no such user to list apps for" do
+      lambda do
+        fbtu %w[users list-apps --app alpha --user 12345], :quiet => true
+      end.should raise_error
+      @err.should include("Unknown user '12345'")
+    end
+  end
+
+  context "with a user and one apps" do
+    before(:each) do
+      alpha = register_app('alpha')
+      @alice = create_user_for(alpha)
+    end
+
+    it "tells you if there was no such app owning the user" do
+      lambda do
+        fbtu %w[users list-apps --app beta --user] + [@alice.id], :quiet => true
+      end.should raise_error
+      @err.should include("Unknown app beta")
+    end
+  end
+end

data/spec/fbtu/users/list_spec.rb

@@ -2,7 +2,7 @@ require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', 'spec_hel
 
 describe "fbtu users list" do
   before(:each) do
-    fbtu %w[apps add --name alpha --app-id 123456 --app-secret abcdef]
+    fbtu %w[apps register --name alpha --app-id 123456 --app-secret abcdef]
     FakeWeb.register_uri(:get,
       'https://graph.facebook.com/oauth/access_token?client_id=123456&client_secret=abcdef&grant_type=client_credentials',
       :body => 'access_token=doublesecret')

data/spec/fbtu/users/nuke_spec.rb

@@ -2,7 +2,7 @@ require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', 'spec_hel
 
 describe "fbtu users nuke" do
   before(:each) do
-    fbtu %w[apps add --name alpha --app-id 123456 --app-secret abcdef]
+    fbtu %w[apps register --name alpha --app-id 123456 --app-secret abcdef]
     FakeWeb.register_uri(:get,
       'https://graph.facebook.com/oauth/access_token?client_id=123456&client_secret=abcdef&grant_type=client_credentials',
       :body => 'access_token=doublesecret')

data/spec/fbtu/users/rm_spec.rb

@@ -2,36 +2,73 @@ require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', 'spec_hel
 
 describe "fbtu users rm" do
   before(:each) do
-    fbtu %w[apps add --name alpha --app-id 123456 --app-secret abcdef]
-    FakeWeb.register_uri(:get,
-      'https://graph.facebook.com/oauth/access_token?client_id=123456&client_secret=abcdef&grant_type=client_credentials',
-      :body => 'access_token=doublesecret')
-
-    user = {
-      "id" => 21055,
-      "access_token" => 9494864868,
-      "login_url" => "https://facebook.example.com/log_this_guy_in?who=thatguy",
+    @alpha = register_app('alpha')
+    @alice = create_user_for(@alpha)
+  end
+
+  it "deletes the user" do
+    delete_url = "https://graph.facebook.com/#{@alice.id}?access_token=#{@alice.access_token}"
+    FakeWeb.register_uri(:delete, delete_url, :body => "true")
+
+    fbtu %w[users rm --app alpha --user] + [@alice.id]
+
+    FakeWeb.should have_requested(:delete,
+      "https://graph.facebook.com/#{@alice.id}?access_token=#{@alice.access_token}")
+    @out.should include("User ID #{@alice.id} removed")
+  end
+
+  it "doesn't delete the user if it's associated with another app" do
+    delete_url = "https://graph.facebook.com/#{@alice.id}?access_token=#{@alice.access_token}"
+    error = "(#2903) Cannot delete this test account because it is associated with other applications."
+    response = {
+      "error" => {
+        "message" => error,
+        "type" => "OAuthException",
+        "code" => 2903
+      }
     }
 
-    FakeWeb.register_uri(:get,
-      'https://graph.facebook.com/123456/accounts/test-users?access_token=doublesecret',
-      :body => {:data => [user]}.to_json)
+    FakeWeb.register_uri(:delete, delete_url, :status => [ '400', 'Bad Request' ],
+                         :body => response.to_json)
+
+    lambda do
+      fbtu %w[users rm --app alpha --user] + [@alice.id], :quiet => true
+    end.should raise_error
+
+    FakeWeb.should have_requested(:delete,
+      "https://graph.facebook.com/#{@alice.id}?access_token=#{@alice.access_token}")
+    @err.should include(error)
+    @err.should include("fbtu users list-apps")
+    @err.should include("fbtu apps rm-user")
+    @err.should include("Then re-run this command.")
   end
 
-  it "deletes the user" do
-    FakeWeb.register_uri(:delete,
-      "https://graph.facebook.com/21055?access_token=9494864868",
-      :body => "true")
+  it "doesn't delete the user if there's another issue" do
+    delete_url = "https://graph.facebook.com/#{@alice.id}?access_token=#{@alice.access_token}"
+    error = "(#9999) Facebook refused for some other reason."
+    response = {
+      "error" => {
+        "message" => error,
+        "type" => "OAuthException",
+        "code" => 9999
+      }
+    }
+
+    FakeWeb.register_uri(:delete, delete_url, :status => [ '400', 'Bad Request' ],
+                         :body => response.to_json)
 
-    fbtu %w[users rm --app alpha  --user 21055]
+    lambda do
+      fbtu %w[users rm --app alpha --user] + [@alice.id], :quiet => true
+    end.should raise_error
 
     FakeWeb.should have_requested(:delete,
-      "https://graph.facebook.com/21055?access_token=9494864868")
+      "https://graph.facebook.com/#{@alice.id}?access_token=#{@alice.access_token}")
+    @err.should include(error)
   end
 
   it "tells you if there was no such user" do
     lambda do
-      fbtu %w[users rm --app alpha  --user bogus], :quiet => true
+      fbtu %w[users rm --app alpha --user bogus], :quiet => true
     end.should raise_error
     @err.should include("Unknown user")
   end