diff options
author | Winnie Hellmann <winnie@gitlab.com> | 2018-05-10 16:28:22 +0200 |
---|---|---|
committer | Winnie Hellmann <winnie@gitlab.com> | 2018-05-10 16:28:22 +0200 |
commit | fe9f500a8065cc614563f9bfc1635577e40e6458 (patch) | |
tree | 1210c2a6591ce804e039f54c7fe0ae772089f573 | |
parent | 63bcac14ca6b61af93074dbd4d285bcf449a9131 (diff) | |
download | gitlab-ce-fe9f500a8065cc614563f9bfc1635577e40e6458.tar.gz |
Fix spec/features/issues/create_issue_for_discussions_in_merge_request_spec.rb
-rw-r--r-- | app/assets/javascripts/notes/components/discussion_counter.vue | 2 | ||||
-rw-r--r-- | spec/features/issues/create_issue_for_discussions_in_merge_request_spec.rb | 12 |
2 files changed, 10 insertions, 4 deletions
diff --git a/app/assets/javascripts/notes/components/discussion_counter.vue b/app/assets/javascripts/notes/components/discussion_counter.vue index cbe4774a360..ba207f079d4 100644 --- a/app/assets/javascripts/notes/components/discussion_counter.vue +++ b/app/assets/javascripts/notes/components/discussion_counter.vue @@ -97,7 +97,7 @@ export default { <a :href="resolveAllDiscussionsIssuePath" v-tooltip - title="Resolve all discussions in new issue" + :title="s__('Resolve all discussions in new issue')" data-container="body" class="new-issue-for-discussion btn btn-default discussion-create-issue-btn"> <span v-html="mrIssueSvg"></span> diff --git a/spec/features/issues/create_issue_for_discussions_in_merge_request_spec.rb b/spec/features/issues/create_issue_for_discussions_in_merge_request_spec.rb index ca01f4e9a9f..52962002c33 100644 --- a/spec/features/issues/create_issue_for_discussions_in_merge_request_spec.rb +++ b/spec/features/issues/create_issue_for_discussions_in_merge_request_spec.rb @@ -6,6 +6,12 @@ feature 'Resolving all open discussions in a merge request from an issue', :js d let(:merge_request) { create(:merge_request, source_project: project) } let!(:discussion) { create(:diff_note_on_merge_request, noteable: merge_request, project: project).to_discussion } + def resolve_all_discussions_link_selector + text = "Resolve all discussions in new issue" + url = new_project_issue_path(project, merge_request_to_resolve_discussions_of: merge_request.iid) + %Q{a[data-original-title="#{text}"][href="#{url}"]} + end + describe 'as a user with access to the project' do before do project.add_master(user) @@ -15,7 +21,7 @@ feature 'Resolving all open discussions in a merge request from an issue', :js d it 'shows a button to resolve all discussions by creating a new issue' do within('.line-resolve-all-container') do - expect(page).to have_link "Resolve all discussions in new issue", href: new_project_issue_path(project, merge_request_to_resolve_discussions_of: merge_request.iid) + expect(page).to have_selector resolve_all_discussions_link_selector end end @@ -25,13 +31,13 @@ feature 'Resolving all open discussions in a merge request from an issue', :js d end it 'hides the link for creating a new issue' do - expect(page).not_to have_link "Resolve all discussions in new issue", href: new_project_issue_path(project, merge_request_to_resolve_discussions_of: merge_request.iid) + expect(page).not_to have_selector resolve_all_discussions_link_selector end end context 'creating an issue for discussions' do before do - click_link "Resolve all discussions in new issue", href: new_project_issue_path(project, merge_request_to_resolve_discussions_of: merge_request.iid) + find(resolve_all_discussions_link_selector).click end it_behaves_like 'creating an issue for a discussion' |