Skip to content

test(codegen): align source-graph IR contracts - #9170

Merged
proggeramlug merged 1 commit into
mainfrom
fix/9164-source-graph-output-contract
Aug 30, 2026
Merged

test(codegen): align source-graph IR contracts#9170
proggeramlug merged 1 commit into
mainfrom
fix/9164-source-graph-output-contract

Conversation

@proggeramlug

Copy link
Copy Markdown
Contributor

Closes #9164.

Full CI exposed two stale assertions in source_graph_export_regressions:

This changes test expectations only; compiler and runtime behavior are unchanged.

Validated on the dedicated Linux server:

  • cargo test -p perry --test source_graph_export_regressions -- --nocapture (16 passed, including forced evacuation)
  • cargo test -p perry-codegen trusted_box_callback_tests:: -- --nocapture (8 passed)
  • cargo fmt --all
  • git diff --check

@coderabbitai

coderabbitai Bot commented Aug 30, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 2 minutes.

View limit details

Limit details: You’ve used all 8 included reviews currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 41d2ab10-9889-4c16-a0ea-6dbc39768f6f

📥 Commits

Reviewing files that changed from the base of the PR and between 5f08e77 and abfbdd9.

📒 Files selected for processing (1)
  • crates/perry/tests/source_graph_export_regressions.rs

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@proggeramlug
proggeramlug merged commit 3411c9d into main Aug 30, 2026
20 checks passed
@proggeramlug
proggeramlug deleted the fix/9164-source-graph-output-contract branch August 30, 2026 13:21
proggeramlug added a commit that referenced this pull request Aug 30, 2026
Co-authored-by: Ralph Küpper <ralph@skelpo.com>
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.

release blocker: source-graph IR assertions reject intentional trusted-box and pshape lowering

1 participant