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 a link to phantomjs, for not-so-js-skilled bootstrap-noobs like me #3347

Closed
wants to merge 1 commit into from
Closed

Conversation

trumbitta
Copy link
Contributor

I'm having issues trying to run make test... one of them was that it was not clear enough to me that I needed to google for and install this phantomjs instead of a nodejs module named phantom.

@eitch
Copy link

eitch commented May 8, 2012

Hi trumbitta. I realized that after trying to install phantomjs using npm, so then i installed it using apt-get, but i still have a problem running make test:

jshint js/.js --config js/.jshintrc
jshint js/tests/unit/
.js --config js/.jshintrc
node js/tests/server.js &
phantomjs js/tests/phantom.js "http://localhost:3000/js/tests"

node.js:201
throw e; // process.nextTick error, or 'error' event on first tick
^
Error: Cannot find module 'connect'
at Function._resolveFilename (module.js:332:11)
at Function._load (module.js:279:25)
at Module.require (module.js:354:17)
at require (module.js:370:17)
at Object. (/data/src/git/bootstrap/js/tests/server.js:6:15)
at Module._compile (module.js:441:26)
at Object..js (module.js:459:10)
at Module.load (module.js:348:32)
at Function._load (module.js:308:12)
at Array.0 (module.js:479:10)
Unable to access network
kill -9 cat js/tests/pid.txt
cat: js/tests/pid.txt: No such file or directory
/bin/sh: 1: kill: Usage: kill [-s sigspec | -signum | -sigspec] [pid | job]... or
kill -l [exitstatus]
make: *** [test] Error 2

Do you by any chance no why it can't find this connect module?

Maybe then this merge request should have some more changes?

@trumbitta
Copy link
Contributor Author

Your error is the very same that lead me to submit this: #3376

@RichardBronosky
Copy link

This would have saved me a lot of time. I also would like to see the README.md corrected as you mention you pull/3376 I have a plugin that I'd like to submit a pull request for but I still haven't managed to run tests yet. Sigh.

fat added a commit that referenced this pull request Jun 3, 2012
@fat
Copy link
Member

fat commented Jun 3, 2012

added a link - thanks! (also, in the future please make pull requests against wip- branches - ❤️)

@fat fat closed this Jun 3, 2012
@trumbitta
Copy link
Contributor Author

Thank you!
About pull requests: lesson learned & noobness defeated ;)

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

Successfully merging this pull request may close these issues.

None yet

4 participants