Skip to content

Fix test using deprecated timeout_minutes field#14889

Merged
pelikhan merged 2 commits intocopilot/standardize-path-validation-patternsfrom
copilot/fix-tests-one-more-time
Feb 11, 2026
Merged

Fix test using deprecated timeout_minutes field#14889
pelikhan merged 2 commits intocopilot/standardize-path-validation-patternsfrom
copilot/fix-tests-one-more-time

Conversation

Copy link
Contributor

Copilot AI commented Feb 11, 2026

Test fixture in TestMaxTurnsCompilation was using the deprecated timeout_minutes field (underscore), causing compilation failures with "Unknown property" errors.

Changes

  • Updated test fixture to use timeout-minutes (hyphen) to match current schema requirements
 engine:
   id: claude
   max-turns: 10
-timeout_minutes: 15
+timeout-minutes: 15
 strict: false

The timeout_minutes field was removed from the schema in #14736, and test fixtures must use the hyphenated version.

Original prompt

Reference: https://github.com/github/gh-aw/actions/runs/21893260769/job/63203935644#step:10:1

Fix tests


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

…t-minutes

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing tests in the project Fix test using deprecated timeout_minutes field Feb 11, 2026
Copilot AI requested a review from pelikhan February 11, 2026 05:13
@pelikhan pelikhan marked this pull request as ready for review February 11, 2026 05:21
Copilot AI review requested due to automatic review settings February 11, 2026 05:21
@pelikhan pelikhan merged commit 5d9d3fd into copilot/standardize-path-validation-patterns Feb 11, 2026
1 check passed
@pelikhan pelikhan deleted the copilot/fix-tests-one-more-time branch February 11, 2026 05:21
Copy link
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

This PR fixes a test compilation failure caused by using the deprecated timeout_minutes field (underscore) instead of the current timeout-minutes field (hyphen) in a test fixture.

Changes:

  • Updated test fixture in TestMaxTurnsCompilation to use timeout-minutes field to match current schema requirements

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

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