Skip to content
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

Add regression testing for the STJv8 source generator. #95352

Merged

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.

@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
108 of 111 checks passed
@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