fix: TryAddPath exact-match check for duplicate bound operation paths (OpenAPI 3.2) #812

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

baywet merged 1 commit into main from fix/tryaddpath-delta-resolution-v3

Apr 14, 2026

+87 −0

Merged

fix: TryAddPath exact-match check for duplicate bound operation paths

SonarQubeCloud / SonarCloud Code Analysis failed Apr 14, 2026 in 39s

Quality Gate failed

Failed conditions
75.0% Coverage on New Code (required ≥ 80%)

Member 'ListToString' does not access instance data and can be marked as static

See more on https://sonarcloud.io/project/issues?id=microsoft_OpenAPI.NET.OData&issues=AZ2JiWavgSVICk-snxYG&open=AZ2JiWavgSVICk-snxYG&pullRequest=812

Do not guard 'HashSet.Add(string)' with 'HashSet.Contains(string)'

See more on https://sonarcloud.io/project/issues?id=microsoft_OpenAPI.NET.OData&issues=AZ2JiWbLgSVICk-snxYI&open=AZ2JiWbLgSVICk-snxYI&pullRequest=812

Member 'AssertGraphBetaModelPaths' does not access instance data and can be marked as static

See more on https://sonarcloud.io/project/issues?id=microsoft_OpenAPI.NET.OData&issues=AZ2JiWavgSVICk-snxYF&open=AZ2JiWavgSVICk-snxYF&pullRequest=812

Prefer 'static readonly' fields over constant array arguments if the called method is called repeatedly and is not mutating the passed array

See more on https://sonarcloud.io/project/issues?id=microsoft_OpenAPI.NET.OData&issues=AZ2JiWavgSVICk-snxYE&open=AZ2JiWavgSVICk-snxYE&pullRequest=812

Refactor this method to reduce its Cognitive Complexity from 18 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=microsoft_OpenAPI.NET.OData&issues=AZ2JiWbLgSVICk-snxYH&open=AZ2JiWbLgSVICk-snxYH&pullRequest=812