-
Notifications
You must be signed in to change notification settings - Fork 945
test(ai): add integration tests #8853
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
Conversation
|
Size Report 1Affected Products
Test Logs |
packages/vertexai/package.json
Outdated
@@ -39,6 +39,7 @@ | |||
"test:ci": "yarn testsetup && node ../../scripts/run_tests_in_ci.js -s test", | |||
"test:skip-clone": "karma start", | |||
"test:browser": "yarn testsetup && karma start", | |||
"test:integration": "karma start --integration", |
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.
You'll have to call this from "test" (line 38) if you want to run it on PRs or do you not want to do that yet?
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.
I would prefer not to add this to test
since then it'll be ran when someone runs test
in the root directory- which is going to fail if someone hasn't defined the config that's used in these tests. Could be annoying for people working on other SDKs
Size Analysis Report 1Affected Products
Test Logs |
346de4e
to
7ad7833
Compare
Vertex AI Mock Responses Check
|
…irebase-js-sdk into dl/vertex-integration
* Version Packages (#9016) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * Revert "Fixed scroll behavior (#8977)" (#9053) * Revert "Fixed scroll behavior (#9043)" This reverts commit 9bcd1ea. * Revert "Add Emulator Overlay (#8977) (#9031)" This reverts commit 8593fa0. * Undid revert of util * Create dirty-crews-cross.md * Removed unnecessary markdown file * Updated to only show banner when calling connect*Emulator (#9055) * Version Packages (#9054) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * Close webchannel instances on terminate (#9041) Explicitly close any unclosed WebChannel instances on Firestore#terminate() * Reorder reporters to prevent log message duplication in CI (#9040) * docs(ai): Fix 'occured' typo in error codes (#9021) * feat(ai): Add `HarmBlockThreshold.OFF` (#9050) * feat(ai): Schema `title`, `maximum`, `minimum`, and `propertyOrdering` (#9047) * feat(ai): Add support for `minItems` and `maxItems` to `Schema` (#9026) * Build out e2e subdirectories (#9065) * Added test project to fdc (#8924) * Version Packages (#9073) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Release version 11.9.0. * test(ai): add integration tests (#8853) * Update issue template to rename VertexAI > AI (#9074) * ci: Fix Firefox error and streamline cross-browser tests (#9015) * Fixed issue where requestSts wasn't including the Studio cookie in it (#9075) * Fix issue where we ping an endpoint that hasn't been implemented (#9059) * Updated firebase auth changeset (#9088) * build(all): Version Packages v11.9.1 (#9089) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> v11.9.1 release. * test(ai): Upgrade mock responses to v14 (#9091) --------- Co-authored-by: Google Open Source Bot <[email protected]> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Maneesh Tewani <[email protected]> Co-authored-by: Mark Duckworth <[email protected]> Co-authored-by: Daniel La Rocque <[email protected]> Co-authored-by: Christina Holland <[email protected]>
Adds initial setup for integration tests, and integration tests for count tokens, generate content, and chat. Each test runs against both backends (Vertex AI and Google AI), for each specified model in the config.
These tests aim to catch issues that are caused by changes in the backend (e.g. A required field suddenly no longer being sent), and to allow us to confidently support new models by just adding them to the set of models we test against.
Future PRs will integrate this into our CI.
Sample output: