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

function names that include "test" at a word boundary #11

Open
pbstark opened this issue Sep 19, 2017 · 1 comment
Open

function names that include "test" at a word boundary #11

pbstark opened this issue Sep 19, 2017 · 1 comment

Comments

@pbstark
Copy link
Collaborator

pbstark commented Sep 19, 2017

There are still several function names that include "test," e.g., "greatest_name" and "test_count_on." Ideally, these should be re-named so that we can use nose for testing without any special configuration. Alternatively, we need a convention for where tests will live, and need to configure Travis CI to run only those tests.

@ron-rivest
Copy link
Owner

In the initial code development, these functions were included right with the
code that was being tested. Now they should be moved into separate module inside
the "tests" directory (where all the nosetests tests go). These include:
audit_orders: test_shuffle, test_audit_orders
election_spec: test_read_election_spec_general, test_read_election_spec_contests,
test_read_election_spec_contest_groups, test_read_election_spec_collections
groups: test_expand, test_expand_2
utils: test_count_on
("greatest_name" is not a test but is caught by the regex filter...)

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

No branches or pull requests

2 participants