fix(tests): adopt master beta gate + new advanced handler keys (homeassistant-ai#1164)

Three failures surfaced after rebasing onto upstream/master:

1. ``test_returns_all_handler_keys`` expected the pre-homeassistant-ai#1164 handler
   set. Add ``get_advanced_settings`` / ``save_advanced_settings``
   to the expected keys.

2. ``test_tools_filesystem.TestFeatureFlag::test_enabled_with_*``
   broke because the master beta gate now forces every beta sub-flag
   False at runtime when ``ENABLE_BETA_FEATURES`` is unset. Set both
   env vars together in the enabling tests so they exercise the
   sub-flag bool parsing in isolation, and add an explicit test for
   the gated behavior so a future regression in
   ``_apply_feature_flag_overrides`` surfaces here too.

3. ``test_yaml_config_tool.enable_flag`` fixture sets
   ``ENABLE_YAML_CONFIG_EDITING`` but didn't set the master, so the
   cached settings landed with the sub-flag forced False — would have
   broken next-up after the filesystem tests. Set
   ``ENABLE_BETA_FEATURES`` alongside.

Author: kingpanther13

tests/src/unit/test_stdio_settings_sidecar.py

@@ -98,6 +98,9 @@ def test_returns_all_handler_keys(self) -> None:
             "policy_post_deny",
             "policy_get_tool_schema",
             "policy_get_value_source",
+            # Advanced settings handlers (#1164).
+            "get_advanced_settings",
+            "save_advanced_settings",
         }
 
     def test_get_tools_reads_cache_when_server_is_none(

tests/src/unit/test_tools_filesystem.py

@@ -45,7 +45,13 @@ def test_filesystem_constants_include_dashboards():
 
 
 class TestFeatureFlag:
-    """Test feature flag functionality."""
+    """Test feature flag functionality.
+
+    ``HAMCP_ENABLE_FILESYSTEM_TOOLS`` is a beta sub-flag (#1164). The
+    master ``ENABLE_BETA_FEATURES`` gate force-sets it False at runtime
+    when the master is off, so every enabling test must set both env
+    vars to exercise the sub-flag's bool parsing in isolation.
+    """
 
     def test_disabled_by_default(self):
         """Feature should be disabled by default."""
@@ -56,22 +62,30 @@ def test_disabled_by_default(self):
 
     def test_enabled_with_true(self):
         """Feature should be enabled when set to 'true'."""
-        with patch.dict(os.environ, {FEATURE_FLAG: "true"}):
+        with patch.dict(
+            os.environ, {FEATURE_FLAG: "true", "ENABLE_BETA_FEATURES": "true"}
+        ):
             assert is_filesystem_tools_enabled() is True
 
     def test_enabled_with_1(self):
         """Feature should be enabled when set to '1'."""
-        with patch.dict(os.environ, {FEATURE_FLAG: "1"}):
+        with patch.dict(
+            os.environ, {FEATURE_FLAG: "1", "ENABLE_BETA_FEATURES": "true"}
+        ):
             assert is_filesystem_tools_enabled() is True
 
     def test_enabled_with_yes(self):
         """Feature should be enabled when set to 'yes'."""
-        with patch.dict(os.environ, {FEATURE_FLAG: "yes"}):
+        with patch.dict(
+            os.environ, {FEATURE_FLAG: "yes", "ENABLE_BETA_FEATURES": "true"}
+        ):
             assert is_filesystem_tools_enabled() is True
 
     def test_enabled_with_on(self):
         """Feature should be enabled when set to 'on'."""
-        with patch.dict(os.environ, {FEATURE_FLAG: "on"}):
+        with patch.dict(
+            os.environ, {FEATURE_FLAG: "on", "ENABLE_BETA_FEATURES": "true"}
+        ):
             assert is_filesystem_tools_enabled() is True
 
     def test_disabled_with_false(self):
@@ -86,11 +100,25 @@ def test_disabled_with_empty_string(self):
 
     def test_case_insensitive(self):
         """Feature flag should be case insensitive."""
-        with patch.dict(os.environ, {FEATURE_FLAG: "TRUE"}):
+        with patch.dict(
+            os.environ, {FEATURE_FLAG: "TRUE", "ENABLE_BETA_FEATURES": "true"}
+        ):
             assert is_filesystem_tools_enabled() is True
-        with patch.dict(os.environ, {FEATURE_FLAG: "True"}):
+        with patch.dict(
+            os.environ, {FEATURE_FLAG: "True", "ENABLE_BETA_FEATURES": "true"}
+        ):
             assert is_filesystem_tools_enabled() is True
 
+    def test_master_off_forces_sub_flag_off(self):
+        """Master beta gate (#1164) forces this sub-flag False even when
+        the sub-flag env var is true. Lock the behavior so a future
+        regression in ``_apply_feature_flag_overrides`` would surface
+        in the filesystem-tools tests, not just the config tests.
+        """
+        with patch.dict(os.environ, {FEATURE_FLAG: "true"}):
+            os.environ.pop("ENABLE_BETA_FEATURES", None)
+            assert is_filesystem_tools_enabled() is False
+
 
 class TestIsMcpToolsAvailable:
     """Test _is_mcp_tools_available function."""

tests/src/unit/test_yaml_config_tool.py

@@ -13,10 +13,17 @@ def enable_flag(monkeypatch):
     If anything in the test process imported the module before this fixture ran,
     the cached settings have ENABLE_YAML_CONFIG_EDITING=False and our env var is
     ignored. Reset the cache before AND after to keep tests hermetic.
+
+    The master beta gate (#1164) also force-sets every beta sub-flag
+    False at runtime when ``ENABLE_BETA_FEATURES`` is unset, so set
+    both env vars together — otherwise the cached settings would land
+    with ``enable_yaml_config_editing=False`` regardless of the
+    sub-flag env var.
     """
     from ha_mcp import config as ha_mcp_config
 
     monkeypatch.setenv("ENABLE_YAML_CONFIG_EDITING", "true")
+    monkeypatch.setenv("ENABLE_BETA_FEATURES", "true")
     monkeypatch.setattr(ha_mcp_config, "_settings", None)
     yield
     # Reset the cache so other tests don't see our enabled flag.