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.

Sign up for GitHub

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

Jump to bottom

Unit tests #21

Open
nathan-osman opened this issue Mar 11, 2014 · 2 comments
Open

Unit tests #21

nathan-osman opened this issue Mar 11, 2014 · 2 comments
Labels
feature important

Comments

@nathan-osman
Copy link
Member

Ideally, we should have unit tests for as much of the code as possible to ensure new commits don't have unexpected side-effects. If set up correctly, we can use Travis CI to run the tests for pull requests.
@nathan-osman
Copy link
Member Author

The unit tests are nowhere near complete so I will reopen this bug.

@nathan-osman nathan-osman reopened this Mar 22, 2014
nathan-osman added a commit that referenced this issue Sep 1, 2014
@nathan-osman
Began adding a simple unit test for article permissions (#21).
a191648
nathan-osman added a commit that referenced this issue Sep 1, 2014
@nathan-osman
Updated first test to check access by anonymous users (#21).
69c80c5
nathan-osman added a commit that referenced this issue Sep 2, 2014
@nathan-osman
Implemented full range of permissions tests for viewing articles (#21).
2ced23a
@nathan-osman
Copy link
Member Author

Another update (since I've been forgetting to reference this bug in commit messages): I just added tests for CMarkdown (our custom markdown extension). The following should be included in the test suite before this bug is closed:

  • scheduled articles
  • form submissions (registration, feedback, article submission)
  • RSS feed and API

These are the big ones that I can think of off the top of my head. I may think of others to add.

nathan-osman added a commit that referenced this issue Feb 9, 2015
@nathan-osman
Added test for the public API (#21).
9c022de
nathan-osman added a commit that referenced this issue Feb 9, 2015
@nathan-osman
Added test for min / max parameters in the API (#21).
c37fb50
nathan-osman added a commit that referenced this issue Feb 9, 2015
@nathan-osman
Completed tests for the API (#21).
b948453
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature important
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@nathan-osman