From ee82abba3c7c8508bccb0896ddacfe469a2fc3b6 Mon Sep 17 00:00:00 2001 From: Guo Xiang Tan Date: Wed, 25 Apr 2018 13:38:37 +0800 Subject: [PATCH] Prefer `save!/create!` over `save/create`. --- spec/models/trust_level3_requirements_spec.rb | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/spec/models/trust_level3_requirements_spec.rb b/spec/models/trust_level3_requirements_spec.rb index 6a7546efa58..b2d9c580f8d 100644 --- a/spec/models/trust_level3_requirements_spec.rb +++ b/spec/models/trust_level3_requirements_spec.rb @@ -30,17 +30,23 @@ describe TrustLevel3Requirements do end it "returns if the user has ever been suspended" do - user.save + user.save! expect(tl3_requirements.penalty_counts.suspended).to eq(0) expect(tl3_requirements.penalty_counts.total).to eq(0) - UserHistory.create(target_user_id: user.id, action: UserHistory.actions[:suspend_user]) + UserHistory.create!( + target_user_id: user.id, + action: UserHistory.actions[:suspend_user] + ) expect(tl3_requirements.penalty_counts.suspended).to eq(1) expect(tl3_requirements.penalty_counts.total).to eq(1) - UserHistory.create(target_user_id: user.id, action: UserHistory.actions[:unsuspend_user]) + UserHistory.create!( + target_user_id: user.id, + action: UserHistory.actions[:unsuspend_user] + ) expect(tl3_requirements.penalty_counts.suspended).to eq(0) expect(tl3_requirements.penalty_counts.total).to eq(0)