Skip to content

Pin CI pytest major version to restore HexManager test compatibility#4

Merged
Robotmad merged 1 commit into
mainfrom
copilot/run-ci-fix-failing-tests
May 13, 2026
Merged

Pin CI pytest major version to restore HexManager test compatibility#4
Robotmad merged 1 commit into
mainfrom
copilot/run-ci-fix-failing-tests

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 13, 2026

CI test runs started failing before executing assertions due to a pytest runtime incompatibility with the simulator’s os.stat shim. This change aligns the CI test dependency to a compatible pytest major version so the suite can run against current code.

  • Problem context

    • The Tests workflow installed latest pytest, which pulled in pytest 9.x and triggered an internal error during test session execution (follow_symlinks arg path).
  • Workflow change

    • Updated the test dependency installation in .github/workflows/tests.yml to constrain pytest to <9 for CI runs.
    • Scope is intentionally limited to workflow dependency resolution; no production or test logic was modified.
# .github/workflows/tests.yml
- pip install "pytest<9"

Agent-Logs-Url: https://github.com/Robotmad/HexManager/sessions/8324d722-1083-4ef3-820b-ab85cad7843c

Co-authored-by: Robotmad <3315650+Robotmad@users.noreply.github.com>
@Robotmad Robotmad marked this pull request as ready for review May 13, 2026 16:52
Copilot AI review requested due to automatic review settings May 13, 2026 16:52
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

This PR restores HexManager CI test compatibility by constraining the pytest version used in the GitHub Actions test workflow.

Changes:

  • Pins CI pytest installation to "pytest<9" to avoid the simulator shim incompatibility with pytest 9.x.

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

@Robotmad Robotmad merged commit 01caf25 into main May 13, 2026
4 checks passed
@Robotmad Robotmad deleted the copilot/run-ci-fix-failing-tests branch May 13, 2026 17:49
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