Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

eiriktsarpalis
Copy link
Member

@eiriktsarpalis eiriktsarpalis commented Nov 28, 2023

Adds .NET 8 source generated sources to complement the existing .NET 6 and .NET 7 regression suites.

@eiriktsarpalis eiriktsarpalis self-assigned this Nov 28, 2023
@ghost
Copy link

ghost commented Nov 28, 2023

Tagging subscribers to this area: @dotnet/area-system-text-json, @gregsdennis
See info in area-owners.md if you want to be subscribed.

Issue Details

null

Author: eiriktsarpalis
Assignees: eiriktsarpalis
Labels:

area-System.Text.Json

Milestone: -

@eiriktsarpalis eiriktsarpalis added this to the 9.0.0 milestone Nov 28, 2023
@eiriktsarpalis eiriktsarpalis added the test-enhancement Improvements of test source code label Nov 28, 2023
Copy link
Member

@jozkee jozkee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Given that Net70RegressionTests.cs is identical to Net80RegressionTests.cs, is this only a snapshot of some generated types on 8.0? Will we continue adding them each release indiscriminately?

@eiriktsarpalis
Copy link
Member Author

Given that Net70RegressionTests.cs is identical to Net80RegressionTests.cs, is this only a snapshot of some generated types on 8.0?

Correct, this is smoke testing functionality of the source generated baselines against the current runtime bits, so what tests are being run doesn't change much.

Will we continue adding them each release indiscriminately?

It depends on whether a particular release makes substantial changes to how it generates code. We have been updating the SG on every release so it made sense to include new regression tests every time.

@eiriktsarpalis eiriktsarpalis merged commit 642c868 into dotnet:main Nov 29, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Dec 29, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-System.Text.Json test-enhancement Improvements of test source code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants