From 569a10db1d4b82975fb4856ac5737f87498373b1 Mon Sep 17 00:00:00 2001 From: Toshi MARUYAMA Date: Mon, 29 Aug 2011 03:57:49 +0000 Subject: [PATCH] remove trailing white-spaces from test/unit/user_test.rb. git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@6715 e93f8b46-1217-0410-a6f0-8f06a7374b81 --- test/unit/user_test.rb | 214 ++++++++++++++++++++--------------------- 1 file changed, 107 insertions(+), 107 deletions(-) diff --git a/test/unit/user_test.rb b/test/unit/user_test.rb index 0fbb7d0c9..1d982241e 100644 --- a/test/unit/user_test.rb +++ b/test/unit/user_test.rb @@ -5,12 +5,12 @@ # modify it under the terms of the GNU General Public License # as published by the Free Software Foundation; either version 2 # of the License, or (at your option) any later version. -# +# # This program is distributed in the hope that it will be useful, # but WITHOUT ANY WARRANTY; without even the implied warranty of # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the # GNU General Public License for more details. -# +# # You should have received a copy of the GNU General Public License # along with this program; if not, write to the Free Software # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. @@ -31,17 +31,17 @@ class UserTest < ActiveSupport::TestCase User.generate_with_protected!(:firstname => 'Testing connection') assert_equal 2, User.count(:all, :conditions => {:firstname => 'Testing connection'}) end - + def test_truth assert_kind_of User, @jsmith end - + def test_mail_should_be_stripped u = User.new u.mail = " foo@bar.com " assert_equal "foo@bar.com", u.mail end - + def test_mail_validation u = User.new u.mail = '' @@ -51,13 +51,13 @@ class UserTest < ActiveSupport::TestCase def test_create user = User.new(:firstname => "new", :lastname => "user", :mail => "newuser@somenet.foo") - + user.login = "jsmith" user.password, user.password_confirmation = "password", "password" # login uniqueness assert !user.save assert_equal 1, user.errors.count - + user.login = "newuser" user.password, user.password_confirmation = "passwd", "password" # password confirmation @@ -79,14 +79,14 @@ class UserTest < ActiveSupport::TestCase end end end - + context "User.login" do should "be case-insensitive." do u = User.new(:firstname => "new", :lastname => "user", :mail => "newuser@somenet.foo") u.login = 'newuser' u.password, u.password_confirmation = "password", "password" assert u.save - + u = User.new(:firstname => "Similar", :lastname => "User", :mail => "similaruser@somenet.foo") u.login = 'NewUser' u.password, u.password_confirmation = "password", "password" @@ -100,7 +100,7 @@ class UserTest < ActiveSupport::TestCase u.login = 'newuser1' u.password, u.password_confirmation = "password", "password" assert u.save - + u = User.new(:firstname => "new", :lastname => "user", :mail => "newUser@Somenet.foo") u.login = 'newuser2' u.password, u.password_confirmation = "password", "password" @@ -115,70 +115,70 @@ class UserTest < ActiveSupport::TestCase @admin.reload assert_equal "john", @admin.login end - + def test_destroy_should_delete_members_and_roles members = Member.find_all_by_user_id(2) ms = members.size rs = members.collect(&:roles).flatten.size - + assert_difference 'Member.count', - ms do assert_difference 'MemberRole.count', - rs do User.find(2).destroy end end - + assert_nil User.find_by_id(2) assert Member.find_all_by_user_id(2).empty? end - + def test_destroy_should_update_attachments attachment = Attachment.create!(:container => Project.find(1), :file => uploaded_test_file("testfile.txt", "text/plain"), :author_id => 2) - + User.find(2).destroy assert_nil User.find_by_id(2) assert_equal User.anonymous, attachment.reload.author end - + def test_destroy_should_update_comments comment = Comment.create!( :commented => News.create!(:project_id => 1, :author_id => 1, :title => 'foo', :description => 'foo'), :author => User.find(2), :comments => 'foo' ) - + User.find(2).destroy assert_nil User.find_by_id(2) assert_equal User.anonymous, comment.reload.author end - + def test_destroy_should_update_issues issue = Issue.create!(:project_id => 1, :author_id => 2, :tracker_id => 1, :subject => 'foo') - + User.find(2).destroy assert_nil User.find_by_id(2) assert_equal User.anonymous, issue.reload.author end - + def test_destroy_should_unassign_issues issue = Issue.create!(:project_id => 1, :author_id => 1, :tracker_id => 1, :subject => 'foo', :assigned_to_id => 2) - + User.find(2).destroy assert_nil User.find_by_id(2) assert_nil issue.reload.assigned_to end - + def test_destroy_should_update_journals issue = Issue.create!(:project_id => 1, :author_id => 2, :tracker_id => 1, :subject => 'foo') issue.init_journal(User.find(2), "update") issue.save! - + User.find(2).destroy assert_nil User.find_by_id(2) assert_equal User.anonymous, issue.journals.first.reload.user end - + def test_destroy_should_update_journal_details_old_value issue = Issue.create!(:project_id => 1, :author_id => 1, :tracker_id => 1, :subject => 'foo', :assigned_to_id => 2) issue.init_journal(User.find(1), "update") @@ -188,12 +188,12 @@ class UserTest < ActiveSupport::TestCase end journal_detail = JournalDetail.first(:order => 'id DESC') assert_equal '2', journal_detail.old_value - + User.find(2).destroy assert_nil User.find_by_id(2) assert_equal User.anonymous.id.to_s, journal_detail.reload.old_value end - + def test_destroy_should_update_journal_details_value issue = Issue.create!(:project_id => 1, :author_id => 1, :tracker_id => 1, :subject => 'foo') issue.init_journal(User.find(1), "update") @@ -203,79 +203,79 @@ class UserTest < ActiveSupport::TestCase end journal_detail = JournalDetail.first(:order => 'id DESC') assert_equal '2', journal_detail.value - + User.find(2).destroy assert_nil User.find_by_id(2) assert_equal User.anonymous.id.to_s, journal_detail.reload.value end - + def test_destroy_should_update_messages board = Board.create!(:project_id => 1, :name => 'Board', :description => 'Board') message = Message.create!(:board_id => board.id, :author_id => 2, :subject => 'foo', :content => 'foo') - + User.find(2).destroy assert_nil User.find_by_id(2) assert_equal User.anonymous, message.reload.author end - + def test_destroy_should_update_news news = News.create!(:project_id => 1, :author_id => 2, :title => 'foo', :description => 'foo') - + User.find(2).destroy assert_nil User.find_by_id(2) assert_equal User.anonymous, news.reload.author end - + def test_destroy_should_delete_private_queries query = Query.new(:name => 'foo', :is_public => false) query.project_id = 1 query.user_id = 2 query.save! - + User.find(2).destroy assert_nil User.find_by_id(2) assert_nil Query.find_by_id(query.id) end - + def test_destroy_should_update_public_queries query = Query.new(:name => 'foo', :is_public => true) query.project_id = 1 query.user_id = 2 query.save! - + User.find(2).destroy assert_nil User.find_by_id(2) assert_equal User.anonymous, query.reload.user end - + def test_destroy_should_update_time_entries entry = TimeEntry.new(:hours => '2', :spent_on => Date.today, :activity => TimeEntryActivity.create!(:name => 'foo')) entry.project_id = 1 entry.user_id = 2 entry.save! - + User.find(2).destroy assert_nil User.find_by_id(2) assert_equal User.anonymous, entry.reload.user end - + def test_destroy_should_delete_tokens token = Token.create!(:user_id => 2, :value => 'foo') - + User.find(2).destroy assert_nil User.find_by_id(2) assert_nil Token.find_by_id(token.id) end - + def test_destroy_should_delete_watchers issue = Issue.create!(:project_id => 1, :author_id => 1, :tracker_id => 1, :subject => 'foo') watcher = Watcher.create!(:user_id => 2, :watchable => issue) - + User.find(2).destroy assert_nil User.find_by_id(2) assert_nil Watcher.find_by_id(watcher.id) end - + def test_destroy_should_update_wiki_contents wiki_content = WikiContent.create!( :text => 'foo', @@ -286,7 +286,7 @@ class UserTest < ActiveSupport::TestCase assert_difference 'WikiContent::Version.count' do wiki_content.save! end - + User.find(2).destroy assert_nil User.find_by_id(2) assert_equal User.anonymous, wiki_content.reload.author @@ -294,15 +294,15 @@ class UserTest < ActiveSupport::TestCase assert_equal User.anonymous, version.reload.author end end - + def test_destroy_should_nullify_issue_categories category = IssueCategory.create!(:project_id => 1, :assigned_to_id => 2, :name => 'foo') - + User.find(2).destroy assert_nil User.find_by_id(2) assert_nil category.reload.assigned_to_id end - + def test_destroy_should_nullify_changesets changeset = Changeset.create!( :repository => Repository::Subversion.create!( @@ -314,31 +314,31 @@ class UserTest < ActiveSupport::TestCase :committer => 'jsmith' ) assert_equal 2, changeset.user_id - + User.find(2).destroy assert_nil User.find_by_id(2) assert_nil changeset.reload.user_id end - + def test_anonymous_user_should_not_be_destroyable assert_no_difference 'User.count' do assert_equal false, User.anonymous.destroy end end - + def test_validate_login_presence @admin.login = "" assert !@admin.save assert_equal 1, @admin.errors.count end - + def test_validate_mail_notification_inclusion u = User.new u.mail_notification = 'foo' u.save assert_not_nil u.errors.on(:mail_notification) end - + context "User#try_to_login" do should "fall-back to case-insensitive if user login is not found as-typed." do user = User.try_to_login("AdMin", "admin") @@ -364,12 +364,12 @@ class UserTest < ActiveSupport::TestCase assert_equal "admin", user.login user.password = "hello" assert user.save - + user = User.try_to_login("admin", "hello") assert_kind_of User, user assert_equal "admin", user.login end - + def test_name_format assert_equal 'Smith, John', @jsmith.name(:lastname_coma_firstname) Setting.user_format = :firstname_lastname @@ -377,18 +377,18 @@ class UserTest < ActiveSupport::TestCase Setting.user_format = :username assert_equal 'jsmith', @jsmith.reload.name end - + def test_lock user = User.try_to_login("jsmith", "jsmith") assert_equal @jsmith, user - + @jsmith.status = User::STATUS_LOCKED assert @jsmith.save - + user = User.try_to_login("jsmith", "jsmith") - assert_equal nil, user + assert_equal nil, user end - + context ".try_to_login" do context "with good credentials" do should "return the user" do @@ -397,21 +397,21 @@ class UserTest < ActiveSupport::TestCase assert_equal "admin", user.login end end - + context "with wrong credentials" do should "return nil" do assert_nil User.try_to_login("admin", "foo") end end end - + if ldap_configured? context "#try_to_login using LDAP" do context "with failed connection to the LDAP server" do should "return nil" do @auth_source = AuthSourceLdap.find(1) AuthSource.any_instance.stubs(:initialize_ldap_con).raises(Net::LDAP::LdapError, 'Cannot connect') - + assert_equal nil, User.try_to_login('edavis', 'wrong') end end @@ -421,7 +421,7 @@ class UserTest < ActiveSupport::TestCase assert_equal nil, User.try_to_login('edavis', 'wrong') end end - + context "on the fly registration" do setup do @auth_source = AuthSourceLdap.find(1) @@ -434,12 +434,12 @@ class UserTest < ActiveSupport::TestCase assert !user.admin? end end - + should "retrieve existing user" do user = User.try_to_login('edavis', '123456') user.admin = true user.save! - + assert_no_difference('User.count') do user = User.try_to_login('edavis', '123456') assert user.admin? @@ -452,7 +452,7 @@ class UserTest < ActiveSupport::TestCase else puts "Skipping LDAP tests." end - + def test_create_anonymous AnonymousUser.delete_all anon = User.anonymous @@ -466,12 +466,12 @@ class UserTest < ActiveSupport::TestCase assert_nil @jsmith.rss_token key = @jsmith.rss_key assert_equal 40, key.length - + @jsmith.reload assert_equal key, @jsmith.rss_key end - + should_have_one :api_token context "User#api_key" do @@ -490,7 +490,7 @@ class UserTest < ActiveSupport::TestCase token = Token.generate!(:action => 'api') user.api_token = token assert user.save - + assert_equal token.value, user.api_key end end @@ -514,7 +514,7 @@ class UserTest < ActiveSupport::TestCase token = Token.generate!(:action => 'api') user.api_token = token user.save - + assert_equal user, User.find_by_api_key(token.value) end end @@ -524,11 +524,11 @@ class UserTest < ActiveSupport::TestCase roles = @jsmith.roles_for_project(Project.find(1)) assert_kind_of Role, roles.first assert_equal "Manager", roles.first.name - + # user with no role assert_nil @dlopper.roles_for_project(Project.find(2)).detect {|role| role.member?} end - + def test_projects_by_role_for_user_with_role user = User.find(2) assert_kind_of Hash, user.projects_by_role @@ -536,12 +536,12 @@ class UserTest < ActiveSupport::TestCase assert_equal [1,5], user.projects_by_role[Role.find(1)].collect(&:id).sort assert_equal [2], user.projects_by_role[Role.find(2)].collect(&:id).sort end - + def test_projects_by_role_for_user_with_no_role user = User.generate! assert_equal({}, user.projects_by_role) end - + def test_projects_by_role_for_anonymous assert_equal({}, User.anonymous.projects_by_role) end @@ -552,13 +552,13 @@ class UserTest < ActiveSupport::TestCase # with memberships assert_equal 6, User.find(2).valid_notification_options.size end - + def test_valid_notification_options_class_method assert_equal 5, User.valid_notification_options.size assert_equal 5, User.valid_notification_options(User.find(7)).size assert_equal 6, User.valid_notification_options(User.find(2)).size end - + def test_mail_notification_all @jsmith.mail_notification = 'all' @jsmith.notified_project_ids = [] @@ -566,7 +566,7 @@ class UserTest < ActiveSupport::TestCase @jsmith.reload assert @jsmith.projects.first.recipients.include?(@jsmith.mail) end - + def test_mail_notification_selected @jsmith.mail_notification = 'selected' @jsmith.notified_project_ids = [1] @@ -574,7 +574,7 @@ class UserTest < ActiveSupport::TestCase @jsmith.reload assert Project.find(1).recipients.include?(@jsmith.mail) end - + def test_mail_notification_only_my_events @jsmith.mail_notification = 'only_my_events' @jsmith.notified_project_ids = [] @@ -582,7 +582,7 @@ class UserTest < ActiveSupport::TestCase @jsmith.reload assert !@jsmith.projects.first.recipients.include?(@jsmith.mail) end - + def test_comments_sorting_preference assert !@jsmith.wants_comments_in_reverse_order? @jsmith.pref.comments_sorting = 'asc' @@ -590,13 +590,13 @@ class UserTest < ActiveSupport::TestCase @jsmith.pref.comments_sorting = 'desc' assert @jsmith.wants_comments_in_reverse_order? end - + def test_find_by_mail_should_be_case_insensitive u = User.find_by_mail('JSmith@somenet.foo') assert_not_nil u assert_equal 'jsmith@somenet.foo', u.mail end - + def test_random_password u = User.new u.random_password @@ -612,7 +612,7 @@ class UserTest < ActiveSupport::TestCase should "delegate to the auth source" do user = User.generate_with_protected! - + allowed_auth_source = AuthSource.generate! def allowed_auth_source.allow_password_changes?; true; end @@ -629,7 +629,7 @@ class UserTest < ActiveSupport::TestCase end end - + context "#allowed_to?" do context "with a unique project" do should "return false if project is archived" do @@ -637,14 +637,14 @@ class UserTest < ActiveSupport::TestCase Project.any_instance.stubs(:status).returns(Project::STATUS_ARCHIVED) assert ! @admin.allowed_to?(:view_issues, Project.find(1)) end - + should "return false if related module is disabled" do project = Project.find(1) project.enabled_module_names = ["issue_tracking"] assert @admin.allowed_to?(:add_issues, project) assert ! @admin.allowed_to?(:view_wiki_pages, project) end - + should "authorize nearly everything for admin users" do project = Project.find(1) assert ! @admin.member_of?(project) @@ -652,7 +652,7 @@ class UserTest < ActiveSupport::TestCase assert @admin.allowed_to?(p.to_sym, project) end end - + should "authorize normal users depending on their roles" do project = Project.find(1) assert @jsmith.allowed_to?(:delete_messages, project) #Manager @@ -664,19 +664,19 @@ class UserTest < ActiveSupport::TestCase should "return false if array is empty" do assert ! @admin.allowed_to?(:view_project, []) end - + should "return true only if user has permission on all these projects" do assert @admin.allowed_to?(:view_project, Project.all) assert ! @dlopper.allowed_to?(:view_project, Project.all) #cannot see Project(2) assert @jsmith.allowed_to?(:edit_issues, @jsmith.projects) #Manager or Developer everywhere assert ! @jsmith.allowed_to?(:delete_issue_watchers, @jsmith.projects) #Dev cannot delete_issue_watchers end - + should "behave correctly with arrays of 1 project" do assert ! User.anonymous.allowed_to?(:delete_issues, [Project.first]) end end - + context "with options[:global]" do should "authorize if user has at least one role that has this permission" do @dlopper2 = User.find(5) #only Developper on a project, not Manager anywhere @@ -689,7 +689,7 @@ class UserTest < ActiveSupport::TestCase end end end - + context "User#notify_about?" do context "Issues" do setup do @@ -703,58 +703,58 @@ class UserTest < ActiveSupport::TestCase @author.update_attribute(:mail_notification, 'all') assert @author.notify_about?(@issue) end - + should "be false for a user with :none" do @author.update_attribute(:mail_notification, 'none') assert ! @author.notify_about?(@issue) end - + should "be false for a user with :only_my_events and isn't an author, creator, or assignee" do @user = User.generate_with_protected!(:mail_notification => 'only_my_events') Member.create!(:user => @user, :project => @project, :role_ids => [1]) assert ! @user.notify_about?(@issue) end - + should "be true for a user with :only_my_events and is the author" do @author.update_attribute(:mail_notification, 'only_my_events') assert @author.notify_about?(@issue) end - + should "be true for a user with :only_my_events and is the assignee" do @assignee.update_attribute(:mail_notification, 'only_my_events') assert @assignee.notify_about?(@issue) end - + should "be true for a user with :only_assigned and is the assignee" do @assignee.update_attribute(:mail_notification, 'only_assigned') assert @assignee.notify_about?(@issue) end - + should "be false for a user with :only_assigned and is not the assignee" do @author.update_attribute(:mail_notification, 'only_assigned') assert ! @author.notify_about?(@issue) end - + should "be true for a user with :only_owner and is the author" do @author.update_attribute(:mail_notification, 'only_owner') assert @author.notify_about?(@issue) end - + should "be false for a user with :only_owner and is not the author" do @assignee.update_attribute(:mail_notification, 'only_owner') assert ! @assignee.notify_about?(@issue) end - + should "be true for a user with :selected and is the author" do @author.update_attribute(:mail_notification, 'selected') assert @author.notify_about?(@issue) end - + should "be true for a user with :selected and is the assignee" do @assignee.update_attribute(:mail_notification, 'selected') assert @assignee.notify_about?(@issue) end - + should "be false for a user with :selected and is not the author or assignee" do @user = User.generate_with_protected!(:mail_notification => 'selected') Member.create!(:user => @user, :project => @project, :role_ids => [1]) @@ -773,9 +773,9 @@ class UserTest < ActiveSupport::TestCase user.salt = nil user.hashed_password = User.hash_password("unsalted") user.save! - + User.salt_unsalted_passwords! - + user.reload # Salt added assert !user.salt.blank? @@ -783,9 +783,9 @@ class UserTest < ActiveSupport::TestCase assert user.check_password?("unsalted") assert_equal user, User.try_to_login(user.login, "unsalted") end - + if Object.const_defined?(:OpenID) - + def test_setting_identity_url normalized_open_id_url = 'http://example.com/' u = User.new( :identity_url => 'http://example.com/' ) @@ -803,18 +803,18 @@ class UserTest < ActiveSupport::TestCase u = User.new( :identity_url => 'example.com' ) assert_equal normalized_open_id_url, u.identity_url end - + def test_setting_blank_identity_url u = User.new( :identity_url => 'example.com' ) u.identity_url = '' assert u.identity_url.blank? end - + def test_setting_invalid_identity_url u = User.new( :identity_url => 'this is not an openid url' ) assert u.identity_url.blank? end - + else puts "Skipping openid tests." end -- 2.39.5