Skip to content

Unquarantine "ee/spec/features/registrations/email_confirmation_spec.rb"

Roy Liu requested to merge 393330-flaky-test-email_confirmation_spec into master

What does this MR do and why?

After this MR: !107302 (merged), the original test case in this flaky test does not exist anymore. Specifically, removing this line changed the test case by removing the stub of soft_email_confirmation

After investigation, we feel like we could turn this test back on and the original error is very likely caused by form validations, so the two wait_for_all_requests are added here.

MR acceptance checklist

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

Related to #393330 (closed)

Edited by Roy Liu

Merge request reports