diff options
author | micael.bergeron <micael.bergeron@solutionstlm.com> | 2017-08-15 16:21:27 +0300 |
---|---|---|
committer | micael.bergeron <micael.bergeron@solutionstlm.com> | 2017-09-06 16:00:57 +0300 |
commit | b44a1bcd0b94a68f680c24d0dfd6d3402af9a881 (patch) | |
tree | b20c07b3bfaf86f00255a27f7428e1000c930fca /spec/models/concerns | |
parent | 45b83ed99afc5cfe24a8d84869894124d93d5b51 (diff) |
rework the contributor badge
- only show in merge-requests
- show as a little glyph
Diffstat (limited to 'spec/models/concerns')
-rw-r--r-- | spec/models/concerns/issuable_spec.rb | 12 |
1 files changed, 2 insertions, 10 deletions
diff --git a/spec/models/concerns/issuable_spec.rb b/spec/models/concerns/issuable_spec.rb index 9948340d976..dc959c58ba1 100644 --- a/spec/models/concerns/issuable_spec.rb +++ b/spec/models/concerns/issuable_spec.rb @@ -539,20 +539,12 @@ describe Issuable do context "for issues" do let(:contributor_issue) { create(:issue, author: contributor, project: project) } let(:first_time_contributor_issue) { create(:issue, author: first_time_contributor, project: project) } - let(:first_time_contributor_issue_other_project) { create(:issue, author: first_time_contributor, project: other_project) } - it "is true when you don't have any merged MR" do + it "is false even without merged MR" do expect(merged_mr).to be - expect(first_time_contributor_issue).to be_first_contribution + expect(first_time_contributor_issue).not_to be_first_contribution expect(contributor_issue).not_to be_first_contribution end - - it "handles multiple projects separately" do - expect(merged_mr).to be - expect(merged_mr_other_project).to be - expect(first_time_contributor_issue).to be_first_contribution - expect(first_time_contributor_issue_other_project).not_to be_first_contribution - end end end end |