- Issue created by @catch
- Status changed to Needs review
over 1 year ago 6:37pm 14 November 2023 - 🇮🇳India pradhumanjainOSL
Replace assertWaitOnAjaxRequest with assertExpectedAjaxRequest
and add the deprecation warning in assertWaitOnAjaxRequest method.
Please review. - Status changed to Needs work
over 1 year ago 6:45pm 14 November 2023 The Needs Review Queue Bot → tested this issue.
While you are making the above changes, we recommend that you convert this patch to a merge request → . Merge requests are preferred over patches. Be sure to hide the old patch files as well. (Converting an issue to a merge request without other contributions to the issue will not receive credit.)
- Status changed to Postponed
over 1 year ago 8:43pm 14 November 2023 - 🇬🇧United Kingdom catch
The changes look mostly fine. This actually requires 📌 Stabilize FunctionalJavascript testing AJAX: add ::assertExpectedAjaxRequest() Fixed to land first, I should have marked it postponed on that and doing so now.
For the new deprecation, it should be from 10.2.0 instead of 10.1.0
- Status changed to Active
11 months ago 7:15am 3 August 2024 - Assigned to ankitv18
- 🇺🇸United States nafisb
I'm Nafis at DrupalCon Atlanta. I plan on working on this issue for the rest of the day.
- Merge request !11663Draft: Deprecate `assertWaitOnAjaxRequest` method in JSWebAssert in favor of `assertExpectedAjaxRequest` → (Closed) created by Unnamed author
- Merge request !1Draft: Deprecate `assertWaitOnAjaxRequest` test method → (Open) created by Unnamed author
- First commit to issue fork.
- Merge request !12108Issue #3401619: Replace assertWaitOnAjaxRequest() with assertExpectedAjaxRequest() → (Open) created by brandonlira
- 🇧🇷Brazil brandonlira
Hi, I've created a new clean branch to address this issue.
The previous branch had too many unrelated commits and was outdated, causing multiple conflicts and making it hard to review.
This new branch includes only the intended change: replacing all usages of the deprecated assertWaitOnAjaxRequest() with assertExpectedAjaxRequest() across the core/ directory.A new merge request has been opened from this clean branch.
Could someone please review and test this updated MR !12108?
Thank you! - Issue was unassigned.
- Status changed to Needs review
about 1 month ago 5:16pm 30 May 2025 - 🇺🇸United States smustgrave
Ticket talks about deprecating a method but don't see the deprecation