test: remove BATCH_CLIENT_PARAMS xfail, now in core

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: Nissan Pow

test/unit/localbatch/test_localbatch.py

@@ -333,13 +333,6 @@ def test_inject_env_is_visible_inside_container(self):
 # ---------------------------------------------------------------------------
 
 
-_NEEDS_CORE_BATCH_PARAMS = pytest.mark.xfail(
-    reason="requires npow/core-deployer-changes: BATCH_CLIENT_PARAMS must be added "
-    "to metaflow_config.py so METAFLOW_BATCH_CLIENT_PARAMS env var is recognized",
-    strict=False,
-)
-
-
 @pytest.mark.docker
 class TestMetaflowE2E:
     """
@@ -357,7 +350,6 @@ def _require_docker(self):
         except Exception:
             pytest.skip("Docker not available")
 
-    @_NEEDS_CORE_BATCH_PARAMS
     def test_batch_step_artifacts_are_persisted(self, simple_batch_run):
         """
         The @batch step writes message='hello from localbatch' and value=42.
@@ -367,11 +359,9 @@ def test_batch_step_artifacts_are_persisted(self, simple_batch_run):
         assert task["message"].data == "hello from localbatch"
         assert task["value"].data == 42
 
-    @_NEEDS_CORE_BATCH_PARAMS
-    def test_run_succeeds(self, simple_batch_run):
+def test_run_succeeds(self, simple_batch_run):
         assert simple_batch_run.successful
 
-    @_NEEDS_CORE_BATCH_PARAMS
-    def test_all_steps_have_tasks(self, simple_batch_run):
+def test_all_steps_have_tasks(self, simple_batch_run):
         step_names = {s.id for s in simple_batch_run.steps()}
         assert {"start", "end"} <= step_names