From: Toshi MARUYAMA <marutosijp2@yahoo.co.jp>
Date: Sat, 19 Dec 2020 17:33:19 +0000 (+0000)
Subject: use 'assert_not' instead of 'assert !' at IssuesControllerTest
X-Git-Tag: 4.2.0~142
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=7955452b6a750963fe19c778c7485cc8d208a06f;p=redmine.git

use 'assert_not' instead of 'assert !' at IssuesControllerTest

git-svn-id: http://svn.redmine.org/redmine/trunk@20680 e93f8b46-1217-0410-a6f0-8f06a7374b81
---

diff --git a/test/functional/issues_controller_test.rb b/test/functional/issues_controller_test.rb
index 14e7fad6d..3ea6d9cef 100644
--- a/test/functional/issues_controller_test.rb
+++ b/test/functional/issues_controller_test.rb
@@ -3060,7 +3060,7 @@ class IssuesControllerTest < Redmine::ControllerTest
     end
 
     # Be sure we don't display inactive IssuePriorities
-    assert ! IssuePriority.find(15).active?
+    assert_not IssuePriority.find(15).active?
     assert_select 'select[name=?]', 'issue[priority_id]' do
       assert_select 'option[value="15"]', 0
     end
@@ -5365,7 +5365,7 @@ class IssuesControllerTest < Redmine::ControllerTest
 
     assert_select 'select[name=?]', 'issue[project_id]'
     # Be sure we don't display inactive IssuePriorities
-    assert ! IssuePriority.find(15).active?
+    assert_not IssuePriority.find(15).active?
     assert_select 'select[name=?]', 'issue[priority_id]' do
       assert_select 'option[value="15"]', 0
     end
@@ -6459,7 +6459,7 @@ class IssuesControllerTest < Redmine::ControllerTest
       assert_select 'select[name=?]', 'issue[custom_field_values][1]'
 
       # Be sure we don't display inactive IssuePriorities
-      assert ! IssuePriority.find(15).active?
+      assert_not IssuePriority.find(15).active?
       assert_select 'select[name=?]', 'issue[priority_id]' do
         assert_select 'option[value="15"]', 0
       end
@@ -6821,7 +6821,7 @@ class IssuesControllerTest < Redmine::ControllerTest
     user = User.find(3)
     action = {:controller => "issues", :action => "bulk_update"}
     assert user.allowed_to?(action, Issue.find(1).project)
-    assert ! user.allowed_to?(action, Issue.find(6).project)
+    assert_not user.allowed_to?(action, Issue.find(6).project)
     post(
       :bulk_update,
       :params => {