devise_masquerade 0.0.4 → 0.0.5

data/Gemfile

@@ -12,11 +12,16 @@ group :test do
   gem 'guard'
   gem 'guard-rspec'
   gem 'guard-bundler'
-  gem 'capybara', '>= 0.4.0'
+  gem 'guard-cucumber'
   gem 'rspec-rails'
   gem 'shoulda'
   gem 'rb-fsevent'
   gem 'factory_girl_rails'
   gem 'database_cleaner'
+  gem 'cucumber'
+  gem 'cucumber-rails'
+  gem 'capybara'
+  gem 'capybara-webkit'
+  gem 'launchy'
 end
 

data/Guardfile

@@ -8,6 +8,12 @@ guard 'rspec', :cli => '--format documentation', :version => 2, :all_after_pass
   watch('spec/spec_helper.rb')                { "spec" }
 end
 
+guard 'cucumber',:all_after_pass => false do
+  watch(%r{^features/.+\.feature$})
+  watch(%r{^features/support/.+$})           { 'features' }
+  watch(%r{^features/step_definitions/(.+)_steps\.rb$}) { |m| Dir[File.join("**/#{m[1]}.feature")][0] || 'features' }
+end
+
 guard 'bundler' do
   watch('Gemfile')
   watch(/^.+\.gemspec/)

data/app/controllers/devise/masquerades_controller.rb

@@ -42,7 +42,7 @@ class Devise::MasqueradesController < DeviseController
   end
 
   def session_key
-    "devise.masquerade.#{resource_name}"
+    "devise_masquerade_#{resource_name}".to_sym
   end
 end
 

data/config/environment.rb

@@ -0,0 +1,2 @@
+require File.expand_path("../../spec/dummy/config/environment.rb",  __FILE__)
+

data/features/back.feature

@@ -0,0 +1,17 @@
+Feature: Use back button for returning to the owner of the masquerade action.
+  In order to back to the owner user
+  As an masquerade user
+  I want to be able to press a simple button on the page
+
+  @devise
+  Scenario: Use back button
+    Given I logged in
+    And I have a user for masquerade
+
+    When I am on the users page
+    And I login as one user
+      Then I should be login as this user
+
+    When I press back masquerade button
+      Then I should be login as owner user
+

data/features/step_definitions/auth_steps.rb

@@ -0,0 +1,11 @@
+Given /^I logged in$/ do
+  @user = create(:user)
+
+  visit '/'
+
+  fill_in 'user[email]', :with => @user.email
+  fill_in 'user[password]', :with => 'password'
+
+  click_on 'Sign in'
+end
+

data/features/step_definitions/back_steps.rb

@@ -0,0 +1,24 @@
+Given /^I have a user for masquerade$/ do
+  @mask = create(:user)
+end
+
+When /^I am on the users page$/ do
+  visit '/'
+end
+
+When /^I login as one user$/ do
+  click_on "Login as"
+end
+
+Then /^I should be login as this user$/ do
+  find('.current_user').should have_content(@mask.email)
+end
+
+When /^I press back masquerade button$/ do
+  click_on "Back masquerade"
+end
+
+Then /^I should be login as owner user$/ do
+  find('.current_user').should have_content(@user.email)
+end
+

data/features/support/env.rb

@@ -0,0 +1,23 @@
+require 'cucumber/rails'
+require 'factory_girl'
+require 'database_cleaner'
+
+Dir[File.join(File.dirname(__FILE__), '..', '..', "spec/support/*.rb")].each {|f| require f}
+
+ENV["RAILS_ENV"] = "test"
+
+Capybara.default_selector = :css
+
+ActionController::Base.allow_rescue = false
+
+World(FactoryGirl::Syntax::Methods)
+
+begin
+  DatabaseCleaner.strategy = :transaction
+rescue NameError
+  raise "You need to add database_cleaner to your Gemfile (in the :test group) if you wish to use it."
+end
+
+Cucumber::Rails::Database.javascript_strategy = :truncation
+Capybara.javascript_driver = :webkit
+

data/lib/devise_masquerade/controllers/helpers.rb

@@ -12,7 +12,15 @@ module DeviseMasquerade
 
             sign_in #{name} if #{name}
           end
+
+          def #{name}_masquerade?
+            session[:"devise_masquerade_#{name}"].present?
+          end
         METHODS
+
+        ActiveSupport.on_load(:action_controller) do
+          helper_method "#{name}_masquerade?"
+        end
       end
     end
   end

data/lib/devise_masquerade/controllers/url_helpers.rb

@@ -6,8 +6,9 @@ module DeviseMasquerade
         send("#{scope}_masquerade_path", resource)
       end
 
-      def back_masquerade_path
-        back_masquerade_path
+      def back_masquerade_path(resource)
+        scope = Devise::Mapping.find_scope!(resource)
+        send("back_#{scope}_masquerade_index_path")
       end
     end
   end

data/lib/devise_masquerade/model.rb

@@ -15,6 +15,9 @@ module Devise
         def self.find_by_masquerade_key(key)
           id = Rails.cache.read("#{self.name.pluralize.downcase}:#{key}:masquerade")
 
+          # clean up the cached masquerade key value
+          Rails.cache.write("#{self.name.pluralize.downcase}:#{key}:masquerade", nil)
+
           self.find_by_id(id)
         end
       end

data/lib/devise_masquerade/routes.rb

@@ -4,9 +4,11 @@ module ActionDispatch::Routing
     protected
 
     def devise_masquerade(mapping, controllers)
