QA Selectors for Batch Comment E2E Automation
This commit is contained in:
parent
5a460397a1
commit
25b658cf65
7 changed files with 34 additions and 8 deletions
|
@ -167,7 +167,7 @@ export default {
|
|||
<button
|
||||
v-if="shouldShowCommentButton"
|
||||
type="button"
|
||||
class="add-diff-note js-add-diff-note-button"
|
||||
class="add-diff-note js-add-diff-note-button qa-diff-comment"
|
||||
title="Add a comment to this line"
|
||||
@click="handleCommentButton"
|
||||
>
|
||||
|
|
|
@ -143,7 +143,7 @@ export default {
|
|||
:show-comment-button="true"
|
||||
:diff-view-type="parallelDiffViewType"
|
||||
line-position="right"
|
||||
class="diff-line-num new_line"
|
||||
class="diff-line-num new_line qa-new-diff-line"
|
||||
/>
|
||||
<td
|
||||
:id="line.right.lineCode"
|
||||
|
|
|
@ -390,7 +390,7 @@ append-right-10 comment-type-dropdown js-comment-type-dropdown droplab-dropdown"
|
|||
:disabled="isSubmitButtonDisabled"
|
||||
name="button"
|
||||
type="button"
|
||||
class="btn comment-btn note-type-toggle js-note-new-discussion dropdown-toggle"
|
||||
class="btn comment-btn note-type-toggle js-note-new-discussion dropdown-toggle qa-note-dropdown"
|
||||
data-display="static"
|
||||
data-toggle="dropdown"
|
||||
aria-label="Open comment type dropdown">
|
||||
|
@ -422,7 +422,7 @@ append-right-10 comment-type-dropdown js-comment-type-dropdown droplab-dropdown"
|
|||
<li :class="{ 'droplab-item-selected': noteType === 'discussion' }">
|
||||
<button
|
||||
type="button"
|
||||
class="btn btn-transparent"
|
||||
class="btn btn-transparent qa-discussion-option"
|
||||
@click.prevent="setNoteType('discussion')">
|
||||
<i
|
||||
aria-hidden="true"
|
||||
|
|
|
@ -187,7 +187,7 @@ export default {
|
|||
:data-supports-quick-actions="!isEditing"
|
||||
name="note[note]"
|
||||
class="note-textarea js-gfm-input js-note-text
|
||||
js-autosize markdown-area js-vue-issue-note-form js-vue-textarea"
|
||||
js-autosize markdown-area js-vue-issue-note-form js-vue-textarea qa-reply-input"
|
||||
aria-label="Description"
|
||||
placeholder="Write a comment or drag your files here…"
|
||||
@keydown.meta.enter="handleUpdate()"
|
||||
|
|
|
@ -369,7 +369,7 @@ Please check your network connection and try again.`;
|
|||
role="group">
|
||||
<button
|
||||
type="button"
|
||||
class="js-vue-discussion-reply btn btn-text-field mr-2"
|
||||
class="js-vue-discussion-reply btn btn-text-field mr-2 qa-discussion-reply"
|
||||
title="Add a reply"
|
||||
@click="showReplyForm">Reply...</button>
|
||||
</div>
|
||||
|
|
|
@ -33,7 +33,7 @@
|
|||
.fade-left= icon('angle-left')
|
||||
.fade-right= icon('angle-right')
|
||||
%ul.merge-request-tabs.nav-tabs.nav.nav-links.scrolling-tabs
|
||||
%li.notes-tab
|
||||
%li.notes-tab.qa-notes-tab
|
||||
= tab_link_for @merge_request, :show, force_link: @commit.present? do
|
||||
Discussion
|
||||
%span.badge.badge-pill= @merge_request.related_notes.user.count
|
||||
|
@ -47,7 +47,7 @@
|
|||
= tab_link_for @merge_request, :pipelines do
|
||||
Pipelines
|
||||
%span.badge.badge-pill.js-pipelines-mr-count= @pipelines.size
|
||||
%li.diffs-tab
|
||||
%li.diffs-tab.qa-diffs-tab
|
||||
= tab_link_for @merge_request, :diffs do
|
||||
Changes
|
||||
%span.badge.badge-pill= @merge_request.diff_size
|
||||
|
|
|
@ -23,6 +23,32 @@ module QA
|
|||
element :squash_checkbox
|
||||
end
|
||||
|
||||
view 'app/views/projects/merge_requests/show.html.haml' do
|
||||
element :notes_tab
|
||||
element :diffs_tab
|
||||
end
|
||||
|
||||
view 'app/assets/javascripts/diffs/components/diff_line_gutter_content.vue' do
|
||||
element :diff_comment
|
||||
end
|
||||
|
||||
view 'app/assets/javascripts/notes/components/comment_form.vue' do
|
||||
element :note_dropdown
|
||||
element :discussion_option
|
||||
end
|
||||
|
||||
view 'app/assets/javascripts/notes/components/note_form.vue' do
|
||||
element :reply_input
|
||||
end
|
||||
|
||||
view 'app/assets/javascripts/notes/components/noteable_discussion.vue' do
|
||||
element :discussion_reply
|
||||
end
|
||||
|
||||
view 'app/assets/javascripts/diffs/components/parallel_diff_table_row.vue' do
|
||||
element :new_diff_line
|
||||
end
|
||||
|
||||
view 'app/views/shared/issuable/_sidebar.html.haml' do
|
||||
element :labels_block
|
||||
end
|
||||
|
|
Loading…
Reference in a new issue