Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Support
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
G
gitlab-ce
Project overview
Project overview
Details
Activity
Releases
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
0
Merge Requests
0
Analytics
Analytics
Repository
Value Stream
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Create a new issue
Commits
Issue Boards
Open sidebar
Jérome Perrin
gitlab-ce
Commits
7c0bf0dc
Commit
7c0bf0dc
authored
8 years ago
by
Douwe Maan
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Remove flaky diff note tests for now
parent
5a77eb15
No related merge requests found
Changes
1
Hide whitespace changes
Inline
Side-by-side
Showing
1 changed file
with
0 additions
and
94 deletions
+0
-94
spec/features/merge_requests/diffs_spec.rb
spec/features/merge_requests/diffs_spec.rb
+0
-94
No files found.
spec/features/merge_requests/diffs_spec.rb
View file @
7c0bf0dc
...
@@ -22,98 +22,4 @@ feature 'Diffs URL', js: true, feature: true do
...
@@ -22,98 +22,4 @@ feature 'Diffs URL', js: true, feature: true do
expect
(
page
).
to
have_css
(
'.diffs.tab-pane.active'
)
expect
(
page
).
to
have_css
(
'.diffs.tab-pane.active'
)
end
end
end
end
context
'when hovering over the parallel view diff file'
do
let
(
:comment_button_class
)
{
'.add-diff-note'
}
before
(
:each
)
do
visit
diffs_namespace_project_merge_request_path
@project
.
namespace
,
@project
,
@merge_request
click_link
'Side-by-side'
@old_line_number
=
first
'.diff-line-num.old_line:not(.empty-cell)'
@new_line_number
=
first
'.diff-line-num.new_line:not(.empty-cell)'
@old_line
=
first
'.line_content[data-line-type="old"]'
@new_line
=
first
'.line_content[data-line-type="new"]'
end
it
'shows a comment button on the old side when hovering over an old line number'
do
@old_line_number
.
hover
expect
(
@old_line_number
).
to
have_css
comment_button_class
expect
(
@new_line_number
).
not_to
have_css
comment_button_class
end
it
'shows a comment button on the old side when hovering over an old line'
do
@old_line
.
hover
expect
(
@old_line_number
).
to
have_css
comment_button_class
expect
(
@new_line_number
).
not_to
have_css
comment_button_class
end
it
'shows a comment button on the new side when hovering over a new line number'
do
@new_line_number
.
hover
expect
(
@new_line_number
).
to
have_css
comment_button_class
expect
(
@old_line_number
).
not_to
have_css
comment_button_class
end
it
'shows a comment button on the new side when hovering over a new line'
do
@new_line
.
hover
expect
(
@new_line_number
).
to
have_css
comment_button_class
expect
(
@old_line_number
).
not_to
have_css
comment_button_class
end
end
context
'when hovering over the inline view diff file'
do
let
(
:comment_button_class
)
{
'.add-diff-note'
}
before
(
:each
)
do
visit
diffs_namespace_project_merge_request_path
@project
.
namespace
,
@project
,
@merge_request
click_link
'Inline'
@old_line_number
=
first
'.diff-line-num.old_line:not(.unfold)'
@new_line_number
=
first
'.diff-line-num.new_line:not(.unfold)'
@new_line
=
first
'.line_content:not(.match)'
end
it
'shows a comment button on the old side when hovering over an old line number'
do
@old_line_number
.
hover
expect
(
@old_line_number
).
to
have_css
comment_button_class
expect
(
@new_line_number
).
not_to
have_css
comment_button_class
end
it
'shows a comment button on the new side when hovering over a new line number'
do
@new_line_number
.
hover
expect
(
@old_line_number
).
to
have_css
comment_button_class
expect
(
@new_line_number
).
not_to
have_css
comment_button_class
end
it
'shows a comment button on the new side when hovering over a new line'
do
@new_line
.
hover
expect
(
@old_line_number
).
to
have_css
comment_button_class
expect
(
@new_line_number
).
not_to
have_css
comment_button_class
end
end
context
'when clicking a comment button'
do
let
(
:test_note_comment
)
{
'this is a test note!'
}
let
(
:note_class
)
{
'.new-note'
}
before
(
:each
)
do
visit
diffs_namespace_project_merge_request_path
@project
.
namespace
,
@project
,
@merge_request
click_link
'Inline'
first
(
'.diff-line-num.old_line:not(.unfold)'
).
hover
find
(
'.add-diff-note'
).
click
end
it
'shows a note form'
do
expect
(
page
).
to
have_css
note_class
end
it
'can be submitted and viewed'
do
fill_in
'note[note]'
,
with: :test_note_comment
click_button
'Comment'
expect
(
page
).
to
have_content
:test_note_comment
end
it
'can be closed'
do
find
(
'.note-form-actions .btn-cancel'
).
click
expect
(
page
).
not_to
have_css
note_class
end
end
end
end
This diff is collapsed.
Click to expand it.
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment