mirror of
https://github.com/home-assistant/core.git
synced 2025-07-30 08:47:09 +00:00
Expand on test coverage TODOs
This commit is contained in:
parent
2808787c84
commit
6f4b51d1ee
@ -6,7 +6,9 @@ rules:
|
||||
common-modules:
|
||||
status: done
|
||||
comment: When we refactor to use a coordinator, be sure to place it in coordinator.py.
|
||||
config-flow-test-coverage: done
|
||||
config-flow-test-coverage:
|
||||
status: todo
|
||||
comment: Use mock calls to check test_urlize_plain_host instead of user_input mod checks, combine test_show_set_form with a happy path flow, finish test_connection_errors and test_login_error with CREATE_ENTRY to check error recovery, move test_success to top and assert unique id in it, split test_reauth to two so we can test incorrect password recovery.
|
||||
config-flow:
|
||||
status: todo
|
||||
comment: Rename ConfigFlowHandler to HuaweiLteConfigFlow.
|
||||
@ -39,7 +41,9 @@ rules:
|
||||
comment: Kind of done, but to be reviewed, there's probably room for improvement. Maybe address this when converting to use a data update coordinator. See also https://github.com/home-assistant/core/issues/55495
|
||||
parallel-updates: todo
|
||||
reauthentication-flow: done
|
||||
test-coverage: todo
|
||||
test-coverage:
|
||||
status: todo
|
||||
comment: Get percentage up there, add missing actual action press invocations in button tests' suspended state tests, rename test_switch.py to test_switch.py + make its functions receive hass as first parameter where applicable.
|
||||
|
||||
# Gold
|
||||
devices: done
|
||||
|
Loading…
x
Reference in New Issue
Block a user