Problem/Motivation
Tests the formatter output with different formatter settings.
Issue fork advanced_text_formatter-3607084
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #3
akshay kashyap commentedI tested this MR locally on a clean Drupal installation and went through all of the scenarios covered by the new kernel test.
I also verified the formatter manually with different settings, including:
In each case, the rendered output matched the expected result, and the new kernel test passed successfully.
I also ran the module's test suite after applying the MR and didn't encounter any regressions or unexpected failures.
Overall, this is a nice improvement. The data provider makes the test easy to read and extend, and it provides good coverage for the formatter's core functionality. This should help catch regressions if changes are made to the formatter in the future.
I only have a couple of minor suggestions:
: voidfor the test method and: arrayfor the data provider) to keep the test consistent with current Drupal coding standards.Other than those minor suggestions, the implementation looks good to me, and all the scenarios I tested locally are working as expected.
Comment #4
akshay kashyap commentedComment #6
igor mashevskyi commented