Skip to content
This repository was archived by the owner on Oct 12, 2023. It is now read-only.

[WIP] verification test for MessageSender abstraction is broken for Path when Send-Via is used #570

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

SeanFeldman
Copy link
Collaborator

DO NOT MERGE

Issue #569

Adding verification tests expose the bug against master.

@SeanFeldman SeanFeldman self-assigned this Sep 12, 2018
@SeanFeldman SeanFeldman requested a review from a team as a code owner September 12, 2018 05:30
@SeanFeldman SeanFeldman changed the title [WIP] Add MessageSender verifications [WIP] verification test for MessageSender abstraction is broken for Path when Send-Via is used Oct 24, 2018
@SeanFeldman SeanFeldman force-pushed the issue-569-repro branch 2 times, most recently from cdd4e22 to a5687eb Compare January 17, 2019 09:21
@axisc
Copy link

axisc commented May 1, 2019

@SeanFeldman - we are moving the underlying issue to the [Azure SDK for .NET] (https://github.com/Azure/azure-sdk-for-net/tree/master/sdk/servicebus/Microsoft.Azure.ServiceBus) repo.

Once we archive this repo, the PR will be read only and we can leverage this test case in the new repo.

@SeanFeldman
Copy link
Collaborator Author

we are moving the underlying issue to the Azure SDK for .NET repo.

Can I suggest to move and then comment for traceability? Right now, anyone looking at this issue would have no lead to the replacement issue in the new repo. Thank you.

@SeanFeldman
Copy link
Collaborator Author

Ah, nevermind. The issue is moved, PR was not. All good @axisc

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

Successfully merging this pull request may close these issues.

2 participants