Ajax-enabled image effect forms do not update to the latest ajax processed configuration

Test-only feature seemed to skip so ran locally

1) Drupal\Tests\image\FunctionalJavascript\ImageAdminStylesTest::testAjaxEnabledEffectForm
Failed asserting that 100 is identical to 400.

/var/www/html/web/core/modules/image/tests/src/FunctionalJavascript/ImageAdminStylesTest.php:90

Code review the change makes sense.

@smustgrave thanks for your review. Sorry, I took it back to add some more assertions because I cannot understand why the parameter value is 100 at that point - it should be 111.

Added assertions to confirm the value of the parameter before and after the AJAX calls.

Ran test locally again

Behat\Mink\Exception\ExpectationException: The field "data[test_parameter]" value is "100", but "111" expected.

Which appears to be the new assertion added.

Yep, #5 the failing value was 100 and not 111 exactly for the same bug, which gets fixed here. Just the existing test was not catching it.

Tricky issue but the explanation and code change makes sense. One comment on the test coverage.

Switched to use assertExpectedAjaxRequest() as suggested.

Looks good!

Committed and pushed 4bfa1e9987e to 11.x and 50692daf255 to 11.0.x and c2bdd141b48 to 10.4.x and a0b547ee0f9 to 10.3.x. Thanks!

nod_ → closed merge request !9481

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