Skip to content

Enable the integration tests for Anthropic #743

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

Merged
merged 2 commits into from
Jan 23, 2025

Conversation

rdimitrov
Copy link
Member

@rdimitrov rdimitrov commented Jan 23, 2025

Fixes: #742

@rdimitrov rdimitrov force-pushed the anthropic-integration-tests branch from 28fd621 to b53eab7 Compare January 23, 2025 13:21
Comment on lines -220 to -228
Hello! I'm CodeGate, your security-focused AI assistant. I can help you with:

- Software security analysis and reviews
- Package security assessments
- Secure coding practices and guidance
- Security vulnerability analysis
- Best practices for secure implementations

How can I assist you with your security needs today?
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had to change this otherwise it was failing the test. I made it similar to the test we did for Copilot.

Of course if we expect to have a codegate-aware answer let me know as we might have a bug 👍

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no, I think this was a remnant of us adding the codegate prompt to everything

- name: Run integration tests - Anthropic
env:
CODEGATE_PROVIDERS: "anthropic"
ENV_ANTHROPIC_KEY: ${{ secrets.anthropic-key }}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we might want to do a matrix in the future, but this is good enough

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was thinking about that too, but I wanted to get everything running first and then do that as a follow up 👍 That way they can run in parallel too

jhrozek
jhrozek previously approved these changes Jan 23, 2025
@rdimitrov rdimitrov merged commit acd4025 into main Jan 23, 2025
4 checks passed
@rdimitrov rdimitrov deleted the anthropic-integration-tests branch January 23, 2025 14:03
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.

Enable the integration tests for Anthropic
2 participants