Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Unable to run test suite using PhantomJS on Mac OS X #309

Closed
roelrz opened this Issue Mar 5, 2014 · 17 comments

Comments

Projects
None yet
5 participants
Contributor

roelrz commented Mar 5, 2014

We are unable to run a test suite (e.g., 100 files) using PhantomJS on Mac OS X successfully.

The test suite runs for a while but at a certain point we get the following the console:

error:  PhantomJS unable to load test URL: http://127.0.0.1:2013/venus-core/41 (timeout)
error:  PhantomJS unable to load test URL: http://127.0.0.1:2013/venus-core/42 (timeout)
error:  PhantomJS unable to load test URL: http://127.0.0.1:2013/venus-core/43 (timeout)
Contributor

roelrz commented Mar 5, 2014

I believe the issue is due to to Mac OS X limiting the number of processes to run.

I tried setting ulimit -n 1024 before running the test suite but that did not work

Contributor

roelrz commented Mar 6, 2014

The issue is not specific just to PhantomJS.

When I try running the test suite in Chrome (e.g., venus run -t mytestsuite/), some of the tests are not able to run at all. For example, if I were to click on the 40th test out of a 100 file test suite, it does not run and throws errors. However, if I run that same test by itself it works as expected.

It definitely seems like a Max OS X specific issue.

Contributor

sethmcl commented Mar 6, 2014

I realize this may be a bit of work, but could you do a PR with a new branch containing a large set of simple tests, which show this issue repro'ing? That would help me find the root cause faster. If not, I will try to create a repro sometime this weekend. Thanks :)

Contributor

roelrz commented Mar 6, 2014

sounds good. will do.

Contributor

roelrz commented Mar 6, 2014

Submitted PR with a new branch container a large test suite: #310

Contributor

sethmcl commented Mar 17, 2014

Hi Roel, I'm having a hard time reproducing this. I checked out your code and loaded all tests in the test/load directory. I ran the tests both in the browser (all at once, by going to the index page), and through phantom. All 100 tests executed.

Contributor

roelrz commented Mar 17, 2014

@sethmcl

Here's the error I'm getting when trying to run the test suite in test/load:

load

Contributor

roelrz commented Mar 17, 2014

Here are my computer settings:

Mac OS X
Version 10.8.5
Processor: 2.13 GHz Intel Core 2 Duo
Memory: 4 GB 1067 MHz DDR3

Contributor

rbakr commented Mar 17, 2014

@roelrz , I think the issue you are having is with the template/config. Do me a favor and try this out:

  1. From the folder you are having trouble running venus from rm -rf .venus/
  2. Run the tests again. Being that you have another config in your /usr/local/*, you should fall back to that, hopefully it works.

I was able to reproduce your issue, and resolve it with the steps above.

Contributor

roelrz commented Mar 17, 2014

@rbakr i tried that but still running into the same issue

Contributor

rbakr commented Mar 17, 2014

@roelrz Since I was able to reproduce this then fix it based on using a working config, I think that is a good place to begin looking. I don't currently have bandwidth but if I get time tonight I'll try to track down the bug in the config or its dependencies. Also, if you'd like to see an example of what I'm doing feel free to swing by.

Contributor

roelrz commented Mar 17, 2014

@rbakr gotcha. no worries. i'll take a deeper look into the config. we can definitely sync up tomorrow during our meeting.

Contributor

roelrz commented Mar 18, 2014

@sethmcl what version of venus are you using?

Was there a resolution to this issue? We're still seeing it.

Contributor

tgvrssanthosh commented Feb 21, 2017

We ran into this issue on linux boxes as well.

Contributor

tgvrssanthosh commented Feb 22, 2017

raised a pull request #362

@tgvrssanthosh tgvrssanthosh reopened this Feb 27, 2017

Contributor

tgvrssanthosh commented Feb 27, 2017

merged #363.

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