-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Run integration tests in CI #691
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
|
|
Heads up, I have this branch from earlier this week: https://github.com/RooVetGit/Roo-Code/tree/feature/integrationTestCI The test worked 100% of the time locally... |
|
@ColemanRoo - It's working! https://github.com/RooVetGit/Roo-Code/actions/runs/13082391740/job/36508410859?pr=691 Might be b/c I'm using MacOS; I haven't tried using Linux like your branch does. |
Awesome! I was trying to avoid using MacOS since those runners cost more, but if that's what makes this work, I think we should go for it. |
| // secrets | ||
|
|
||
| private async storeSecret(key: SecretKey, value?: string) { | ||
| public async storeSecret(key: SecretKey, value?: string) { |
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.
We call this from the integration test, so it has to be public, but it seems like it shouldn't be. I'll try to clean this up in a follow-up.
|
@mrubens - I think this is ready to go if you have time to take a quick look. |
🤔 - Seems to work on both MacOS and Ubuntu: https://github.com/RooVetGit/Roo-Code/actions/runs/13083240642/job/36510729987?pr=691 We can just use Ubuntu if it's cheaper. |
Description
Clean up our integration tests a bit and run them in CI.
Type of change
How Has This Been Tested?
Checklist:
Additional context
Related Issues
Reviewers
Important
Add integration tests to CI workflow, update pre-commit hooks, and enhance
ClineProviderfor testing.integration-testjob to.github/workflows/code-qa.ymlto run integration tests onubuntu-latest.workflow_dispatchto trigger CI manually..husky/pre-committo runnpm run compile,npm run lint, andnpm run check-types..env.integration.examplefor integration test environment variables.tsconfig.integration.jsonfor integration test TypeScript configuration.flake.nixandflake.lockfor Nix development environment.test:integrationscript inpackage.jsonto run integration tests.src/test/task.test.tsfor task-related integration tests.src/test/extension.test.tsto include environment variable checks.ClineProviderinClineProvider.tsto support integration tests withviewLaunchedandmessagesgetters.This description was created by
for 2e094b4. It will automatically update as commits are pushed.