merb-auth-slice-password-reset 1.1.0

data/lib/merb-auth-slice-password-reset/mixins/senile_user/mm_senile_user.rb

@@ -0,0 +1,22 @@
+module Merb
+  class Authentication
+    module Mixins
+      module SenileUser
+        module MMClassMethods
+          def self.extended(base)
+            base.class_eval do
+              key :password_reset_code, String
+            end # base.class_eval
+            def find_with_password_reset_code(code)
+              find(:first, :conditions => {:password_reset_code => code})
+            end
+            
+            def find_with_login_param(param_name, value)
+              find(:first, :conditions => {param_name => value})
+            end
+          end # self.extended
+        end # MMClassMethods
+      end # SenileUser
+    end # Mixins
+  end # Authentication
+end #Merb

data/lib/merb-auth-slice-password-reset/mixins/senile_user/sq_senile_user.rb

@@ -0,0 +1,20 @@
+module Merb
+  class Authentication
+    module Mixins
+      module SenileUser
+        module SQClassMethods
+          def self.extended(base)
+            def find_with_password_reset_code(code)
+              self[:password_reset_code => code]
+            end
+            def find_with_login_param(param_name, value)
+              self[param_name => value]
+            end
+          end # self.extended
+        end # SQClassMethods
+        module SQInstanceMethods
+        end # SQInstanceMethods
+      end # SenileUser
+    end # Mixins
+  end # Authentication
+end # Merb

data/lib/merb-auth-slice-password-reset/slicetasks.rb

@@ -0,0 +1,18 @@
+namespace :slices do
+  namespace :"merb-auth-slice-password-reset" do 
+    
+    # add your own merb-auth-slice-password-reset tasks here
+    
+    # implement this to test for structural/code dependencies
+    # like certain directories or availability of other files
+    desc "Test for any dependencies"
+    task :preflight do
+    end
+    
+    # implement this to perform any database related setup steps
+    desc "Migrate the database"
+    task :migrate do
+    end
+    
+  end
+end

data/lib/merb-auth-slice-password-reset/spectasks.rb

@@ -0,0 +1,75 @@
+namespace :slices do
+  namespace :"merb-auth-slice-password-reset" do
+
+    desc "Run slice specs within the host application context"
+    task :spec => [ "spec:explain", "spec:default" ]
+
+    namespace :spec do
+
+      slice_root = File.expand_path(File.join(File.dirname(__FILE__), '..', '..'))
+
+      task :explain do
+        puts "\nNote: By running MerbAuthSlicePasswordReset specs inside the application context any\n" +
+        "overrides could break existing specs. This isn't always a problem,\n" +
+        "especially in the case of views. Use these spec tasks to check how\n" +
+        "well your application conforms to the original slice implementation."
+      end
+
+      Spec::Rake::SpecTask.new('default') do |t|
+        t.spec_opts = ["--format", "specdoc", "--colour"]
+        t.spec_files = Dir["#{slice_root}/spec/**/*_spec.rb"].sort
+      end
+
+      desc "Run all model specs, run a spec for a specific Model with MODEL=MyModel"
+      Spec::Rake::SpecTask.new('model') do |t|
+        t.spec_opts = ["--format", "specdoc", "--colour"]
+        if(ENV['MODEL'])
+          t.spec_files = Dir["#{slice_root}/spec/models/**/#{ENV['MODEL']}_spec.rb"].sort
+        else
+          t.spec_files = Dir["#{slice_root}/spec/models/**/*_spec.rb"].sort
+        end
+      end
+
+      desc "Run all controller specs, run a spec for a specific Controller with CONTROLLER=MyController"
+      Spec::Rake::SpecTask.new('controller') do |t|
+        t.spec_opts = ["--format", "specdoc", "--colour"]
+        if(ENV['CONTROLLER'])
+          t.spec_files = Dir["#{slice_root}/spec/controllers/**/#{ENV['CONTROLLER']}_spec.rb"].sort
+        else
+          t.spec_files = Dir["#{slice_root}/spec/controllers/**/*_spec.rb"].sort
+        end
+      end
+
+      desc "Run all view specs, run specs for a specific controller (and view) with CONTROLLER=MyController (VIEW=MyView)"
+      Spec::Rake::SpecTask.new('view') do |t|
+        t.spec_opts = ["--format", "specdoc", "--colour"]
+        if(ENV['CONTROLLER'] and ENV['VIEW'])
+          t.spec_files = Dir["#{slice_root}/spec/views/**/#{ENV['CONTROLLER']}/#{ENV['VIEW']}*_spec.rb"].sort
+        elsif(ENV['CONTROLLER'])
+          t.spec_files = Dir["#{slice_root}/spec/views/**/#{ENV['CONTROLLER']}/*_spec.rb"].sort
+        else
+          t.spec_files = Dir["#{slice_root}/spec/views/**/*_spec.rb"].sort
+        end
+      end
+
+      desc "Run all mailer specs, run a spec for a specific Mailer with MAILER=MyMailer"
+      Spec::Rake::SpecTask.new('mailer') do |t|
+        t.spec_opts = ["--format", "specdoc", "--colour"]
+        if(ENV['MAILER'])
+          t.spec_files = Dir["#{slice_root}/spec/mailer/**/#{ENV['MAILER']}_spec.rb"].sort
+        else
+          t.spec_files = Dir["#{slice_root}/spec/mailers/**/*_spec.rb"].sort
+        end
+      end
+
+      desc "Run all specs and output the result in html"
+      Spec::Rake::SpecTask.new('html') do |t|
+        t.spec_opts = ["--format", "html"]
+        t.libs = ['lib', 'server/lib' ]
+        t.spec_files = Dir["#{slice_root}/spec/**/*_spec.rb"].sort
+      end
+
+    end
+
+  end
+end

data/public/stylesheets/master.css

@@ -0,0 +1,2 @@
+html, body { margin: 0; padding: 0; }
+#container { width: 800px; margin: 4em auto; padding: 4em 4em 6em 4em; background: #DDDDDD; }

data/spec/mailers/password_reset_mailer_spec.rb

@@ -0,0 +1,50 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe "PasswordResetMailer" do
+
+  before(:all) do
+    Merb::Router.prepare { add_slice(:merb_auth_slice_password_reset)}
+    User.auto_migrate!
+  end
+
+  after(:all) do
+    Merb::Router.reset!
+  end
+
+  describe MerbAuthSlicePasswordReset::PasswordResetMailer do
+
+    def deliver(action, mail_opts= {},opts = {})
+      MerbAuthSlicePasswordReset::PasswordResetMailer.dispatch_and_deliver action, mail_opts, opts
+      @delivery = Merb::Mailer.deliveries.last
+    end
+
+    before(:each) do
+      @u = User.new(:email => "homer@simpsons.com", :login => "homer")
+      @u.send(:password_reset_code=, "12345")
+      @mailer_params = { :from => "info@mysite.com", :to => @u.email, :subject => "Welcome to MySite.com" }
+    end
+
+    after(:each) do
+      Merb::Mailer.deliveries.clear
+    end
+
+    it "should send mail to homer@simpsons.com for the password reset email" do
+      deliver(:password_reset, @mailer_params, :user => @u)
+      @delivery.assigns(:headers).should include("to: homer@simpsons.com")
+    end
+
+    it "should send the mail from 'info@mysite.com' for the the password reset email" do
+      deliver(:password_reset, @mailer_params, :user => @u)
+      @delivery.assigns(:headers).should include("from: info@mysite.com")
+    end
+
+    it "should mention the password reset link in the the password reset emails" do
+      deliver(:password_reset, @mailer_params, :user => @u)
+      the_url = MerbAuthSlicePasswordReset::PasswordResetMailer.new.slice_url(:reset_password, :password_reset_code => @u.password_reset_code)
+      the_url.should_not be_nil
+      @delivery.text.should include(the_url)
+    end
+
+  end
+
+end

