[service-bus] Mitigate session-overlap test #16183
Conversation
ghost
added
the
|
/azp run js - service-bus - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run js - service-bus - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run js - service-bus - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
richardpark-msft
changed the title
|
/azp run js - service-bus - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run js - service-bus - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run js - service-bus - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run js - service-bus - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run js - service-bus - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run js - service-bus - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run js - service-bus - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run js - service-bus - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run js - service-bus - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
richardpark-msft
force-pushed
the
sb-flakytests-sessions-new
branch
from
f912646 to
5eb9160
Compare
|
/azp run js - service-bus - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run js - service-bus - tests |
|
/azp run js - service-bus - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
richardpark-msft
force-pushed
the
sb-flakytests-sessions-new
branch
from
e1a8aac to
67c2aca
Compare
richardpark-msft
changed the title
|
/azp run js - service-bus - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
… maxConcurrentCalls was passed properly. Replaced with a smaller test that can (with the right parameters, manually specified) still cause the session overlap bug. I don't have a solution for that one yet but this will allow us to investigate it when we're ready. Part of the investigation for Azure#14441
richardpark-msft
force-pushed
the
sb-flakytests-sessions-new
branch
from
67c2aca to
f5f5b54
Compare
|
/azp run js - service-bus - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run js - service-bus - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run js - service-bus - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run js - service-bus - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
(removing some of the debug logging I'd added in a previous PR. Prior live test run passed but I noticed it was chattier than it should have been) |
…empty entity so if the message that has been sent has not yet appeared (there can sometimes be a delay) then this test would have failed. The fix is to use one of our blocking calls (receiveMessages), put the message back and then peek.
|
An unrelated test failed but the assumption in it was bad (it expects to immediately peek a message after it's been sent, but there can be a delay). Fixed to receiveMessages() first, then put it back, then peekMessage(). |
|
/azp run js - service-bus - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Thank you "Chris" |
Removing three tests which had more code than needed to test that the maxConcurrentCalls was passed properly.
Replaced with a smaller test that can (with the right parameters, manually specified) still cause the session overlap bug. I don't have a solution for that one yet but this will allow us to investigate it when we're ready.
Part of the investigation for #14441