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

Initial updates #1607

Merged
merged 8 commits into from
Sep 10, 2024
Merged

Initial updates #1607

merged 8 commits into from
Sep 10, 2024

Conversation

IEvangelist
Copy link
Member

@IEvangelist IEvangelist commented Sep 6, 2024

Summary

Update the existing top-level testing doc. Fixes #1114, note that this doesn't include any example tests with the FakeLogger<T> as I couldn't figure out how to get the to work.


Internal previews

📄 File 🔗 Preview link
docs/fundamentals/testing.md docs/fundamentals/testing

@dotnet-bot dotnet-bot added the fundamentals Content related to fundamentals. label Sep 6, 2024
@dotnet-policy-service dotnet-policy-service bot added the okr-freshness OKR: Freshness of content label Sep 9, 2024
@IEvangelist IEvangelist marked this pull request as ready for review September 9, 2024 19:02
Copy link
Contributor

@tdykstra tdykstra left a comment

Choose a reason for hiding this comment

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

Looks great. I left a few minor suggestions.

docs/fundamentals/testing.md Outdated Show resolved Hide resolved
docs/fundamentals/testing.md Outdated Show resolved Hide resolved
docs/fundamentals/testing.md Outdated Show resolved Hide resolved
docs/fundamentals/testing.md Outdated Show resolved Hide resolved
@IEvangelist IEvangelist merged commit 7c7010b into dotnet:main Sep 10, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fundamentals Content related to fundamentals. okr-freshness OKR: Freshness of content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Aspire.Hosting.Testing doc to cover new changes in 8.1
3 participants