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

nosetests issue with /test-script/ in the base level script #28

Closed
sanguineturtle opened this issue Jul 18, 2014 · 4 comments
Closed

nosetests issue with /test-script/ in the base level script #28

sanguineturtle opened this issue Jul 18, 2014 · 4 comments
Assignees
Labels

Comments

@sanguineturtle
Copy link
Contributor

Update folder name so it isn't picked up by nosetests

Current: test-script
Suggestion: scripts

@sanguineturtle sanguineturtle self-assigned this Jul 18, 2014
@cc7768
Copy link
Member

cc7768 commented Jul 18, 2014

Sounds like a good solution, anyone opposed to it?

@jstac
Copy link
Contributor

jstac commented Jul 19, 2014

This is a temporary replacement for unit testing, isn't it? In which case can we get rid of it? Tests are coming soon and we can always recover it if we need to. Correct me if I'm wrong.

@sanguineturtle
Copy link
Contributor Author

The little script ran all py files in a directory (as a test for the website examples). But we could run all of those in a test_websiteexamples.py as part of the testing infrastructure OR even better have a tests/ folder in the quant-econ/examples folder for the website.

Let's delete test-scripts and create a quant-econ/examples/tests/ for unit testing of the website examples.

@jstac
Copy link
Contributor

jstac commented Jul 19, 2014

Good plan. Done.

@jstac jstac closed this as completed Jul 19, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants