docs(config): unify config.get reads and remove ALLOW_* gates

tests/test_adapter_profile_policy.py

@@ -5,6 +5,8 @@ from __future__ import annotations
 import os
 import unittest
 
+from jiangchang_skill_core import config
+
 from adapter_test_utils import (
     ALLOWED_TEST_TARGETS,
     FakeAdapter,
@@ -24,34 +26,22 @@ def _restore_env(saved: dict[str, str | None]) -> None:
             os.environ.pop(k, None)
         else:
             os.environ[k] = v
+    config.reset_cache()
 
 
-_ENV_KEYS = (
-    "OPENCLAW_TEST_TARGET",
-    "OPENCLOW_TEST_TARGET",
-    "ALLOW_REAL_API",
-    "ALLOW_REAL_RPA",
-    "ALLOW_WRITE_ACTIONS",
-)
+_ENV_KEYS = ("OPENCLAW_TEST_TARGET",)
 
 
 class TestAdapterProfilePolicy(unittest.TestCase):
-    def test_get_test_target_reads_opencylow_typo_alias(self) -> None:
-        """兼容环境变量历史拼写 OPENCLOW_TEST_TARGET。"""
-        saved = _save_env(_ENV_KEYS)
-        try:
-            os.environ.pop("OPENCLAW_TEST_TARGET", None)
-            os.environ.pop("OPENCLOW_TEST_TARGET", None)
-            os.environ["OPENCLOW_TEST_TARGET"] = "mock"
-            self.assertEqual(get_test_target(), "mock")
-        finally:
-            _restore_env(saved)
+    def tearDown(self) -> None:
+        config.reset_cache()
 
     def test_get_test_target_defaults_to_unit(self) -> None:
         saved = _save_env(_ENV_KEYS)
         try:
             for k in _ENV_KEYS:
                 os.environ.pop(k, None)
+            config.reset_cache()
             self.assertEqual(get_test_target(), "unit")
         finally:
             _restore_env(saved)
@@ -62,6 +52,7 @@ class TestAdapterProfilePolicy(unittest.TestCase):
             for k in _ENV_KEYS:
                 os.environ.pop(k, None)
             os.environ["OPENCLAW_TEST_TARGET"] = "not_a_valid_target"
+            config.reset_cache()
             with self.assertRaises(ValueError):
                 get_test_target()
         finally:
@@ -74,6 +65,7 @@ class TestAdapterProfilePolicy(unittest.TestCase):
         saved = _save_env(_ENV_KEYS)
         try:
             os.environ["OPENCLAW_TEST_TARGET"] = "simulator_api"
+            config.reset_cache()
             skip, _ = should_skip_profile("simulator_api")
             self.assertFalse(skip)
         finally:
@@ -84,6 +76,7 @@ class TestAdapterProfilePolicy(unittest.TestCase):
         try:
             for k in _ENV_KEYS:
                 os.environ.pop(k, None)
+            config.reset_cache()
             self.assertEqual(get_test_target(), "unit")
             for prof in ("simulator_api", "simulator_rpa", "real_api", "real_rpa"):
                 skip, reason = should_skip_profile(prof)
@@ -93,46 +86,23 @@ class TestAdapterProfilePolicy(unittest.TestCase):
         finally:
             _restore_env(saved)
 
-    def test_real_api_requires_allow_flag(self) -> None:
+    def test_real_api_runs_when_target_matches(self) -> None:
         saved = _save_env(_ENV_KEYS)
         try:
             os.environ["OPENCLAW_TEST_TARGET"] = "real_api"
-            os.environ.pop("ALLOW_REAL_API", None)
-            skip, reason = should_skip_profile("real_api")
-            self.assertTrue(skip)
-            self.assertIn("ALLOW_REAL_API", reason)
-            os.environ["ALLOW_REAL_API"] = "1"
-            skip2, _ = should_skip_profile("real_api", write=False)
-            self.assertFalse(skip2)
+            config.reset_cache()
+            skip, _ = should_skip_profile("real_api", write=True)
+            self.assertFalse(skip)
         finally:
             _restore_env(saved)
 
-    def test_real_api_write_requires_allow_write_actions(self) -> None:
-        saved = _save_env(_ENV_KEYS)
-        try:
-            os.environ["OPENCLAW_TEST_TARGET"] = "real_api"
-            os.environ["ALLOW_REAL_API"] = "1"
-            os.environ.pop("ALLOW_WRITE_ACTIONS", None)
-            skip, reason = should_skip_profile("real_api", write=True)
-            self.assertTrue(skip)
-            self.assertIn("ALLOW_WRITE_ACTIONS", reason)
-            os.environ["ALLOW_WRITE_ACTIONS"] = "1"
-            skip2, _ = should_skip_profile("real_api", write=True)
-            self.assertFalse(skip2)
-        finally:
-            _restore_env(saved)
-
-    def test_real_rpa_requires_allow_rpa_flag(self) -> None:
+    def test_real_rpa_runs_when_target_matches(self) -> None:
         saved = _save_env(_ENV_KEYS)
         try:
             os.environ["OPENCLAW_TEST_TARGET"] = "real_rpa"
-            os.environ.pop("ALLOW_REAL_RPA", None)
-            skip, reason = should_skip_profile("real_rpa")
-            self.assertTrue(skip)
-            self.assertIn("ALLOW_REAL_RPA", reason)
-            os.environ["ALLOW_REAL_RPA"] = "1"
-            skip2, _ = should_skip_profile("real_rpa", write=False)
-            self.assertFalse(skip2)
+            config.reset_cache()
+            skip, _ = should_skip_profile("real_rpa", write=True)
+            self.assertFalse(skip)
         finally:
             _restore_env(saved)
 
@@ -146,10 +116,10 @@ class TestAdapterProfilePolicy(unittest.TestCase):
             FakeAdapter("unauthorized").call({})
 
     def test_assert_no_real_network_env(self) -> None:
-        saved = _save_env(("ALLOW_REAL_API", "ALLOW_REAL_RPA"))
+        saved = _save_env(_ENV_KEYS)
         try:
-            os.environ.pop("ALLOW_REAL_API", None)
-            os.environ.pop("ALLOW_REAL_RPA", None)
+            os.environ.pop("OPENCLAW_TEST_TARGET", None)
+            config.reset_cache()
             assert_no_real_network_env()
         finally:
             _restore_env(saved)