Skip to content

fix(models): preserve specification extensions in OpenApiPaths and OpenApiResponses copy constructors (#3162) - #3163

Merged
Vincent Biret (baywet) merged 1 commit into
support/v2from
fix/copy-to-v2
Nov 6, 2026
Merged

Vincent Biret (baywet) merged 1 commit into
support/v2from
fix/copy-to-v2

Conversation

@baywet

Copy link
Copy Markdown
Member

port of 3162

…enApiResponses copy constructors (#3162)

* fix(models): preserve extensions in OpenApiPaths copy constructor

Signed-off-by: Hardik K <hardikkaurani1@gmail.com>

* test(models): add unit tests for OpenApiPaths copy constructor extensions

Signed-off-by: Hardik K <hardikkaurani1@gmail.com>

* test(models): verify OpenApiDocument copy constructor preserves Paths extensions

Signed-off-by: Hardik K <hardikkaurani1@gmail.com>

* fix(models): preserve extensions in OpenApiResponses copy constructor

Signed-off-by: Hardik K <hardikkaurani1@gmail.com>

* test(models): add unit tests for OpenApiResponses copy constructor extensions

Signed-off-by: Hardik K <hardikkaurani1@gmail.com>

* test(models): verify OpenApiOperation copy constructor preserves Responses extensions

Signed-off-by: Hardik K <hardikkaurani1@gmail.com>

* test(models): add edge case tests for null and empty extensions in Paths and Responses copy constructors

Signed-off-by: Hardik K <hardikkaurani1@gmail.com>

* docs(models): update XML documentation on OpenApiPaths and OpenApiResponses copy constructors

Signed-off-by: Hardik K <hardikkaurani1@gmail.com>

---------

Signed-off-by: Hardik K <hardikkaurani1@gmail.com>
@baywet
Vincent Biret (baywet) requested a review from a team as a code owner November 6, 2026 14:57
@baywet Vincent Biret (baywet) self-assigned this Nov 6, 2026
@baywet
Vincent Biret (baywet) enabled auto-merge (squash) November 6, 2026 14:57
@github-code-quality

Copy link
Copy Markdown

Code coverage overview

Languages: C#

C# / code-coverage/dotnet

support/v2 a68f8be fix/copy-to-v2 2b069b2 +/-
93% 24% -69%
View coverage of the 10 most impacted files.
File support/v2 a68f8be fix/copy-to-v2 2b069b2 +/-
_/src/Verify/Se.../ScrubEngine.cs 0% 29% +29%
_/src/Verify/Ve...rifier_Inner.cs 0% 35% +35%
_/src/Verify/Na...fierSettings.cs 0% 45% +45%
_/src/Verify/Ve...nnerVerifier.cs 0% 46% +46%
_/src/Verify/At...fierSettings.cs 0% 55% +55%
_/src/Verify/Se...fierSettings.cs 0% 64% +64%
_/src/Verify/Se...tionSettings.cs 0% 65% +65%
_/src/Verify/Fi...eNameCleaner.cs 0% 85% +85%
_/src/Verify/Se...eScrubberSet.cs 0% 85% +85%
_/src/EmptyFile...leExtensions.cs 0% 86% +86%

ℹ️ Calculations represent line coverage. Results may change as calculations are updated.

@sonarqubecloud

sonarqubecloud Bot commented Nov 6, 2026

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

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

See analysis details on SonarQube Cloud

@baywet
Vincent Biret (baywet) merged commit 3301ba7 into support/v2 Nov 6, 2026
16 of 17 checks passed
@baywet
Vincent Biret (baywet) deleted the fix/copy-to-v2 branch November 6, 2026 19:25
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.

3 participants