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

Jump to bottom

UI tests fail with phantomjs has crashed #7147

Closed
mattab opened this issue Feb 6, 2015 · 5 comments
Closed

UI tests fail with phantomjs has crashed #7147

mattab opened this issue Feb 6, 2015 · 5 comments
Labels
c: Tests & QA For issues related to automated tests or making it easier to QA & test issues. Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change. worksforme The issue cannot be reproduced and things work as intended.
Milestone
Backlog (Help wan...

Comments

@mattab
Copy link
Member

mattab commented Feb 6, 2015

The goal of this issue is to investigate and solve the Segmentation fault in phantomjs.

UI tests CI job sometime fail with:

[...]
✓ should show the updated segment after page reload (28791ms)

PhantomJS has crashed. Please read the crash reporting guide at https://github.com/ariya/phantomjs/wiki/Crash-Reporting and file a bug report at https://github.com/ariya/phantomjs/issues/new with the crash dump file attached: /tmp/4fc62382-9b22-a9c5-2c28353d-0e63dcb7.dmp

/home/travis/build/piwik/piwik/tests/travis/travis.sh: line 76: 16423 Segmentation fault phantomjs ../lib/screenshot-testing/run-tests.js --store-in-ui-tests-repo --persist-fixture-data --assume-artifacts --core

for example this job: https://travis-ci.org/piwik/piwik/jobs/49804130

(in the same build the 5.3.3 job failed with DB connection errors... maybe the build server is unstable at times)
@mattab mattab added Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change. c: Tests & QA For issues related to automated tests or making it easier to QA & test issues. labels Feb 6, 2015
@mattab mattab added this to the Short term milestone Feb 6, 2015
@mattab
Copy link
Member Author

mattab commented Feb 13, 2015

Recently getting those crashes more and more often eg https://travis-ci.org/piwik/piwik/jobs/50577885

Maybe it would help to upgrade to newer version of phantomjs if one is available?

@mattab mattab modified the milestones: Short term, Mid term Apr 7, 2015
@mnapoli
Copy link
Contributor

mnapoli commented Jun 2, 2015

Getting segmentation faults with phantomjs lately (on Travis), we'll see if it lasts.

@tsteur
Copy link
Member

tsteur commented Jun 2, 2015

Same here on my branch https://travis-ci.org/piwik/piwik/jobs/65056870 . That's the UI test for my branch but it works in the PR build that runs with master

@mattab
Copy link
Member Author

mattab commented Jun 9, 2015

btw: we're not the only users to experience phantomjs segfaults and crashes: https://github.com/ariya/phantomjs/issues?utf8=%E2%9C%93&q=is%3Aissue+is%3Aopen+crash+1.9.8

Travis is currently using 1.9.8 version of phantomjs. have we considered upgrading to 2.0.0, if we're lucky it may help with the crash/seg fault? (although likely it will not help)

@diosmosis diosmosis mentioned this issue Jun 9, 2015
Merged
@mattab mattab mentioned this issue Jun 18, 2015
Closed
@mattab mattab modified the milestones: Long term, Mid term Dec 5, 2016
@mattab
Copy link
Member Author

mattab commented Feb 28, 2017

not seeing this problem anymore

@mattab mattab closed this as completed Feb 28, 2017
@mattab mattab added the worksforme The issue cannot be reproduced and things work as intended. label Feb 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: Tests & QA For issues related to automated tests or making it easier to QA & test issues. Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change. worksforme The issue cannot be reproduced and things work as intended.
Projects
None yet
Milestone
Backlog (Help wanted)
Development

No branches or pull requests
3 participants
@tsteur @mattab @mnapoli