Skip to content

googletest --> catch2 - #2579

Closed
samfreund wants to merge 1 commit into
PhotonVision:mainfrom
samfreund:catch2
Closed

googletest --> catch2#2579
samfreund wants to merge 1 commit into
PhotonVision:mainfrom
samfreund:catch2

Conversation

@samfreund

Copy link
Copy Markdown
Member

Description

What changed?

Switches c++ test framework from googletest --> catch2

Why?

WPILib is doing it, and as they lead we follow. (They also won't publish the artifacts anymore soooo)

Testing

  • I have tested this change locally
  • Test evidence (screenshots, videos, or test results):

AI Disclosure

  • This PR was authored entirely by me
  • This PR includes AI-generated code (e.g., from GitHub Copilot, ChatGPT)
    • If yes, I have reviewed all AI-generated code for correctness
    • If yes, please describe which parts were AI-assisted:

Describe AI involvement here if applicable

AI did the mass edits for changing the framework.


Merge Checklist

Additional Checks (if applicable)

  • User-facing changes? User documentation is updated
  • Breaking changes? Migration guide is included in description and migrations are marked with // MIGRATION: <dataYear> where <dataYear> is the last season the pre-migration data was used in
  • Bug fix? Regression test is added
  • New dependency? License compatibility is verified and steps have been taken to follow it
  • Serde changes? All messages are regenerated with no unexpected hash changes
  • Configuration changes? Changes are backwards compatible with previous season's last release
  • Pipeline/data exchange changes? Frontend types are updated in ./photon-client/src/types

Signed-off-by: samfreund <samf.236@proton.me>
@samfreund
samfreund requested a review from a team as a code owner August 5, 2026 20:52
@github-actions github-actions Bot added the photonlib Things related to the PhotonVision library label Aug 5, 2026
@samfreund

samfreund commented Aug 5, 2026

Copy link
Copy Markdown
Member Author

The new version of native-utils would pull in some other stuff too, so we may as well just close this and fold into #2566

@samfreund samfreund closed this Aug 5, 2026
@samfreund
samfreund deleted the catch2 branch August 5, 2026 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

photonlib Things related to the PhotonVision library

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant