devise-better_routes 0.0.3 → 0.0.4

data/README.md

@@ -75,6 +75,11 @@ Do you like this one?
 
 ## Changes
 
+### 0.0.4
+
+* __Add password routes__
+* Add URL helpers for session & password
+
 ### 0.0.3
 
 * __Add session routes__

data/lib/devise/better_routes.rb

@@ -6,6 +6,25 @@ require 'devise/better_routes/version'
 module Devise
   module Controllers
     module UrlHelpers
+      def self.override_current_resource_helpers!(routes = {})
+        routes.each do |module_name, actions|
+          [:path, :url].each do |path_or_url|
+            actions.each do |action|
+              action = action ? "#{action}_" : ""
+              method = "#{action}#{module_name}_#{path_or_url}"
+
+              class_eval <<-URL_HELPERS, __FILE__, __LINE__ + 1
+                def #{method}(resource_or_scope, *args)
+                  scope = Devise::Mapping.find_scope!(resource_or_scope)
+                  current_resource_name = _devise_path_name(scope, "current_\#{scope}")
+                  _devise_route_context.send("#{action}\#{current_resource_name}_#{module_name}_#{path_or_url}", *args)
+                end
+              URL_HELPERS
+            end
+          end
+        end
+      end
+
       def self.override_registration_helpers!(registration_routes = nil)
         helper_mappings = {
           new_registration: 'new_#{scope}',
@@ -19,9 +38,9 @@ module Devise
               scope = Devise::Mapping.find_scope!(resource_or_scope)
               current_resource_name = _devise_path_name(scope, "current_\#{scope}")
               if respond_to?(:controller_name) && controller_name == _devise_registration_controller(scope, current_resource_name)
-                send("\#{current_resource_name}_#{path_or_url}", *args)
+                _devise_route_context.send("\#{current_resource_name}_#{path_or_url}", *args)
               else
-                send("\#{scope.to_s.pluralize}_#{path_or_url}", *args)
+                _devise_route_context.send("\#{scope.to_s.pluralize}_#{path_or_url}", *args)
               end
             end
           URL_HELPERS
@@ -31,17 +50,18 @@ module Devise
               def #{method}(resource_or_scope, *args)
                 scope = Devise::Mapping.find_scope!(resource_or_scope)
                 current_resource_name = _devise_path_name(scope, "current_\#{scope}")
-                send("#{new_name}_#{path_or_url}", *args)
+                _devise_route_context.send("#{new_name}_#{path_or_url}", *args)
               end
             URL_HELPERS
           end
         end
       end
-      override_registration_helpers!(Devise::URL_HELPERS[:registration])
 
       def self.run_generate_hooks!
+        self.override_current_resource_helpers!(Devise::URL_HELPERS.slice(:session, :password))
         self.override_registration_helpers!(Devise::URL_HELPERS[:registration])
       end
+      run_generate_hooks!
 
       def _devise_path_name(scope, name)
         Devise.mappings[scope].path_names[name.to_sym]
@@ -84,6 +104,17 @@ module ActionDispatch
           @scope[:path], @scope[:as] = path, as
         end
 
+        def devise_password(mapping, controllers)
+          current_resource_name = mapping.path_names["current_#{mapping.singular}".to_sym]
+          path, as, @scope[:path], @scope[:as] = @scope[:path], @scope[:as], nil, nil
+          resource current_resource_name, only: [] do
+            resource :password, :only => [:new, :create, :edit, :update],
+              :path => mapping.path_names[:password], :controller => controllers[:passwords]
+          end
+        ensure
+          @scope[:path], @scope[:as] = path, as
+        end
+
         def devise_registration(mapping, controllers)
           path, as, @scope[:path], @scope[:as] = @scope[:path], @scope[:as], nil, nil
           current_resource_name = mapping.path_names["current_#{mapping.name}".to_sym]

data/lib/devise/better_routes/version.rb

@@ -1,5 +1,5 @@
 module Devise
   module BetterRoutes
-    VERSION = "0.0.3"
+    VERSION = "0.0.4"
   end
 end

data/spec/devise/better_routes_helpers_spec.rb

@@ -0,0 +1,215 @@
+require 'spec_helper'
+
+describe Devise::BetterRoutes, 'helpers', type: :helper do
+  describe %|devise_for :users| do
+    context 'registration' do
+      specify { expect(users_path).to eq '/users' }
+      specify { expect(new_user_path).to eq '/users/new' }
+      specify { expect(cancel_current_user_path).to eq '/current_user/cancel' }
+      specify { expect(edit_current_user_path).to eq '/current_user/edit' }
+      specify { expect(current_user_path).to eq '/current_user' }
+      it 'registration_path is delegated to users_path' do
+        expect(registration_path(:user)).to eq users_path
+      end
+      it 'registration_path is delegated to current_user_path on current_users' do
+        instance_eval { def controller_name; 'current_users' end }
+        expect(registration_path(:user)).to eq current_user_path
+        instance_eval { undef controller_name }
+      end
+      it 'new_registration_path is delegated to new_user_path' do
+        expect(new_registration_path(:user)).to eq new_user_path
+      end
+      it 'cancel_registration_path is delegated to cancel_current_user_path' do
+        expect(cancel_registration_path(:user)).to eq cancel_current_user_path
+      end
+      it 'edit_registration_path is delegated to edit_current_user_path' do
+        expect(edit_registration_path(:user)).to eq edit_current_user_path
+      end
+    end
+
+    context 'session' do
+      specify { expect(current_user_session_path).to eq '/current_user/session' }
+      specify { expect(new_current_user_session_path).to eq '/current_user/session/new' }
+      it 'session_path is delegated to current_user_session_path' do
+        expect(session_path(:user)).to eq current_user_session_path
+      end
+      it 'new_session_path is delegated to new_current_user_session_path' do
+        expect(new_session_path(:user)).to eq new_current_user_session_path
+      end
+    end
+
+    context 'password' do
+      specify { expect(current_user_password_path).to eq '/current_user/password' }
+      specify { expect(new_current_user_password_path).to eq '/current_user/password/new' }
+      specify { expect(edit_current_user_password_path).to eq '/current_user/password/edit' }
+      it 'password_path is delegated to current_user_password_path' do
+        expect(password_path(:user)).to eq current_user_password_path
+      end
+      it 'new_password_path is delegated to new_current_user_password_path' do
+        expect(new_password_path(:user)).to eq new_current_user_password_path
+      end
+      it 'edit_password_path is delegated to edit_current_user_password_path' do
+        expect(edit_password_path(:user)).to eq edit_current_user_password_path
+      end
+    end
+  end
+
+  describe %|devise_for :rails_programmers, path_names: {current_rails_programmer: 'me'}| do
+    context 'registration' do
+      specify { expect(rails_programmers_path).to eq '/rails_programmers' }
+      specify { expect(new_rails_programmer_path).to eq '/rails_programmers/new' }
+      specify { expect(cancel_me_path).to eq '/me/cancel' }
+      specify { expect(edit_me_path).to eq '/me/edit' }
+      specify { expect(me_path).to eq '/me' }
+      it 'registration_path is delegated to rails_programmers_path' do
+        expect(registration_path(:rails_programmer)).to eq rails_programmers_path
+      end
+      it 'registration_path is delegated to me_path on me' do
+        instance_eval { def controller_name; 'me' end }
+        expect(registration_path(:rails_programmer)).to eq me_path
+        instance_eval { undef controller_name }
+      end
+      it 'new_registration_path is delegated to new_rails_programmer_path' do
+        expect(new_registration_path(:rails_programmer)).to eq new_rails_programmer_path
+      end
+      it 'cancel_registration_path is delegated to cancel_me_path' do
+        expect(cancel_registration_path(:rails_programmer)).to eq cancel_me_path
+      end
+      it 'edit_registration_path is delegated to edit_me_path' do
+        expect(edit_registration_path(:rails_programmer)).to eq edit_me_path
+      end
+    end
+
+    context 'session' do
+      specify { expect(me_session_path).to eq '/me/session' }
+      specify { expect(new_me_session_path).to eq '/me/session/new' }
+      it 'session_path is delegated to me_session_path' do
+        expect(session_path(:rails_programmer)).to eq me_session_path
+      end
+      it 'new_session_path is delegated to new_me_session_path' do
+        expect(new_session_path(:rails_programmer)).to eq new_me_session_path
+      end
+    end
+
+    context 'password' do
+      specify { expect(me_password_path).to eq '/me/password' }
+      specify { expect(new_me_password_path).to eq '/me/password/new' }
+      specify { expect(edit_me_password_path).to eq '/me/password/edit' }
+      it 'password_path is delegated to me_password_path' do
+        expect(password_path(:rails_programmer)).to eq me_password_path
+      end
+      it 'new_password_path is delegated to new_user_password_path' do
+        expect(new_password_path(:rails_programmer)).to eq new_me_password_path
+      end
+      it 'edit_password_path is delegated to edit_me_password_path' do
+        expect(edit_password_path(:rails_programmer)).to eq edit_me_password_path
+      end
+    end
+  end
+
+  describe %|devise_for :programmers, controllers: {programmers: 'rails_programmers', current_programmer: 'me'}| do
+    context 'registration' do
+      specify { expect(programmers_path).to eq '/programmers' }
+      specify { expect(new_programmer_path).to eq '/programmers/new' }
+      specify { expect(cancel_current_programmer_path).to eq '/current_programmer/cancel' }
+      specify { expect(edit_current_programmer_path).to eq '/current_programmer/edit' }
+      specify { expect(current_programmer_path).to eq '/current_programmer' }
+      it 'registration_path is delegated to programmers_path' do
+        expect(registration_path(:programmer)).to eq programmers_path
+      end
+      it 'registration_path is delegated to current_programmer_path on me' do
+        instance_eval { def controller_name; 'me' end }
+        expect(registration_path(:programmer)).to eq current_programmer_path
+        instance_eval { undef controller_name }
+      end
+      it 'new_registration_path is delegated to new_programmer_path' do
+        expect(new_registration_path(:programmer)).to eq new_programmer_path
+      end
+      it 'cancel_registration_path is delegated to cancel_current_programmer_path' do
+        expect(cancel_registration_path(:programmer)).to eq cancel_current_programmer_path
+      end
+      it 'edit_registration_path is delegated to edit_current_programmer_path' do
+        expect(edit_registration_path(:programmer)).to eq edit_current_programmer_path
+      end
+    end
+
+    context 'session' do
+      specify { expect(current_programmer_session_path).to eq '/current_programmer/session' }
+      specify { expect(new_current_programmer_session_path).to eq '/current_programmer/session/new' }
+      it 'session_path is delegated to current_programmer_session_path' do
+        expect(session_path(:programmer)).to eq current_programmer_session_path
+      end
+      it 'new_session_path is delegated to new_current_programmer_session_path' do
+        expect(new_session_path(:programmer)).to eq new_current_programmer_session_path
+      end
+    end
+
+    context 'password' do
+      specify { expect(current_programmer_password_path).to eq '/current_programmer/password' }
+      specify { expect(new_current_programmer_password_path).to eq '/current_programmer/password/new' }
+      specify { expect(edit_current_programmer_password_path).to eq '/current_programmer/password/edit' }
+      it 'password_path is delegated to current_programmer_password_path' do
+        expect(password_path(:programmer)).to eq current_programmer_password_path
+      end
+      it 'new_password_path is delegated to new_current_programmer_password_path' do
+        expect(new_password_path(:programmer)).to eq new_current_programmer_password_path
+      end
+      it 'edit_password_path is delegated to edit_current_programmer_password_path' do
+        expect(edit_password_path(:programmer)).to eq edit_current_programmer_password_path
+      end
+    end
+  end
+
+  describe %|devise_for :engineers, controllers: {registrations: 'rails_programmers'}| do
+    context 'registration' do
+      specify { expect(engineers_path).to eq '/engineers' }
+      specify { expect(new_engineer_path).to eq '/engineers/new' }
+      specify { expect(cancel_current_engineer_path).to eq '/current_engineer/cancel' }
+      specify { expect(edit_current_engineer_path).to eq '/current_engineer/edit' }
+      specify { expect(current_engineer_path).to eq '/current_engineer' }
+      it 'registration_path is delegated to engineers_path' do
+        expect(registration_path(:engineer)).to eq engineers_path
+      end
+      it 'registration_path is delegated to current_engineer_path on me' do
+        instance_eval { def controller_name; 'rails_programmers' end }
+        expect(registration_path(:engineer)).to eq current_engineer_path
+        instance_eval { undef controller_name }
+      end
+      it 'new_registration_path is delegated to new_engineer_path' do
+        expect(new_registration_path(:engineer)).to eq new_engineer_path
+      end
+      it 'cancel_registration_path is delegated to cancel_current_engineer_path' do
+        expect(cancel_registration_path(:engineer)).to eq cancel_current_engineer_path
+      end
+      it 'edit_registration_path is delegated to edit_current_engineer_path' do
+        expect(edit_registration_path(:engineer)).to eq edit_current_engineer_path
+      end
+    end
+
+    context 'session' do
+      specify { expect(current_engineer_session_path).to eq '/current_engineer/session' }
+      specify { expect(new_current_engineer_session_path).to eq '/current_engineer/session/new' }
+      it 'session_path is delegated to current_engineer_session_path' do
+        expect(session_path(:engineer)).to eq current_engineer_session_path
+      end
+      it 'new_session_path is delegated to new_current_engineer_session_path' do
+        expect(new_session_path(:engineer)).to eq new_current_engineer_session_path
+      end
+    end
+
+    context 'password' do
+      specify { expect(current_engineer_password_path).to eq '/current_engineer/password' }
+      specify { expect(new_current_engineer_password_path).to eq '/current_engineer/password/new' }
+      specify { expect(edit_current_engineer_password_path).to eq '/current_engineer/password/edit' }
+      it 'password_path is delegated to current_engineer_password_path' do
+        expect(password_path(:engineer)).to eq current_engineer_password_path
+      end
+      it 'new_password_path is delegated to new_current_engineer_password_path' do
+        expect(new_password_path(:engineer)).to eq new_current_engineer_password_path
+      end
+      it 'edit_password_path is delegated to edit_current_engineer_password_path' do
+        expect(edit_password_path(:engineer)).to eq edit_current_engineer_password_path
+      end
+    end
+  end
+end

data/spec/devise/better_routes_routing_spec.rb

@@ -0,0 +1,207 @@
+require 'spec_helper'
+
+describe Devise::BetterRoutes, 'routing', type: :routing do
+  describe %|devise_for :users| do
+    context 'registration' do
+      it 'routes to users#create' do
+        expect(post('/users')).to route_to('users#create')
+      end
+      it 'routes to users#new' do
+        expect(get('/users/new')).to route_to('users#new')
+      end
+      it 'routes to current_users#cancel' do
+        expect(get('/current_user/cancel')).to route_to('current_users#cancel')
+      end
+      it 'routes to current_users#edit' do
+        expect(get('/current_user/edit')).to route_to('current_users#edit')
+      end
+      it 'routes to current_users#update' do
+        expect(put('/current_user')).to route_to('current_users#update')
+        #expect(patch('/current_user')).to route_to('current_users#update')
+      end
+      it 'routes to current_users#destroy' do
+        expect(delete('/current_user')).to route_to('current_users#destroy')
+      end
+    end
+
+    context 'session' do
+      it 'routes to devise/sessions#create' do
+        expect(post('/current_user/session')).to route_to('devise/sessions#create')
+      end
+      it 'routes to devise/sessions#destroy' do
+        expect(delete('/current_user/session')).to route_to('devise/sessions#destroy')
+      end
+      it 'routes to devise/sessions#new' do
+        expect(get('/current_user/session/new')).to route_to('devise/sessions#new')
+      end
+    end
+
+    context 'password' do
+      it 'routes to devise/passwords#create' do
+        expect(post('/current_user/password')).to route_to('devise/passwords#create')
+      end
+      it 'routes to devise/passwords#update' do
+        expect(put('/current_user/password')).to route_to('devise/passwords#update')
+      end
+      it 'routes to devise/passwords#new' do
+        expect(get('/current_user/password/new')).to route_to('devise/passwords#new')
+      end
+      it 'routes to devise/passwords#edit' do
+        expect(get('/current_user/password/edit')).to route_to('devise/passwords#edit')
+      end
+    end
+  end
+
+  describe %|devise_for :rails_programmers, path_names: {current_rails_programmer: 'me'}| do
+    context 'registration' do
+      it 'routes to rails_programmers#create' do
+        expect(post('/rails_programmers')).to route_to('rails_programmers#create')
+      end
+      it 'routes to rails_programmers#new' do
+        expect(get('/rails_programmers/new')).to route_to('rails_programmers#new')
+      end
+      it 'routes to me#cancel' do
+        expect(get('/me/cancel')).to route_to('me#cancel')
+      end
+      it 'routes to me#edit' do
+        expect(get('/me/edit')).to route_to('me#edit')
+      end
+      it 'routes to me#update' do
+        expect(put('/me')).to route_to('me#update')
+        #expect(patch('/me')).to route_to('me#update')
+      end
+      it 'routes to me#destroy' do
+        expect(delete('/me')).to route_to('me#destroy')
+      end
+    end
+
+    context 'session' do
+      it 'routes to devise/sessions#create' do
+        expect(post('/me/session')).to route_to('devise/sessions#create')
+      end
+      it 'routes to devise/sessions#destroy' do
+        expect(delete('/me/session')).to route_to('devise/sessions#destroy')
+      end
+      it 'routes to devise/sessions#new' do
+        expect(get('/me/session/new')).to route_to('devise/sessions#new')
+      end
+    end
+
+    context 'password' do
+      it 'routes to devise/passwords#create' do
+        expect(post('/me/password')).to route_to('devise/passwords#create')
+      end
+      it 'routes to devise/passwords#update' do
+        expect(put('/me/password')).to route_to('devise/passwords#update')
+      end
+      it 'routes to devise/passwords#new' do
+        expect(get('/me/password/new')).to route_to('devise/passwords#new')
+      end
+      it 'routes to devise/passwords#edit' do
+        expect(get('/me/password/edit')).to route_to('devise/passwords#edit')
+      end
+    end
+  end
+
+  describe %|devise_for :programmers, controllers: {programmers: 'rails_programmers', current_programmer: 'me'}| do
+    context 'registration' do
+      it 'routes to rails_programmers#create' do
+        expect(post('/programmers')).to route_to('rails_programmers#create')
+      end
+      it 'routes to rails_programmers#new' do
+        expect(get('/programmers/new')).to route_to('rails_programmers#new')
+      end
+      it 'routes to me#cancel' do
+        expect(get('/current_programmer/cancel')).to route_to('me#cancel')
+      end
+      it 'routes to me#edit' do
+        expect(get('/current_programmer/edit')).to route_to('me#edit')
+      end
+      it 'routes to me#update' do
+        expect(put('/current_programmer')).to route_to('me#update')
+        #expect(patch('/current_programmer')).to route_to('me#update')
+      end
+      it 'routes to me#destroy' do
+        expect(delete('/current_programmer')).to route_to('me#destroy')
+      end
+    end
+
+    context 'session' do
+      it 'routes to devise/sessions#create' do
+        expect(post('/current_programmer/session')).to route_to('devise/sessions#create')
+      end
+      it 'routes to devise/sessions#destroy' do
+        expect(delete('/current_programmer/session')).to route_to('devise/sessions#destroy')
+      end
+      it 'routes to devise/sessions#new' do
+        expect(get('/current_programmer/session/new')).to route_to('devise/sessions#new')
+      end
+    end
+
+    context 'password' do
+      it 'routes to devise/passwords#create' do
+        expect(post('/current_programmer/password')).to route_to('devise/passwords#create')
+      end
+      it 'routes to devise/passwords#update' do
+        expect(put('/current_programmer/password')).to route_to('devise/passwords#update')
+      end
+      it 'routes to devise/passwords#new' do
+        expect(get('/current_programmer/password/new')).to route_to('devise/passwords#new')
+      end
+      it 'routes to devise/passwords#edit' do
+        expect(get('/current_programmer/password/edit')).to route_to('devise/passwords#edit')
+      end
+    end
+  end
+
+  describe %|devise_for :engineers, controllers: {registrations: 'rails_programmers'}| do
+    context 'registration' do
+      it 'routes to rails_programmers#create' do
+        expect(post('/engineers')).to route_to('rails_programmers#create')
+      end
+      it 'routes to rails_programmers#new' do
+        expect(get('/engineers/new')).to route_to('rails_programmers#new')
+      end
+      it 'routes to rails_programmers#cancel' do
+        expect(get('/current_engineer/cancel')).to route_to('rails_programmers#cancel')
+      end
+      it 'routes to rails_programmers#edit' do
+        expect(get('/current_engineer/edit')).to route_to('rails_programmers#edit')
+      end
+      it 'routes to rails_programmers#update' do
+        expect(put('/current_engineer')).to route_to('rails_programmers#update')
+        #expect(patch('/current_engineer')).to route_to('rails_programmers#update')
+      end
+      it 'routes to rails_programmers#destroy' do
+        expect(delete('/current_engineer')).to route_to('rails_programmers#destroy')
+      end
+    end
+
+    context 'session' do
+      it 'routes to devise/sessions#create' do
+        expect(post('/current_engineer/session')).to route_to('devise/sessions#create')
+      end
+      it 'routes to devise/sessions#destroy' do
+        expect(delete('/current_engineer/session')).to route_to('devise/sessions#destroy')
+      end
+      it 'routes to devise/sessions#new' do
+        expect(get('/current_engineer/session/new')).to route_to('devise/sessions#new')
+      end
+    end
+
+    context 'password' do
+      it 'routes to devise/passwords#create' do
+        expect(post('/current_engineer/password')).to route_to('devise/passwords#create')
+      end
+      it 'routes to devise/passwords#update' do
+        expect(put('/current_engineer/password')).to route_to('devise/passwords#update')
+      end
+      it 'routes to devise/passwords#new' do
+        expect(get('/current_engineer/password/new')).to route_to('devise/passwords#new')
+      end
+      it 'routes to devise/passwords#edit' do
+        expect(get('/current_engineer/password/edit')).to route_to('devise/passwords#edit')
+      end
+    end
+  end
+end

data/spec/devise/better_routes_spec.rb

@@ -4,283 +4,4 @@ describe Devise::BetterRoutes do
   it 'expect to have a version number' do
     expect(Devise::BetterRoutes::VERSION).not_to be_nil
   end
-
-  context 'routing' do
-    include RSpec::Rails::RoutingExampleGroup
-    describe 'users' do
-      context 'registration' do
-        it 'routes to users#create' do
-          expect(post('/users')).to route_to('users#create')
-        end
-        it 'routes to users#new' do
-          expect(get('/users/new')).to route_to('users#new')
-        end
-        it 'routes to current_users#cancel' do
-          expect(get('/current_user/cancel')).to route_to('current_users#cancel')
-        end
-        it 'routes to current_users#edit' do
-          expect(get('/current_user/edit')).to route_to('current_users#edit')
-        end
-        it 'routes to current_users#update' do
-          expect(put('/current_user')).to route_to('current_users#update')
-          #expect(patch('/current_user')).to route_to('current_users#update')
-        end
-        it 'routes to current_users#destroy' do
-          expect(delete('/current_user')).to route_to('current_users#destroy')
-        end
-      end
-
-      context 'session' do
-        it 'routes to devise/sessions#create' do
-          expect(post('/current_user/session')).to route_to('devise/sessions#create')
-        end
-        it 'routes to devise/sessions#destroy' do
-          expect(delete('/current_user/session')).to route_to('devise/sessions#destroy')
-        end
-        it 'routes to devise/sessions#new' do
-          expect(get('/current_user/session/new')).to route_to('devise/sessions#new')
-        end
-      end
-    end
-
-    describe 'rails_programmers' do
-      context 'registration' do
-        it 'routes to rails_programmers#create' do
-          expect(post('/rails_programmers')).to route_to('rails_programmers#create')
-        end
-        it 'routes to rails_programmers#new' do
-          expect(get('/rails_programmers/new')).to route_to('rails_programmers#new')
-        end
-        it 'routes to me#cancel' do
-          expect(get('/me/cancel')).to route_to('me#cancel')
-        end
-        it 'routes to me#edit' do
-          expect(get('/me/edit')).to route_to('me#edit')
-        end
-        it 'routes to me#update' do
-          expect(put('/me')).to route_to('me#update')
-          #expect(patch('/me')).to route_to('me#update')
-        end
-        it 'routes to me#destroy' do
-          expect(delete('/me')).to route_to('me#destroy')
-        end
-      end
-
-      context 'session' do
-        it 'routes to devise/sessions#create' do
-          expect(post('/me/session')).to route_to('devise/sessions#create')
-        end
-        it 'routes to devise/sessions#destroy' do
-          expect(delete('/me/session')).to route_to('devise/sessions#destroy')
-        end
-        it 'routes to devise/sessions#new' do
-          expect(get('/me/session/new')).to route_to('devise/sessions#new')
-        end
-      end
-    end
-
-    describe 'programmers' do
-      context 'registration' do
-        it 'routes to rails_programmers#create' do
-          expect(post('/programmers')).to route_to('rails_programmers#create')
-        end
-        it 'routes to rails_programmers#new' do
-          expect(get('/programmers/new')).to route_to('rails_programmers#new')
-        end
-        it 'routes to me#cancel' do
-          expect(get('/current_programmer/cancel')).to route_to('me#cancel')
-        end
-        it 'routes to me#edit' do
-          expect(get('/current_programmer/edit')).to route_to('me#edit')
-        end
-        it 'routes to me#update' do
-          expect(put('/current_programmer')).to route_to('me#update')
-          #expect(patch('/current_programmer')).to route_to('me#update')
-        end
-        it 'routes to me#destroy' do
-          expect(delete('/current_programmer')).to route_to('me#destroy')
-        end
-      end
-
-      context 'session' do
-        it 'routes to devise/sessions#create' do
-          expect(post('/current_programmer/session')).to route_to('devise/sessions#create')
-        end
-        it 'routes to devise/sessions#destroy' do
-          expect(delete('/current_programmer/session')).to route_to('devise/sessions#destroy')
-        end
-        it 'routes to devise/sessions#new' do
-          expect(get('/current_programmer/session/new')).to route_to('devise/sessions#new')
-        end
-      end
-    end
-
-    describe 'engineers' do
-      context 'registration' do
-        it 'routes to rails_programmers#create' do
-          expect(post('/engineers')).to route_to('rails_programmers#create')
-        end
-        it 'routes to rails_programmers#new' do
-          expect(get('/engineers/new')).to route_to('rails_programmers#new')
-        end
-        it 'routes to rails_programmers#cancel' do
-          expect(get('/current_engineer/cancel')).to route_to('rails_programmers#cancel')
-        end
-        it 'routes to rails_programmers#edit' do
-          expect(get('/current_engineer/edit')).to route_to('rails_programmers#edit')
-        end
-        it 'routes to rails_programmers#update' do
-          expect(put('/current_engineer')).to route_to('rails_programmers#update')
-          #expect(patch('/current_engineer')).to route_to('rails_programmers#update')
-        end
-        it 'routes to rails_programmers#destroy' do
-          expect(delete('/current_engineer')).to route_to('rails_programmers#destroy')
-        end
-      end
-
-      context 'session' do
-        it 'routes to devise/sessions#create' do
-          expect(post('/current_engineer/session')).to route_to('devise/sessions#create')
-        end
-        it 'routes to devise/sessions#destroy' do
-          expect(delete('/current_engineer/session')).to route_to('devise/sessions#destroy')
-        end
-        it 'routes to devise/sessions#new' do
-          expect(get('/current_engineer/session/new')).to route_to('devise/sessions#new')
-        end
-      end
-    end
-  end
-
-  context 'URL helpers' do
-    include Rails.application.routes.url_helpers
-    include Devise::Controllers::UrlHelpers
-    describe 'users' do
-      context 'registration' do
-        specify { expect(users_path).to eq '/users' }
-        specify { expect(new_user_path).to eq '/users/new' }
-        specify { expect(cancel_current_user_path).to eq '/current_user/cancel' }
-        specify { expect(edit_current_user_path).to eq '/current_user/edit' }
-        specify { expect(current_user_path).to eq '/current_user' }
-        it 'registration_path is delegated to users_path' do
-          expect(registration_path(:user)).to eq users_path
-        end
-        it 'registration_path is delegated to current_user_path on current_users' do
-          instance_eval { def controller_name; 'current_users' end }
-          expect(registration_path(:user)).to eq current_user_path
-          instance_eval { undef controller_name }
-        end
-        it 'new_registration_path is delegated to new_user_path' do
-          expect(new_registration_path(:user)).to eq new_user_path
-        end
-        it 'cancel_registration_path is delegated to cancel_current_user_path' do
-          expect(cancel_registration_path(:user)).to eq cancel_current_user_path
-        end
-        it 'edit_registration_path is delegated to edit_current_user_path' do
-          expect(edit_registration_path(:user)).to eq edit_current_user_path
-        end
-      end
-
-      context 'session' do
-        specify { expect(current_user_session_path).to eq '/current_user/session' }
-        specify { expect(new_current_user_session_path).to eq '/current_user/session/new' }
-      end
-    end
-
-    describe 'rails_programmers' do
-      context 'registration' do
-        specify { expect(rails_programmers_path).to eq '/rails_programmers' }
-        specify { expect(new_rails_programmer_path).to eq '/rails_programmers/new' }
-        specify { expect(cancel_me_path).to eq '/me/cancel' }
-        specify { expect(edit_me_path).to eq '/me/edit' }
-        specify { expect(me_path).to eq '/me' }
-        it 'registration_path is delegated to rails_programmers_path' do
-          expect(registration_path(:rails_programmer)).to eq rails_programmers_path
-        end
-        it 'registration_path is delegated to me_path on me' do
-          instance_eval { def controller_name; 'me' end }
-          expect(registration_path(:rails_programmer)).to eq me_path
-          instance_eval { undef controller_name }
-        end
-        it 'new_registration_path is delegated to new_rails_programmer_path' do
-          expect(new_registration_path(:rails_programmer)).to eq new_rails_programmer_path
-        end
-        it 'cancel_registration_path is delegated to cancel_me_path' do
-          expect(cancel_registration_path(:rails_programmer)).to eq cancel_me_path
-        end
-        it 'edit_registration_path is delegated to edit_me_path' do
-          expect(edit_registration_path(:rails_programmer)).to eq edit_me_path
-        end
-      end
-
-      context 'session' do
-        specify { expect(me_session_path).to eq '/me/session' }
-        specify { expect(new_me_session_path).to eq '/me/session/new' }
-      end
-    end
-
-    describe 'programmers' do
-      context 'registration' do
-        specify { expect(programmers_path).to eq '/programmers' }
-        specify { expect(new_programmer_path).to eq '/programmers/new' }
-        specify { expect(cancel_current_programmer_path).to eq '/current_programmer/cancel' }
-        specify { expect(edit_current_programmer_path).to eq '/current_programmer/edit' }
-        specify { expect(current_programmer_path).to eq '/current_programmer' }
-        it 'registration_path is delegated to programmers_path' do
-          expect(registration_path(:programmer)).to eq programmers_path
-        end
-        it 'registration_path is delegated to current_programmer_path on me' do
-          instance_eval { def controller_name; 'me' end }
-          expect(registration_path(:programmer)).to eq current_programmer_path
-          instance_eval { undef controller_name }
-        end
-        it 'new_registration_path is delegated to new_programmer_path' do
-          expect(new_registration_path(:programmer)).to eq new_programmer_path
-        end
-        it 'cancel_registration_path is delegated to cancel_current_programmer_path' do
-          expect(cancel_registration_path(:programmer)).to eq cancel_current_programmer_path
-        end
-        it 'edit_registration_path is delegated to edit_current_programmer_path' do
-          expect(edit_registration_path(:programmer)).to eq edit_current_programmer_path
-        end
-      end
-
-      context 'session' do
-        specify { expect(current_programmer_session_path).to eq '/current_programmer/session' }
-        specify { expect(new_current_programmer_session_path).to eq '/current_programmer/session/new' }
-      end
-    end
-
-    describe 'engineers' do
-      context 'registration' do
-        specify { expect(engineers_path).to eq '/engineers' }
-        specify { expect(new_engineer_path).to eq '/engineers/new' }
-        specify { expect(cancel_current_engineer_path).to eq '/current_engineer/cancel' }
-        specify { expect(edit_current_engineer_path).to eq '/current_engineer/edit' }
-        specify { expect(current_engineer_path).to eq '/current_engineer' }
-        it 'registration_path is delegated to engineers_path' do
-          expect(registration_path(:engineer)).to eq engineers_path
-        end
-        it 'registration_path is delegated to current_engineer_path on me' do
-          instance_eval { def controller_name; 'rails_programmers' end }
-          expect(registration_path(:engineer)).to eq current_engineer_path
-          instance_eval { undef controller_name }
-        end
-        it 'new_registration_path is delegated to new_engineer_path' do
-          expect(new_registration_path(:engineer)).to eq new_engineer_path
-        end
-        it 'cancel_registration_path is delegated to cancel_current_engineer_path' do
-          expect(cancel_registration_path(:engineer)).to eq cancel_current_engineer_path
-        end
-        it 'edit_registration_path is delegated to edit_current_engineer_path' do
-          expect(edit_registration_path(:engineer)).to eq edit_current_engineer_path
-        end
-      end
-
-      context 'session' do
-        specify { expect(current_engineer_session_path).to eq '/current_engineer/session' }
-        specify { expect(new_current_engineer_session_path).to eq '/current_engineer/session/new' }
-      end
-    end
-  end
 end

data/spec/rails_app/app/models/user.rb

@@ -1,5 +1,5 @@
 class User < ActiveRecord::Base
-  devise :database_authenticatable, :registerable
+  devise :database_authenticatable, :registerable, :recoverable
 end
 class RailsProgrammer < User; end
 class Programmer < User; end

data/spec/spec_helper.rb

@@ -6,5 +6,6 @@ require 'rspec/rails'
 require 'rspec/autorun'
 
 RSpec.configure do |config|
-  # some config ...
+  config.include Rails.application.routes.url_helpers, type: :helper
+  config.include Devise::Controllers::UrlHelpers, type: :helper
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: devise-better_routes
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-07-12 00:00:00.000000000 Z
+date: 2013-07-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: devise
@@ -140,6 +140,8 @@ files:
 - devise-better_routes.gemspec
 - lib/devise/better_routes.rb
 - lib/devise/better_routes/version.rb
+- spec/devise/better_routes_helpers_spec.rb
+- spec/devise/better_routes_routing_spec.rb
 - spec/devise/better_routes_spec.rb
 - spec/rails_app/Rakefile
 - spec/rails_app/app/assets/images/.keep
@@ -198,7 +200,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -574275781529839639
+      hash: -4129266091696730390
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -207,7 +209,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -574275781529839639
+      hash: -4129266091696730390
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.24
@@ -215,6 +217,8 @@ signing_key:
 specification_version: 3
 summary: Better routes patch for Devise
 test_files:
+- spec/devise/better_routes_helpers_spec.rb
+- spec/devise/better_routes_routing_spec.rb
 - spec/devise/better_routes_spec.rb
 - spec/rails_app/Rakefile
 - spec/rails_app/app/assets/images/.keep