diff options
author | Sean McGivern <sean@gitlab.com> | 2019-07-31 14:32:13 +0300 |
---|---|---|
committer | Sean McGivern <sean@gitlab.com> | 2019-07-31 14:32:13 +0300 |
commit | fff0fc7b4c2475a432fd4e1b025be66f9f5ba519 (patch) | |
tree | 83ee6971083b9ce41efc2a6ab7238af13ed5f294 | |
parent | 533237a097281dbe4fb1c821d5823c4de8c8f6af (diff) | |
parent | 89a8d02c350a18b23e1ba0f00080e4549d9770ec (diff) |
Merge branch '65317-adapt-tae-role-name-change' into 'master'
Fix Danger finding a test reviewer
Closes #65317
See merge request gitlab-org/gitlab-ce!31292
-rw-r--r-- | lib/gitlab/danger/teammate.rb | 2 | ||||
-rw-r--r-- | spec/controllers/projects/error_tracking_controller_spec.rb | 2 | ||||
-rw-r--r-- | spec/features/display_system_header_and_footer_bar_spec.rb | 2 | ||||
-rw-r--r-- | spec/lib/gitlab/danger/teammate_spec.rb | 8 |
4 files changed, 11 insertions, 3 deletions
diff --git a/lib/gitlab/danger/teammate.rb b/lib/gitlab/danger/teammate.rb index b44f134f2c1..74cbcc11255 100644 --- a/lib/gitlab/danger/teammate.rb +++ b/lib/gitlab/danger/teammate.rb @@ -39,7 +39,7 @@ module Gitlab def has_capability?(project, category, kind, labels) case category when :test - area = role[/Test Automation Engineer, (\w+)/, 1] + area = role[/Test Automation Engineer(?:.*?, (\w+))/, 1] area && labels.any?(area) if kind == :reviewer else diff --git a/spec/controllers/projects/error_tracking_controller_spec.rb b/spec/controllers/projects/error_tracking_controller_spec.rb index 844c61f1ace..d11ef24ef96 100644 --- a/spec/controllers/projects/error_tracking_controller_spec.rb +++ b/spec/controllers/projects/error_tracking_controller_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require 'rails_helper' +require 'spec_helper' describe Projects::ErrorTrackingController do set(:project) { create(:project) } diff --git a/spec/features/display_system_header_and_footer_bar_spec.rb b/spec/features/display_system_header_and_footer_bar_spec.rb index af9d9a5834f..e32da1a02bc 100644 --- a/spec/features/display_system_header_and_footer_bar_spec.rb +++ b/spec/features/display_system_header_and_footer_bar_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -require 'rails_helper' +require 'spec_helper' describe 'Display system header and footer bar' do let(:header_message) { "Foo" } diff --git a/spec/lib/gitlab/danger/teammate_spec.rb b/spec/lib/gitlab/danger/teammate_spec.rb index 6a6cf1429c8..171f2344e82 100644 --- a/spec/lib/gitlab/danger/teammate_spec.rb +++ b/spec/lib/gitlab/danger/teammate_spec.rb @@ -40,6 +40,14 @@ describe Gitlab::Danger::Teammate do it '#maintainer? returns false' do expect(subject.maintainer?(project, :test, labels)).to be_falsey end + + context 'when hyperlink is mangled in the role' do + let(:role) { '<a href="#">Test Automation Engineer</a>, Create' } + + it '#reviewer? returns true' do + expect(subject.reviewer?(project, :test, labels)).to be_truthy + end + end end context 'when role is Test Automation Engineer, Manage' do |