openskip-repim 0.1.8 → 0.1.9

data/ChangeLog

@@ -1,3 +1,9 @@
+== 0.1.9 / 2009-07-01
+* change default single access mode
+
+== 0.1.8 / 2009-06-30
+* add single access mode
+
 == 0.1.7 / 2009-05-22
 * fix up old style rspec test (generator's)
 * add how to use. (patch from yhara)

data/generators/relying_party/relying_party_generator.rb

@@ -6,6 +6,8 @@ class RelyingPartyGenerator < Rails::Generator::NamedBase
            "Specify User class name defailt is [User]"){|v| options[:user_klass] = "User" }
     opt.on("--user-management=generation_type",
            "'model' for generate (rspec_)model, 'singnup' for controller using Repim::Signup. default is 'signup'"){|v| options[:user_model_only] = (v == "model") }
+    opt.on("--without-single-access",
+           "Allow user login with multiple browser."){|v| options[:without_single_access] = v }
     opt.on("--openid-migration=migration_name",
            "Specify openid migration name, default is 'open_id_authentication_tables'. If value is 'none' generate nothing.") do |v|
       options[:openid_migration] = v
@@ -41,7 +43,7 @@ class RelyingPartyGenerator < Rails::Generator::NamedBase
       end
 
       generate_user_management(m, !options[:skip_sessions_spec]) unless options[:user_model_only]
-      m.dependency(care_rspec("model"), [user_klass_name, "identity_url:string", @args].flatten.compact)
+      m.dependency(care_rspec("model"), [user_klass_name, user_klass_columns, @args].flatten.compact)
 
       # FIXME very veriy dirty
       # "sleep 3" is for changing timestamp of 'create_user' or 'open_id_authentication_tables'
@@ -68,6 +70,12 @@ class RelyingPartyGenerator < Rails::Generator::NamedBase
       detect{|c| File.exists?(File.expand_path(c, RAILS_ROOT)) }
   end
 
+  def user_klass_columns
+    columns = ["identity_url:string"]
+    columns << "single_access_token:string" unless options[:without_single_access]
+    columns
+  end
+
   def care_rspec(base); using_rspec? ? "rspec_#{base}" : base end
 
   def using_rspec?; File.directory?( File.expand_path("spec", RAILS_ROOT) ) end

data/generators/relying_party/templates/spec/application_controller_spec.rb

@@ -3,6 +3,7 @@ require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
 describe ApplicationController do
   before :all do
     ApplicationController.user_klass = mock("User-Klass")
+    ApplicationController.with_single_access = false
   end
   describe "#logged in" do
     before do
@@ -15,3 +16,19 @@ describe ApplicationController do
   end
 end
 
+describe ApplicationController, "with_single_access" do
+  before :all do
+    ApplicationController.user_klass = mock("User-Klass")
+    ApplicationController.with_single_access = true
+  end
+  describe "#logged in" do
+    before do
+      session[:single_access_token] = 12345
+      ApplicationController.user_klass.should_receive(:find_by_single_access_token).with(12345).and_return(@user = mock("user"))
+    end
+
+    it{ controller.should be_signed_in }
+    it{ controller.current_user.should == @user }
+  end
+end
+

data/generators/relying_party/templates/spec/sessions_controller_spec.rb

@@ -3,6 +3,7 @@ require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
 describe SessionsController do
   before :all do
     SessionsController.user_klass = mock("User-Klass")
+    SessionsController.with_single_access = false
   end
 
   #Delete this example and add some real ones
@@ -77,3 +78,83 @@ describe SessionsController do
     it{ flash[:error].should_not be_blank }
   end
 end
+
+describe SessionsController, "with_single_access" do
+  before :all do
+    SessionsController.user_klass = mock("User-Klass")
+    SessionsController.with_single_access = true
+  end
+
+  #Delete this example and add some real ones
+  it "should use SessionsController" do
+    controller.should be_an_instance_of(SessionsController)
+  end
+
+  it "get new should be success" do
+    get :new
+    response.should be_success
+  end
+
+  def mock_out_authenticattion(controller, url, is_success)
+    result = mock("result")
+    result.should_receive(:successful?).and_return is_success
+
+    controller.
+      should_receive(:authenticate_with_open_id).
+      with(url, an_instance_of(Hash)).
+      and_return{|url, ax, block| block.call(result, url, ax) }
+  end
+
+  describe "authentication success" do
+    before do
+      url = "---openid-url---"
+
+      @user = mock("user", :id => 12345, :single_access_token => "12345", :single_access_token= => "12345", :save => true)
+      SessionsController.user_klass.should_receive(:find_by_identity_url).with(url).and_return(@user)
+
+      controller.should_receive(:root_path).and_return("/")
+      mock_out_authenticattion(controller, url, true)
+
+      post :create, :openid_url =>url
+    end
+
+    it{ response.should redirect_to("/") }
+    it{ session[:single_access_token].should == "12345" }
+    it{ controller.current_user.should == @user }
+    it{ controller.should be_signed_in }
+  end
+
+  describe "authentication success but user not found then render users/new" do
+    before do
+      url = "---openid-url---"
+      SessionsController.user_klass.should_receive(:find_by_identity_url).with(url).and_return(nil)
+      @user = mock("user", :id => 12345, :single_access_token => "12345", :single_access_token= => "12345", :save => true)
+      SessionsController.user_klass.should_receive(:new).with({}).and_return(@user)
+
+      mock_out_authenticattion(controller, url, true)
+
+      post :create, :openid_url =>url
+    end
+
+    it{ response.should render_template("users/new") }
+    it{ controller.current_user.should be_nil }
+    it{ controller.should_not be_signed_in }
+    it{ assigns[:user].should == @user }
+  end
+
+  describe "authentication failed" do
+    before do
+      url = "---openid-url---"
+
+      mock_out_authenticattion(controller, url, false)
+
+      post :create, :openid_url =>url
+    end
+
+    it{ response.should render_template("sessions/new") }
+    it{ assigns[:user].should be_nil }
+    it{ controller.current_user.should be_nil }
+    it{ session[:single_access_token].should be_nil }
+    it{ flash[:error].should_not be_blank }
+  end
+end

data/generators/relying_party/templates/spec/users_controller_spec.rb

@@ -51,6 +51,7 @@ describe UsersController do
   describe "responding to POST create" do
     describe "with valid params" do
       before do
+        UsersController.with_single_access = false
         User.should_receive(:new).with({'these' => 'params'}).and_return(mock_user(:save => true))
 
         @identity_url = session[:identity_url] = "http://moro.openid.example.com/"

data/lib/repim.rb

@@ -1,3 +1,3 @@
 module Repim
-  Version = '0.1.8'
+  Version = '0.1.9'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: openskip-repim
 version: !ruby/object:Gem::Version 
-  version: 0.1.8
+  version: 0.1.9
 platform: ruby
 authors: 
 - MOROHASHI Kyosuke
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-06-30 00:00:00 -07:00
+date: 2009-07-01 00:00:00 -07:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency