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

Add Automated Tests for Other Data Scripts #89

Open
alexkcode opened this issue Apr 3, 2024 · 3 comments
Open

Add Automated Tests for Other Data Scripts #89

alexkcode opened this issue Apr 3, 2024 · 3 comments
Labels
tech debt tests Related to automated testing changes & additions

Comments

@alexkcode
Copy link
Collaborator

Building off of #41, we would like to refactor and add test suites for the rest of the python scripts:

  • src/data/scripts/add_context_by_property_type.py
  • src/data/scripts/building_utils.py
  • src/data/scripts/clean_and_pare_down_data_current_year.py
  • src/data/scripts/process_data.py

Which should then test the following things (or at least the data backend of these features):

  • Buildings rendering properly with a wide variety of properties (new data, old data, with district chilling, without, with natural gas use an without)
    
  • Validating our data process works properly and generates correct calculations (we could make a small test CSV with 100 buildings and check calculations)
    
@gaylem
Copy link
Collaborator

gaylem commented Apr 17, 2024

@alexkcode Is this something you're planning on doing? I'd be interested in helping either way. I don't know much about testing but I'd like to learn! Let me know!

@alexkcode
Copy link
Collaborator Author

@alexkcode Is this something you're planning on doing? I'd be interested in helping either way. I don't know much about testing but I'd like to learn! Let me know!

@gaylem hey! if you're on tonight I can go over this with you, otherwise let's connect sometime on Slack about it.

@gaylem
Copy link
Collaborator

gaylem commented Apr 24, 2024

@alexkcode hey let's about it next Tuesday! The Docker stuff is done so I'm looking for my next mission :)

@vkoves vkoves added tests Related to automated testing changes & additions tech debt labels Dec 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tech debt tests Related to automated testing changes & additions
Projects
Status: No status
Development

When branches are created from issues, their pull requests are automatically linked.

3 participants