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

Updated editor tests to wait for edit() promise to resolve #903

Closed
wants to merge 3 commits into from

Conversation

edhager
Copy link
Contributor

@edhager edhager commented Apr 21, 2014

I updated the editor column plugin tests to wait for the promise returned by grid.edit() to resolve. Some of the tests were trying to test multiple columns so I divided them.

edhager added 3 commits April 21, 2014 09:29
…ilures) because the test was not waiting for the promise from Editor#edit to resolve before deleting the grid.
… I fixed that. In doing so, I found two problems. 1) The tests active an editOn editor and then later try to move it. During the move (done via a DOM appendChild call), the blur event handler was firing and removing the editor from the DOM. That combo was causing an error in the browser. 2) The editor plug-in module defined variables in which it tracked active editor state. Because they were defined at the module level, that state was carrying over between tests.
@kfranqueiro
Copy link
Member

This has been incorporated via 97f5832 with additional fixes in 716bdec and a983594. Thanks!

@edhager edhager deleted the editor-test-edit-promise branch November 17, 2014 15:47
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.

2 participants