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

Better tests #35

Open
captn3m0 opened this issue Nov 30, 2015 · 2 comments
Open

Better tests #35

captn3m0 opened this issue Nov 30, 2015 · 2 comments

Comments

@captn3m0
Copy link
Member

We can do so much more with travis:

  1. Make sure that the couches are in _couches directory
  2. Make sure all files have valid extensions
  3. Anti-spam mechanisms via akismet. Run all the couch content for spam checks and fail the build if the spam scores are too high.
  4. Validate the couch structure itself (required/optional fields)
@captn3m0 captn3m0 mentioned this issue Nov 30, 2015
@ppaeps
Copy link
Collaborator

ppaeps commented Nov 30, 2015

On 2015-12-01 01:24:06 (+0530), Nemo notifications@github.com wrote:

We can do so much more with travis:

  1. Make sure that the couches are in _couches directory
  2. Make sure all files have valid extensions

I've made a start on 2 in pull request #36. Not sure how to go about
1. Is this travis contraption like a Git pre-receive hook? Just
adding a check for new files outside _couches is probably a lot too
naive. ;)

@captn3m0
Copy link
Member Author

Yes, travis just runs the cibuild script and fails the build if it returns non-zero.

Yeah, checking outside couches seems a bit excessive. I should pay a bit more attention before clicking the Merge button, thats all.

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

No branches or pull requests

3 participants