code layout clean up test/unit/workflow_test.rb

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

diff --git a/test/unit/workflow_test.rb b/test/unit/workflow_test.rb
index 138992d249a9101bf714b09aa146999366f83cd4..3d963e23636ac5c99751d0679b19e9fa920f158c 100644 (file)
--- a/test/unit/workflow_test.rb
+++ b/test/unit/workflow_test.rb
@@ -28,21 +28,33 @@ class WorkflowTest < ActiveSupport::TestCase
 
   def test_copy
     WorkflowTransition.delete_all
-    WorkflowTransition.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 1, :new_status_id => 2)
-    WorkflowTransition.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 1, :new_status_id => 3, :assignee => true)
-    WorkflowTransition.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 1, :new_status_id => 4, :author => true)
+    WorkflowTransition.create!(:role_id => 1, :tracker_id => 2,
+                               :old_status_id => 1, :new_status_id => 2)
+    WorkflowTransition.create!(:role_id => 1, :tracker_id => 2,
+                               :old_status_id => 1, :new_status_id => 3,
+                               :assignee => true)
+    WorkflowTransition.create!(:role_id => 1, :tracker_id => 2,
+                               :old_status_id => 1, :new_status_id => 4,
+                               :author => true)
 
     assert_difference 'WorkflowTransition.count', 3 do
       WorkflowTransition.copy(Tracker.find(2), Role.find(1), Tracker.find(3), Role.find(2))
     end
 
-    assert WorkflowTransition.where(:role_id => 2, :tracker_id => 3, :old_status_id => 1, :new_status_id => 2, :author => false, :assignee => false).first
-    assert WorkflowTransition.where(:role_id => 2, :tracker_id => 3, :old_status_id => 1, :new_status_id => 3, :author => false, :assignee => true).first
-    assert WorkflowTransition.where(:role_id => 2, :tracker_id => 3, :old_status_id => 1, :new_status_id => 4, :author => true, :assignee => false).first
+    assert WorkflowTransition.where(:role_id => 2, :tracker_id => 3,
+                                    :old_status_id => 1, :new_status_id => 2,
+                                    :author => false, :assignee => false).first
+    assert WorkflowTransition.where(:role_id => 2, :tracker_id => 3,
+                                    :old_status_id => 1, :new_status_id => 3,
+                                    :author => false, :assignee => true).first
+    assert WorkflowTransition.where(:role_id => 2, :tracker_id => 3,
+                                    :old_status_id => 1, :new_status_id => 4,
+                                    :author => true, :assignee => false).first
   end
 
   def test_workflow_permission_should_validate_rule
-    wp = WorkflowPermission.new(:role_id => 1, :tracker_id => 1, :old_status_id => 1, :field_name => 'due_date')
+    wp = WorkflowPermission.new(:role_id => 1, :tracker_id => 1,
+                                :old_status_id => 1, :field_name => 'due_date')
     assert !wp.save
 
     wp.rule = 'foo'
@@ -56,7 +68,8 @@ class WorkflowTest < ActiveSupport::TestCase
   end
 
   def test_workflow_permission_should_validate_field_name
-    wp = WorkflowPermission.new(:role_id => 1, :tracker_id => 1, :old_status_id => 1, :rule => 'required')
+    wp = WorkflowPermission.new(:role_id => 1, :tracker_id => 1,
+                                :old_status_id => 1, :rule => 'required')
     assert !wp.save
 
     wp.field_name = 'foo'