Skip to content

Add integration tests for coordinator-mode TypeScript tasks#69328

Open
guan404ming wants to merge 2 commits into
apache:mainfrom
guan404ming:ts-coordinator-integration-tests
Open

Add integration tests for coordinator-mode TypeScript tasks#69328
guan404ming wants to merge 2 commits into
apache:mainfrom
guan404ming:ts-coordinator-integration-tests

Conversation

@guan404ming

@guan404ming guan404ming commented Jul 3, 2026

Copy link
Copy Markdown
Member

Related Issue

closes #69290

Why

Coordinator-mode TypeScript tasks had no Airflow-level end-to-end coverage and ts-sdk changes triggered no integration tests in CI

How

  • Add a ts_sdk e2e test mode mirroring the Go SDK's: build the ts-sdk example bundle with pnpm in an ephemeral Node container, deploy it with its stub Dag, and assert Dag success, Python <-> TS XCom round-trips, Connection/Variable reads, and coordinator logs reaching the task-log store
  • Provide the worker's Node.js runtime via a node-provider compose service in docker/ts.yml
  • Wire CI through a new TS_SDK_E2E_FILES selective-checks group -> run-ts-sdk-e2e-tests -> a dedicated e2e job, with doc and test updates

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)
    Claude Code with Fable 5

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

@guan404ming guan404ming force-pushed the ts-coordinator-integration-tests branch 2 times, most recently from 9a48f05 to 697c34c Compare July 4, 2026 08:30
@guan404ming guan404ming marked this pull request as ready for review July 6, 2026 07:08
@guan404ming guan404ming force-pushed the ts-coordinator-integration-tests branch from 697c34c to 26ad7c1 Compare July 9, 2026 04:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TypeScript SDK: Airflow-level integration tests for coordinator-mode tasks

1 participant