diff --git a/homeassistant/components/mqtt/binary_sensor.py b/homeassistant/components/mqtt/binary_sensor.py index 9143b804c60b..7ab2e9ebf90e 100644 --- a/homeassistant/components/mqtt/binary_sensor.py +++ b/homeassistant/components/mqtt/binary_sensor.py @@ -42,7 +42,6 @@ from .mixins import ( MqttAvailability, MqttEntity, async_setup_entity_entry_helper, - validate_sensor_entity_category, write_state_on_attr_change, ) from .models import MqttValueTemplate, ReceiveMessage @@ -56,7 +55,7 @@ DEFAULT_PAYLOAD_ON = "ON" DEFAULT_FORCE_UPDATE = False CONF_EXPIRE_AFTER = "expire_after" -_PLATFORM_SCHEMA_BASE = MQTT_RO_SCHEMA.extend( +PLATFORM_SCHEMA_MODERN = MQTT_RO_SCHEMA.extend( { vol.Optional(CONF_DEVICE_CLASS): vol.Any(DEVICE_CLASSES_SCHEMA, None), vol.Optional(CONF_EXPIRE_AFTER): cv.positive_int, @@ -68,15 +67,7 @@ _PLATFORM_SCHEMA_BASE = MQTT_RO_SCHEMA.extend( } ).extend(MQTT_ENTITY_COMMON_SCHEMA.schema) -DISCOVERY_SCHEMA = vol.All( - validate_sensor_entity_category(binary_sensor.DOMAIN, discovery=True), - _PLATFORM_SCHEMA_BASE.extend({}, extra=vol.REMOVE_EXTRA), -) - -PLATFORM_SCHEMA_MODERN = vol.All( - validate_sensor_entity_category(binary_sensor.DOMAIN, discovery=False), - _PLATFORM_SCHEMA_BASE, -) +DISCOVERY_SCHEMA = PLATFORM_SCHEMA_MODERN.extend({}, extra=vol.REMOVE_EXTRA) async def async_setup_entry( diff --git a/homeassistant/components/mqtt/mixins.py b/homeassistant/components/mqtt/mixins.py index 412664ceedf7..8047af17050d 100644 --- a/homeassistant/components/mqtt/mixins.py +++ b/homeassistant/components/mqtt/mixins.py @@ -27,9 +27,8 @@ from homeassistant.const import ( CONF_NAME, CONF_UNIQUE_ID, CONF_VALUE_TEMPLATE, - EntityCategory, ) -from homeassistant.core import CALLBACK_TYPE, HomeAssistant, async_get_hass, callback +from homeassistant.core import CALLBACK_TYPE, HomeAssistant, callback from homeassistant.helpers import ( config_validation as cv, device_registry as dr, @@ -207,62 +206,6 @@ def validate_device_has_at_least_one_identifier(value: ConfigType) -> ConfigType ) -def validate_sensor_entity_category( - domain: str, discovery: bool -) -> Callable[[ConfigType], ConfigType]: - """Check the sensor's entity category is not set to `config` which is invalid for sensors.""" - - # A guard was added to the core sensor platform with HA core 2023.11.0 - # See: https://github.com/home-assistant/core/pull/101471 - # A developers blog from october 2021 explains the correct uses of the entity category - # See: - # https://developers.home-assistant.io/blog/2021/10/26/config-entity/?_highlight=entity_category#entity-categories - # - # To limitate the impact of the change we use a grace period - # of 3 months for user to update there configs. - - def _validate(config: ConfigType) -> ConfigType: - if ( - CONF_ENTITY_CATEGORY in config - and config[CONF_ENTITY_CATEGORY] == EntityCategory.CONFIG - ): - config_str: str - if not discovery: - config_str = yaml_dump(config) - config.pop(CONF_ENTITY_CATEGORY) - _LOGGER.warning( - "Entity category `config` is invalid for sensors, ignoring. " - "This stops working from HA Core 2024.2.0" - ) - # We only open an issue if the user can fix it - if discovery: - return config - config_file = getattr(config, "__config_file__", "?") - line = getattr(config, "__line__", "?") - hass = async_get_hass() - async_create_issue( - hass, - domain=DOMAIN, - issue_id="invalid_entity_category", - breaks_in_ha_version="2024.2.0", - is_fixable=False, - severity=IssueSeverity.WARNING, - translation_key="invalid_entity_category", - learn_more_url=( - f"https://www.home-assistant.io/integrations/{domain}.mqtt/" - ), - translation_placeholders={ - "domain": domain, - "config": config_str, - "config_file": config_file, - "line": line, - }, - ) - return config - - return _validate - - MQTT_ENTITY_DEVICE_INFO_SCHEMA = vol.All( cv.deprecated(CONF_DEPRECATED_VIA_HUB, CONF_VIA_DEVICE), vol.Schema( diff --git a/homeassistant/components/mqtt/sensor.py b/homeassistant/components/mqtt/sensor.py index 2c173f801fa2..9d1ed964be3f 100644 --- a/homeassistant/components/mqtt/sensor.py +++ b/homeassistant/components/mqtt/sensor.py @@ -44,7 +44,6 @@ from .mixins import ( MqttAvailability, MqttEntity, async_setup_entity_entry_helper, - validate_sensor_entity_category, write_state_on_attr_change, ) from .models import ( @@ -88,7 +87,6 @@ PLATFORM_SCHEMA_MODERN = vol.All( # Deprecated in HA Core 2021.11.0 https://github.com/home-assistant/core/pull/54840 # Removed in HA Core 2023.6.0 cv.removed(CONF_LAST_RESET_TOPIC), - validate_sensor_entity_category(sensor.DOMAIN, discovery=False), _PLATFORM_SCHEMA_BASE, ) @@ -96,7 +94,6 @@ DISCOVERY_SCHEMA = vol.All( # Deprecated in HA Core 2021.11.0 https://github.com/home-assistant/core/pull/54840 # Removed in HA Core 2023.6.0 cv.removed(CONF_LAST_RESET_TOPIC), - validate_sensor_entity_category(sensor.DOMAIN, discovery=True), _PLATFORM_SCHEMA_BASE.extend({}, extra=vol.REMOVE_EXTRA), ) diff --git a/homeassistant/components/mqtt/strings.json b/homeassistant/components/mqtt/strings.json index fac2f32d284d..3194806a2213 100644 --- a/homeassistant/components/mqtt/strings.json +++ b/homeassistant/components/mqtt/strings.json @@ -20,10 +20,6 @@ "title": "MQTT entities with auxiliary heat support found", "description": "Entity `{entity_id}` has auxiliary heat support enabled, which has been deprecated for MQTT climate devices. Please adjust your configuration and remove deprecated config options from your configuration and restart Home Assistant to fix this issue." }, - "invalid_entity_category": { - "title": "An MQTT {domain} with an invalid entity category was found", - "description": "Home Assistant detected a manually configured MQTT `{domain}` entity that has an `entity_category` set to `config`. \nConfiguration file: **{config_file}**\nNear line: **{line}**\n\nConfig with invalid setting:\n\n```yaml\n{config}\n```\n\nWhen set, make sure `entity_category` for a `{domain}` is set to `diagnostic` or `None`. Update your YAML configuration and restart Home Assistant to fix this issue." - }, "invalid_platform_config": { "title": "Invalid config found for mqtt {domain} item", "description": "Home Assistant detected an invalid config for a manually configured item.\n\nPlatform domain: **{domain}**\nConfiguration file: **{config_file}**\nNear line: **{line}**\nConfiguration found:\n```yaml\n{config}\n```\nError: **{error}**.\n\nMake sure the configuration is valid and [reload](/developer-tools/yaml) the manually configured MQTT items or restart Home Assistant to fix this issue." diff --git a/tests/components/mqtt/test_init.py b/tests/components/mqtt/test_init.py index 98e2c9b71fe4..3d7b349712ee 100644 --- a/tests/components/mqtt/test_init.py +++ b/tests/components/mqtt/test_init.py @@ -31,12 +31,7 @@ from homeassistant.const import ( import homeassistant.core as ha from homeassistant.core import CoreState, HomeAssistant, callback from homeassistant.exceptions import HomeAssistantError -from homeassistant.helpers import ( - device_registry as dr, - entity_registry as er, - issue_registry as ir, - template, -) +from homeassistant.helpers import device_registry as dr, entity_registry as er, template from homeassistant.helpers.entity import Entity from homeassistant.helpers.entity_platform import async_get_platforms from homeassistant.helpers.typing import ConfigType @@ -49,7 +44,6 @@ from .test_common import help_all_subscribe_calls from tests.common import ( MockConfigEntry, MockEntity, - async_capture_events, async_fire_mqtt_message, async_fire_time_changed, mock_restore_cache, @@ -2159,98 +2153,6 @@ async def test_setup_manual_mqtt_with_invalid_config( assert "required key not provided" in caplog.text -@pytest.mark.parametrize( - ("hass_config", "entity_id"), - [ - ( - { - mqtt.DOMAIN: { - "sensor": { - "name": "test", - "state_topic": "test-topic", - "entity_category": "config", - } - } - }, - "sensor.test", - ), - ( - { - mqtt.DOMAIN: { - "binary_sensor": { - "name": "test", - "state_topic": "test-topic", - "entity_category": "config", - } - } - }, - "binary_sensor.test", - ), - ], -) -@patch( - "homeassistant.components.mqtt.PLATFORMS", [Platform.BINARY_SENSOR, Platform.SENSOR] -) -async def test_setup_manual_mqtt_with_invalid_entity_category( - hass: HomeAssistant, - mqtt_mock_entry: MqttMockHAClientGenerator, - caplog: pytest.LogCaptureFixture, - entity_id: str, -) -> None: - """Test set up a manual sensor item with an invalid entity category.""" - events = async_capture_events(hass, ir.EVENT_REPAIRS_ISSUE_REGISTRY_UPDATED) - assert await mqtt_mock_entry() - assert "Entity category `config` is invalid for sensors, ignoring" in caplog.text - state = hass.states.get(entity_id) - assert state is not None - assert len(events) == 1 - - -@pytest.mark.parametrize( - ("config", "entity_id"), - [ - ( - { - "name": "test", - "state_topic": "test-topic", - "entity_category": "config", - }, - "binary_sensor.test", - ), - ( - { - "name": "test", - "state_topic": "test-topic", - "entity_category": "config", - }, - "sensor.test", - ), - ], -) -@patch( - "homeassistant.components.mqtt.PLATFORMS", [Platform.BINARY_SENSOR, Platform.SENSOR] -) -async def test_setup_discovery_mqtt_with_invalid_entity_category( - hass: HomeAssistant, - mqtt_mock_entry: MqttMockHAClientGenerator, - caplog: pytest.LogCaptureFixture, - config: dict[str, Any], - entity_id: str, -) -> None: - """Test set up a discovered sensor item with an invalid entity category.""" - events = async_capture_events(hass, ir.EVENT_REPAIRS_ISSUE_REGISTRY_UPDATED) - assert await mqtt_mock_entry() - - domain = entity_id.split(".")[0] - json_config = json.dumps(config) - async_fire_mqtt_message(hass, f"homeassistant/{domain}/bla/config", json_config) - await hass.async_block_till_done() - assert "Entity category `config` is invalid for sensors, ignoring" in caplog.text - state = hass.states.get(entity_id) - assert state is not None - assert len(events) == 0 - - @patch("homeassistant.components.mqtt.PLATFORMS", []) @pytest.mark.parametrize( ("mqtt_config_entry_data", "protocol"),