-
Notifications
You must be signed in to change notification settings - Fork 7
refactor: modularize OpenEVSE client by splitting core logic into dedicated modules and updating test suite #541
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
Closed
Closed
Changes from 2 commits
Commits
Show all changes
58 commits
Select commit
Hold shift + click to select a range
ef94d94
refactor: modularize OpenEVSE client by splitting core logic into ded…
firstof9 6fa13c0
refactor: delegate send_command to requester and expand test coverage…
firstof9 ecab0e1
changes based on feedback
firstof9 0565f62
test: remove unused mock_create_task variable in websocket tests
firstof9 eb1d99e
refactor: normalize non-JSON responses to dictionaries and improve we…
firstof9 9255f97
refactor: include status and success state in error responses and upd…
firstof9 b851f23
refactor: improve HTTP error logging and command response handling, a…
firstof9 979ca78
refactor: remove unused asyncio import and clean up whitespace in cov…
firstof9 6e1124e
add null safety checks for websocket and config states and update uni…
firstof9 7481ebe
improve error handling for callbacks, divert mode, and current limit …
firstof9 5c40132
prevent concurrent update callback execution and improve test cleanup…
firstof9 59f0db0
improve error handling and cache preservation during API updates and …
firstof9 2a72244
fix log capture scope for UnsupportedFeature exceptions in override …
firstof9 60815b9
improve error handling for write-refresh callbacks and serial detecti…
firstof9 19198aa
implement request coalescing for update callbacks in Requester and i…
firstof9 4a3d36a
improve error handling and validation for API responses and override …
firstof9 c212365
improve error handling and validation for override states and command…
firstof9 cc46758
improve robustness of divert mode, websocket payload handling, and ov…
firstof9 80be5c7
reorder websocket tests and update trigger key selection logic
firstof9 d51a3d6
improve divert mode validation and handle unknown EVSE state during o…
firstof9 79890cc
nullify websocket handle on close and return success status for curr…
firstof9 a5b683a
standardize RAPI response logging and add legacy firmware test fixtures
firstof9 be734d7
add state coercion for override toggling and fix websocket listening …
firstof9 186973d
improve error handling for non-JSON responses, validate claims data, …
firstof9 8720951
clear log buffers and fix test execution order in client and override…
firstof9 f99cc84
consolidate coverage tests into existing test suites and remove redun…
firstof9 003af07
ignore message-only responses during update and ensure non-dict respo…
firstof9 2bad3d3
simplify error response handling and improve websocket state test cov…
firstof9 4fef52a
improve status boolean casting, strengthen API response validation, a…
firstof9 de380de
enforce strict 'ok' status checks on commands and improve override st…
firstof9 96c8591
prevent concurrent payload types in requester and add exit condition …
firstof9 26819db
improve type casting for client properties and expand version check …
firstof9 39d1347
improve event loop handling, harden current limit validation, and up…
firstof9 485c264
improve temperature parsing and enhance override test coverage
firstof9 3e3f6ec
handle manual override sleeping state, allow empty RAPI requests, and…
firstof9 8c38c87
add comprehensive unit tests to improve code coverage for OpenEVSE cl…
firstof9 98d5292
improve test coverage and documentation for firmware and override fun…
firstof9 f3cb30e
update requester to support forced full refreshes and expand trigger …
firstof9 c4f2224
improve response validation, handle non-dict responses, and update ta…
firstof9 26e2afd
change websocket task storage from set to list to ensure ordered task…
firstof9 4aff377
add coverage for non-dict response handling and full_refresh callback…
firstof9 2893cc3
trigger full refresh on command execution and include delete method i…
firstof9 f61adcb
improve override state validation and refine RAPI command response ha…
firstof9 77a155d
add test case for set_override failure handling when POST returns err…
firstof9 c3a4d87
allow empty passwords for authentication and improve websocket state …
firstof9 0533c74
ignore unused mock_connect arguments in websocket tests
firstof9 1901a7a
serialize callback execution with a lock and coerce None passwords to…
firstof9 81035bc
improve event loop handling in websocket client by caching loop refer…
firstof9 45cc653
add test case for OpenEVSEWebsocket initialization without a running …
firstof9 08012be
prevent callback deadlocks with reentrancy handling and normalize req…
firstof9 46680cb
improve websocket loop handling and support read-only mapping payloads
firstof9 6db90cb
improve RAPI response handling and force full status refresh for lega…
firstof9 7249839
mock update method in toggle_override tests to prevent unnecessary ne…
firstof9 c31b1bb
improve websocket thread-safety and update status refresh error handling
firstof9 764396b
improve override toggle reliability, enhance error logging, and resol…
firstof9 741f1c9
update divert mode and override toggle tests to handle state dependen…
firstof9 a6e4222
add coverage for state setter scheduling failure scenarios
firstof9 858a65d
improve callback state management, update trigger logic, and refine t…
firstof9 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1,25 @@ | ||
| """Provide a package for python-openevse-http.""" | ||
|
|
||
| from .client import OpenEVSE | ||
| from .exceptions import ( | ||
| AlreadyListening, | ||
| AuthenticationError, | ||
| InvalidType, | ||
| MissingMethod, | ||
| MissingSerial, | ||
| ParseJSONError, | ||
| UnknownError, | ||
| UnsupportedFeature, | ||
| ) | ||
|
|
||
| __all__ = [ | ||
| "AlreadyListening", | ||
| "AuthenticationError", | ||
| "InvalidType", | ||
| "MissingMethod", | ||
| "MissingSerial", | ||
| "OpenEVSE", | ||
| "ParseJSONError", | ||
| "UnknownError", | ||
| "UnsupportedFeature", | ||
| ] | ||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.