-      resources :masquerade, :only => :show,
+      resources :masquerade,
+        :only => :show,
         :path => mapping.path_names[:masquerade],
         :controller => controllers[:masquerades] do
+
         get :back, :on => :collection
       end
     end

data/lib/devise_masquerade/version.rb

@@ -1,3 +1,3 @@
 module DeviseMasquerade
-  VERSION = "0.0.4"
+  VERSION = "0.0.5"
 end

data/spec/dummy/app/controllers/application_controller.rb

@@ -1,6 +1,7 @@
 class ApplicationController < ActionController::Base
   protect_from_forgery
 
+  before_filter :authenticate_user!
   before_filter :masquerade_user!
 end
 

data/spec/dummy/app/controllers/dashboard_controller.rb

@@ -2,7 +2,7 @@ class DashboardController < ApplicationController
   before_filter :masquerade_user!
 
   def index
-    render :text => "text to render"
+    @users = User.where("users.id != ?", current_user.id).all
   end
 end
 

data/spec/dummy/app/views/dashboard/index.html.erb

@@ -0,0 +1,3 @@
+<h1>Users</h1>
+
+<%= render @users %>

data/spec/dummy/app/views/layouts/application.html.erb

@@ -1,14 +1,20 @@
 <!DOCTYPE html>
 <html>
-<head>
-  <title>Dummy</title>
-  <%= stylesheet_link_tag :all %>
-  <%= javascript_include_tag :defaults %>
-  <%= csrf_meta_tag %>
-</head>
-<body>
+  <head>
+    <title>Dummy</title>
+    <%= stylesheet_link_tag :all %>
+    <%= javascript_include_tag :defaults %>
+    <%= csrf_meta_tag %>
+  </head>
+  <body>
+    <% if signed_in? %>
+      <h1 class='current_user'><%= current_user.email %></h1>
 
-<%= yield %>
+      <% if user_masquerade? %>
+        <%= link_to "Back masquerade", back_masquerade_path(current_user) %>
+      <% end %>
+    <% end %>
 
-</body>
+    <%= yield %>
+  </body>
 </html>

data/spec/dummy/app/views/users/_user.html.erb

@@ -0,0 +1,6 @@
+<p>
+  <%= user.email %>
+
+  <%= link_to "Login as", masquerade_path(user) %>
+</p>
+

data/spec/models/user_spec.rb

@@ -16,6 +16,7 @@ describe User do
       user.masquerade!
 
       Rails.cache.should_receive(:read).with("users:#{user.masquerade_key}:masquerade").and_return(user.id)
+      Rails.cache.should_receive(:write).with("users:#{user.masquerade_key}:masquerade", nil)
 
       new_user = User.find_by_masquerade_key(user.masquerade_key)
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: devise_masquerade
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.5
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-11-23 00:00:00.000000000 Z
+date: 2012-11-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -76,7 +76,12 @@ files:
 - README.md
 - Rakefile
 - app/controllers/devise/masquerades_controller.rb
+- config/environment.rb
 - devise_masquerade.gemspec
+- features/back.feature
+- features/step_definitions/auth_steps.rb
+- features/step_definitions/back_steps.rb
+- features/support/env.rb
 - lib/devise_masquerade.rb
 - lib/devise_masquerade/controllers/helpers.rb
 - lib/devise_masquerade/controllers/url_helpers.rb
@@ -91,7 +96,9 @@ files:
 - spec/dummy/app/controllers/dashboard_controller.rb
 - spec/dummy/app/helpers/application_helper.rb
 - spec/dummy/app/models/user.rb
+- spec/dummy/app/views/dashboard/index.html.erb
 - spec/dummy/app/views/layouts/application.html.erb
+- spec/dummy/app/views/users/_user.html.erb
 - spec/dummy/config.ru
 - spec/dummy/config/application.rb
 - spec/dummy/config/boot.rb
@@ -111,6 +118,7 @@ files:
 - spec/dummy/config/routes.rb
 - spec/dummy/db/migrate/20121119085620_devise_create_users.rb
 - spec/dummy/db/schema.rb
+- spec/dummy/public/.empty
 - spec/dummy/script/rails
 - spec/models/user_spec.rb
 - spec/orm/active_record.rb
@@ -142,6 +150,10 @@ signing_key:
 specification_version: 3
 summary: use for login as functionallity on your admin users pages
 test_files:
+- features/back.feature
+- features/step_definitions/auth_steps.rb
+- features/step_definitions/back_steps.rb
+- features/support/env.rb
 - spec/controllers/dashboard_controller_spec.rb
 - spec/controllers/devise/masquerades_controller_spec.rb
 - spec/dummy/Rakefile
@@ -149,7 +161,9 @@ test_files:
 - spec/dummy/app/controllers/dashboard_controller.rb
 - spec/dummy/app/helpers/application_helper.rb
 - spec/dummy/app/models/user.rb
+- spec/dummy/app/views/dashboard/index.html.erb
 - spec/dummy/app/views/layouts/application.html.erb
+- spec/dummy/app/views/users/_user.html.erb
 - spec/dummy/config.ru
 - spec/dummy/config/application.rb
 - spec/dummy/config/boot.rb
@@ -169,6 +183,7 @@ test_files:
 - spec/dummy/config/routes.rb
 - spec/dummy/db/migrate/20121119085620_devise_create_users.rb
 - spec/dummy/db/schema.rb
+- spec/dummy/public/.empty
 - spec/dummy/script/rails
 - spec/models/user_spec.rb
 - spec/orm/active_record.rb