-
Notifications
You must be signed in to change notification settings - Fork 457
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
Failing SqlServer
, and SqlClient
tests
#5057
Labels
Comments
dotnet-issue-labeler
bot
added
the
area-integrations
Issues pertaining to Aspire Integrations packages
label
Jul 24, 2024
radical
added a commit
to radical/aspire
that referenced
this issue
Jul 24, 2024
radical
added a commit
to radical/aspire
that referenced
this issue
Jul 24, 2024
radical
changed the title
Failing tests
Failing Jul 24, 2024
Aspire.Microsoft.EntityFrameworkCore.SqlServer.Tests
SqlServer
, and SqlClient
tests
More info in testcontainers/testcontainers-dotnet#1220 and microsoft/mssql-docker#892 . |
The tests are not really "disabled", but a workaround has been applied. I'm adding |
Merged
@radical - can this be closed? We worked around the issue. |
16 tasks
radical
pushed a commit
that referenced
this issue
Sep 4, 2024
updates Testcontainers to the latest version `3.10.0`. This version includes a fix for the regression/changes made in the latest MSSQL Docker image (microsoft/mssql-docker#892). The recently added workaround (#5057, #5058) is no longer necessary. Fixes #5057
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
Update: The tests have been updated to work around this issue. But once the image is fixed, that should be reverted.
Build: https://dev.azure.com/dnceng-public/cbb18261-c48f-4abb-8651-8cdcb5474649/_build/results?buildId=753141
Build error leg or test failing: Aspire.Microsoft.EntityFrameworkCore.SqlServer.Tests.WorkItemExecution
Pull request: #5043
This is always failing on CI now.
Log trails with:
Also,
Aspire.Microsoft.Data.SqlClient.Tests
(log) is failing.cc @eerhardt @joperezr
Report
Summary
The text was updated successfully, but these errors were encountered: