Skip to content

Commit 9f8babe

Browse files
fkdkentRedmine Patch Meetup
authored andcommitted
Use update
1 parent aa1a63f commit 9f8babe

File tree

1 file changed

+3
-6
lines changed

1 file changed

+3
-6
lines changed

test/unit/issue_test.rb

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2928,24 +2928,21 @@ def test_saving_twice_should_not_duplicate_journal_details
29282928

29292929
with_settings :issue_done_ratio => 'issue_field' do
29302930
issue = Issue.generate!(:status_id => 1, :done_ratio => 30)
2931-
issue.status = closed_status
2932-
issue.save!
2931+
issue.update!(:status => closed_status)
29332932

29342933
assert_equal 30, issue.read_attribute(:done_ratio)
29352934
end
29362935

29372936
with_settings :issue_done_ratio => 'issue_status' do
29382937
issue = Issue.generate!(:status_id => 1, :done_ratio => 30)
2939-
issue.status = closed_status
2940-
issue.save!
2938+
issue.update!(:status => closed_status)
29412939

29422940
assert_equal 90, issue.read_attribute(:done_ratio)
29432941
end
29442942

29452943
with_settings :issue_done_ratio => 'issue_field_and_closed_status' do
29462944
issue = Issue.generate!(:status_id => 1, :done_ratio => 30)
2947-
issue.status = closed_status
2948-
issue.save!
2945+
issue.update!(:status => closed_status)
29492946

29502947
assert_equal 100, issue.read_attribute(:done_ratio)
29512948
end

0 commit comments

Comments
 (0)