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

Issue5 test suite #8

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

Conversation

wd15
Copy link
Member

@wd15 wd15 commented Apr 22, 2015

No description provided.

wd15 added 6 commits April 10, 2015 12:15
Address data-driven-science#5

The DBHandler class works with two test cases. The dbhandle.py script
uses DBHandler for all its functions.

 * The tests in the test/ directory are broken.

 * There are still issues when the tests shutdown their own database
   instance. It is probably better to start the database and then test
   rather than allow the tests to start their own DB instances
Address data-driven-science#5

Currently the tests run 5 times. This is an issue with Flask testing
and nosetest working together. Each test class seems to rerun the
entire test suite.
Address data-driven-science#5

 - Use test/config.py for all test configuration.

 - Use Handler to run the database daemon.

 - Delete the database on tear down.

 - Use TestCase instead of LiveServerTestCase to avoid running the
   tests multiple times.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants