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 range tests + other #1423

Merged
merged 1 commit into from
Feb 11, 2025
Merged

Conversation

huss
Copy link
Member

@huss huss commented Feb 11, 2025

Description

  • Replace older maxmin test code with range. This only tests min/max and not the reading.
  • L23 was incorrectly in raw and is now in quantity.
  • Misc minor edits.

Type of change

  • Note merging this changes the database configuration.
  • This change requires a documentation update (already done)

Checklist

  • I have followed the OED pull request ideas
  • I have removed text in ( ) from the issue request
  • You acknowledge that every person contributing to this work has signed the OED Contributing License Agreement and each author is listed in the Description section.

Limitations

None known

- Replace older maxmin test code with range. This only tests min/max
and not the reading.
- L23 was incorrectly in raw and is now in quantity.
- Misc minor edits.
@huss
Copy link
Member Author

huss commented Feb 11, 2025

This only involves testing and mostly placeholders for future tests. Given this, I am approving my own work.

@huss huss merged commit a59cfd2 into OpenEnergyDashboard:development Feb 11, 2025
3 checks passed
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.

1 participant