Skip to content
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

Tests: Add a bunch of input-element tests (tests only, no code) #3866

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sideshowbarker
Copy link
Contributor

Importing these tests now because they are for input-element types that have requirements related to the constraint-validation API — which we’ve been implementing recently.

This commit only imports tests, without any changes to our code.

@sideshowbarker sideshowbarker force-pushed the tests-add-the-input-element branch 8 times, most recently from 7eb58b4 to 8b3ce5a Compare March 10, 2025 17:30
@sideshowbarker
Copy link
Contributor Author

All these tests (and all other tests) are passing when I run the test suite locally.

I am hitting the VERIFICATION FAILED: root_element->layout_node() at /Users/mike/workspace/ladybird/Libraries/LibWeb/CSS/Length.cpp:157 thing locally, but that seems to happen after all the tests have been run successfully (except for the one timeout)

Done!
==========================================================
Pass: 3442, Fail: 0, Skipped: 269, Timeout: 1, Crashed: 0
==========================================================
Timeout: Text/input/wpt-import/html/rendering/replaced-elements/svg-inline-sizing/svg-inline.html

@sideshowbarker sideshowbarker force-pushed the tests-add-the-input-element branch 3 times, most recently from 742b5ae to cef22dd Compare March 17, 2025 06:08
Importing these tests now because they are for input-element types that
have requirements related to the constraint-validation API — which we’ve
been implementing recently.

This commit only imports tests, without any changes to our code.
@sideshowbarker sideshowbarker force-pushed the tests-add-the-input-element branch from cef22dd to c8a2a8d Compare March 17, 2025 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant