Skip to content

Commit a368b2b

Browse files
Nissan Powclaude
andcommitted
test: remove xfails now that deployer code is included
Remove xfail markers for sfn-batch (ResultSelector/sfn-local compat) and airflow-kubernetes (AirflowDeployer) since the deployer implementation is now in this PR. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent d1fb3a2 commit a368b2b

2 files changed

Lines changed: 0 additions & 13 deletions

File tree

test/ux/core/test_sfn_compilation.py

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -201,11 +201,6 @@ def test_linear_flow(self):
201201
result.get("result", "OK") == "OK"
202202
), f"Validation failed: {result.get('diagnostics', result)}"
203203

204-
@pytest.mark.xfail(
205-
reason="requires npow/core-deployer-changes: step_functions.py must add "
206-
"ResultSelector to Parallel states for sfn-local compatibility",
207-
strict=False,
208-
)
209204
def test_branch_flow(self):
210205
"""Parallel branch flow produces valid Parallel states with ResultSelector."""
211206
definition = _compile_flow_to_json("dag/branch_flow.py")

test/ux/ux_test_config.yaml

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -35,19 +35,11 @@ backends:
3535
cluster: null
3636
decospec: "batch:image=python:3.9"
3737
enabled: true
38-
# xfail: requires core changes from npow/core-deployer-changes (step_functions.py
39-
# sfn-local compatibility fixes: ResultSelector on Parallel states, no ProcessorConfig
40-
# on Map states). Remove once that PR is merged into master.
41-
xfail_reason: "requires npow/core-deployer-changes (sfn-local compatibility fixes in step_functions.py)"
4238

4339
# Apache Airflow + Kubernetes (devstack: minikube + airflow helm chart)
4440
- name: airflow-kubernetes
4541
scheduler_type: airflow
4642
cluster: default
4743
decospec: "kubernetes:image=python:3.9"
4844
enabled: true
49-
# xfail: requires AirflowDeployer from npow/core-deployer-changes (__init__.py
50-
# registration + airflow_deployer.py + airflow_client.py).
51-
# Remove once that PR is merged into master.
52-
xfail_reason: "requires npow/core-deployer-changes (AirflowDeployer not yet in core)"
5345

0 commit comments

Comments
 (0)