summaryrefslogtreecommitdiff
path: root/spec/features
diff options
context:
space:
mode:
authorGitLab Bot <gitlab-bot@gitlab.com>2020-06-10 09:08:35 +0000
committerGitLab Bot <gitlab-bot@gitlab.com>2020-06-10 09:08:35 +0000
commit23ff717a29540bb1d4b0068f164b5f9df99386bf (patch)
tree1d31f1766f3a5e4748cd806cf655d72d35cb59dc /spec/features
parentc2879da99d02b4e49a1394f5699a2963f2651f18 (diff)
downloadgitlab-ce-23ff717a29540bb1d4b0068f164b5f9df99386bf.tar.gz
Add latest changes from gitlab-org/gitlab@master
Diffstat (limited to 'spec/features')
-rw-r--r--spec/features/action_cable_logging_spec.rb10
-rw-r--r--spec/features/issues/issue_sidebar_spec.rb3
-rw-r--r--spec/features/merge_request/user_edits_assignees_sidebar_spec.rb2
3 files changed, 14 insertions, 1 deletions
diff --git a/spec/features/action_cable_logging_spec.rb b/spec/features/action_cable_logging_spec.rb
index ba99405a51a..ce7c0e03aad 100644
--- a/spec/features/action_cable_logging_spec.rb
+++ b/spec/features/action_cable_logging_spec.rb
@@ -33,5 +33,15 @@ RSpec.describe 'ActionCable logging', :js do
gitlab_sign_in(user)
visit project_issue_path(project, issue)
+
+ # Because there is no visual indicator for Capybara to wait on before closing the browser,
+ # we need to test an actual feature to ensure that the subscription was already established.
+
+ expect(page.find('.assignee')).to have_content 'None'
+
+ fill_in 'note[note]', with: "/assign #{user.username}"
+ click_button 'Comment'
+
+ expect(page.find('.assignee')).to have_content user.name
end
end
diff --git a/spec/features/issues/issue_sidebar_spec.rb b/spec/features/issues/issue_sidebar_spec.rb
index 8f52a7541f4..1230fc76413 100644
--- a/spec/features/issues/issue_sidebar_spec.rb
+++ b/spec/features/issues/issue_sidebar_spec.rb
@@ -13,7 +13,6 @@ describe 'Issue Sidebar' do
let!(:xss_label) { create(:label, project: project, title: '&lt;script&gt;alert("xss");&lt;&#x2F;script&gt;') }
before do
- stub_feature_flags(save_issuable_health_status: false)
sign_in(user)
end
@@ -56,6 +55,8 @@ describe 'Issue Sidebar' do
it 'sees link to invite members' do
page.within '.dropdown-menu-user' do
expect(page).to have_link('Invite Members', href: project_project_members_path(project))
+ expect(page).to have_selector('[data-track-event="click_invite_members"]')
+ expect(page).to have_selector("[data-track-label='edit_assignee']")
end
end
end
diff --git a/spec/features/merge_request/user_edits_assignees_sidebar_spec.rb b/spec/features/merge_request/user_edits_assignees_sidebar_spec.rb
index 27b3ff3e085..2caa5de7e3d 100644
--- a/spec/features/merge_request/user_edits_assignees_sidebar_spec.rb
+++ b/spec/features/merge_request/user_edits_assignees_sidebar_spec.rb
@@ -112,6 +112,8 @@ describe 'Merge request > User edits assignees sidebar', :js do
it 'sees link to invite members' do
page.within '.dropdown-menu-user' do
expect(page).to have_link('Invite Members', href: project_project_members_path(project))
+ expect(page).to have_selector('[data-track-event="click_invite_members"]')
+ expect(page).to have_selector("[data-track-label='edit_assignee']")
end
end
end