Skip to content

Quarantine HubConnectionCanSendAndReceiveGroupMessages test #62436

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Jun 21, 2025

The HubConnectionCanSendAndReceiveGroupMessages test in the SignalR StackExchange Redis tests is failing due to Docker infrastructure issues in the CI environment. The test is experiencing Docker container creation failures with the error:

Command '/usr/bin/docker run --rm -p 6379:6379 --name redisTestContainer -d redis' failed with exit code '127'. Output:
docker: open /datadisks/disk1/docker/tmp/GetImageBlob3428286899: no such file or directory.

This change quarantines the test by adding the QuarantinedTest attribute to temporarily disable it until the underlying Docker infrastructure issue can be resolved. This follows the established pattern used by other quarantined tests in the same file and across the repository.

The change is minimal and surgical - adding only one line to mark the test as quarantined while preserving all existing test logic for when the infrastructure issue is fixed.

Fixes #62435.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

@javiercn
Copy link
Member

Apply the Quarantine attribute using #62435 as the Url. Look at how is done in other samples on the repo

@Copilot Copilot AI changed the title [WIP] Quarantine HubConnectionCanSendAndReceiveGroupMessages Quarantine HubConnectionCanSendAndReceiveGroupMessages test Jun 21, 2025
@Copilot Copilot AI requested a review from javiercn June 21, 2025 16:17
Copilot finished work on behalf of javiercn June 21, 2025 16:17
Copy link
Contributor Author

Copilot AI commented Jun 21, 2025

Apply the Quarantine attribute using #62435 as the Url. Look at how is done in other samples on the repo

Fixed the attribute placement to follow the standard pattern used throughout the repository. The [QuarantinedTest] attribute is now positioned after the other test attributes, matching the pattern used by other quarantined tests. Commit: a2d398d

Copilot finished work on behalf of javiercn June 21, 2025 16:23
@javiercn javiercn marked this pull request as ready for review June 21, 2025 18:00
@javiercn
Copy link
Member

Quarantine all these tests with the same URL

Microsoft.AspNetCore.SignalR.StackExchangeRedis.Tests.RedisEndToEndTests.HubConnectionCanSendAndReceiveGroupMessages
Microsoft.AspNetCore.SignalR.StackExchangeRedis.Tests.RedisEndToEndTests.StatefulReconnectPreservesMessageFromOtherServer
Microsoft.AspNetCore.SignalR.StackExchangeRedis.Tests.RedisEndToEndTests.CanSendAndReceiveUserMessagesWhenOneConnectionWithUserDisconnects
Microsoft.AspNetCore.SignalR.StackExchangeRedis.Tests.RedisEndToEndTests.HubConnectionCanSendAndReceiveGroupMessagesGroupNameWithPatternIsTreatedAsLiteral
Microsoft.AspNetCore.SignalR.StackExchangeRedis.Tests.RedisEndToEndTests.HubConnectionCanSendAndReceiveMessages

Copy link
Contributor Author

Copilot AI commented Jun 21, 2025

Quarantine all these tests with the same URL

Microsoft.AspNetCore.SignalR.StackExchangeRedis.Tests.RedisEndToEndTests.HubConnectionCanSendAndReceiveGroupMessages
...

All five tests have been quarantined with the same URL as requested. Each test now has the [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/62435")] attribute. Commit: 20ca2069

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Quarantine HubConnectionCanSendAndReceiveGroupMessages
2 participants