Commit 48fd1e9c authored by Walmyr Lima's avatar Walmyr Lima

Remove unnecessary assertions from end-to-end test

The removed assertions sometimes cause this test to be flaky, due
to the automatic discussions not being rendered in time, and so,
it's better not to have them and have this test providing reliable
feedback, than have extra assertions that cause flakiness.

In any case, we'll still have the assertions that the issue was
added/removed to/from the epic, in the epic page.

Finally, this change save us five lines of code.
parent 9de12ecd
......@@ -63,12 +63,7 @@ module QA
Page::Project::Issue::Show.perform do |show|
show.wait_for_related_issues_to_load
show.comment("/epic #{issue.project.group.web_url}/-/epics/#{epic.iid}")
expect(show).to have_content('added to epic')
show.comment("/remove_epic")
expect(show).to have_content('removed from epic')
end
epic.visit!
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment