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.

Sign up for GitHub

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

Jump to bottom

Fix Reolink DHCP IP update #103654

Merged
merged 6 commits into from Nov 10, 2023
Merged

Fix Reolink DHCP IP update #103654

Changes from 2 commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
9b26bce
fix DHCP
starkillerOG Nov 8, 2023
98ba29b
fix Mappingproxy to normal dict
starkillerOG Nov 8, 2023
d433133
Add tests
starkillerOG Nov 9, 2023
b5c443a
Simplify
starkillerOG Nov 9, 2023
3b91103
use call_args_list
starkillerOG Nov 10, 2023
964d66a
remove duplicate test
starkillerOG Nov 10, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
4 changes: 3 additions & 1 deletion homeassistant/components/reolink/config_flow.py
View file
Open in desktop
Expand Up @@ -113,7 +113,9 @@ async def async_step_dhcp(self, discovery_info: dhcp.DhcpServiceInfo) -> FlowRes
raise AbortFlow("already_configured")

# check if the camera is reachable at the new IP
host = ReolinkHost(self.hass, existing_entry.data, existing_entry.options)
new_config = dict(existing_entry.data)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems like this is missing test coverage if it can be changed without updating tests

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well there is a check that goes over this code, but since the test controlls if a ConnectionError occurs or not it does not mather if the IP is valid or not.
The IP that is eventually saved to the config is checked, but that is correct, see line 142:
self._abort_if_unique_id_configured(updates={CONF_HOST: discovery_info.ip})
Where the correct discovery_info.ip is used

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just added some extra assertions (assert_called_with) in the tests so that it would now fail if this would be changed in the future.

new_config[CONF_HOST] = discovery_info.ip
host = ReolinkHost(self.hass, new_config, existing_entry.options)
try:
await host.api.get_state("GetLocalLink")
await host.api.logout()
Expand Down