Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 21, 2026

CI failing on testifylint rule violation in pkg/workflow/http_mcp_env_vars_test.go:70.

Change

Replace assert.Equal with assert.Len for checking collection length:

// Before
assert.Equal(t, 3, len(envVars), "Should have exactly 3 environment variables")

// After  
assert.Len(t, envVars, 3, "Should have exactly 3 environment variables")

The assert.Len helper provides better failure messages and is the idiomatic way to assert collection lengths in testify.

Original prompt

Reference: https://github.com/githubnext/gh-aw/actions/runs/21197211966/job/60975657682#step:7:1


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix workflow issue in GitHub Actions Fix testifylint violation: use assert.Len for collection length checks Jan 21, 2026
Copilot AI requested a review from pelikhan January 21, 2026 06:51
@pelikhan pelikhan marked this pull request as ready for review January 21, 2026 12:33
@pelikhan pelikhan merged commit 9a6bae6 into copilot/stabilize-daily-news-gateway Jan 21, 2026
@pelikhan pelikhan deleted the copilot/fix-workflow-issue branch January 21, 2026 12:33
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