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 acceptance test framework based on node jasmine #3

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

heruka-urgyen
Copy link
Member

This is an experiment on solving #1 using node jasmine

TODOs:

  • create infrastructure to run tests
  • add URL as an argument to npm test (acceptance_test wrapper will pass it)

Prerequisites:

  1. Installed Node
  2. Installed NPM

How to run:

  1. Run npm install to install a local copy of jasmine-node and run tests (which are set up as a post-install script)
  2. Consecutive tests can be run with npm test

Tests are located in spec directory. Each filename must end with .spec.js

Pros:

  • only one prerequisite
  • simple directory structure and minimal setup needed

Cons:

  • supports only BDD test style out of the box

@bzz
Copy link
Member

bzz commented Dec 24, 2014

Updated description to list all dependencies.
Do you think it might be worth moving node stuff to spec itself as that's the only part node-dependant in this branch?

@bzz
Copy link
Member

bzz commented Dec 24, 2014

Did not make a PR for this by mistake! Sorry.
Also I think it is good to links PRs back to issues, so doing it right now #1 :)

Added TODO: how about a URL as parameter (with localhost by default)?
So we ca run it accross different instances\implementation local and remote

@heruka-urgyen
Copy link
Member Author

npm comes together with node, so

Installed NPM

is for free after you installed node.

@heruka-urgyen
Copy link
Member Author

how about a URL as parameter

I'm not exactly sure how to do it. Perhaps, through an environment variable?

@bzz
Copy link
Member

bzz commented Dec 24, 2014

Env var sounds good if there is no other way to pass arguments to
test-runner.
Please feel free to modify ./test_spec I'v updated it with the URL
selection logic

On Wed, Dec 24, 2014 at 1:25 PM, felizbear [email protected] wrote:

how about a URL as parameter
I'm not exactly sure how to do it. Perhaps, through an environment
variable?


Reply to this email directly or view it on GitHub
#3 (comment)
.

Kind regards,
Alexander.

@bzz bzz mentioned this pull request Jan 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants