Skip to content
This repository was archived by the owner on May 15, 2026. It is now read-only.

chore: remove integration tests - #11598

Merged
hannesrudolph merged 2 commits into
mainfrom
chore/remove-integration-tests
Feb 19, 2026
Merged

chore: remove integration tests#11598
hannesrudolph merged 2 commits into
mainfrom
chore/remove-integration-tests

Conversation

@ghost

@ghostghost commented Feb 19, 2026

Copy link
Copy Markdown

Remove all integration tests from the repository.

Changes

Integration test files deleted

  • packages/cloud/src/__tests__/CloudService.integration.test.ts
  • src/core/tools/__tests__/executeCommandTimeout.integration.spec.ts
  • src/core/webview/__tests__/webviewMessageHandler.imageMentions.integration.spec.ts

Workflow change needed (manual step)

The GitHub App token lacks workflows permission, so the following change to .github/workflows/code-qa.yml must be applied manually:

Remove the check-openrouter-api-key job (lines 62-75) and the integration-test job (lines 77-123) from the code-qa.yml workflow. These jobs run integration tests against the OpenRouter API and are no longer needed.


View task on Roo Code Cloud

Start a new Roo Code Cloud session on this branch

@dosubotdosubotBot added the lgtm This PR has been approved by a maintainer label Feb 19, 2026
@hannesrudolph
hannesrudolph marked this pull request as ready for review February 19, 2026 08:04
@dosubotdosubotBot added the size:XL This PR changes 500-999 lines, ignoring generated files. label Feb 19, 2026
@hannesrudolph
hannesrudolph merged commit aff46b1 into mainFeb 19, 2026
18 checks passed
@hannesrudolph
hannesrudolph deleted the chore/remove-integration-tests branch February 19, 2026 08:16
@github-actionsgithub-actionsBot mentioned this pull request Feb 19, 2026
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtmThis PR has been approved by a maintainersize:XLThis PR changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@hannesrudolph@roomote