adva_user 0.0.1

data/test/integration/memberships_test.rb

@@ -0,0 +1,52 @@
+require File.expand_path(File.join(File.dirname(__FILE__), '..', 'test_helper' ))
+
+module IntegrationTests
+  class MembershipsTest < ActionController::IntegrationTest
+    def setup
+      super
+      @site = use_site! 'site with pages'
+    end
+
+    test "when superuser creates a new user on the backend,
+          the new user should have a membership to the page it was created on (fix for bug #288)" do
+      login_as_superuser
+      visit_new_user_form
+      post_the_new_user_form
+    end
+    
+    test "when superuser creates a new superuser on the backend the new superuser does not need to 
+          be member of any site" do
+      login_as_superuser
+      visit_new_user_form
+      post_the_new_user_form_for_superuser
+    end
+    
+    def visit_new_user_form
+      visit "/admin/sites/#{@site.id}/users/new"
+      renders_template "admin/users/new"
+    end
+    
+    def post_the_new_user_form
+      fill_in :user_first_name, :with => 'John'
+      fill_in :user_last_name,  :with => 'Doe'
+      fill_in :user_email,      :with => 'memberships@test.org'
+      fill_in :user_password,   :with => 'pass'
+      click_button :save
+      
+      user = User.find_by_email('memberships@test.org')
+      assert !user.memberships.blank?
+    end
+    
+    def post_the_new_user_form_for_superuser
+      fill_in :user_first_name, :with => 'John'
+      fill_in :user_last_name,  :with => 'Superuser'
+      fill_in :user_email,      :with => 'superusers.memberships@test.org'
+      fill_in :user_password,   :with => 'pass'
+      check   :role_superuser
+      click_button :save
+      
+      user = User.find_by_email('superusers.memberships@test.org')
+      assert user.memberships.empty?
+    end
+  end
+end

data/test/integration/user_deletion_test.rb

@@ -0,0 +1,27 @@
+require File.expand_path(File.join(File.dirname(__FILE__), '..', 'test_helper' ))
+
+module IntegrationTests
+  class UserDeletionTest < ActionController::IntegrationTest
+    def setup
+      super
+      @site = use_site! 'site with pages'
+    end
+    
+    test "A user deletes his account" do
+      # user logs in to the site
+      user = login_as_user
+      
+      # TODO
+      # there's no user profile page so far
+      # visit user_path
+      # click_link 'Edit'
+      # click_link 'Delete'
+
+      # user deletes the own profile
+      delete user_path
+
+      # user should not be there anymore
+      lambda { User.find(user.id) }.should raise_error(ActiveRecord::RecordNotFound)
+    end
+  end
+end

data/test/integration/user_login_test.rb

@@ -0,0 +1,53 @@
+require File.expand_path(File.join(File.dirname(__FILE__), '..', 'test_helper' ))
+
+module IntegrationTests
+  class UserLoginTest < ActionController::IntegrationTest
+    def setup
+      super
+      @site = use_site! 'site with pages'
+    end
+  
+    test "A verified user logs in with valid credentials" do
+      visit '/login'
+
+      fill_in 'user_email', :with => 'a-user@example.com'
+      fill_in 'user_password', :with => 'a password'
+      click_button 'login'
+
+      controller.current_user.should_not be_nil
+      controller.current_user.email.should == 'a-user@example.com'
+    end
+  
+    test "A verified user logs in with invalid credentials" do
+      visit '/login'
+      fill_in 'user_email', :with => 'a-user@example.com'
+      fill_in 'user_password', :with => 'a wrong password'
+      click_button 'login'
+
+      controller.current_user.should be_nil
+      renders_template 'session/new'
+    end
+      
+    test "An unverified user logs in with valid credentials" do
+      visit '/login'
+      fill_in 'user_email', :with => 'an-unverified-user@example.com'
+      fill_in 'user_password', :with => 'a password'
+      click_button 'login'
+
+      # FIXME we don't provide any more specific feedback?
+      controller.current_user.should be_nil
+      renders_template 'session/new'
+    end
+      
+    test "An unverified user logs in with invalid credentials" do
+      visit '/login'
+      fill_in 'user_email', :with => 'an-unverified-user@example.com'
+      fill_in 'user_password', :with => 'a wrong password'
+      click_button 'login'
+
+      # FIXME we don't provide any more specific feedback?
+      controller.current_user.should be_nil
+      renders_template 'session/new'
+    end
+  end
+end

data/test/integration/user_login_with_remember_me_test.rb

@@ -0,0 +1,20 @@
+require File.expand_path(File.join(File.dirname(__FILE__), '..', 'test_helper' ))
+
+module IntegrationTests
+  class UserLoginWithRememberMeTest < ActionController::IntegrationTest
+    def setup
+      super
+      @site = use_site! 'site with pages'
+      @user = User.find_by_email('a-superuser@example.com')
+      token = @user.assign_token!('remember me')
+      cookies[:remember_me] = Rack::Utils.escape("#{@user.id};#{token}")
+      cookies[:uid] = @user.id.to_s
+      cookies[:uname] = @user.name
+    end
+    
+    test "User with remember me cookie does not need to login again" do
+      visit '/admin'
+      assert_template 'admin/sites/show'
+    end
+  end
+end

data/test/integration/user_registration_test.rb

@@ -0,0 +1,64 @@
+require File.expand_path(File.join(File.dirname(__FILE__), '..', 'test_helper' ))
+
+module IntegrationTests
+  class UserRegistrationTest < ActionController::IntegrationTest
+    def setup
+      super
+      @site = use_site! 'site with pages'
+    end
+
+    test "User clicks through section frontend section index pages" do
+      registers_to_site_membership
+      verify_the_email_adress
+      check_logged_in
+    end
+
+    def registers_to_site_membership
+      visit "/user/new"
+      renders_template "user/new"
+
+      fill_in "user_first_name", :with => 'John'
+      fill_in "user_email", :with => 'john-doe@test.com'
+      fill_in "user_password", :with => 'password'
+      click_button "register"
+
+      renders_template "user/verification_sent"
+
+      # user record should exist but not be verified
+      john_doe.should_not be_nil
+      john_doe.verified?.should be_false
+
+      # should have sent an email notification
+      verification_email.to.should include(john_doe.email)
+    end
+
+    def verify_the_email_adress
+      # extract the verification url from the email
+      # http://www.example.com/user/verify?token=1%3Bbd69ea84ed49b61623da2c4d74de2936eb0b0229
+      verification_email.body =~ /^(http:.*)$/
+      visit $1
+
+      john_doe.reload.verified?.should be_true
+    end
+
+    def check_logged_in
+      controller.current_user.should == john_doe
+
+      session[:uid].should == john_doe.id
+      cookies['uid'].should == john_doe.id.to_s
+      cookies['uname'].should == CGI.escape(john_doe.name)
+    end
+
+
+    def john_doe
+      @john_doe ||= User.find_by_email('john-doe@test.com')
+    end
+
+    def verification_email
+      @verification_email ||= begin
+        ActionMailer::Base.deliveries.should_not be_empty
+        ActionMailer::Base.deliveries.last
+      end
+    end
+  end
+end

data/test/test_helper.rb

@@ -0,0 +1 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../adva_cms/test/test_helper')

data/test/unit/cells/user_cell_test.rb

@@ -0,0 +1,13 @@
+# FIXME Commented out test! Convert!
+#
+# require File.dirname(__FILE__) + '/../spec_helper'
+# 
+# describe UserCell do
+#   it "renders" do
+#     site = mock('site', :id => 1)
+#     section = mock('section', :id => 1, :track_method_calls => nil) # TODO: use real object?
+#     controller = mock('controller', :perform_caching => false, :site => site, :section => section)
+#     cell = UserCell.new(controller, nil)
+#     cell.render_state(:recent).should =~ /recent users/i
+#   end
+# end

data/test/unit/helpers/users_helper_test.rb

@@ -0,0 +1,52 @@
+require File.expand_path(File.dirname(__FILE__) + "/../../test_helper")
+
+class UsersHelperTest < ActionView::TestCase
+  include UsersHelper
+
+  def setup
+    super
+    
+    @user = User.first
+    @email = 'email@gravatar.com'
+    @md5 = 'eafcd5f6d59e86088dfcf706831b297e'
+
+    stub(self).request.returns(ActionController::TestRequest.new)
+  end
+
+  # who
+  test "#who returns 'You' if the given user is the current user" do
+    stub(self).current_user.returns(@user)
+    who(@user).should == 'You'
+  end
+
+  test "#who returns the given user's name if the given user is not the current user" do
+    stub(self).current_user.returns(nil)
+    who(@user).should == @user.name
+  end
+
+  test "#gravatar_img returns an image tag with the class 'avatar' merged to the given options" do
+    gravatar_img(@user).should have_tag('img[src^=?][class=?]', 'http://www.gravatar.com', 'avatar')
+  end
+
+  test "#gravatar_img adds the gravatar_url for the given user's email adress" do
+    mock(self).gravatar_url(@user.email).returns('gravatar_url')
+    gravatar_img(@user)
+  end
+
+  # gravatar_url
+  test "#gravatar_url returns a default image url if the given email adress is blank" do
+    gravatar_url.should == '/assets/adva_cms/avatar.gif'
+  end
+
+  test "#gravatar_url the resulting gravatar url includes the md5/hexdigested email adress as the gravatar_id" do
+    gravatar_url(@email).should =~ /gravatar_id=#{@md5}/
+  end
+
+  test "#gravatar_url the resulting gravatar url includes a size parameter with the given size" do
+    gravatar_url(@email, 50).should =~ /size=50/
+  end
+
+  test "#gravatar_url the resulting gravatar url includes a default parameter" do
+    gravatar_url(@email).should =~ %r(default=http://test.host/assets/adva_cms/avatar.gif)
+  end
+end

data/test/unit/models/account_test.rb

@@ -0,0 +1,21 @@
+require File.expand_path(File.dirname(__FILE__) + "/../../test_helper")
+
+class AccountTest < ActiveSupport::TestCase
+  def setup
+    super
+    @user1 = User.find_by_first_name('user1')
+    @user2 = User.find_by_first_name('user2')
+    @user3 = User.find_by_first_name('user3')
+    @user4 = User.find_by_first_name('user4')
+
+    @account = Account.find_by_name('an account')
+  end
+
+  test 'creation' do
+    @account.name.should == 'an account'
+    @account.users.should include(@user1)
+    @account.users.should include(@user2)
+    @account.users.should include(@user3)
+    @account.users.should exclude(@user4)
+  end
+end

data/test/unit/models/anonymous_test.rb

@@ -0,0 +1,54 @@
+require File.expand_path(File.dirname(__FILE__) + "/../../test_helper")
+
+class AnonymousTest < ActiveSupport::TestCase
+  def setup
+    super
+    @anonymous = User.anonymous :name => 'John Doe'
+  end
+
+  test 'acts as authenticated user (with single token authentication)' do
+    User.should act_as_authenticated_user
+  end
+
+  # VALIDATIONS
+
+  test 'validates the presence of a name' do
+    @anonymous.should validate_presence_of(:first_name)
+  end
+
+  test 'validates the presence of an email' do
+    @anonymous.should validate_presence_of(:email)
+  end
+
+  test 'validates the length of the name (3-40 chars)' do
+    @anonymous.should validate_length_of(:first_name, :within => 3..40)
+  end
+
+  test 'email format validation succeeds with a valid email address' do
+    @anonymous.email = 'valid@email.org'
+    @anonymous.should be_valid
+  end
+
+  test 'email format validation fails with an invalid email address' do
+    @anonymous.email = 'invalid-email.org'
+    @anonymous.should_not be_valid
+  end
+
+  # INSTANCE METHODS
+
+  test '#has_role? returns true when the passed role is Role::Anonymous' do
+    @anonymous.should have_role(:anonymous)
+  end
+
+  test '#has_role? returns false when the passed role is not Role::Anonymous' do
+    @anonymous.should_not have_role(:user)
+  end
+
+  test '#anonymous? returns true' do
+    @anonymous.should be_anonymous
+  end
+
+  test '#registered? returns false' do
+    @anonymous.should_not be_registered
+  end
+end

data/test/unit/models/password_mailer_test.rb

@@ -0,0 +1,26 @@
+require File.expand_path(File.dirname(__FILE__) + "/../../test_helper")
+
+class PasswordMailerTest < ActiveSupport::TestCase
+
+  test "observes events" do
+    Event.observers.should include('PasswordMailer')
+  end
+
+  test "implements #handle_user_password_reset_requested!" do
+    PasswordMailer.should respond_to(:handle_user_password_reset_requested!)
+  end
+
+  test "implements #handle_user_password_updated!" do
+    PasswordMailer.should respond_to(:handle_user_password_updated!)
+  end
+
+  test "receives #handle_user_password_reset_requested! when a :user_password_reset_requested event is triggered" do
+    mock(PasswordMailer).handle_user_password_reset_requested!.with(anything)
+    Event.trigger(:user_password_reset_requested, User.new, self)
+  end
+
+  test "receives #handle_user_password_updated! when a :user_password_updated event is triggered" do
+    mock(PasswordMailer).handle_user_password_updated!.with(anything)
+    Event.trigger(:user_password_updated, User.new, self)
+  end
+end

data/test/unit/models/user_mailer_test.rb

@@ -0,0 +1,16 @@
+require File.expand_path(File.dirname(__FILE__) + "/../../test_helper")
+
+class UserMailerTest < ActiveSupport::TestCase
+  test "observes events" do
+    Event.observers.should include('UserMailer')
+  end
+
+  test "implements #handle_user_registered!" do
+    UserMailer.should respond_to(:handle_user_registered!)
+  end
+
+  test "receives #handle_user_registered! when a :user_registered event is triggered" do
+    mock(UserMailer).handle_user_registered!.with(anything)
+    Event.trigger(:user_registered, User.new, self)
+  end
+end

data/test/unit/models/user_test.rb

@@ -0,0 +1,173 @@
+require File.expand_path(File.dirname(__FILE__) + "/../../test_helper")
+
+class UserTest < ActiveSupport::TestCase
+  def setup
+    super
+    @user = User.find_by_first_name('a user')
+    @credentials = { :email => @user.email, :password => 'a password' }
+    @valid_user_params = { :email => 'test@example.org', :password => 'test', :first_name => 'name' }
+  end
+
+  test 'acts as authenticated user' do
+    User.should act_as_authenticated_user
+  end
+
+  # ASSOCIATIONS
+
+  test "has many sites" do
+    @user.should have_many(:sites)
+  end
+
+  test "has many memberships" do
+    @user.should have_many(:memberships)
+  end
+
+  test "has many roles" do
+    @user.should have_many(:roles)
+  end
+
+  # VALIDATIONS
+
+  test "validates the presence of a first name" do
+    @user.should validate_presence_of(:first_name)
+  end
+
+  test "validates the presence of an email adress" do
+    @user.should validate_presence_of(:email)
+  end
+
+  test "validates the uniqueness of the email" do
+    @user.should validate_uniqueness_of(:email)
+  end
+
+  test "validates the presence of a password if the password is required" do
+    @user.should validate_presence_of(:password, :if => :password_required?)
+  end
+
+  test "validates the length of the last name" do
+    @user.should validate_length_of(:last_name, :within => 0..40)
+  end
+
+  test "creates a user with blank last name" do
+    @user.last_name = ''
+    @user.save.should be_true
+  end
+
+  test "validates the length of the first name" do
+    @user.should validate_length_of(:first_name, :within => 1..40)
+  end
+
+  test "validates the length of the password" do
+    @user.should validate_length_of(:password, :within => 4..40)
+  end
+
+  # CLASS METHODS
+
+  # User.authenticate
+  test 'User.authenticate returns the user if user#authenticate succeeds' do
+    User.authenticate(@credentials).should == @user
+  end
+
+  test 'User.authenticate returns false if user#authenticate fails' do
+    @credentials[:email] = 'does_not_exist@email.org'
+    User.authenticate(@credentials).should be_false
+  end
+
+  test 'User.authenticate returns false if no user with the given email exists' do
+    @credentials[:password] = 'wrong password'
+    User.authenticate(@credentials).should be_false
+  end
+  
+  # User.admins_and_superusers
+  test 'User.admins_and_superusers returns all site admins and superusers' do
+    admin = User.find_by_first_name('an admin')
+    superuser = User.find_by_first_name('a superuser')
+
+    result = User.admins_and_superusers
+    result.should include(admin)
+    result.should include(superuser)
+    # result.size.should == 2
+  end
+
+  # User.create_superuser
+  test 'User.create_superuser defaults :first_name to the name part of the email' do
+    user = User.create_superuser(:email => 'first.name@example.com')
+    user.first_name == 'first.name'
+  end
+
+  test 'User.create_superuser uses default values for email and password' do
+    user = User.create_superuser(:email => nil, :password => nil, :first_name => nil)
+    user.email.should == 'admin@example.org'
+    user.password.should == 'admin'
+  end
+
+  test 'User.create_superuser uses params values if given' do
+    user = User.create_superuser(@valid_user_params)
+    user.email.should == 'test@example.org'
+    user.password.should == 'test'
+    user.first_name == 'name'
+  end
+
+  test 'User.create_superuser verifies the user' do
+    user = User.create_superuser(@valid_user_params)
+    user.verified?.should be_true
+  end
+
+  test 'User.create_superuser saves the user' do
+    user = User.create_superuser(@valid_user_params)
+    user.new_record?.should be_false
+  end
+
+  test 'assigns the password hash' do
+    user = User.create_superuser(@valid_user_params)
+    user.password_hash.should_not be_blank
+    user.password_salt.should_not be_blank
+  end
+  
+  # INSTANCE METHODS
+  
+  test '#verify! sets the verified_at timestamp and saves the user' do
+    @user.update_attributes(:verified_at => nil)
+    @user.verify!
+    @user.reload.verified_at.should_not be_nil
+  end
+
+  # test '#restore! restores the user record' do
+  #   @user.deleted_at = @time_now
+  #   @user.should_receive(:update_attributes).with :deleted_at => nil
+  #   @user.restore!
+  # end
+
+  test '#anonymous? returns true when anonymous is true' do
+    User.new(:anonymous => true).anonymous?.should be_true
+    User.new.anonymous?.should be_false
+  end
+
+  # registered?
+  test '#registered? returns true when the record is not new' do
+    @user.registered?.should be_true
+    User.new.registered?.should be_false
+  end
+
+  test '#to_s returns the name' do # FIXME hu? where's that used?
+    @user.to_s.should == @user.name
+  end
+
+  test "#homepage returns the http://homepage is homepage is set to 'homepage'" do
+    @user.homepage ='homepage'
+    @user.homepage.should == 'http://homepage'
+  end
+
+  test "#homepage returns the http://homepage is homepage is set to 'http://homepage'" do
+    @user.homepage ='http://homepage'
+    @user.homepage.should == 'http://homepage'
+  end
+
+  test "#homepage returns nil if homepage is not set" do
+    @user.homepage.should be_nil
+  end
+
+  test "#email_with_name returns formatted string to use with email headers" do
+    @user.email_with_name.should == "a user <a-user@example.com>"
+  end
+end