From 6ad58913c8beeeffeee77645e41860848085806d Mon Sep 17 00:00:00 2001
From: Guo Xiang Tan <tgx_world@hotmail.com>
Date: Tue, 4 Jun 2019 16:40:10 +0800
Subject: [PATCH] DEV: Prefer `save!` over `save`.
---
spec/components/email/receiver_spec.rb | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/spec/components/email/receiver_spec.rb b/spec/components/email/receiver_spec.rb
index 918767786c9..e794579e48a 100644
--- a/spec/components/email/receiver_spec.rb
+++ b/spec/components/email/receiver_spec.rb
@@ -451,7 +451,7 @@ describe Email::Receiver do
category.email_in = "category@bar.com"
category.email_in_allow_strangers = true
category.set_permissions(Group[:trust_level_4] => :full)
- category.save
+ category.save!
expect { process(:staged_reply_restricted) }.to change { topic.posts.count }
end
@@ -860,7 +860,7 @@ describe Email::Receiver do
group.save
category.set_permissions(group => :create_post)
- category.save
+ category.save!
# raises an InvalidAccess when the user doesn't have the privileges to create a topic
expect { process(:existing_user) }.to raise_error(Discourse::InvalidAccess)
@@ -933,7 +933,7 @@ describe Email::Receiver do
Fabricate(:user, email: "tl4@bar.com", trust_level: TrustLevel[4])
category.set_permissions(Group[:trust_level_4] => :full)
- category.save
+ category.save!
Group.refresh_automatic_group!(:trust_level_4)
@@ -1264,7 +1264,7 @@ describe Email::Receiver do
context "read-only category" do
before do
category.set_permissions(everyone: :readonly)
- category.save
+ category.save!
Fabricate(:user, email: "alice@foo.com")
Fabricate(:user, email: "bob@bar.com")