drupalPostAjaxForm() does not exist anymore

Open on Drupal.org →

Created on 10 November 2024, 4 months ago

Turning on phpstan evaluation of the test cases (see 📌 Add declare(strict_types=1) to all test modules Active ) reveals that we still have code in the tests that uses drupalPostAjaxForm(). This method was part of Simpletest and was removed from Drupal core more than 5 years ago by #2809161: Convert Javascript/AJAX testing to use JavascriptTestBase →

This method is called from LinkTypeFieldEntryTest::doFlagUiFieldPlugin() which appears to be a utility function for internal use in Flag tests (but it's not declared protected for some reason). However, this utility function is never called so in practice the non-existent drupalPostAjaxForm() never gets called either. That's why this problem only shows up with static analysis, and not in the PHPUnit test output.

This problem was pointed out years ago in #2751053: Convert Simpletests to PHPUnit → but it seems to have been overlooked in the eventual commits.

The simplest thing would be to just delete doFlagUiFieldPlugin() entirely. My preference would be to first determine what that function was intended to test, and before deleting this ensure that Flag does have a WebDriver test that was ported from this Simpletest method at some point.

🐛 Bug report

Status

Active

Version

4.0

Component

Flag core

Created by

🇺🇸United States tr Cascadia

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Merge Requests

!122drupalPostAjaxForm() does not exist anymore
Merged
ivnish
updated 18 days ago

Comments & Activities

Issue created by @tr
Comment about 1 month ago →
🇺🇸United States tr Cascadia
LinkTypeFieldEntryTest::doFlagUiFieldPlugin() was first created as LinkTypeFieldEntryTest::doFlagUIfieldPlugin() (different capitalization) back in 2016 by this issue: #2701503: Refactor test classes to take advantage of FlagTestBase. → . While the method was declared, it wasn't used even back then.

I'm posting this as documentation and to bring it back near the top of my to-do list. My next step will be to see where the code in that method was located in the commit prior to this, and see if it was in use or whether the refactoring just forgot to use it when it was put into the new function. Baby steps ...
Comment 18 days ago →
ivnish Kazakhstan
Comment 18 days ago →
ivnish Kazakhstan
Yes, the test with this function doesn't execute. Let's remove it
Merge request !122Issue #3486527 by ivnish, tr: drupalPostAjaxForm() does not exist anymore → (Merged) created by ivnish
Comment 18 days ago →
System Message

ivnish → committed 82b1accf on 8.x-4.x
Issue #3486527 by ivnish, tr: drupalPostAjaxForm() does not exist...
Comment 18 days ago →
ivnish Kazakhstan
Comment 4 days ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024