Feature/275 masstransit rabbitmq client integration

[MichielBrys]

**Closes #275 **

Host & client projects for masstransit rabbitmq. And example project. Readme's are in the src projects.

PR Checklist

• Created a feature/dev branch in your fork (vs. submitting directly from a commit on main)
• Based off latest main branch of toolkit
• PR doesn't include merge commits (always rebase on top of our main, if needed)
• New integration
  • Docs are written
  • Added description of major feature to project description for NuGet package (4000 total character limit, so don't push entire description over that)
• Tests for the changes have been added (for bug fixes / features) (if applicable)
• Contains NO breaking changes
• Every new API (including internal ones) has full XML docs
• Code follows all style conventions

Other information

[MichielBrys]

@dotnet-policy-service agree

[fredimachado]

Hi @MichielBrys , I noticed the AddMassTransitClient method and I was just wondering what would happen if, in the future we also have Aspire packages for other transports like CommunityToolkit.Aspire.Client.MassTransit.ServiceBus for example.

Maybe it could be named AddMassTransitRabbitMqClient in order to not make it ambiguous?

Also, maybe the settings class could have a more specific name as well like MassTransitRabbitMqClientSettings.

[aaronpowell]

Left some comments and suggestions to bring it inline with more of the design of the Toolkit projects.

Also, you'll need to ensure that the NuGet packages are added to the Directory.Packages.props with their correct versions since we use CPM to ensure everything is consistently versioned.

[anoordover]

Can you explain what this solution adds on the ease of use as compared to https://github.com/anoordover/MassTransitJobConsumer?
What would the impact of using this component be for my example?

[MichielBrys]

Thanks for the reviews, I read them and indeed realised the hosting package isn't doing anything extra so I got rid of it. Currently only have the client which uses the connection string from the .AddRabbitMQ().

I'm thinking it would be pretty usefull to have in the community toolkit and I'd like to expand it to support all the implementations that work with masstransit so people have a nice wrapper to use with aspire to easily set up masstransit

I also fixed the nuget versioning to use the Directory.Packages.props file.

[aaronpowell]

Thanks for the reviews, I read them and indeed realised the hosting package isn't doing anything extra so I got rid of it. Currently only have the client which uses the connection string from the .AddRabbitMQ().

I'm thinking it would be pretty usefull to have in the community toolkit and I'd like to expand it to support all the implementations that work with masstransit so people have a nice wrapper to use with aspire to easily set up masstransit

I also fixed the nuget versioning to use the Directory.Packages.props file.

Given that it doesn't provide any functionality over the standard RabbitMQ integration, I don't see that there's a need for the hosting integration.

The client integration should be able to use the connection string provided by the standard RabbitMQ integration just fine.

[MichielBrys]

Thanks for the reviews, I read them and indeed realised the hosting package isn't doing anything extra so I got rid of it. Currently only have the client which uses the connection string from the .AddRabbitMQ().
I'm thinking it would be pretty usefull to have in the community toolkit and I'd like to expand it to support all the implementations that work with masstransit so people have a nice wrapper to use with aspire to easily set up masstransit
I also fixed the nuget versioning to use the Directory.Packages.props file.

Given that it doesn't provide any functionality over the standard RabbitMQ integration, I don't see that there's a need for the hosting integration.

The client integration should be able to use the connection string provided by the standard RabbitMQ integration just fine.

Agreed its the client package I meant that could be nice to have in the toolkit, the hosting one I already removed yesterday.

[aaronpowell]

It's been removed from the solution, but the files are still on disk:

These can be deleted, along with the .idea folder.

[aaronpowell]

Also, can the examples be moved into a sub-folder of examples, such as examples/masstransit-rabbitmq (and a solution folder to match), so that it's consistent with the others.

[MichielBrys]

Could someone merge the pr? I don't have write access I think. Thanks :)

[aaronpowell]

Could someone merge the pr? I don't have write access I think. Thanks :)

Yeah. CodeQL didn't run for some reason so I'll have to override that to merge.

[aaronpowell]

I just noticed that there's no tests.

Can you add some tests for the integration. Since this is a client integration, check out https://github.com/CommunityToolkit/Aspire/tree/main/tests/CommunityToolkit.Aspire.Meilisearch.Tests as an example for how to write client integration tests.