Skip to content

Add integration tests for coordinator-mode TypeScript tasks#69328

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

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 from 13317a1 to 79ce3ab Compare July 3, 2026 13:39
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