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

Unit tests #149

Merged
merged 12 commits into from
Feb 21, 2024
Merged

Unit tests #149

merged 12 commits into from
Feb 21, 2024

Conversation

erika-n
Copy link
Collaborator

@erika-n erika-n commented Feb 21, 2024

Finished writing unit tests and made sure 11 test files passed.

jkwan2011 and others added 12 commits January 3, 2024 02:15
Co-authored-by: Erika Nesse <[email protected]>
Co-authored-by: dwindleduck <[email protected]>
Co-authored-by: Alan Pinkert <[email protected]>
Co-authored-by: David Egan <[email protected]>
Co-authored-by: thatoldplatitude <[email protected]>
Co-authored-by: dwindleduck <[email protected]>
Co-authored-by: David Egan <[email protected]>
Co-authored-by: harry <[email protected]>
Co-authored-by: Debajyoti Debnath <[email protected]>
…engine

Co-authored-by: dwindleduck <[email protected]>
Co-authored-by: thatoldplatitude <[email protected]>
Co-authored-by: Alan Pinkert <[email protected]>
Co-authored-by: David Egan <[email protected]>
Co-authored-by: Debajyoti Debnath <[email protected]>
Co-authored-by: Jonathan Kwan <[email protected]>
Co-authored-by: dwindleduck <[email protected]>
Co-authored-by: Erika Nesse <[email protected]>
Co-authored-by: Debajyoti Debnath <[email protected]>
Co-authored-by: David Egan <[email protected]>
Co-authored-by: Alan Pinkert <[email protected]>
Co-authored-by: thatoldplatitude <[email protected]>
Co-authored-by: dwindleduck <[email protected]>
Co-authored-by: Debajyoti Debnath <[email protected]>
Co-authored-by: harry <[email protected]>
Co-authored-by: Alan Pinkert <[email protected]>
Co-authored-by: Jonathan Kwan <[email protected]>
Co-authored-by: David Egan <[email protected]>
Co-authored-by: dwindleduck <[email protected]>
Co-authored-by: Debajyoti Debnath <[email protected]>
Co-authored-by: harry <[email protected]>
Co-authored-by: Alan Pinkert <[email protected]>
Co-authored-by: Jonathan Kwan <[email protected]>
Co-authored-by: David Egan <[email protected]>
Co-authored-by: Tyler Monaghan <[email protected]>
Co-authored-by: Chad Stoughton <[email protected]>
Co-authored-by: thatoldplatitude <[email protected]>
Co-authored-by: dwindleduck <[email protected]>
Co-authored-by: Debajyoti Debnath <[email protected]>
Co-authored-by: harry <[email protected]>
Co-authored-by: Alan Pinkert <[email protected]>
Co-authored-by: Jonathan Kwan <[email protected]>
Co-authored-by: David Egan <[email protected]>
Co-authored-by: Tyler Monaghan <[email protected]>
Co-authored-by: Chad Stoughton <[email protected]>
Co-authored-by: thatoldplatitude <[email protected]>
Co-authored-by: dwindleduck <[email protected]>
Co-authored-by: Debajyoti Debnath <[email protected]>
Co-authored-by: Chad Stoughton <[email protected]>
Co-authored-by: David Egan <[email protected]>
Co-authored-by: thatoldplatitude <[email protected]>
Co-authored-by: Alan Pinkert <[email protected]>
Co-authored-by: harry <[email protected]>
Co-authored-by: David Egan <[email protected]>
Co-authored-by: thatoldplatitude <[email protected]>
Co-authored-by: dwindleduck <[email protected]>
Co-authored-by: AdamFinkle <[email protected]>
@erika-n erika-n merged commit 176a9cb into main Feb 21, 2024
12 checks passed
@erika-n erika-n deleted the unit_tests branch February 21, 2024 01:28
AdamFinkle added a commit to AdamFinkle/home-energy-analysis-tool that referenced this pull request May 22, 2024
* Updated test_engine to use JSON inputs

Co-authored-by: Erika Nesse <[email protected]>
Co-authored-by: dwindleduck <[email protected]>
Co-authored-by: Alan Pinkert <[email protected]>
Co-authored-by: David Egan <[email protected]>
Co-authored-by: thatoldplatitude <[email protected]>

* Collected BalancePointGraph info within Home

Co-authored-by: dwindleduck <[email protected]>
Co-authored-by: David Egan <[email protected]>
Co-authored-by: harry <[email protected]>
Co-authored-by: Debajyoti Debnath <[email protected]>

* Added test_get_outputs_natural gas to test_examples, refactored test_engine

Co-authored-by: dwindleduck <[email protected]>
Co-authored-by: thatoldplatitude <[email protected]>
Co-authored-by: Alan Pinkert <[email protected]>
Co-authored-by: David Egan <[email protected]>
Co-authored-by: Debajyoti Debnath <[email protected]>
Co-authored-by: Jonathan Kwan <[email protected]>

* Fixed bug in determining summer months.

Co-authored-by: dwindleduck <[email protected]>
Co-authored-by: Erika Nesse <[email protected]>
Co-authored-by: Debajyoti Debnath <[email protected]>
Co-authored-by: David Egan <[email protected]>
Co-authored-by: Alan Pinkert <[email protected]>
Co-authored-by: thatoldplatitude <[email protected]>

* Updated test_engine test cases based on spreadsheet

Co-authored-by: dwindleduck <[email protected]>
Co-authored-by: Debajyoti Debnath <[email protected]>
Co-authored-by: harry <[email protected]>
Co-authored-by: Alan Pinkert <[email protected]>
Co-authored-by: Jonathan Kwan <[email protected]>
Co-authored-by: David Egan <[email protected]>

* Added asssertions for UA and standard deviation

* Fixed issues with balance point differences from test cases

Co-authored-by: dwindleduck <[email protected]>
Co-authored-by: Debajyoti Debnath <[email protected]>
Co-authored-by: harry <[email protected]>
Co-authored-by: Alan Pinkert <[email protected]>
Co-authored-by: Jonathan Kwan <[email protected]>
Co-authored-by: David Egan <[email protected]>
Co-authored-by: Tyler Monaghan <[email protected]>
Co-authored-by: Chad Stoughton <[email protected]>
Co-authored-by: thatoldplatitude <[email protected]>

* Removed print statement

Co-authored-by: dwindleduck <[email protected]>
Co-authored-by: Debajyoti Debnath <[email protected]>
Co-authored-by: harry <[email protected]>
Co-authored-by: Alan Pinkert <[email protected]>
Co-authored-by: Jonathan Kwan <[email protected]>
Co-authored-by: David Egan <[email protected]>
Co-authored-by: Tyler Monaghan <[email protected]>
Co-authored-by: Chad Stoughton <[email protected]>
Co-authored-by: thatoldplatitude <[email protected]>

* Changed point where balance point graph data is collected.

Co-authored-by: dwindleduck <[email protected]>
Co-authored-by: Debajyoti Debnath <[email protected]>
Co-authored-by: Chad Stoughton <[email protected]>
Co-authored-by: David Egan <[email protected]>
Co-authored-by: thatoldplatitude <[email protected]>
Co-authored-by: Alan Pinkert <[email protected]>

* Finished writing tests for summary input

Co-authored-by: harry <[email protected]>
Co-authored-by: David Egan <[email protected]>
Co-authored-by: thatoldplatitude <[email protected]>
Co-authored-by: dwindleduck <[email protected]>
Co-authored-by: AdamFinkle <[email protected]>

* reformatted

---------

Co-authored-by: Jonathan Kwan <[email protected]>
Co-authored-by: Erika Nesse <[email protected]>
Co-authored-by: dwindleduck <[email protected]>
Co-authored-by: Alan Pinkert <[email protected]>
Co-authored-by: David Egan <[email protected]>
Co-authored-by: thatoldplatitude <[email protected]>
Co-authored-by: harry <[email protected]>
Co-authored-by: Debajyoti Debnath <[email protected]>
Co-authored-by: Tyler Monaghan <[email protected]>
Co-authored-by: Chad Stoughton <[email protected]>
Co-authored-by: AdamFinkle <[email protected]>
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

Successfully merging this pull request may close these issues.

2 participants