-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Add characterization tests for basic langchain text responses and stream data. #676
Conversation
const response = new StreamingTextResponse(stream); | ||
|
||
expect(await readAllChunks(response)).toEqual([ | ||
'', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With Langchain, there is an empty text chunk at the beginning and the end. Is this desired?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, but it shouldn't be a problem and not worth investigating until we finalize the complex mode API imo
The error is caused by the following chain of events:
I’ve tried setting up Jest transformIgnorePatterns, but it does not help. |
Tests are not run currently. |
Overview
x-flush-delay
header in mock server to speed up tests (default to 5ms to speed up existing tests)Note
The langchain stream test does not work in the edge environment and is deactivated in that environment. The errror is caused by the following chain of events:
Unexpected token 'export'
with Jest (on Cloudflare Workers) uuidjs/uuid#678 (that’s open for almost a year)Details: