Skip to content

Don't deduplicate Method Parameter Types#454

Merged
mak638 merged 2 commits intomainfrom
447-methodmemberparameters-doesnt-return-the-expected-amount-of-entries-when-having-the-same-type-for-parameters
Mar 4, 2026
Merged

Don't deduplicate Method Parameter Types#454
mak638 merged 2 commits intomainfrom
447-methodmemberparameters-doesnt-return-the-expected-amount-of-entries-when-having-the-same-type-for-parameters

Conversation

@alexanderlinne
Copy link
Collaborator

Fixes #447

Support verification of parameter types and counts by deduplicating types when instantiating dependencies rather than the parameters themselves.

This PR also inlines some functions that do not have multiple users.

Signed-off-by: Alexander Linne <alexander.linne@tngtech.com>
@alexanderlinne alexanderlinne changed the title feat: dont deduplicate method parameter types Don't deduplicate Method Parameter Types Mar 2, 2026
@codecov-commenter
Copy link

codecov-commenter commented Mar 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.17%. Comparing base (dbf0a49) to head (179f88c).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #454      +/-   ##
==========================================
- Coverage   77.20%   77.17%   -0.04%     
==========================================
  Files         275      275              
  Lines       14411    14385      -26     
  Branches     1337     1337              
==========================================
- Hits        11126    11101      -25     
  Misses       2866     2866              
+ Partials      419      418       -1     

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

…-expected-amount-of-entries-when-having-the-same-type-for-parameters
@mak638 mak638 enabled auto-merge March 4, 2026 18:19
@mak638 mak638 merged commit e50b6dd into main Mar 4, 2026
9 checks passed
@mak638 mak638 deleted the 447-methodmemberparameters-doesnt-return-the-expected-amount-of-entries-when-having-the-same-type-for-parameters branch March 4, 2026 18:23
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.

MethodMember.Parameters doesn't return the expected amount of entries when having the same type for parameters

3 participants