code cleanup: rubocop: fix Layout/LeadingCommentSpace in test/unit/user_test.rb

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

diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
index f24a61c6d24c80b19261c142078a681d8efc63df..1515c9fb796aaf32fbaf30c444ea342dcdeb71f9 100644 (file)
--- a/.rubocop_todo.yml
+++ b/.rubocop_todo.yml
@@ -294,7 +294,6 @@ Layout/LeadingCommentSpace:
   Exclude:
     - 'lib/diff.rb'
     - 'lib/redmine/wiki_formatting/textile/redcloth3.rb'
-    - 'test/unit/user_test.rb'
 
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle.

diff --git a/test/unit/user_test.rb b/test/unit/user_test.rb
index da9fbe7edd4ea504e198b5730fad3af7181db86a..20e34d99c6a6ef1f3e6f08c8ab96f0b482b0fef3 100644 (file)
--- a/test/unit/user_test.rb
+++ b/test/unit/user_test.rb
@@ -1164,8 +1164,10 @@ class UserTest < ActiveSupport::TestCase
 
   test "#allowed_to? for normal users" do
     project = Project.find(1)
-    assert_equal true, @jsmith.allowed_to?(:delete_messages, project)    #Manager
-    assert_equal false, @dlopper.allowed_to?(:delete_messages, project) #Developer
+    # Manager
+    assert_equal true, @jsmith.allowed_to?(:delete_messages, project)
+    # Developer
+    assert_equal false, @dlopper.allowed_to?(:delete_messages, project)
   end
 
   test "#allowed_to? with empty array should return false" do
@@ -1174,13 +1176,17 @@ class UserTest < ActiveSupport::TestCase
 
   test "#allowed_to? with multiple projects" do
     assert_equal true, @admin.allowed_to?(:view_project, Project.all.to_a)
-    assert_equal false, @dlopper.allowed_to?(:view_project, Project.all.to_a) #cannot see Project(2)
-    assert_equal true, @jsmith.allowed_to?(:edit_issues, @jsmith.projects.to_a) #Manager or Developer everywhere
-    assert_equal false, @jsmith.allowed_to?(:delete_issue_watchers, @jsmith.projects.to_a) #Dev cannot delete_issue_watchers
+    # cannot see Project(2)
+    assert_equal false, @dlopper.allowed_to?(:view_project, Project.all.to_a)
+    # Manager or Developer everywhere
+    assert_equal true, @jsmith.allowed_to?(:edit_issues, @jsmith.projects.to_a)
+    # Dev cannot delete_issue_watchers
+    assert_equal false, @jsmith.allowed_to?(:delete_issue_watchers, @jsmith.projects.to_a)
   end
 
   test "#allowed_to? with with options[:global] should return true if user has one role with the permission" do
-    @dlopper2 = User.find(5) #only Developer on a project, not Manager anywhere
+    # only Developer on a project, not Manager anywhere
+    @dlopper2 = User.find(5)
     @anonymous = User.find(6)
     assert_equal true, @jsmith.allowed_to?(:delete_issue_watchers, nil, :global => true)
     assert_equal false, @dlopper2.allowed_to?(:delete_issue_watchers, nil, :global => true)
@@ -1191,7 +1197,8 @@ class UserTest < ActiveSupport::TestCase
 
   # this is just a proxy method, the test only calls it to ensure it doesn't break trivially
   test "#allowed_to_globally?" do
-    @dlopper2 = User.find(5) #only Developer on a project, not Manager anywhere
+    # only Developer on a project, not Manager anywhere
+    @dlopper2 = User.find(5)
     @anonymous = User.find(6)
     assert_equal true, @jsmith.allowed_to_globally?(:delete_issue_watchers)
     assert_equal false, @dlopper2.allowed_to_globally?(:delete_issue_watchers)