data/spec/mixins/senile_user_spec.rb

@@ -0,0 +1,111 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe "Senile User" do
+
+  before(:all) do
+    Merb::Router.prepare { add_slice(:merb_auth_slice_password_reset) }
+  end
+
+  after(:all) do
+    Merb::Router.reset!
+  end
+
+  describe "SenileUser Mixin" do
+
+    include SenileUserSpecHelper
+
+    before(:all) do
+      User.auto_migrate!
+    end
+
+    before(:each) do
+      @user = User.new(user_attributes)
+    end
+
+    after(:each) do
+      User.all.destroy!
+    end
+
+    it "should add the 'password_reset_code' property to the user model" do
+      @user.should respond_to(:password_reset_code)
+      @user.should respond_to(:password_reset_code=)
+    end
+
+  end
+
+
+  # describe "SenileUser Mixin Activation Code" do
+  #
+  #   include SenileUserSpecHelper
+  #
+  #   before(:all) do
+  #     User.auto_migrate!
+  #   end
+  #
+  #   after(:each) do
+  #     User.all.destroy!
+  #   end
+  #
+  #   before(:each) do
+  #     @user = User.new(user_attributes)
+  #   end
+  #
+  #   it "should set the activation_code" do
+  #     @user.activation_code.should be_nil
+  #     @user.save
+  #     @user.activation_code.should_not be_nil
+  #   end
+  # end
+  #
+  #
+  # describe "Activation" do
+  #
+  #   include SenileUserSpecHelper
+  #
+  #   before(:all) do
+  #     User.auto_migrate!
+  #   end
+  #
+  #   before(:each) do
+  #     User.all.destroy!
+  #     @user = User.create(user_attributes)
+  #   end
+  #
+  #   after(:each) do
+  #     User.all.destroy!
+  #   end
+  #
+  #   it "should mark users as active" do
+  #     @user.should_not be_active
+  #     @user.activate
+  #     @user.should be_active
+  #     @user.reload
+  #     @user.should be_active
+  #   end
+  #
+  #   it "should mark user as (just) activated" do
+  #     @user.activate
+  #     @user.should be_recently_activated
+  #   end
+  #
+  #   it "should set the activated_at property to the current date and time" do
+  #     now = DateTime.now
+  #     DateTime.should_receive(:now).and_return(now)
+  #     @user.activate
+  #     @user.activated_at.should == now
+  #   end
+  #
+  #   it "should clear out the activation code" do
+  #     @user.activation_code.should_not be_nil
+  #     @user.activate
+  #     @user.activation_code.should be_nil
+  #   end
+  #
+  #   it "should send out the activation notification" do
+  #     @user.should_receive(:send_activation_notification)
+  #     @user.activate
+  #   end
+  #
+  # end
+
+end

data/spec/requests/passwords_spec.rb

@@ -0,0 +1,85 @@
+require File.join(File.dirname(__FILE__), '..', 'spec_helper.rb')
+ 
+describe "passwords" do
+  # It seems I can't spec requests unless I add an application.html.erb file.
+  # So the following two lines and the associated code in the before and after
+  # blocks create and delete that file.
+  LAYOUT_FILE = Merb.root / "app/views/layout/application.html.erb"
+  @layout_file_exits = false
+
+  before(:all) do
+    Merb::Router.reset!
+    Merb::Router.prepare { slice(:merb_auth_slice_password_reset) }
+    User.auto_migrate!
+    
+    @u = User.create(:email => "homer@simpsons.com", :login => "homer")
+    @u.generate_password_reset_code
+    @code = @u.password_reset_code
+    
+    if File.exists?(LAYOUT_FILE)
+      @layout_file_exits = true
+    else
+      File.open(LAYOUT_FILE, 'w') {|f| f.write("<%= catch_content :for_layout %>") }
+    end
+  end
+
+  after(:all) do
+    Merb::Router.reset!
+    
+    File.delete(LAYOUT_FILE) unless @layout_file_exits
+  end
+
+  describe "reset" do
+    
+    before(:each) do
+      @response = request("/reset_password/#{@code}")
+    end
+        
+    it "should respond successfully" do
+      @response.should be_successful
+    end
+    
+    it "should return a form to the user" do
+      @response.should have_xpath("//form")
+    end
+    
+    it "should send the completed form params to /reset_check" do
+      @response.should have_xpath("//form[@action='/reset_check/#{@code}']")
+    end
+    
+    it "should have password and password_confirmation attributes wrapped in a user object" do
+      @response.should have_xpath("//input[@name='user[password]']")
+      @response.should have_xpath("//input[@name='user[password_confirmation]']")      
+    end
+    
+    it "should raise a 404 if the code doesn't match a user" do
+      request("/reset_password/nonsense").status.should == 404
+    end
+        
+  end
+
+  describe "reset_check" do
+
+    before(:each) do
+      @response = request("/reset_check/#{@code}", :method => "POST",
+        :params => { :user => { :password => :blah, :password_confirmation => :blah } } )
+    end
+
+    it "should raise a 404 if the code doesn't match a user" do
+      request("/reset_check/nonsense").status.should == 404
+    end
+
+    it "should redirect to root if the user is saved" do
+      @response.should redirect_to("/", :message => {:notice => "Your password has been changed"})
+    end
+
+    it "should render the reset form if the user can't be saved" do
+      class User; def update(val); return false; end; end
+      response = request("/reset_check/#{@code}", :method => "POST",
+        :params => { :user => { :password => :blah, :password_confirmation => :blah } } )
+      response.should have_xpath("//form[@action='/reset_check/#{@code}']")
+    end
+    
+  end
+
+end

data/spec/spec_helper.rb

@@ -0,0 +1,61 @@
+require 'rubygems'
+require 'merb-core'
+require 'merb-slices'
+require 'spec'
+require 'dm-core'
+require 'dm-validations'
+
+# Add merb-auth-slice-password-reset.rb to the search path
+Merb::Plugins.config[:merb_slices][:auto_register] = true
+Merb::Plugins.config[:merb_slices][:search_path]   = File.join(File.dirname(__FILE__), '..', 'lib', 'merb-auth-slice-password-reset.rb')
+
+# Using Merb.root below makes sure that the correct root is set for
+# - testing standalone, without being installed as a gem and no host application
+# - testing from within the host application; its root will be used
+Merb.start_environment(
+  :testing => true, 
+  :adapter => 'runner', 
+  :environment => ENV['MERB_ENV'] || 'test',
+  :merb_root => Merb.root,
+  :session_store => :memory,
+  :exception_details => true
+)
+
+module Merb
+  module Test
+    module SliceHelper
+      
+      # The absolute path to the current slice
+      def current_slice_root
+        @current_slice_root ||= File.expand_path(File.join(File.dirname(__FILE__), '..'))
+      end
+      
+      # Whether the specs are being run from a host application or standalone
+      def standalone?
+        Merb.root == ::MerbAuthSlicePasswordReset.root
+      end
+      
+    end
+  end
+end
+
+module SenileUserSpecHelper
+  def user_attributes(options = {})
+    { :login => 'fred',
+      :email => 'fred@example.com'
+    }.merge(options)
+  end
+end
+
+class Merb::Mailer
+  self.delivery_method = :test_send
+end
+
+Spec::Runner.configure do |config|
+  config.include(Merb::Test::ViewHelper)
+  config.include(Merb::Test::RouteHelper)
+  config.include(Merb::Test::ControllerHelper)
+  config.include(Merb::Test::SliceHelper)
+  config.include(SenileUserSpecHelper)
+  config.before(:all){ User.auto_migrate! }
+end