-
Notifications
You must be signed in to change notification settings - Fork 16
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
Added support for Azure Event Hub #98
Added support for Azure Event Hub #98
Conversation
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
<AssemblyName>Squadron.AzureCloudEventHub.Tests</AssemblyName> | ||
</PropertyGroup> | ||
|
||
<ItemGroup> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove empty itemgroup
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed
<CopyToOutputDirectory>Always</CopyToOutputDirectory> | ||
</None> | ||
</ItemGroup> | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove empty line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed
|
||
namespace Squadron.AzureCloudEventHub.Tests | ||
{ | ||
public class AzureNewEventHubResourceTests |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add test also for existing eventhub
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A test has been added in class AzureExistingEventHubResourceTests
<ProjectReference Include="..\Core\Core.csproj" /> | ||
</ItemGroup> | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove emtpy line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed
private EventHubOptionsBuilder() | ||
: base() | ||
{ | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove empty line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed
/// <returns></returns> | ||
public static EventHubOptionsBuilder New() => new EventHubOptionsBuilder(); | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove empty line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed
- add test for existing event hub namespace - remove blank lines and empty itemgroup
"SubscriptionId": "", | ||
"ResourceGroup": "", | ||
"DefaultLocation": "", | ||
"SubscriptionId": "3764e1f0-3a1b-4a28-a340-1ed0d8d388fd", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ups... remove these
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks - must have gone too fast wehen I committed. All secrets are removed
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
@glucaci it seems like RabbitMQ and ReavenDB tests are failing, which are tests that shouldn't be affected by this PR |
Kudos, SonarCloud Quality Gate passed!
|
This PR adds support in Squadron for Azure Event Hubs. It is possible to create an EventHub namespace with one or more EventHubs in it. Test shows how to publish and consume events on a new event hub and assert that the message published is similar to what this consumed.