Skip to content

Comments

Validate injectivity of use/defs#2631

Draft
toinehartman wants to merge 3 commits intomainfrom
fix/field-use-defs
Draft

Validate injectivity of use/defs#2631
toinehartman wants to merge 3 commits intomainfrom
fix/field-use-defs

Conversation

@toinehartman
Copy link
Member

@toinehartman toinehartman commented Feb 5, 2026

This PR modifies the tests in two related ways.

  1. Check that use/defs are injective, i.e. any use only maps to definitions of overloaded names.
  2. Add a test use/defs of fields of syntax constructors.

Closes #2632

@codecov
Copy link

codecov bot commented Feb 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 46%. Comparing base (0d4f663) to head (9c1f55d).
⚠️ Report is 20 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##              main   #2631   +/-   ##
=======================================
  Coverage       46%     46%           
- Complexity    6672    6682   +10     
=======================================
  Files          795     795           
  Lines        65878   65893   +15     
  Branches      9870    9878    +8     
=======================================
+ Hits         30679   30701   +22     
+ Misses       32824   32819    -5     
+ Partials      2375    2373    -2     

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TModels contain extra use/defs for fields

1 participant