jro_apps/redmine Commit - r15678:bf5dade8df89

0

test/functional/project_enumerations_controller_test.rb +1 -1

                 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

0

test/functional/users_controller_test.rb +1 -1

                   :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

0

test/integration/admin_test.rb +1 -1

                 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

0

test/integration/api_test/issues_test.rb +2 -2

                 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_nil json['issue']['spent_hours']
-                assert_equal nil, json['issue']['total_spent_hours']
+                assert_nil json['issue']['total_spent_hours']
               end
               test "POST /issues.xml should create an issue with the attributes" do

0

test/unit/auth_source_ldap_test.rb +3 -3

                 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

0

test/unit/changeset_test.rb +1 -1

                                   :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

0

test/unit/custom_field_test.rb +2 -2

               def test_float_cast_blank_value_should_return_nil
                 field = CustomField.new(:field_format => 'float')
-                assert_equal nil, field.cast_value(nil)
+                assert_nil field.cast_value(nil)
-                assert_equal nil, field.cast_value('')
+                assert_nil field.cast_value('')
               end
               def test_float_cast_valid_value_should_return_float

0

test/unit/custom_field_user_format_test.rb +3 -3

               end
               def test_cast_blank_value
-                assert_equal nil, @field.cast_value(nil)
+                assert_nil @field.cast_value(nil)
-                assert_equal nil, @field.cast_value("")
+                assert_nil @field.cast_value("")
               end
               def test_cast_valid_value
               end
               def test_cast_invalid_value
-                assert_equal nil, @field.cast_value("187")
+                assert_nil @field.cast_value("187")
               end
             end

0

test/unit/custom_field_version_format_test.rb +3 -3

               end
               def test_cast_blank_value
-                assert_equal nil, @field.cast_value(nil)
+                assert_nil @field.cast_value(nil)
-                assert_equal nil, @field.cast_value("")
+                assert_nil @field.cast_value("")
               end
               def test_cast_valid_value
               end
               def test_cast_invalid_value
-                assert_equal nil, @field.cast_value("187")
+                assert_nil @field.cast_value("187")
               end
             end

0

test/unit/group_test.rb +1 -1

                 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

0

test/unit/helpers/application_helper_test.rb +1 -1

                 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

0

test/unit/issue_test.rb +2 -2

                 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
                 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

0

test/unit/journal_test.rb +1 -1

               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

0

test/unit/lib/redmine/codeset_util_test.rb +1 -1

               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

0

test/unit/lib/redmine/export/pdf_test.rb +2 -2

                   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

0

test/unit/lib/redmine/menu_manager/mapper_test.rb +1 -1

                 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

0

test/unit/lib/redmine/scm/adapters/git_adapter_test.rb +3 -3

                   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
                   assert entries1
                   assert_equal 3, entries1.size
                   f1 = entries1[1]
-                  assert_equal nil, f1.name
+                  assert_nil f1.name
-                  assert_equal nil, f1.path
+                  assert_nil f1.path
                   assert_equal 'file', f1.kind
                 end

0

test/unit/project_test.rb +2 -2

                 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_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 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

0

test/unit/repository_git_test.rb +1 -1

                   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

0

test/unit/user_test.rb +3 -3

                 @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
                   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

0

test/unit/version_test.rb +2 -2

                 # 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

0

test/unit/wiki_page_test.rb +1 -1

                 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

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages