Skip to content

Commit 2dc81ed

Browse files
Force int value on port in P1Monitor (#130084)
1 parent c4762f3 commit 2dc81ed

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

homeassistant/components/p1_monitor/config_flow.py

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -57,10 +57,13 @@ async def async_step_user(
5757
data_schema=vol.Schema(
5858
{
5959
vol.Required(CONF_HOST): TextSelector(),
60-
vol.Required(CONF_PORT, default=80): NumberSelector(
61-
NumberSelectorConfig(
62-
mode=NumberSelectorMode.BOX,
63-
)
60+
vol.Required(CONF_PORT, default=80): vol.All(
61+
NumberSelector(
62+
NumberSelectorConfig(
63+
min=1, max=65535, mode=NumberSelectorMode.BOX
64+
),
65+
),
66+
vol.Coerce(int),
6467
),
6568
}
6669
),

tests/components/p1_monitor/test_config_flow.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,7 @@ async def test_full_user_flow(hass: HomeAssistant) -> None:
3636
assert result2.get("type") is FlowResultType.CREATE_ENTRY
3737
assert result2.get("title") == "P1 Monitor"
3838
assert result2.get("data") == {CONF_HOST: "example.com", CONF_PORT: 80}
39+
assert isinstance(result2["data"][CONF_PORT], int)
3940

4041
assert len(mock_setup_entry.mock_calls) == 1
4142
assert len(mock_p1monitor.mock_calls) == 1

0 commit comments

Comments
 (0)