From: Jean-Philippe Lang Date: Sun, 11 Dec 2016 15:26:41 +0000 (+0000) Subject: Use assert_nil instead of assert_equal. X-Git-Tag: 3.4.0~516 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=bf5dade8df89bd4e519b59846a9007bd0a5911ca;p=redmine.git Use assert_nil instead of assert_equal. git-svn-id: http://svn.redmine.org/redmine/trunk@16060 e93f8b46-1217-0410-a6f0-8f06a7374b81 --- diff --git a/test/functional/project_enumerations_controller_test.rb b/test/functional/project_enumerations_controller_test.rb index 1ab95e4ec..c098b2b30 100644 --- a/test/functional/project_enumerations_controller_test.rb +++ b/test/functional/project_enumerations_controller_test.rb @@ -83,7 +83,7 @@ class ProjectEnumerationsControllerTest < Redmine::ControllerTest assert_equal "1", previously_inactive.custom_value_for(billable_field).value # ... QA - assert_equal nil, project.time_entry_activities.find_by_name("QA"), "Custom QA activity created when it wasn't modified" + assert_nil project.time_entry_activities.find_by_name("QA"), "Custom QA activity created when it wasn't modified" end def test_update_will_update_project_specific_activities diff --git a/test/functional/users_controller_test.rb b/test/functional/users_controller_test.rb index f899e631a..0dbd12a81 100644 --- a/test/functional/users_controller_test.rb +++ b/test/functional/users_controller_test.rb @@ -463,7 +463,7 @@ class UsersControllerTest < Redmine::ControllerTest :user => {:auth_source_id => '', :password => 'newpass123', :password_confirmation => 'newpass123'} } - assert_equal nil, u.reload.auth_source + assert_nil u.reload.auth_source assert u.check_password?('newpass123') end diff --git a/test/integration/admin_test.rb b/test/integration/admin_test.rb index 73f89139d..f16f10bd1 100644 --- a/test/integration/admin_test.rb +++ b/test/integration/admin_test.rb @@ -48,7 +48,7 @@ class AdminTest < Redmine::IntegrationTest put "/users/#{user.id}", :id => user.id, :user => { :status => User::STATUS_LOCKED } assert_redirected_to "/users/#{ user.id }/edit" locked_user = User.try_to_login("psmith", "psmith09") - assert_equal nil, locked_user + assert_nil locked_user end test "Add a user as an anonymous user should fail" do diff --git a/test/integration/api_test/issues_test.rb b/test/integration/api_test/issues_test.rb index d4040201e..94f42287a 100644 --- a/test/integration/api_test/issues_test.rb +++ b/test/integration/api_test/issues_test.rb @@ -414,8 +414,8 @@ class Redmine::ApiTest::IssuesTest < Redmine::ApiTest::Base json = ActiveSupport::JSON.decode(response.body) assert_equal parent.estimated_hours, json['issue']['estimated_hours'] assert_equal (parent.estimated_hours.to_f + 3.0), json['issue']['total_estimated_hours'] - assert_equal nil, json['issue']['spent_hours'] - assert_equal nil, json['issue']['total_spent_hours'] + assert_nil json['issue']['spent_hours'] + assert_nil json['issue']['total_spent_hours'] end test "POST /issues.xml should create an issue with the attributes" do diff --git a/test/unit/auth_source_ldap_test.rb b/test/unit/auth_source_ldap_test.rb index bf549218b..eac13ff70 100644 --- a/test/unit/auth_source_ldap_test.rb +++ b/test/unit/auth_source_ldap_test.rb @@ -95,17 +95,17 @@ class AuthSourceLdapTest < ActiveSupport::TestCase test '#authenticate with an invalid LDAP user should return nil' do auth = AuthSourceLdap.find(1) - assert_equal nil, auth.authenticate('nouser','123456') + assert_nil auth.authenticate('nouser','123456') end test '#authenticate without a login should return nil' do auth = AuthSourceLdap.find(1) - assert_equal nil, auth.authenticate('','123456') + assert_nil auth.authenticate('','123456') end test '#authenticate without a password should return nil' do auth = AuthSourceLdap.find(1) - assert_equal nil, auth.authenticate('edavis','') + assert_nil auth.authenticate('edavis','') end test '#authenticate without filter should return any user' do diff --git a/test/unit/changeset_test.rb b/test/unit/changeset_test.rb index 179c515b4..780bc0bd2 100644 --- a/test/unit/changeset_test.rb +++ b/test/unit/changeset_test.rb @@ -566,7 +566,7 @@ class ChangesetTest < ActiveSupport::TestCase :committer => nil) assert( c.save ) assert_equal "", c.comments - assert_equal nil, c.committer + assert_nil c.committer assert_equal "UTF-8", c.comments.encoding.to_s end diff --git a/test/unit/custom_field_test.rb b/test/unit/custom_field_test.rb index c805e170e..461af85e4 100644 --- a/test/unit/custom_field_test.rb +++ b/test/unit/custom_field_test.rb @@ -321,8 +321,8 @@ class CustomFieldTest < ActiveSupport::TestCase def test_float_cast_blank_value_should_return_nil field = CustomField.new(:field_format => 'float') - assert_equal nil, field.cast_value(nil) - assert_equal nil, field.cast_value('') + assert_nil field.cast_value(nil) + assert_nil field.cast_value('') end def test_float_cast_valid_value_should_return_float diff --git a/test/unit/custom_field_user_format_test.rb b/test/unit/custom_field_user_format_test.rb index f5d4e66fe..d9a1cfb2c 100644 --- a/test/unit/custom_field_user_format_test.rb +++ b/test/unit/custom_field_user_format_test.rb @@ -62,8 +62,8 @@ class CustomFieldUserFormatTest < ActiveSupport::TestCase end def test_cast_blank_value - assert_equal nil, @field.cast_value(nil) - assert_equal nil, @field.cast_value("") + assert_nil @field.cast_value(nil) + assert_nil @field.cast_value("") end def test_cast_valid_value @@ -73,6 +73,6 @@ class CustomFieldUserFormatTest < ActiveSupport::TestCase end def test_cast_invalid_value - assert_equal nil, @field.cast_value("187") + assert_nil @field.cast_value("187") end end diff --git a/test/unit/custom_field_version_format_test.rb b/test/unit/custom_field_version_format_test.rb index a394dabc8..71a835e04 100644 --- a/test/unit/custom_field_version_format_test.rb +++ b/test/unit/custom_field_version_format_test.rb @@ -45,8 +45,8 @@ class CustomFieldVersionFormatTest < ActiveSupport::TestCase end def test_cast_blank_value - assert_equal nil, @field.cast_value(nil) - assert_equal nil, @field.cast_value("") + assert_nil @field.cast_value(nil) + assert_nil @field.cast_value("") end def test_cast_valid_value @@ -56,6 +56,6 @@ class CustomFieldVersionFormatTest < ActiveSupport::TestCase end def test_cast_invalid_value - assert_equal nil, @field.cast_value("187") + assert_nil @field.cast_value("187") end end diff --git a/test/unit/group_test.rb b/test/unit/group_test.rb index b7302ad68..4f33e5fc6 100644 --- a/test/unit/group_test.rb +++ b/test/unit/group_test.rb @@ -130,7 +130,7 @@ class GroupTest < ActiveSupport::TestCase assert group.destroy assert group.destroyed? - assert_equal nil, Issue.find(1).assigned_to_id + assert_nil Issue.find(1).assigned_to_id end def test_builtin_groups_should_be_created_if_missing diff --git a/test/unit/helpers/application_helper_test.rb b/test/unit/helpers/application_helper_test.rb index f1d98cbd8..299488a2f 100644 --- a/test/unit/helpers/application_helper_test.rb +++ b/test/unit/helpers/application_helper_test.rb @@ -1020,7 +1020,7 @@ EXPECTED assert_equal 'test1/test2', to_path_param('test1/test2') assert_equal 'test1/test2', to_path_param('/test1/test2/') assert_equal 'test1/test2', to_path_param('//test1/test2/') - assert_equal nil, to_path_param('/') + assert_nil to_path_param('/') end def test_wiki_links_in_tables diff --git a/test/unit/issue_test.rb b/test/unit/issue_test.rb index 7adc53ec4..e4e611153 100644 --- a/test/unit/issue_test.rb +++ b/test/unit/issue_test.rb @@ -1643,7 +1643,7 @@ class IssueTest < ActiveSupport::TestCase issue.reload assert_equal 2, issue.project_id # Cleared fixed_version - assert_equal nil, issue.fixed_version + assert_nil issue.fixed_version end def test_move_to_another_project_should_keep_fixed_version_when_shared_with_the_target_project @@ -1665,7 +1665,7 @@ class IssueTest < ActiveSupport::TestCase issue.reload assert_equal 5, issue.project_id # Cleared fixed_version - assert_equal nil, issue.fixed_version + assert_nil issue.fixed_version end def test_move_to_another_project_should_keep_fixed_version_when_shared_systemwide diff --git a/test/unit/journal_test.rb b/test/unit/journal_test.rb index f26b284e7..054bdb450 100644 --- a/test/unit/journal_test.rb +++ b/test/unit/journal_test.rb @@ -201,7 +201,7 @@ class JournalTest < ActiveSupport::TestCase def test_custom_field_should_return_nil_for_non_cf_detail d = JournalDetail.new(:property => 'subject') - assert_equal nil, d.custom_field + assert_nil d.custom_field end def test_visible_details_should_include_relations_to_visible_issues_only diff --git a/test/unit/lib/redmine/codeset_util_test.rb b/test/unit/lib/redmine/codeset_util_test.rb index f5a6d6336..5c88e0f75 100644 --- a/test/unit/lib/redmine/codeset_util_test.rb +++ b/test/unit/lib/redmine/codeset_util_test.rb @@ -51,7 +51,7 @@ class Redmine::CodesetUtilTest < ActiveSupport::TestCase def test_to_utf8_by_setting_blank_string assert_equal "", Redmine::CodesetUtil.to_utf8_by_setting("") - assert_equal nil, Redmine::CodesetUtil.to_utf8_by_setting(nil) + assert_nil Redmine::CodesetUtil.to_utf8_by_setting(nil) end def test_to_utf8_by_setting_returns_ascii_as_utf8 diff --git a/test/unit/lib/redmine/export/pdf_test.rb b/test/unit/lib/redmine/export/pdf_test.rb index d19bdeb14..f1f9056a3 100644 --- a/test/unit/lib/redmine/export/pdf_test.rb +++ b/test/unit/lib/redmine/export/pdf_test.rb @@ -94,9 +94,9 @@ class PdfTest < ActiveSupport::TestCase assert a2.readable? assert (! a2.visible?) aa1 = Redmine::Export::PDF::RDMPdfEncoding::attach(Attachment.all, "Testfile.PNG", "UTF-8") - assert_equal nil, aa1 + assert_nil aa1 aa2 = Redmine::Export::PDF::RDMPdfEncoding::attach(Attachment.all, "test#{str2}.png", encoding) - assert_equal nil, aa2 + assert_nil aa2 set_tmp_attachments_directory end diff --git a/test/unit/lib/redmine/menu_manager/mapper_test.rb b/test/unit/lib/redmine/menu_manager/mapper_test.rb index fbffc5d12..b1a61075b 100644 --- a/test/unit/lib/redmine/menu_manager/mapper_test.rb +++ b/test/unit/lib/redmine/menu_manager/mapper_test.rb @@ -156,7 +156,7 @@ class Redmine::MenuManager::MapperTest < ActiveSupport::TestCase menu_mapper.push :test_overview, { :controller => 'projects', :action => 'show'}, {} item = menu_mapper.find(:nothing) - assert_equal nil, item + assert_nil item end def test_delete diff --git a/test/unit/lib/redmine/scm/adapters/git_adapter_test.rb b/test/unit/lib/redmine/scm/adapters/git_adapter_test.rb index 6757d5ba5..4e09b9df6 100644 --- a/test/unit/lib/redmine/scm/adapters/git_adapter_test.rb +++ b/test/unit/lib/redmine/scm/adapters/git_adapter_test.rb @@ -336,7 +336,7 @@ class GitAdapterTest < ActiveSupport::TestCase assert_equal 15, revs1.length assert_equal "7234cb2750b63f47bff735edc50a1c0a433c2518", revs1[0].identifier - assert_equal nil, revs1[0].parents + assert_nil revs1[0].parents assert_equal "899a15dba03a3b350b89c3f537e4bbe02a03cdc9", revs1[1].identifier assert_equal 1, revs1[1].parents.length @@ -477,8 +477,8 @@ class GitAdapterTest < ActiveSupport::TestCase assert entries1 assert_equal 3, entries1.size f1 = entries1[1] - assert_equal nil, f1.name - assert_equal nil, f1.path + assert_nil f1.name + assert_nil f1.path assert_equal 'file', f1.kind end diff --git a/test/unit/project_test.rb b/test/unit/project_test.rb index b134c67cc..3809662f7 100644 --- a/test/unit/project_test.rb +++ b/test/unit/project_test.rb @@ -353,8 +353,8 @@ class ProjectTest < ActiveSupport::TestCase issue_with_hierarchy_fixed_version.reload assert_equal 4, issue_with_local_fixed_version.fixed_version_id, "Fixed version was not keep on an issue local to the moved project" - assert_equal nil, issue_with_hierarchy_fixed_version.fixed_version_id, "Fixed version is still set after moving the Project out of the hierarchy where the version is defined in" - assert_equal nil, parent_issue.fixed_version_id, "Fixed version is still set after moving the Version out of the hierarchy for the issue." + assert_nil issue_with_hierarchy_fixed_version.fixed_version_id, "Fixed version is still set after moving the Project out of the hierarchy where the version is defined in" + assert_nil parent_issue.fixed_version_id, "Fixed version is still set after moving the Version out of the hierarchy for the issue." end def test_parent diff --git a/test/unit/repository_git_test.rb b/test/unit/repository_git_test.rb index 7b56b4a79..beb3d6fea 100644 --- a/test/unit/repository_git_test.rb +++ b/test/unit/repository_git_test.rb @@ -149,7 +149,7 @@ class RepositoryGitTest < ActiveSupport::TestCase assert_equal 3, commit.filechanges.count change = commit.filechanges.sort_by(&:path).first assert_equal "README", change.path - assert_equal nil, change.from_path + assert_nil change.from_path assert_equal "A", change.action assert_equal NUM_HEAD, @repository.extra_info["heads"].size diff --git a/test/unit/user_test.rb b/test/unit/user_test.rb index d39694e94..eb9df3cef 100644 --- a/test/unit/user_test.rb +++ b/test/unit/user_test.rb @@ -611,7 +611,7 @@ class UserTest < ActiveSupport::TestCase @jsmith.save! user = User.try_to_login("jsmith", "jsmith") - assert_equal nil, user + assert_nil user end def test_try_to_login_with_locked_user_and_not_active_only_should_return_user @@ -645,11 +645,11 @@ class UserTest < ActiveSupport::TestCase 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') + assert_nil User.try_to_login('edavis', 'wrong') end test "#try_to_login using LDAP" do - assert_equal nil, User.try_to_login('edavis', 'wrong') + assert_nil User.try_to_login('edavis', 'wrong') end test "#try_to_login using LDAP binding with user's account" do diff --git a/test/unit/version_test.rb b/test/unit/version_test.rb index 470207be2..bd0590cb5 100644 --- a/test/unit/version_test.rb +++ b/test/unit/version_test.rb @@ -243,12 +243,12 @@ class VersionTest < ActiveSupport::TestCase # Project 1 now out of the shared scope project_1_issue.reload - assert_equal nil, project_1_issue.fixed_version, + assert_nil project_1_issue.fixed_version, "Fixed version is still set after changing the Version's sharing" # Project 5 now out of the shared scope project_5_issue.reload - assert_equal nil, project_5_issue.fixed_version, + assert_nil project_5_issue.fixed_version, "Fixed version is still set after changing the Version's sharing" # Project 2 issue remains diff --git a/test/unit/wiki_page_test.rb b/test/unit/wiki_page_test.rb index a0819e53c..1f5e27080 100644 --- a/test/unit/wiki_page_test.rb +++ b/test/unit/wiki_page_test.rb @@ -107,7 +107,7 @@ class WikiPageTest < ActiveSupport::TestCase child.wiki_id = 2 child.save! - assert_equal nil, child.reload.parent_id + assert_nil child.reload.parent_id end def test_move_parent_should_move_child_page