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

Running Tests #8

Open
dataspider opened this issue Nov 2, 2024 · 2 comments
Open

Running Tests #8

dataspider opened this issue Nov 2, 2024 · 2 comments

Comments

@dataspider
Copy link

It seems that to run any of the tests, I need an API key for the census data – although there is quite a bit of functionality that could be tested simply with the test data you deliver with the package.
Please separate the tests into those that handle data-retrieval functionality (which will require an API key) and tests that handle computational functionality, given locally available data.

openjournals/joss-reviews#7212

@bkrayfield
Copy link

I agree with this, a small testing file would be useful.

@mdp0023
Copy link
Owner

mdp0023 commented Nov 10, 2024

Thank you for the suggestion, I also agree that would add a lot of functionality to the tests. I split the original tests into two separate files, one to test functionality for data that comes with the package, and another that tests the data acquisition functions. I also set them up to remove intermediary files after tests are conducted to reduce unnecessary file sizes.”

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

3 participants