Commit 704ad67

authored

Ensure hue tests add config entry before updating it (home-assistant#110428)

1 parent a51ecd0 commit 704ad67Copy full SHA for 704ad67

File tree

-1

lines changed

-1

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -178,10 +178,10 @@ async def setup_bridge(hass: HomeAssistant, mock_bridge_v1):`
`178`	`178`	`"""Load the Hue light platform with the provided bridge."""`
`179`	`179`	`hass.config.components.add(hue.DOMAIN)`
`180`	`180`	`config_entry = create_config_entry()`
	`181`	`+ config_entry.add_to_hass(hass)`
`181`	`182`	`hass.config_entries.async_update_entry(`
`182`	`183`	`config_entry, options={CONF_ALLOW_HUE_GROUPS: True}`
`183`	`184`	`)`
`184`		`- config_entry.add_to_hass(hass)`
`185`	`185`	`config_entry.mock_state(hass, ConfigEntryState.LOADED)`
`186`	`186`	`mock_bridge_v1.config_entry = config_entry`
`187`	`187`	`hass.data[hue.DOMAIN] = {config_entry.entry_id: mock_bridge_v1}`

Lines changed: 2 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -14,6 +14,7 @@ async def test_migrate_api_key(hass: HomeAssistant) -> None:`
`14`	`14`	`domain=hue.DOMAIN,`
`15`	`15`	`data={"host": "0.0.0.0", "api_version": 2, "username": "abcdefgh"},`
`16`	`16`	`)`
	`17`	`+ config_entry.add_to_hass(hass)`
`17`	`18`	`await hue.migration.check_migration(hass, config_entry)`
`18`	`19`	`# the username property should have been migrated to api_key`
`19`	`20`	`assert config_entry.data == {`
`@@ -29,6 +30,7 @@ async def test_auto_switchover(hass: HomeAssistant) -> None:`
`29`	`30`	`domain=hue.DOMAIN,`
`30`	`31`	`data={"host": "0.0.0.0", "api_version": 1, "username": "abcdefgh"},`
`31`	`32`	`)`
	`33`	`+ config_entry.add_to_hass(hass)`
`32`	`34`
`33`	`35`	`with patch.object(hue.migration, "is_v2_bridge", retun_value=True), patch.object(`
`34`	`36`	`hue.migration, "handle_v2_migration"`

Comments

(0)