-
Notifications
You must be signed in to change notification settings - Fork 40
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
Data standards #149
Data standards #149
Conversation
Let's get Shreya's initialize_city test changes into master before we merge this so we can decide whether you should flesh out the test you wrote or just update her test |
The test_brisbane_config file is getting written to src/. Can you create a temp directory and write there instead? |
@j-t-t thanks, path updated and CI tests running now. |
Sorry, I should have been more specific. Best to use a tmpdir created for this test. Test functions can take an argument called tmpdir and you can use that. https://docs.pytest.org/en/latest/tmpdir.html |
@j-t-t thanks, updated with proper use of tmpdir fixture. |
set default city lat/lng in new config from geocoded response