Skip to content

[EventHubs] Improving Event Hubs unit tests coverage#9761

Merged
kinelski merged 23 commits intoAzure:masterfrom
kinelski:tests
Feb 19, 2020
Merged

[EventHubs] Improving Event Hubs unit tests coverage#9761
kinelski merged 23 commits intoAzure:masterfrom
kinelski:tests

Conversation

@kinelski
Copy link
Copy Markdown
Contributor

@kinelski kinelski commented Feb 1, 2020

Fixes #9227.

  • Multiple test scenarios added.
  • I reordered some tests in EventProcessorClientTests to keep them more organized.
  • Some tests removed from EventProcessorClientLiveTests because they were duplicates of the new tests.

@kinelski kinelski added Test Debt Do Not Merge Event Hubs Client This issue is related to a non-management package labels Feb 1, 2020
@kinelski kinelski self-assigned this Feb 1, 2020
@kinelski kinelski marked this pull request as ready for review February 18, 2020 14:02
@kinelski kinelski requested a review from jsquire as a code owner February 18, 2020 14:02
@kinelski kinelski merged commit 610ad4a into Azure:master Feb 19, 2020
@kinelski kinelski deleted the tests branch February 19, 2020 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Client This issue is related to a non-management package Event Hubs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Address gaps in Event Processor test coverage

2 participants