Merge branch 'dev' into hassfest_trigger_target

script/hassfest/services.py

@@ -43,13 +43,21 @@ def unique_field_validator(fields: Any) -> Any:
     return fields
 
 
-CORE_INTEGRATION_FIELD_SCHEMA = vol.Schema(
+CUSTOM_INTEGRATION_EXTRA_SCHEMA_DICT = {
-    {
+    vol.Optional("description"): str,
+    vol.Optional("name"): str,
+}
+
+
+CORE_INTEGRATION_NOT_TARGETED_FIELD_SCHEMA_DICT = {
     vol.Optional("example"): exists,
     vol.Optional("default"): exists,
     vol.Optional("required"): bool,
     vol.Optional("advanced"): bool,
     vol.Optional(CONF_SELECTOR): selector.validate_selector,
+}
+
+FIELD_FILTER_SCHEMA_DICT = {
     vol.Optional("filter"): {
         vol.Exclusive("attribute", "field_filter"): {
             vol.Required(str): [vol.All(str, service.validate_attribute_option)],
@@ -57,90 +65,95 @@ CORE_INTEGRATION_FIELD_SCHEMA = vol.Schema(
         vol.Exclusive("supported_features", "field_filter"): [
             vol.All(str, service.validate_supported_feature)
         ],
-        },
     }
-)
+}
 
-CORE_INTEGRATION_SECTION_SCHEMA = vol.Schema(
+
-    {
+def _field_schema(targeted: bool, custom: bool) -> vol.Schema:
+    """Return the field schema."""
+    schema_dict = CORE_INTEGRATION_NOT_TARGETED_FIELD_SCHEMA_DICT.copy()
+
+    # Filters are only allowed for targeted services because they rely on the presence
+    # of a `target` field to determine the scope of the service call. Non-targeted
+    # services do not have a `target` field, making filters inapplicable.
+    if targeted:
+        schema_dict |= FIELD_FILTER_SCHEMA_DICT
+
+    if custom:
+        schema_dict |= CUSTOM_INTEGRATION_EXTRA_SCHEMA_DICT
+
+    return vol.Schema(schema_dict)
+
+
+def _section_schema(targeted: bool, custom: bool) -> vol.Schema:
+    """Return the section schema."""
+    schema_dict = {
         vol.Optional("collapsed"): bool,
-        vol.Required("fields"): vol.Schema({str: CORE_INTEGRATION_FIELD_SCHEMA}),
+        vol.Required("fields"): vol.Schema(
-    }
-)
-
-CUSTOM_INTEGRATION_FIELD_SCHEMA = CORE_INTEGRATION_FIELD_SCHEMA.extend(
             {
-        vol.Optional("description"): str,
+                str: _field_schema(targeted, custom),
-        vol.Optional("name"): str,
             }
-)
-
-CUSTOM_INTEGRATION_SECTION_SCHEMA = vol.Schema(
-    {
-        vol.Optional("description"): str,
-        vol.Optional("name"): str,
-        vol.Optional("collapsed"): bool,
-        vol.Required("fields"): vol.Schema({str: CUSTOM_INTEGRATION_FIELD_SCHEMA}),
-    }
-)
-
-
-CORE_INTEGRATION_SERVICE_SCHEMA = vol.Any(
-    vol.Schema(
-        {
-            vol.Optional("target"): vol.Any(
-                selector.TargetSelector.CONFIG_SCHEMA, None
         ),
+    }
+
+    if custom:
+        schema_dict |= CUSTOM_INTEGRATION_EXTRA_SCHEMA_DICT
+
+    return vol.Schema(schema_dict)
+
+
+def _service_schema(targeted: bool, custom: bool) -> vol.Schema:
+    """Return the service schema."""
+    schema_dict = {
         vol.Optional("fields"): vol.All(
             vol.Schema(
                 {
                     str: vol.Any(
-                            CORE_INTEGRATION_FIELD_SCHEMA,
+                        _field_schema(targeted, custom),
-                            CORE_INTEGRATION_SECTION_SCHEMA,
+                        _section_schema(targeted, custom),
-                        )
+                    ),
                 }
             ),
             unique_field_validator,
-            ),
+        )
     }
-    ),
+
+    if targeted:
+        schema_dict[vol.Required("target")] = vol.Any(
+            selector.TargetSelector.CONFIG_SCHEMA, None
+        )
+
+    if custom:
+        schema_dict |= CUSTOM_INTEGRATION_EXTRA_SCHEMA_DICT
+
+    return vol.Schema(schema_dict)
+
+
+CORE_INTEGRATION_SERVICE_SCHEMA = vol.Any(
+    _service_schema(targeted=True, custom=False),
+    _service_schema(targeted=False, custom=False),
     None,
 )
 
 CUSTOM_INTEGRATION_SERVICE_SCHEMA = vol.Any(
-    vol.Schema(
+    _service_schema(targeted=True, custom=True),
-        {
+    _service_schema(targeted=False, custom=True),
-            vol.Optional("description"): str,
-            vol.Optional("name"): str,
-            vol.Optional("target"): vol.Any(
-                selector.TargetSelector.CONFIG_SCHEMA, None
-            ),
-            vol.Optional("fields"): vol.All(
-                vol.Schema(
-                    {
-                        str: vol.Any(
-                            CUSTOM_INTEGRATION_FIELD_SCHEMA,
-                            CUSTOM_INTEGRATION_SECTION_SCHEMA,
-                        )
-                    }
-                ),
-                unique_field_validator,
-            ),
-        }
-    ),
     None,
 )
 
+
 CORE_INTEGRATION_SERVICES_SCHEMA = vol.Schema(
     {
         vol.Remove(vol.All(str, service.starts_with_dot)): object,
         cv.slug: CORE_INTEGRATION_SERVICE_SCHEMA,
     }
 )
+
 CUSTOM_INTEGRATION_SERVICES_SCHEMA = vol.Schema(
     {cv.slug: CUSTOM_INTEGRATION_SERVICE_SCHEMA}
 )
 
+
 VALIDATE_AS_CUSTOM_INTEGRATION = {
     # Adding translations would be a breaking change
     "foursquare",

tests/components/group/test_notify.py

@@ -199,7 +199,8 @@ async def test_send_message_with_data(hass: HomeAssistant, tmp_path: Path) -> No
                     },
                 },
             ),
-        ]
+        ],
+        any_order=True,
     )