E2E test: dequarantine approval rules spec
What does this MR do and why?
The reason this test was quarantined has been fixed: #425787 (closed) / !132206 (merged)
The test had another issue whereby staging had too many groups, which caused the search to fail due to this bug, failure issue: #423899 (closed). The groups now cleaned on a schedule in staging, but to be on the safe side, I have removed :reliable
so it doesn't block deployments if it fails due to this reason again.
How to set up and validate locally
QA_LOG_LEVEL=debug bundle exec rspec qa/specs/features/ee/browser_ui/3_create/merge_request/approval_rules_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.
-
I have evaluated the MR acceptance checklist for this MR.
Edited by Jay McCure