New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add CEReactions tests #14711
Add CEReactions tests #14711
Conversation
- HTMLMapElement - HTMLCanvasElement - HTMLDataElement - HTMLDetailsElement - HTMLModElement - HTMLOListElement - HTMLQuoteElement - HTMLSlotElement - HTMLTimeElement
w3c-test:mirror |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests are wrong with boolean attributes; unclear what's going on with
|
||
<script> | ||
|
||
testReflectAttribute('open', 'open', 'true', 'false', 'open on HTMLDetailsElement', 'details', HTMLDetailsElement); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Boolean attributes don't get reflected by the strings "true" and "false". They get reflected by the removal or addition of the attribute.
|
||
<script> | ||
|
||
testReflectAttribute('cite', 'cite', '../resources/custom-elements-helpers.js', './resources/reactions.js', 'cite on ins use HTMLModElement', 'ins', HTMLModElement); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This test fails on Chrome. I believe it is because the attribute name is datetime
, not dateTime
.
|
||
<script> | ||
|
||
testReflectAttribute('reversed', 'reversed', 'true', 'false', 'reversed on HTMLOListElement', 'ol', HTMLOListElement); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Boolean attributes problem again
|
||
<script> | ||
|
||
testReflectAttribute('dateTime', 'dateTime', '2018-12-10', '2018-12-12', 'dateTime on HTMLTimeElement', 'time', HTMLTimeElement); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also fails on Chrome, also I guess for the same reason.
- Use "datetime" as the attribute name. - Modify the tests for boolean attributes.
@domenic, many thanks for your review! I just submit a new commit to address your comments, PTAL. |
w3c-test:mirror |
part of issue #5727