Since AjaxThemeTest was committed assertWaitOnAjaxRequest was added to test for AJAX completion. Lets remove the local function and use the core one for consistency.

CommentFileSizeAuthor
#2 update_ajaxthemetest-2783819-2.patch999 bytesLendude
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Lendude created an issue. See original summary.

Lendude’s picture

Lendude
Primary language Dutch
Location Amsterdam
CreditAttribution: Lendude as a volunteer and at Noctilaris commented
Status: Active » Needs review
FileSize
update_ajaxthemetest-2783819-2.patch999 bytes

there we go

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner as a volunteer and at Chapter Three commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +Quickfix

+1

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed bd2772d to 8.3.x and cbf8e9d to 8.2.x. Thanks!

  • alexpott committed bd2772d on 8.3.x 
    Issue #2783819 by Lendude: Update AjaxThemeTest to use...

  • alexpott committed cbf8e9d on 8.2.x 
    Issue #2783819 by Lendude: Update AjaxThemeTest to use...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.