diff options
author | Yorick Peterse <yorickpeterse@gmail.com> | 2019-03-15 19:34:37 +0300 |
---|---|---|
committer | Yorick Peterse <yorickpeterse@gmail.com> | 2019-03-18 16:40:09 +0300 |
commit | 3eee0426c5bee0edd5d65291e2ea32a02b906792 (patch) | |
tree | 2406ff8df44097d64123d2674146ffc25e784623 /spec/lib | |
parent | 96eefa7f28c189d308ec2bf129fec4d048f35edc (diff) |
Backport EE changes to UserExtractor to CE
This backports the changes EE made to Gitlab::UserExtractor, removing
the need for an EE specific module.
Diffstat (limited to 'spec/lib')
-rw-r--r-- | spec/lib/gitlab/user_extractor_spec.rb | 20 |
1 files changed, 20 insertions, 0 deletions
diff --git a/spec/lib/gitlab/user_extractor_spec.rb b/spec/lib/gitlab/user_extractor_spec.rb index fcc05ab3a0c..b86ec5445b8 100644 --- a/spec/lib/gitlab/user_extractor_spec.rb +++ b/spec/lib/gitlab/user_extractor_spec.rb @@ -38,6 +38,18 @@ describe Gitlab::UserExtractor do expect(extractor.users).to include(user) end + + context 'input as array of strings' do + it 'is treated as one string' do + extractor = described_class.new(text.lines) + + user_1 = create(:user, username: "USER-1") + user_4 = create(:user, username: "USER-4") + user_email = create(:user, email: 'user@gitlab.org') + + expect(extractor.users).to contain_exactly(user_1, user_4, user_email) + end + end end describe '#matches' do @@ -48,6 +60,14 @@ describe Gitlab::UserExtractor do it 'includes all mentioned usernames' do expect(extractor.matches[:usernames]).to contain_exactly('user-1', 'user-2', 'user-4') end + + context 'input has no matching e-mail or usernames' do + it 'returns an empty list of users' do + extractor = described_class.new('My test') + + expect(extractor.users).to be_empty + end + end end describe '#references' do |