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 PhantomJS 2.x to test suite #10

Closed
jamesshore opened this issue Feb 16, 2015 · 8 comments
Closed

Add PhantomJS 2.x to test suite #10

jamesshore opened this issue Feb 16, 2015 · 8 comments

Comments

@jamesshore
Copy link
Owner

Add PhantomJS to the list of supported browsers.

Blocked by Medium/phantomjs#288.

@jamesshore
Copy link
Owner Author

PhantomJS 1.9.x does not work with Quixote because PhantomJS uses an old version of WebKit that doesn't support "load" events on <link> tags. (See ariya/phantomjs#11721.) The load event is needed for stylesheets, which we use heavily in our tests.

The new 2.0 version of PhantomJS will hopefully fix this problem. We're waiting for the PhantomJS npm module to come out before supporting it, as it makes our build automation cleaner.

The PhantomJS npm module is blocked on Medium/phantomjs#288. When this issue is fixed, we should be able to support PhantomJS.

@wolframkriesing
Copy link

nice to see :)
awesome that you picked it up so fast

@AkeemMcLennon
Copy link

I assume it's still possible to test against PhantomJS 2.0 if the binary is already installed? From my understanding, the npm module just downloads it from the website.

@jamesshore
Copy link
Owner Author

Definitely. This is just about adding it to our test suite and ensuring any cross-browser incompatibilities are addressed.

Right now, PhantomJS 2.0 should work (even 1.x works for some people's needs) but it hasn't gone through our test suite.

@AkeemMcLennon
Copy link

Great! I'll be sure to give it a try later today.

@jamesshore jamesshore changed the title Support PhantomJS Add PhantomJS 2.x to test suite Nov 18, 2015
@jamesshore
Copy link
Owner Author

It looks like the phantomjs installer we were waiting on has been forked, so it may be possible to implement this now.

  • phantomjs2 supports installing on Mac
  • phantomjs2.0 (a fork of phantomjs2) supports installing on Mac and Linux

It seems like the original phantomjs installer is still waiting on Windows binary support, but double-check Medium/phantomjs#288 before implementing.

@jamesshore
Copy link
Owner Author

To do this, we need a way for PhantomJS to fit into our workflow. That probably means having a way of starting and stopping PhantomJS from the Jakefile.

This isn't very high on my priority list, so I'm looking for someone who wants this to submit a pull request.

@jamesshore
Copy link
Owner Author

I've decided to stop officially supporting headless browsers as a general rule. The purpose of Quixote is to check how sites display to users on their actual browsers, so testing against a headless browser defeats the purpose. With tools like Karma, it isn't necessary to use a headless browser.

That said, we do test against Safari, which uses WebKit just like PhantomJS does. It's likely that PhantomJS will work just fine. If it doesn't, it's probably a bug in PhantomJS (or an outdated version of WebKit).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants