Skip to content

chore: [Orchestration] Add E2E test for grounding via Sharepoint #458

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
Jun 4, 2025

Conversation

Jonas-Isr
Copy link
Member

@Jonas-Isr Jonas-Isr commented Jun 3, 2025

Context

AI/ai-sdk-java-backlog#249

This PR adds an e2e test and an endpoint to the sample app showing how to use grounding via Sharepoint. The sharepoint used for this is the one set up by the JS team.

Feature scope:

  • E2E test added
  • Endpoint to sample app added

Definition of Done

@Jonas-Isr Jonas-Isr self-assigned this Jun 3, 2025
@Jonas-Isr Jonas-Isr added the please-review Request to review a pull-request label Jun 3, 2025
@Jonas-Isr Jonas-Isr changed the title chore: [Grounding] Add E2E test for grounding via Sharepoint chore: [Orchestration] Add E2E test for grounding via Sharepoint Jun 4, 2025
@Jonas-Isr Jonas-Isr enabled auto-merge (squash) June 4, 2025 08:14
/**
* Using grounding via a sharepoint repository to provide additional context to the AI model.
*
* @link <a href="https://help.sap.com/docs/sap-ai-core/sap-ai-core-service-guide/grounding">SAP
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(Comment)

Not a fan of putting volatile URLs in JavaDoc, but since this is a test app and already somewhat-established practice in this class - i won't complain further ;)

@Jonas-Isr Jonas-Isr merged commit ffbbe41 into main Jun 4, 2025
7 of 8 checks passed
@Jonas-Isr Jonas-Isr deleted the grounding-test-sharepoint branch June 4, 2025 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
please-review Request to review a pull-request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants