8000 Initial Test Scaffolding by ericclemmons · Pull Request #96 · evolution/genesis-wordpress · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Initial Test Scaffolding #96

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

Merged
merged 21 commits into from
Dec 19, 2013
Merged

Initial Test Scaffolding #96

merged 21 commits into from
Dec 19, 2013

Conversation

ericclemmons
Copy link
Contributor
  • Test generation with Yeoman
  • Initial tests with Mocha
  • Convert to Travis CI compatible tests

Refs #69

- "0.10"
addons:
hosts:
- local.generatortest.com
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was the crux to it all working.

@ericclemmons
Copy link
Contributor Author

@EvanK I'm ready to merge this down & tag it, just for testing with Travis to work! To run tests locally:

$ ./test/support/generate
$ cd ./test/temp
$ vagrant up
$ cd -
$ npm test

So, the generate script will scaffold a genesis wordpress site config for you to vagrant up. The IP/hostname/etc. is all hard-coded for testing. Mocha then runs functional tests against it, and even installs the site via Zombie.js!

On Travis, the same process takes place, except without the vagrant up.

@jimmynotjim
Copy link
Contributor

You don't need to merge it for Travis testing do you? It's tested it already (and reporting that it fails).

@EvanK
Copy link
Contributor
EvanK commented Dec 19, 2013

Looks like it errored out on the lamp-server install. Getting something to build in Travis always seems to take extra dicking around

@ericclemmons
Copy link
Contributor Author

Yea, I have to merge it, which will result in successful builds like:

https://travis-ci.org/genesis/wordpress/builds/15692665

After that passed, I corrected the code to prepare it for master. So, code-wise, does everything look good?

@EvanK
Copy link
Contributor
EvanK commented Dec 19, 2013

👍

ericclemmons added a commit that referenced this pull request Dec 19, 2013
@ericclemmons ericclemmons merged commit 8faf496 into master Dec 19, 2013
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.

3 participants
0