Skip to content

[ZEPPELIN-6760] Add an end-to-end test for local assistant tool testing - #5593

Open
seung-00 wants to merge 1 commit into
apache:masterfrom
seung-00:feature/ZEPPELIN-6760
Open

seung-00 wants to merge 1 commit into
apache:masterfrom
seung-00:feature/ZEPPELIN-6760

Conversation

@seung-00

@seung-00 seung-00 commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

What is this PR for?

Add a local E2E test using a real model to verify tool selection, chained tool calls, and the final response.
The test is skipped by default and explicitly enabled with an API key.

What type of PR is it?

Improvement

Todos

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-6760

How should this be tested?

Run locally with an API key:

./mvnw test -pl zeppelin-server -Dtest=AssistantE2ETest \
  -Dassistant.e2e.apiKey=<API_KEY>

Verify both tools are called and the response contains the expected output. Without the API key, verify the test is skipped.

Screenshots (if appropriate)

Questions:

  • Does the license files need to update? N
  • Is there breaking changes for older versions? N
  • Does this needs documentation? N

@seung-00
seung-00 force-pushed the feature/ZEPPELIN-6760 branch from ba9bcef to bb6c433 Compare October 27, 2026 04:02

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant