Skip to content

Adds 'circular' tests for the two branches present#2060

Open
Matt-Carre wants to merge 9 commits into
mainfrom
2042_inversion_tests
Open

Adds 'circular' tests for the two branches present#2060
Matt-Carre wants to merge 9 commits into
mainfrom
2042_inversion_tests

Conversation

@Matt-Carre
Copy link
Copy Markdown
Contributor

@Matt-Carre Matt-Carre commented May 18, 2026

Fixes #2042

This adds tests that check that f(g(x)) == g(f(x)), where f and g are inverses of each other (e.g., f=converting cm to mm, g=converting mm to cm), and x is an input value.

Also adds in Bools when not included.

@Matt-Carre Matt-Carre requested a review from a team as a code owner May 18, 2026 08:33
@Matt-Carre
Copy link
Copy Markdown
Contributor Author

This will be expanded once #2049 is merged into main.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.12%. Comparing base (edfa7d6) to head (dd54869).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2060   +/-   ##
=======================================
  Coverage   99.12%   99.12%           
=======================================
  Files         336      336           
  Lines       13073    13073           
=======================================
  Hits        12959    12959           
  Misses        114      114           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

Extend beamline-relevant arithmetic with "Circularity" tests of conversions

1 participant