Skip to content

[e2e][java] Validate ReActAgent output without schema#927

Merged
wenjin272 merged 1 commit into
apache:mainfrom
wenjin272:codex/validate-react-no-schema-output
Jul 25, 2026
Merged

[e2e][java] Validate ReActAgent output without schema#927
wenjin272 merged 1 commit into
apache:mainfrom
wenjin272:codex/validate-react-no-schema-output

Conversation

@wenjin272

Copy link
Copy Markdown
Contributor

Related to #837 and the review discussion in #838.

Purpose of change

Strengthen the Java ReActAgent regression test for the no-output-schema path.

The test previously only printed the output and therefore passed as long as the
job executed without throwing. This change collects the result and verifies
that the agent emits an output and that the output is a String, matching the
no-schema contract and the existing Python coverage.

Tests

  • mvn --batch-mode --no-transfer-progress spotless:check -pl e2e-test/flink-agents-end-to-end-tests-integration
  • mvn --batch-mode --no-transfer-progress test-compile -pl e2e-test/flink-agents-end-to-end-tests-integration -am -DskipTests
  • ./tools/ut.sh --java --e2e --flink 2.3 (29 tests, 0 failures, 0 errors, 8 skipped)

API

No public API changes.

Documentation

  • doc-not-needed

@github-actions github-actions Bot added doc-not-needed Your PR changes do not impact docs fixVersion/0.4.0 priority/major Default priority of the PR or issue. labels Jul 25, 2026
@wenjin272
wenjin272 marked this pull request as ready for review July 25, 2026 04:26
@wenjin272
wenjin272 merged commit 0ae6e7d into apache:main Jul 25, 2026
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs fixVersion/0.4.0 priority/major Default priority of the PR or issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant