Skip to content

Conversation

Tristan-WorkGH
Copy link
Contributor

@Tristan-WorkGH Tristan-WorkGH commented Aug 11, 2025

Show the conditions detected by coverage that cannot happen in code of #264.

Unblock #273

@Tristan-WorkGH Tristan-WorkGH self-assigned this Aug 11, 2025
@Tristan-WorkGH
Copy link
Contributor Author

Tristan-WorkGH commented Aug 11, 2025

Hi @basseche, look at testDtoHasLimitAfterMapped() tests, it show cases that is not possible in mergeCurrentLimits() with hasLimits() because of how operationalLimitsGroupToMapDataCurrentLimits() convert.

@Tristan-WorkGH Tristan-WorkGH added bug Something isn't working help wanted Extra attention is needed WIP labels Aug 11, 2025
Copy link

sonarqubecloud bot commented Aug 12, 2025

@Tristan-WorkGH Tristan-WorkGH added PR: corrected PR: waiting-for-review and removed bug Something isn't working help wanted Extra attention is needed WIP labels Aug 12, 2025
@Tristan-WorkGH Tristan-WorkGH merged commit eef11c4 into main Aug 13, 2025
4 checks passed
@Tristan-WorkGH Tristan-WorkGH deleted the fix/b32f39 branch August 13, 2025 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants