-
-
Notifications
You must be signed in to change notification settings - Fork 84
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
test tooling (phantomjs support?) #43
Comments
I ran into this issue and was able to use this polyfill for phantomjs |
ah great :) |
If it helps, I installed this polyfill
and then updated my with
|
raytiley
added a commit
to raytiley/ember-collection
that referenced
this issue
Jan 30, 2016
This is mostly to allow tests to run in PhantomJS without requring a 'requestAnimationFrame' pollyfill. Closes adopted-ember-addons#43.
raytiley
added a commit
to raytiley/ember-collection
that referenced
this issue
Jan 30, 2016
This is mostly to allow tests to run in PhantomJS without requring a 'requestAnimationFrame' pollyfill. Closes adopted-ember-addons#43.
raytiley
added a commit
to raytiley/ember-collection
that referenced
this issue
Jan 31, 2016
This is mostly to allow tests to run in PhantomJS without requring a 'requestAnimationFrame' pollyfill. Closes adopted-ember-addons#43. The weirdness in the test is because there is no way currently to make sure the component is cleared and destroyed before the module's teardown hook is called. Wrapping the component in an if is a workaround until this: https://github.com/switchfly/ember-test-helpers/pull/147/files lands.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The fact that
ember-collection
doesn't run in phantomjs tripped me up for a time; at the least the documentation should mention that, as phantomjs is standard config for ember-cli apps.Better would be to get it to work in phantomjs, though; we need a
requestAnimationFrame
polyfill. I tried https://github.com/cagosta/requestAnimationFrame but couldn't get it to work. Any ideas?The text was updated successfully, but these errors were encountered: