\Drupal\Tests\ckeditor5\Kernel\ValidatorsTest::test() obscures 2nd, 3rd etc violation message for the same property path

All green. Looking at the MR nothing stands out as bad.

Tried running tests without the fix but just got Error : Call to undefined method Drupal\Tests\ckeditor5\Kernel\ValidatorsTest::violationsToArray()

But think this is good to move on.

This makes the original issue a bit simpler and it is a clear improvement. RTBC+1

Saving issue credits.

Pushed this to 11.x to unblock progress on 📌 Configuration schema & required keys Fixed , but the following seems redundant:

+    $actual_violations = self::violationsToArray($violations);
+    $this->assertSame($expected_violations, self::violationsToArray($violations));

Anyone want to file a quick issue to clean that up?

You mean that the variable $actual_violations is redundant?

If so: I respectfully disagree: it's very helpful for setting conditional breakpoints 😇

It doesn't cost us performance, nor are end users affected since this is only for tests. So, I'm kinda hoping this is fine as-is. Still, happy to do what you're requesting if you disagree! 👍

You mean that the variable $actual_violations is redundant?

No. Just that the 2nd line can be simplified to use that variable.

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