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

[FEATURE ember-testing-wait-hooks] #3433

Merged
merged 1 commit into from Sep 26, 2013
Merged

Conversation

ef4
Copy link
Contributor

@ef4 ef4 commented Sep 17, 2013

In integration testing, sometimes it's necessary to make wait respect
other kinds of pending actions that are not in its default
implementation. For example, if your application is waiting for a CSS3
transition or an IndexDB action, the default wait implementaiton will
ignore those things and move on with the test prematurely.

This change lets you register additional functions that wait will call
to see whether it's really time to move on.

In integration testing, sometimes it's necessary to make `wait` respect
other kinds of pending actions that are not in its default
implementation. For example, if your application is waiting for a CSS3
transition or an IndexDB action, the default `wait` implementaiton will
ignore those things and move on with the test prematurely.

This change lets you register additional functions that `wait` will call
to see whether it's really time to move on.
@stefanpenner
Copy link
Member

👍 Awesome, I've been itching to role this. Thanks bro.

stefanpenner added a commit that referenced this pull request Sep 26, 2013
[FEATURE ember-testing-wait-hooks]
@stefanpenner stefanpenner merged commit 9341a94 into emberjs:master Sep 26, 2013
@stefanpenner
Copy link
Member

some recent concern about the api naming. (cc @tomdale @wycats)

@ghost ghost assigned stefanpenner Oct 28, 2013
@stefanpenner stefanpenner mentioned this pull request Jan 3, 2014
8 tasks
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.

None yet

2 participants