fix: sync test-harness files to source tree for dev and CI testing

The hatch build hook includes files in sdist/wheel via force_include,
but tests run from the source tree. Add a sync script that copies
files from the test-harness submodule, called by poe before test/cov.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Simon Schrottner <simon.schrottner@gmail.com>

Author: aepfli

providers/openfeature-provider-flagd/src/openfeature/contrib/provider/flagd/resolvers/process/flags.py

@@ -25,13 +25,12 @@ def __init__(
         super().__init__()
         self._emit_provider_configuration_changed = emit_provider_configuration_changed
 
-    def update(self, flags_data: dict) -> list[str]:
+    def update(self, flags_data: dict) -> None:
         changed_keys = super().update(flags_data)
         if self._emit_provider_configuration_changed is not None:
             self._emit_provider_configuration_changed(
                 ProviderEventDetails(
-                    flags_changed=list(self.flags.keys()),
+                    flags_changed=changed_keys,
                     metadata=dict(self.flag_set_metadata),
                 )
             )
-        return changed_keys

tools/openfeature-flagd-api-testkit/hatch_build.py

@@ -1,40 +1,39 @@
 """Hatch build hook to copy test-harness evaluator files into the package."""
 
+import shutil
 from pathlib import Path
 
 from hatchling.builders.hooks.plugin.interface import BuildHookInterface
 
-# Relative to the testkit package root
-TEST_HARNESS_EVALUATOR = Path(
-    "../../providers/openfeature-provider-flagd/openfeature/test-harness/evaluator"
-)
-DEST_BASE = "src/openfeature/contrib/tools/flagd/testkit"
+DEST_BASE = Path("src/openfeature/contrib/tools/flagd/testkit")
 
 
 class TestHarnessCopyHook(BuildHookInterface):
     PLUGIN_NAME = "test-harness-copy"
 
     def initialize(self, version: str, build_data: dict) -> None:
         root = Path(self.root)
-        src = root / TEST_HARNESS_EVALUATOR
-
-        # When building wheel from sdist, the submodule isn't available
-        # but the files were already force-included in the sdist.
-        if not src.exists():
-            return
-
-        force = build_data.setdefault("force_include", {})
-
-        # Map submodule gherkin files -> package features/
-        gherkin_src = src / "gherkin"
-        for path in gherkin_src.rglob("*"):
-            if path.is_file():
-                rel = path.relative_to(gherkin_src)
-                force[str(path)] = f"{DEST_BASE}/features/{rel}"
-
-        # Map submodule flag files -> package flag_data/
-        flags_src = src / "flags"
-        for path in flags_src.rglob("*"):
-            if path.is_file():
-                rel = path.relative_to(flags_src)
-                force[str(path)] = f"{DEST_BASE}/flag_data/{rel}"
+        features_dest = root / DEST_BASE / "features"
+        flags_dest = root / DEST_BASE / "flag_data"
+
+        # Copy from submodule if files don't exist yet
+        if not features_dest.exists() or not flags_dest.exists():
+            src = (
+                root
+                / "../../providers/openfeature-provider-flagd"
+                / "openfeature/test-harness/evaluator"
+            )
+            if src.exists():
+                if not features_dest.exists():
+                    shutil.copytree(src / "gherkin", features_dest)
+                if not flags_dest.exists():
+                    shutil.copytree(src / "flags", flags_dest)
+
+        # Force-include gitignored files into both sdist and wheel
+        if features_dest.exists() and flags_dest.exists():
+            force = build_data.setdefault("force_include", {})
+            for dest_dir in (features_dest, flags_dest):
+                for path in dest_dir.rglob("*"):
+                    if path.is_file():
+                        rel = str(path.relative_to(root))
+                        force[rel] = rel

tools/openfeature-flagd-api-testkit/hatch_build_sync.py

@@ -0,0 +1,34 @@
+"""Copy test-harness evaluator files into the package source tree.
+
+Used by `poe sync-test-harness` for local development and CI testing.
+The hatch build hook (hatch_build.py) handles inclusion in wheel/sdist.
+"""
+
+import shutil
+from pathlib import Path
+
+ROOT = Path(__file__).parent
+TEST_HARNESS_EVALUATOR = (
+    ROOT / "../../providers/openfeature-provider-flagd/openfeature/test-harness/evaluator"
+).resolve()
+DEST_BASE = ROOT / "src/openfeature/contrib/tools/flagd/testkit"
+
+
+def sync() -> None:
+    if not TEST_HARNESS_EVALUATOR.exists():
+        msg = (
+            f"Test harness evaluator not found at {TEST_HARNESS_EVALUATOR}. "
+            "Make sure submodules are initialized."
+        )
+        raise FileNotFoundError(msg)
+
+    for src_name, dest_name in [("gherkin", "features"), ("flags", "flag_data")]:
+        src = TEST_HARNESS_EVALUATOR / src_name
+        dest = DEST_BASE / dest_name
+        if dest.exists():
+            shutil.rmtree(dest)
+        shutil.copytree(src, dest)
+
+
+if __name__ == "__main__":
+    sync()

tools/openfeature-flagd-api-testkit/pyproject.toml

@@ -68,8 +68,9 @@ disallow_any_generics = false
 omit = ["tests/**"]
 
 [tool.poe.tasks]
-test = "pytest tests"
-test-cov = "coverage run -m pytest tests"
+sync-test-harness = "python hatch_build_sync.py"
+test = ["sync-test-harness", {cmd = "pytest tests"}]
+test-cov = ["sync-test-harness", {cmd = "coverage run -m pytest tests"}]
 cov-report = "coverage xml"
 cov = [
     "test-cov",