-
Notifications
You must be signed in to change notification settings - Fork 19
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
Investigate PhantomJS alternatives #1449
Comments
why do you think it would be so hard to get sniffybara working with Selenium/Chrome? We're losing much more than 8% of our time atm dealing with test failures. I don't see any obvious benefits of keeping phantomJS in our stack. I'd personally prefer to get it entirely out of our stack. But if it's truly easier to set up the chrome fallback, we can certainly try it first |
Another point. How well does Sniffybara work with SPAs? I can't remember it ever failing to be honest. I'd love to know a little bit of the history and where it's helped us in the past. |
Bumping the retries to 10 probably shaves off some of the problem but doesn't seem like a fix to me. I've observed failures even with 10 retries. It seems like phantom can get in a state where it just crashes over and over. No idea what's up with that. |
@amprokop Yeah, they def work with react, and they are pretty important for our ability to deploy daily. They caught a bunch of stuff with Reader (https://github.com/department-of-veterans-affairs/caseflow/pull/1379/files#r108937012) @joofsh maybe it's not as bad as I think :/. The current implementation is pretty coupled to Poltergeist, and I don't really know how the Selenium capybara driver rolls. |
@amprokop yeah that's so weird, sounds like retry 10 isn't really a good stop gap. Will try and get the Chrome fallback solution within the week, and then we can look into going all Chrome (with sniffy converted over to Chrome) |
Findings after a little bit of research
PhantomJS / Poltergeist / Sniffybara
Capybara-webdriver
Selenium / Chrome
Suggestion
Because it'd be such a pain to set up sniffy to work with the Selenium plugin, an
I'm leaning toward the Chrome fallback plan described here (https://blog.shakacode.com/yak-shaving-failing-integration-tests-with-react-a93444886c8c). We keep most of the benifits of PhantomJS, while mitigating the biggest drawback (the crashes). We could keep that going for a little while and see where to go from there.
Another easier option is to bump the retries to 10. I know its a little jank, but I don't see the downside (besides adding 10-15 seconds to some builds).
The text was updated successfully, but these errors were encountered: