-
Notifications
You must be signed in to change notification settings - Fork 4
Add missing unit tests #29
Comments
Add missing unit tests in organization/tests.py #28 |
I like this! GitHub tip: You can make checkboxes using GitHub markdown. Check out https://guides.github.com/features/mastering-markdown/ in the Task Lists section. |
@willingc It looks very cool, thank you! |
Nice. It is pretty cool :) |
@willingc @vubo Would It be a good idea to include the functional tests in this repo or should I include it in AUT repo. Benefit in first case is that I can integrate it with CI so all new code pushed passes throught these tests also. |
Add unit test for delete_organization and if statement Addresses one task in issue #29
Hmm...good questions. Nice job asking for thoughts and feedback.
Would you both raise these questions on your weekly reports for discussion? It looks like Systers/portal is already using Python 3.4 and Django 1.7.7. I would recommend that VMS match their Django infrastructure, but this really is a discussion for a larger audience :) Please feel free to continue discussing thoughts or ideas here too. |
Add missing unit tests in event/tests.py: #36 |
Add missing unit tests in job/tests.py: #38 |
@vubo You are making good progress on this issue. Let's try to get the remaining two done this week. I'll do a detailed review this weekend. You may wish to do a quick skim of coverage.py docs, tox, and some of Harry Percival's Test Driven Development book (http://obeythetestinggoat.com). We'll likely use all of these when adding new features. |
@willingc yes, I will finish the tests before Sunday then. |
Add missing unit tests in shift/tests.py: #43 |
Add missing unit tests in volunteer/tests.py: #47 |
The text was updated successfully, but these errors were encountered: