From 62e626e6801e3b4fe9ef55565f4f6065325f608e Mon Sep 17 00:00:00 2001 From: Toshi MARUYAMA Date: Wed, 4 Dec 2019 13:04:23 +0000 Subject: [PATCH] code layout clean up test/unit/workflow_test.rb git-svn-id: http://svn.redmine.org/redmine/trunk@19332 e93f8b46-1217-0410-a6f0-8f06a7374b81 --- test/unit/workflow_test.rb | 29 +++++++++++++++++++++-------- 1 file changed, 21 insertions(+), 8 deletions(-) diff --git a/test/unit/workflow_test.rb b/test/unit/workflow_test.rb index 138992d24..3d963e236 100644 --- 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' -- 2.39.5