diff options
author | Felipe Artur <felipefac@gmail.com> | 2016-12-14 17:57:14 -0200 |
---|---|---|
committer | Felipe Artur <felipefac@gmail.com> | 2016-12-20 17:55:14 -0200 |
commit | 77deeb12f74b857f9356168ccdf92612fc85fe84 (patch) | |
tree | eae473639b98b794b1249861af68f9f26efb4e3d /spec | |
parent | b6d069c10fbf0ef13ad78bb22cfc965a278adcea (diff) | |
download | gitlab-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.rb | 20 |
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") } |