Skip to content

Fix add_batch_comments_in_merge_request spec on staging

Anastasia McDonald requested to merge qa-fix-add-batch-comments-spec into master

What does this MR do and why?

Describe in detail what your merge request does and why.

E2E test add_batch_comments_in_merge_request_spec.rb is failing on staging as test environment broadcast banner is covering Submit button (see #364073 (closed)). This MR modifies the flow so that we click Submit button in the comment which is synchronised with the footer and has exactly the same effect.

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

Expand for screenshot image

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

bundle exec bin/qa Test::Instance::All http://localhost:3000/ -- qa/specs/features/ee/browser_ui/3_create/merge_request/add_batch_comments_in_merge_request_spec.rb

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports

Loading