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

Continuous Deployment and Tests #25

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

Continuous Deployment and Tests #25

wants to merge 9 commits into from

Conversation

rdohms
Copy link
Member

@rdohms rdohms commented Jun 27, 2016

This PR introduces acceptance testing that makes sure the raffler is running well. These are fully functional and not unit tests, mostly due to easier coverage and the fact that our usual failures are due to external factors or coordination of the whole. So win, win.

These tests are written in nightmare.js but some things are still in progress.

- added gulp and node-sass to remove ruby deps
- moved in compass replacements
- added proper asset building with gulp
- moved source stuff to source
- added circle and deploy scripts, untested yet.
For some weird reason the cache seems to always be populated with the full event list, not the filtered one.
Moved the filtering to afterwards, so its always filtered before render, this way I don't care about cache.

There I fixed it™

/gif deal with it.
Nightmare tests simulate usage of raffler to ensure everything is ok.
Runs against the PHP Web Server.
@rdohms rdohms changed the title Continuos Deployment and Tests Continuous Deployment and Tests Jun 27, 2016
@lastmjs
Copy link

lastmjs commented Apr 6, 2017

You can run headless Electron with xvfb. Let me know if you have any questions.

@rdohms
Copy link
Member Author

rdohms commented Apr 6, 2017

@lastmjs i tried a couple versions of that and none worked out, could you fork from this branch and get our circle.yml in shape? I would appreciate that very much. Thanks

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

Successfully merging this pull request may close these issues.

2 participants