Skip to content

Fix failing QA spec

Sincheol (David) Kim requested to merge fix-approval-settings-qa-failure into master

What does this MR do and why?

It fixes failing ./qa/specs/features/ee/browser_ui/4_verify/new_discussion_not_dropping_merge_trains_mr_spec.rb after enabling support_group_level_merge_checks_setting feature flag.

It's caused by the change in the placement of data-qa-selector which was moved to the the div instead of the checkbox input when feature flag is enabled. I think we should move the attribute to checkbox input.

Related to #402987 (comment 1332960839) and #377723 (closed)

MR acceptance checklist

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

Edited by Sincheol (David) Kim

Merge request reports