Skip to content

test(e2e): Add eve test for Jev evaluate calls - #25034

Merged
andreiborza merged 2 commits into
developfrom
ab/eve-jev-test
Oct 5, 2026
Merged

andreiborza merged 2 commits into
developfrom
ab/eve-jev-test

Conversation

@andreiborza

Copy link
Copy Markdown
Member

What

Adds an e2e test to the eve app. The test makes an agent tool call a TypeSafe Jev evaluation.

Why

Make sure a Jev evaluation in an eve tool records a gen_ai.evaluate span under the tool span.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 1099aca. Configure here.

Comment thread dev-packages/e2e-tests/test-applications/node-eve/tests/eve.test.ts
@andreiborza
andreiborza marked this pull request as ready for review October 5, 2026 10:12

@nicohrubec nicohrubec left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nice

@andreiborza
andreiborza merged commit 2998151 into develop Oct 5, 2026
687 of 695 checks passed
@andreiborza
andreiborza deleted the ab/eve-jev-test branch October 5, 2026 10:26
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.

2 participants