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

Add missing } in core ajax tests #1990

Closed
wants to merge 10 commits into from
Closed

Add missing } in core ajax tests #1990

wants to merge 10 commits into from

Conversation

rbri
Copy link
Contributor

@rbri rbri commented Nov 11, 2023

Found while working on improving the js support of HtmlUnit.

1cg and others added 10 commits November 7, 2023 10:02
`keyup` only works with a keyboard, and ignores pasting with a right click or programmatically updating the value with JS. 

`input` is a better event for detecting any "inputs to the input".

Co-authored-by: Sam Eaton <sam@eaton.party>
* docs: php (symfony sdk)

* docs: twig support for fragments

---------

Co-authored-by: Cristian Toma <cristian.toma@vivre.eu>
* Homogenize configuration docs

* Remove source change
Could not figure out how to import directly from docs.md with zola.
rbri added a commit to HtmlUnit/htmlunit that referenced this pull request Nov 11, 2023
@rbri
Copy link
Contributor Author

rbri commented Nov 11, 2023

Looks like this bug reduces the number of test in the test suite. After fixing the 1.9.7 release test suite changed from 584 to 653 tests.

@alexpetros
Copy link
Collaborator

Huh, odd, thanks. Can you rebase against dev please?

@alexpetros alexpetros changed the title add missing } Add missing } in core ajax tests Nov 13, 2023
@rbri
Copy link
Contributor Author

rbri commented Nov 14, 2023

@alexpetros rebased to test - is it ok now? or do you like to have a pr against the dev branch?

@alexpetros
Copy link
Collaborator

Yes, the PR should be against the dev branch

@rbri
Copy link
Contributor Author

rbri commented Nov 14, 2023

So i have to create a new one...

@alexpetros
Copy link
Collaborator

alexpetros commented Nov 14, 2023

Nope, you can retarget this PR, just rebase it against dev and then edit the PR!

@rbri rbri changed the base branch from master to dev November 14, 2023 17:59
@rbri
Copy link
Contributor Author

rbri commented Nov 14, 2023

like this?

@alexpetros
Copy link
Collaborator

Yup, but now you have to rebase it against dev so it's only the one commit

@rbri
Copy link
Contributor Author

rbri commented Nov 14, 2023

Sorry but i will create a new PR.

@rbri
Copy link
Contributor Author

rbri commented Nov 14, 2023

replaced by #1998

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.

6 participants