qunit-dom icon indicating copy to clipboard operation
qunit-dom copied to clipboard

feature: support IDOMElementDescriptors

Open bendemboski opened this issue 1 year ago • 2 comments

Implement support for RFC #726

bendemboski avatar Dec 09 '23 19:12 bendemboski

This PR does change existing error messages, as can be seen from the tests, which isn't an explicit goal of this change. Here's the explanation.

If an assertion couldn't find its element (whether because of a non-matching selector or because it was passed null), previously the error messaging would be inconsistent between its usage of <not found> vs. <unknown>. The code that produced those messages was refactored and is now driven by the dom element descriptor description functionality, so it would require extra code to restore the inconsistent behavior and keep the error message the same as it was before this PR.

bendemboski avatar Dec 09 '23 19:12 bendemboski

Drafted https://github.com/mainmatter/qunit-dom/pull/2101 as per feedback, would this help dom-element-descriptors to create a "plugin" for qunit-dom?

BobrImperator avatar Apr 08 '24 19:04 BobrImperator

@BobrImperator this is rebased and, pending your decision on the assertion messages change or any other feedback you might have, ready to merge!

bendemboski avatar Jun 27 '24 05:06 bendemboski

https://github.com/mainmatter/qunit-dom/releases/tag/v3.2.0 :rocket:

BobrImperator avatar Jun 27 '24 18:06 BobrImperator