Skip to content

test: Add regression tests for constant folding panics#1476

Merged
aborgna-q merged 2 commits intomainfrom
ab/const-fold-custom-consts
Mar 26, 2026
Merged

test: Add regression tests for constant folding panics#1476
aborgna-q merged 2 commits intomainfrom
ab/const-fold-custom-consts

Conversation

@aborgna-q
Copy link
Copy Markdown
Collaborator

@aborgna-q aborgna-q commented Mar 26, 2026

See #1473

The actual fixes are in Quantinuum/hugr#2986

@aborgna-q aborgna-q requested a review from a team as a code owner March 26, 2026 12:02
@aborgna-q aborgna-q requested a review from ss2165 March 26, 2026 12:02
/// Check that opaque constants don't cause panics when they fail to be folded.
//
// TODO: The disabled tests here are waiting for
//<https://github.com/Quantinuum/hugr/pull/2986> to be merged and released.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

expect fail better than disable?

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.62%. Comparing base (424ab87) to head (460b2a6).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1476      +/-   ##
==========================================
+ Coverage   83.60%   83.62%   +0.01%     
==========================================
  Files         187      187              
  Lines       28791    28796       +5     
  Branches    27800    27805       +5     
==========================================
+ Hits        24071    24080       +9     
+ Misses       3556     3553       -3     
+ Partials     1164     1163       -1     
Flag Coverage Δ
python 93.00% <ø> (ø)
rust 83.29% <100.00%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

@aborgna-q aborgna-q requested a review from ss2165 March 26, 2026 13:08
@aborgna-q aborgna-q added this pull request to the merge queue Mar 26, 2026
Merged via the queue into main with commit fce6495 Mar 26, 2026
24 checks passed
@aborgna-q aborgna-q deleted the ab/const-fold-custom-consts branch March 26, 2026 13:37
@hugrbot hugrbot mentioned this pull request Mar 26, 2026
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