Skip to content

fix: update trial mode tests to match generateYAML's 4-value return signature#25138

Merged
pelikhan merged 1 commit intomainfrom
copilot/fix-tests
Apr 7, 2026
Merged

fix: update trial mode tests to match generateYAML's 4-value return signature#25138
pelikhan merged 1 commit intomainfrom
copilot/fix-tests

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 7, 2026

generateYAML was updated to return (string, []string, []string, error) but 5 call sites across the trial mode test files still destructured only 2 values, causing a build failure in CI integration tests.

Changes

  • trial_mode_test.go (3 sites) and trial_issue_mode_test.go (2 sites): updated generateYAML calls from lockContent, err := to lockContent, _, _, err :=

@pelikhan pelikhan marked this pull request as ready for review April 7, 2026 18:31
Copilot AI review requested due to automatic review settings April 7, 2026 18:31
@pelikhan pelikhan merged commit 721bf29 into main Apr 7, 2026
@pelikhan pelikhan deleted the copilot/fix-tests branch April 7, 2026 18:31
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates integration test call sites to match generateYAML’s updated 4-value return signature, fixing CI build failures caused by incorrect destructuring in trial-mode tests.

Changes:

  • Updated generateYAML call sites in trial_mode_test.go (3 occurrences) to destructure (string, []string, []string, error).
  • Updated generateYAML call sites in trial_issue_mode_test.go (2 occurrences) to destructure (string, []string, []string, error).
Show a summary per file
File Description
pkg/workflow/trial_mode_test.go Adjusts generateYAML destructuring to include the two newly returned slices (ignored via _).
pkg/workflow/trial_issue_mode_test.go Same destructuring fix for issue-trigger trial mode integration tests.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 2/2 changed files
  • Comments generated: 0

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.

3 participants