-
Notifications
You must be signed in to change notification settings - Fork 10.4k
Enable test logging for ProjectTemplate tests #26507
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we consider targeting this for release/5.0
to aid diagnostics there?
If I get general consensus that this is a good change/correct way to do it, then I can update the other tests and move it |
I didn't add LoggedTest to Components and BlazorTemplates as they have other complex infrastructure that I am unfamiliar with. But I did get rid of the Xunit.TestFramework that they were using and moved them to the common one we use everywhere else. |
2d4f326
to
cfbb362
Compare
Any feedback? |
So the new logs are these ones?
|
Yep, they didn't exist before. |
@dotnet/aspnet-build please merge when available! |
Possible change to templates tests so we have log artifacts. Only applied it to a couple tests to show the idea first.