-
Notifications
You must be signed in to change notification settings - Fork 543
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
Create integration tests for the rest of the Components #1566
Comments
Is this referring to: aspire/tests/Aspire.EndToEnd.Tests/IntegrationServicesTests.cs Lines 18 to 34 in eb08a4a
So, essentially connect, and try some basic operation? |
Correct. There are components that aren't covered in those tests - like the EF postgres and sql server components. |
@eerhardt is this issue stale? Are we still missing some test coverage here? If so, is 8.1 the right milestone? I don't think that Ankit is currently looking into this. |
I'm not sure what you mean by "stale", but this issue is still relevant. We don't have these integration tests yet. So yes we are missing some test coverage here.
Yes, I believe so. We want better test coverage in 8.1. |
Last ones completed by #4758 . |
We are missing the EntityFramework components in our functional integration tests added in #1217. We should add them, plus any other missing components that would be valuable.
TODO:
The text was updated successfully, but these errors were encountered: