moro-repim 0.1.6 → 0.1.7

data/ChangeLog

@@ -1,3 +1,7 @@
+== 0.1.7 / 2009-05-22
+* fix up old style rspec test (generator's)
+* add how to use. (patch from yhara)
+
 == 0.1.6 / 2009-04-02
 * change default after_create_url to root_url
 

data/README.rdoc

@@ -1,10 +1,10 @@
 
 = repim
 
-
 == Description
 
-Relying Party in minutes.
+repim (RElying Party In Minutes) is a rails plugin to
+create websites using OpenID authentication easily.
 
 == Installation
 
@@ -22,6 +22,19 @@ Relying Party in minutes.
 
 == Synopsis
 
+Generate template files
+
+  $ ruby script/generate relying_party Sessions name:string display_name:string
+
+Edit config/environment.rb to load repim gem
+
+  config.gem 'moro-repim', :lib => 'repim', :source => 'http://gems.github.com/'
+
+Start server
+  
+  $ ruby script/server
+
+You will see the login form in http://localhost:3000/signin .
 
 == Copyright
 

data/generators/relying_party/relying_party_generator.rb

@@ -34,9 +34,10 @@ class RelyingPartyGenerator < Rails::Generator::NamedBase
       unless options[:skip_sessions_spec]
         m.directory "spec/controllers"
         m.file "spec/application_controller_spec.rb", "spec/controllers/application_controller_spec.rb"
-
         m.file "spec/sessions_controller_spec.rb", "spec/controllers/#{controller_file_name}_spec.rb"
-        m.file "spec/sessions_routing_spec.rb", "spec/controllers/#{plural_name}_routing_spec.rb"
+
+        m.directory "spec/routing"
+        m.file "spec/sessions_routing_spec.rb", "spec/routing/#{plural_name}_routing_spec.rb"
       end
 
       generate_user_management(m, !options[:skip_sessions_spec]) unless options[:user_model_only]
@@ -108,7 +109,7 @@ EOS
 
     if with_spec
       m.file "spec/users_controller_spec.rb", "spec/controllers/#{user_controller_name}_spec.rb"
-      m.file "spec/users_routing_spec.rb", "spec/controllers/#{users}_routing_spec.rb"
+      m.file "spec/users_routing_spec.rb", "spec/routing/#{users}_routing_spec.rb"
     end
   end
 end

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

@@ -15,6 +15,16 @@ describe SessionsController do
     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---"
@@ -22,11 +32,8 @@ describe SessionsController do
       @user = mock("user", :id => 12345)
       SessionsController.user_klass.should_receive(:find_by_identity_url).with(url).and_return(@user)
 
-      result = mock("result")
-      result.should_receive(:successful?).and_return true
-
       controller.should_receive(:root_path).and_return("/")
-      controller.should_receive(:authenticate_with_open_id).with(url, {}).and_yield(result,url, {})
+      mock_out_authenticattion(controller, url, true)
 
       post :create, :openid_url =>url
     end
@@ -43,10 +50,7 @@ describe SessionsController do
       SessionsController.user_klass.should_receive(:find_by_identity_url).with(url).and_return(nil)
       SessionsController.user_klass.should_receive(:new).with({}).and_return(@user = mock("user"))
 
-      result = mock("result")
-      result.should_receive(:successful?).and_return true
-
-      controller.should_receive(:authenticate_with_open_id).with(url, {}).and_yield(result,url, {})
+      mock_out_authenticattion(controller, url, true)
 
       post :create, :openid_url =>url
     end
@@ -61,10 +65,7 @@ describe SessionsController do
     before do
       url = "---openid-url---"
 
-      result = mock("result")
-      result.should_receive(:successful?).and_return false
-
-      controller.should_receive(:authenticate_with_open_id).with(url, {}).and_yield(result,url, {})
+      mock_out_authenticattion(controller, url, false)
 
       post :create, :openid_url =>url
     end

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

@@ -7,11 +7,11 @@ describe SessionsController do
     end
 
     it "should map #destroy -> /signout" do
-      route_for(:controller => "sessions", :action => "destroy").should == "/signout"
+      route_for(:controller => "sessions", :action => "destroy").should == {:path => "/signout", :method => "DELETE"}
     end
 
-    it "should map #show" do
-      route_for(:controller => "sessions", :action => "show").should == "/session"
+    it "should map #create" do
+      route_for(:controller => "sessions", :action => "create").should == {:path => "/session", :method => "POST"}
     end
   end
 

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

@@ -3,27 +3,27 @@ require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
 describe UsersController do
   describe "route generation" do
     it "should map #index" do
-      route_for(:controller => "users", :action => "index").should == "/users"
+      route_for(:controller => "users", :action => "index").should == {:path => "/users", :method => "GET"}
     end
   
     it "should map #new" do
-      route_for(:controller => "users", :action => "new").should == "/users/new"
+      route_for(:controller => "users", :action => "new").should == {:path => "/users/new", :method => "GET"}
     end
   
     it "should map #show" do
-      route_for(:controller => "users", :action => "show", :id => 1).should == "/users/1"
+      route_for(:controller => "users", :action => "show", :id => "1").should == {:path => "/users/1", :method => "GET"}
     end
   
     it "should map #edit" do
-      route_for(:controller => "users", :action => "edit", :id => 1).should == "/users/1/edit"
+      route_for(:controller => "users", :action => "edit", :id => "1").should == {:path => "/users/1/edit", :method => "GET"}
     end
   
     it "should map #update" do
-      route_for(:controller => "users", :action => "update", :id => 1).should == "/users/1"
+      route_for(:controller => "users", :action => "update", :id => "1").should == {:path => "/users/1", :method => "PUT"}
     end
   
     it "should map #destroy" do
-      route_for(:controller => "users", :action => "destroy", :id => 1).should == "/users/1"
+      route_for(:controller => "users", :action => "destroy", :id => "1").should == {:path => "/users/1", :method => "DELETE"}
     end
   end
 

data/lib/repim.rb

@@ -1,3 +1,3 @@
 module Repim
-  Version = '0.1.6'
+  Version = '0.1.7'
 end

data/lib/repim/signup.rb

@@ -13,7 +13,7 @@ module Repim
           flash[:notice] = 'User was successfully created.'
           reset_session
           self.current_user = @user
-          format.html { redirect_to(after_create_url) }
+          format.html { redirect_to(after_create_url(@user)) }
         else
           format.html { render :action => "new" }
         end
@@ -21,8 +21,8 @@ module Repim
     end
 
     private
-    def after_create_url
-      root_url
+    def after_create_url(created)
+      respond_to?(:root_url) ? root_url : user_url(created)
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: moro-repim
 version: !ruby/object:Gem::Version 
-  version: 0.1.6
+  version: 0.1.7
 platform: ruby
 authors: 
 - MOROHASHI Kyosuke
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-04-02 00:00:00 -07:00
+date: 2009-05-22 00:00:00 -07:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency