Skip to content

chore(ci): add additional test token - #7227

Merged
PeterSchafer merged 1 commit into
mainfrom
chore/add_test_token
Sep 7, 2026
Merged

PeterSchafer merged 1 commit into
mainfrom
chore/add_test_token

Conversation

@PeterSchafer

@PeterSchafer PeterSchafer commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Pull Request Submission Checklist

  • Follows CONTRIBUTING guidelines
  • Commit messages
    are release-note ready, emphasizing
    what was changed, not how.
  • Includes detailed description of changes
  • Contains risk assessment (Low | Medium | High)
  • Highlights breaking API changes (if applicable)
  • Links to automated tests covering new functionality
  • Includes manual testing instructions (if necessary)
  • Updates relevant GitBook documentation (PR link: ___)
  • Includes product update to be announced in the next stable release notes

What does this PR do?

Add additional test token

Where should the reviewer start?

How should this be manually tested?

What's the product update that needs to be communicated to CLI users?


Note

Low Risk
Test-only setup change with no impact on CLI runtime or production auth paths.

Overview
Adds TEST_SNYK_TOKEN_4 to the TOKEN_ENV_VARS list in test/setup.js, so Jest global setup can pick a fourth optional API token when more than one is defined in the environment.

Behavior is unchanged otherwise: selectRandomToken() still chooses randomly among whichever of the listed env vars are set and non-empty, then configures the CLI for integration tests.

Reviewed by Cursor Bugbot for commit 9bd7139. Bugbot is set up for automated code reviews on this repo. Configure here.

@PeterSchafer
PeterSchafer requested a review from a team as a code owner September 4, 2026 17:22
@snyk-io

snyk-io Bot commented Sep 4, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@snyk-pr-review-bot

This comment has been minimized.

@snyk-pr-review-bot

Copy link
Copy Markdown

PR Reviewer Guide 🔍

🧪 PR contains tests
🔒 No security concerns identified
⚡ No major issues detected
📚 Repository Context Analyzed

This review considered 3 relevant code sections from 3 files (average relevance: 0.73)

🤖 Repository instructions applied (from AGENTS.md)

@PeterSchafer
PeterSchafer merged commit 5c8aeed into main Sep 7, 2026
10 of 11 checks passed
@PeterSchafer
PeterSchafer deleted the chore/add_test_token branch September 7, 2026 09:50
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