-
Notifications
You must be signed in to change notification settings - Fork 19
Support HTML5 form attribute #1
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
Conversation
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.
Thanks, could you implement a test for this feature?
Set up code coverage
Yes I will; but to get an idea of current test status I want to set up coverage first. Could you got to scrutinizer and add this repository? (I can set up the rest) |
Done! |
As it turns out, the current implementation is actually pretty broken. <html>
<body>
<form id="form1">
<a href="https://google.nl">
<button form="invalid" name="abc" >This will link</button>
</a>
<a href="https://google.nl">
<button name="abc" >This will submit</button>
</a>
</form>
<a href="https://google.nl">
<button form="form1" name="abcdef" >This will submit</button>
</a>
</body>
</html> From testing on chrome: The codeception implemention will always use an anchor if that is found first. I will rewrite the implementation of the button. |
Please review the code carefully. Note: I do not throw an error when a Note that an empty <form><a href="#"><button form="">Test</button></a></form> |
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.
Looks good to me.
… type string, int given
Fixes Codeception/Codeception#5477