summaryrefslogtreecommitdiff
path: root/spec
diff options
context:
space:
mode:
authorFelipe Artur <felipefac@gmail.com>2016-12-14 17:57:14 -0200
committerFelipe Artur <felipefac@gmail.com>2016-12-20 17:55:14 -0200
commit77deeb12f74b857f9356168ccdf92612fc85fe84 (patch)
treeeae473639b98b794b1249861af68f9f26efb4e3d /spec
parentb6d069c10fbf0ef13ad78bb22cfc965a278adcea (diff)
downloadgitlab-ce-77deeb12f74b857f9356168ccdf92612fc85fe84.tar.gz
Fix issuable assignee update bug when previous assignee is null
Diffstat (limited to 'spec')
-rw-r--r--spec/models/concerns/issuable_spec.rb20
1 files changed, 20 insertions, 0 deletions
diff --git a/spec/models/concerns/issuable_spec.rb b/spec/models/concerns/issuable_spec.rb
index 4fa06a8c60a..3cc96816cb0 100644
--- a/spec/models/concerns/issuable_spec.rb
+++ b/spec/models/concerns/issuable_spec.rb
@@ -44,6 +44,26 @@ describe Issue, "Issuable" do
it { expect(described_class).to respond_to(:assigned) }
end
+ describe "after_save" do
+ describe "#update_cache_counts" do
+ context "when previous assignee exists" do
+ it "user updates cache counts" do
+ expect(user).to receive(:update_cache_counts)
+
+ issue.update(assignee: user)
+ end
+ end
+
+ context "when previous assignee does not exist" do
+ it "does not raise error" do
+ issue.update(assignee_id: "")
+
+ expect { issue.update(assignee_id: user) }.not_to raise_error
+ end
+ end
+ end
+ end
+
describe ".search" do
let!(:searchable_issue) { create(:issue, title: "Searchable issue") }