-
Notifications
You must be signed in to change notification settings - Fork 29
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
Initial Testcases #14
Conversation
@hhsecond let me know when this is ready for a code review. thanks for all the work! |
Fixed many errors around managing singleton contexts, temporarily disabled doctests until we are able to configure autodoc correctly.
hey @hhsecond, As you'll see I pushed some changes on my end to update the tests and the test-runners. The travis-ci build is now passing on all tests (the appveyor config still needs some work, i'm not really sure what this issue with windows is right now). In general, really nice job! Definitely on our way to having decent test coverage here. I'll send in some more changes/comments when I have the next free minute, but this is a great start! |
see also #20 which we will want to merge and rebase this branch on top of. |
I'm going to merge this so that focused test development can occur in isolated PRs. |
No description provided.