Skip to content

Refactor user approval qa-selector

Jannik Lehmann requested to merge jnnkl-qa-selector-approval-modal into master

What does this MR do and why?

This MR solves an Issue that came up after merging this MR: !73848 (merged) It was mentioned here: !73848 (comment 747689504)

A missing qa-selector caused the tests to brake, this qa-selector is reintroduced in this MR.

Related isue: #346648 (closed)

Screenshots or screen recordings

No visual changes.

How to set up and validate locally

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

  1. cd $PATHTOGDK/gitlab/qa
  2. QA_DEBUG=true CHROME_HEADLESS=false bundle exec bin/qa Test::Instance::All http://localhost:3000 qa/specs/features/browser_ui/1_manage/login/register_spec.rb:109

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 Sanad Liaquat

Merge request reports