Skip to content

chore: update ci. remove connectors & summarize tests - #139

Open
daniel-cohere wants to merge 2 commits into
mainfrom
daniel/remove-connectors-summarize-from-ci
Open

chore: update ci. remove connectors & summarize tests#139
daniel-cohere wants to merge 2 commits into
mainfrom
daniel/remove-connectors-summarize-from-ci

Conversation

@daniel-cohere

@daniel-cohere daniel-cohere commented Nov 4, 2025

Copy link
Copy Markdown

This PR updates the TestNewClient function in the sdk_test.go file, replacing the base model with the command-a-03-2025 model in the TokenizeRequest and adding the same model to the DetokenizeRequest. It also removes the TestSummarize and TestConnectorCRUD test cases.

  • The Model field in the TokenizeRequest is updated from "base" to "command-a-03-2025".
  • A new Model field is added to the DetokenizeRequest with the value "command-a-03-2025".
  • The TestSummarize test case is removed.
  • The TestConnectorCRUD test case is removed.

Note

Low Risk
Test-only changes that remove coverage for two endpoints and update a model string; no production logic is affected.

Overview
Updates the SDK integration tests by switching the tokenize/detokenize coverage to use command-a-03-2025 and explicitly passing the model for detokenization.

Removes the TestSummarize and TestConnectorCRUD subtests from tests/sdk_test.go, reducing CI reliance on endpoints that are flaky or hard to exercise in test runs (e.g., OAuth/connector lifecycle).

Written by Cursor Bugbot for commit c0d7372. This will update automatically on new commits. Configure here.

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