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-lazy-routing] #3695

Merged

Conversation

rwjblue
Copy link
Member

@rwjblue rwjblue commented Nov 5, 2013

  • setupForTesting sets the application into testing mode (which is used to defer readiness).
  • Adds an initializer that defers readiness if the application is in testing mode.
  • visit helper has been updated to advance readiness upon first call.
  • App.reset() now behaves the same after setupForTesting (in a deferred state).

* `setupForTesting` sets the application into testing mode (which is
  used to defer readiness.
* Adds an initializer that defers readiness if the application is in
  testing mode.
* `visit` helper has been updated to advance readiness upon first call.
* `App.reset()` now behaves the same after `setupForTesting` (in a
  deferred state).
@stefanpenner
Copy link
Member

nice, going to try this in an app today. I'll merge or provide feedback once i've done so.

@ghost ghost assigned stefanpenner Nov 5, 2013
@ryanto
Copy link
Contributor

ryanto commented Nov 5, 2013

I just put this in my app and it seems to work great, got about a 10% time to run improvement on integration tests and it helps clean up a lot of LOG_TRANSITIONS noise.

stefanpenner added a commit that referenced this pull request Nov 5, 2013
@stefanpenner stefanpenner merged commit 52d8217 into emberjs:master Nov 5, 2013
@stefanpenner
Copy link
Member

@ryanto seems good to me.

I've been craving this feature for some time thanks @rjackson for landing it.

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.

3 participants