Skip to content

Update visit to use wait_for_requests

Coung Ngo requested to merge cngo-update-visit-requests into master

What does this MR do and why?

This MR updates visit to use wait_for_requests

Use wait_for_requests instead of wait_for_all_requests because a user only expects to wait for JS requests before interacting and not rack requests. Since we are no longer waiting for rack requests, this should shorten feature spec execution time and possibly reduce timeouts.

This is similar to !130084 (merged)

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

How to set up and validate locally

Edited by Coung Ngo

Merge request reports