Adding public API test coverage

src/Components/Aspire.Azure.Messaging.EventHubs/AspireEventHubsExtensions.cs

@@ -33,6 +33,9 @@ public static void AddAzureEventProcessorClient(
         Action<AzureMessagingEventHubsProcessorSettings>? configureSettings = null,
         Action<IAzureClientBuilder<EventProcessorClient, EventProcessorClientOptions>>? configureClientBuilder = null)
     {
+        ArgumentNullException.ThrowIfNull(builder);
+        ArgumentException.ThrowIfNullOrEmpty(connectionName);
+
         new EventProcessorClientComponent()
             .AddClient(builder, DefaultConfigSectionName + nameof(EventProcessorClient),
                 configureSettings, configureClientBuilder, connectionName, serviceKey: null);
@@ -53,6 +56,7 @@ public static void AddKeyedAzureEventProcessorClient(
         Action<AzureMessagingEventHubsProcessorSettings>? configureSettings = null,
         Action<IAzureClientBuilder<EventProcessorClient, EventProcessorClientOptions>>? configureClientBuilder = null)
     {
+        ArgumentNullException.ThrowIfNull(builder);
         ArgumentException.ThrowIfNullOrEmpty(name);
 
         string configurationSectionName = EventProcessorClientComponent
@@ -79,6 +83,9 @@ public static void AddAzurePartitionReceiverClient(
         Action<AzureMessagingEventHubsPartitionReceiverSettings>? configureSettings = null,
         Action<IAzureClientBuilder<PartitionReceiver, PartitionReceiverOptions>>? configureClientBuilder = null)
     {
+        ArgumentNullException.ThrowIfNull(builder);
+        ArgumentException.ThrowIfNullOrEmpty(connectionName);
+
         new PartitionReceiverClientComponent()
             .AddClient(builder, DefaultConfigSectionName + nameof(PartitionReceiver),
                 configureSettings, configureClientBuilder, connectionName, serviceKey: null);
@@ -99,6 +106,7 @@ public static void AddKeyedAzurePartitionReceiverClient(
         Action<AzureMessagingEventHubsPartitionReceiverSettings>? configureSettings = null,
         Action<IAzureClientBuilder<PartitionReceiver, PartitionReceiverOptions>>? configureClientBuilder = null)
     {
+        ArgumentNullException.ThrowIfNull(builder);
         ArgumentException.ThrowIfNullOrEmpty(name);
 
         string configurationSectionName = PartitionReceiverClientComponent
@@ -125,6 +133,9 @@ public static void AddAzureEventHubProducerClient(
         Action<AzureMessagingEventHubsProducerSettings>? configureSettings = null,
         Action<IAzureClientBuilder<EventHubProducerClient, EventHubProducerClientOptions>>? configureClientBuilder = null)
     {
+        ArgumentNullException.ThrowIfNull(builder);
+        ArgumentException.ThrowIfNullOrEmpty(connectionName);
+
         new EventHubProducerClientComponent()
             .AddClient(builder, DefaultConfigSectionName + nameof(EventHubProducerClient),
                 configureSettings, configureClientBuilder, connectionName, serviceKey: null);
@@ -145,6 +156,7 @@ public static void AddKeyedAzureEventHubProducerClient(
         Action<AzureMessagingEventHubsProducerSettings>? configureSettings = null,
         Action<IAzureClientBuilder<EventHubProducerClient, EventHubProducerClientOptions>>? configureClientBuilder = null)
     {
+        ArgumentNullException.ThrowIfNull(builder);
         ArgumentException.ThrowIfNullOrEmpty(name);
 
         string configurationSectionName = EventHubProducerClientComponent
@@ -172,6 +184,9 @@ public static void AddAzureEventHubBufferedProducerClient(
         Action<IAzureClientBuilder<EventHubBufferedProducerClient, EventHubBufferedProducerClientOptions>>?
             configureClientBuilder = null)
     {
+        ArgumentNullException.ThrowIfNull(builder);
+        ArgumentException.ThrowIfNullOrEmpty(connectionName);
+
         new EventHubBufferedProducerClientComponent()
             .AddClient(builder, DefaultConfigSectionName + nameof(EventHubBufferedProducerClient), configureSettings,
                 configureClientBuilder, connectionName, serviceKey: null);
@@ -193,6 +208,7 @@ public static void AddKeyedAzureEventHubBufferedProducerClient(
         Action<AzureMessagingEventHubsBufferedProducerSettings>? configureSettings = null,
         Action<IAzureClientBuilder<EventHubBufferedProducerClient, EventHubBufferedProducerClientOptions>>? configureClientBuilder = null)
     {
+        ArgumentNullException.ThrowIfNull(builder);
         ArgumentException.ThrowIfNullOrEmpty(name);
 
         string configurationSectionName = EventHubBufferedProducerClientComponent
@@ -218,6 +234,9 @@ public static void AddAzureEventHubConsumerClient(
         Action<AzureMessagingEventHubsConsumerSettings>? configureSettings = null,
         Action<IAzureClientBuilder<EventHubConsumerClient, EventHubConsumerClientOptions>>? configureClientBuilder = null)
     {
+        ArgumentNullException.ThrowIfNull(builder);
+        ArgumentException.ThrowIfNullOrEmpty(connectionName);
+
         new EventHubConsumerClientComponent()
             .AddClient(builder, DefaultConfigSectionName + nameof(EventHubConsumerClient),
                 configureSettings, configureClientBuilder, connectionName, serviceKey: null);
@@ -239,6 +258,7 @@ public static void AddKeyedAzureEventHubConsumerClient(
         Action<AzureMessagingEventHubsConsumerSettings>? configureSettings = null,
         Action<IAzureClientBuilder<EventHubConsumerClient, EventHubConsumerClientOptions>>? configureClientBuilder = null)
     {
+        ArgumentNullException.ThrowIfNull(builder);
         ArgumentException.ThrowIfNullOrEmpty(name);
 
         string configurationSectionName = EventHubConsumerClientComponent

src/Components/Aspire.Azure.Messaging.ServiceBus/AspireServiceBusExtensions.cs

@@ -37,6 +37,9 @@ public static void AddAzureServiceBusClient(
         Action<AzureMessagingServiceBusSettings>? configureSettings = null,
         Action<IAzureClientBuilder<ServiceBusClient, ServiceBusClientOptions>>? configureClientBuilder = null)
     {
+        ArgumentNullException.ThrowIfNull(builder);
+        ArgumentException.ThrowIfNullOrEmpty(connectionName);
+
         new MessageBusComponent().AddClient(builder, DefaultConfigSectionName, configureSettings, configureClientBuilder, connectionName, serviceKey: null);
     }
 
@@ -55,6 +58,7 @@ public static void AddKeyedAzureServiceBusClient(
         Action<AzureMessagingServiceBusSettings>? configureSettings = null,
         Action<IAzureClientBuilder<ServiceBusClient, ServiceBusClientOptions>>? configureClientBuilder = null)
     {
+        ArgumentNullException.ThrowIfNull(builder);
         ArgumentException.ThrowIfNullOrEmpty(name);
 
         string configurationSectionName = MessageBusComponent.GetKeyedConfigurationSectionName(name, DefaultConfigSectionName);

src/Components/Aspire.Azure.Messaging.WebPubSub/AspireWebPubSubExtensions.cs

@@ -36,6 +36,9 @@ public static void AddAzureWebPubSubServiceClient(
         Action<AzureMessagingWebPubSubSettings>? configureSettings = null,
         Action<IAzureClientBuilder<WebPubSubServiceClient, WebPubSubServiceClientOptions>>? configureClientBuilder = null)
     {
+        ArgumentNullException.ThrowIfNull(builder);
+        ArgumentException.ThrowIfNullOrEmpty(connectionName);
+
         new WebPubSubComponent().AddClient(builder, DefaultConfigSectionName, configureSettings, configureClientBuilder, connectionName, serviceKey: null);
     }
 
@@ -56,6 +59,7 @@ public static void AddKeyedAzureWebPubSubServiceClient(
         Action<AzureMessagingWebPubSubSettings>? configureSettings = null,
         Action<IAzureClientBuilder<WebPubSubServiceClient, WebPubSubServiceClientOptions>>? configureClientBuilder = null)
     {
+        ArgumentNullException.ThrowIfNull(builder);
         ArgumentException.ThrowIfNullOrEmpty(connectionName);
         ArgumentException.ThrowIfNullOrEmpty(serviceKey);
 

src/Components/Aspire.Azure.Storage.Blobs/AspireBlobStorageExtensions.cs

@@ -37,7 +37,13 @@ public static void AddAzureBlobClient(
         Action<AzureStorageBlobsSettings>? configureSettings = null,
         Action<IAzureClientBuilder<BlobServiceClient, BlobClientOptions>>? configureClientBuilder = null)
     {
-        new BlobStorageComponent().AddClient(builder, DefaultConfigSectionName, configureSettings, configureClientBuilder, connectionName, serviceKey: null);
+        AddAzureBlobClientInternal(
+            builder,
+            DefaultConfigSectionName,
+            connectionName,
+            serviceKey: null,
+            configureSettings,
+            configureClientBuilder);
     }
 
     /// <summary>
@@ -60,7 +66,27 @@ public static void AddKeyedAzureBlobClient(
 
         string configurationSectionName = BlobStorageComponent.GetKeyedConfigurationSectionName(name, DefaultConfigSectionName);
 
-        new BlobStorageComponent().AddClient(builder, configurationSectionName, configureSettings, configureClientBuilder, connectionName: name, serviceKey: name);
+        AddAzureBlobClientInternal(
+            builder,
+            configurationSectionName,
+            connectionName: name,
+            serviceKey: name,
+            configureSettings,
+            configureClientBuilder);
+    }
+
+    private static void AddAzureBlobClientInternal(
+        IHostApplicationBuilder builder,
+        string configurationSectionName,
+        string connectionName,
+        string? serviceKey,
+        Action<AzureStorageBlobsSettings>? configureSettings = null,
+        Action<IAzureClientBuilder<BlobServiceClient, BlobClientOptions>>? configureClientBuilder = null)
+    {
+        ArgumentNullException.ThrowIfNull(builder);
+        ArgumentException.ThrowIfNullOrEmpty(connectionName);
+
+        new BlobStorageComponent().AddClient(builder, configurationSectionName, configureSettings, configureClientBuilder, connectionName: connectionName, serviceKey: serviceKey);
     }
 
     private sealed class BlobStorageComponent : AzureComponent<AzureStorageBlobsSettings, BlobServiceClient, BlobClientOptions>

src/Components/Aspire.Microsoft.Data.SqlClient/AspireSqlServerSqlClientExtensions.cs

@@ -42,15 +42,20 @@ public static void AddSqlServerClient(this IHostApplicationBuilder builder, stri
     /// <exception cref="InvalidOperationException">If required <see cref="MicrosoftDataSqlClientSettings.ConnectionString"/> is not provided in configuration section.</exception>
     public static void AddKeyedSqlServerClient(this IHostApplicationBuilder builder, string name, Action<MicrosoftDataSqlClientSettings>? configureSettings = null)
     {
-        ArgumentNullException.ThrowIfNull(name);
+        ArgumentException.ThrowIfNullOrEmpty(name);
 
         AddSqlClient(builder, $"{DefaultConfigSectionName}:{name}", configureSettings, connectionName: name, serviceKey: name);
     }
 
-    private static void AddSqlClient(IHostApplicationBuilder builder, string configurationSectionName,
-        Action<MicrosoftDataSqlClientSettings>? configure, string connectionName, object? serviceKey)
+    private static void AddSqlClient(
+        IHostApplicationBuilder builder,
+        string configurationSectionName,
+        Action<MicrosoftDataSqlClientSettings>? configure,
+        string connectionName,
+        object? serviceKey)
     {
         ArgumentNullException.ThrowIfNull(builder);
+        ArgumentException.ThrowIfNullOrEmpty(connectionName);
 
         MicrosoftDataSqlClientSettings settings = new();
         builder.Configuration.GetSection(configurationSectionName).Bind(settings);

src/Components/Aspire.Microsoft.EntityFrameworkCore.Cosmos/AspireAzureEFCoreCosmosExtensions.cs

@@ -43,6 +43,8 @@ public static class AspireAzureEFCoreCosmosExtensions
         Action<DbContextOptionsBuilder>? configureDbContextOptions = null) where TContext : DbContext
     {
         ArgumentNullException.ThrowIfNull(builder);
+        ArgumentException.ThrowIfNullOrEmpty(connectionName);
+        ArgumentException.ThrowIfNullOrEmpty(databaseName);
 
         builder.EnsureDbContextNotRegistered<TContext>();
 

src/Components/Aspire.Microsoft.EntityFrameworkCore.SqlServer/AspireSqlServerEFCoreSqlClientExtensions.cs

@@ -41,6 +41,7 @@ public static class AspireSqlServerEFCoreSqlClientExtensions
         Action<DbContextOptionsBuilder>? configureDbContextOptions = null) where TContext : DbContext
     {
         ArgumentNullException.ThrowIfNull(builder);
+        ArgumentException.ThrowIfNullOrEmpty(connectionName);
 
         builder.EnsureDbContextNotRegistered<TContext>();
 

src/Components/Aspire.MySqlConnector/AspireMySqlConnectorExtensions.cs

@@ -7,7 +7,6 @@
 using Microsoft.Extensions.Configuration;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Diagnostics.HealthChecks;
-using Microsoft.Extensions.Logging;
 using MySqlConnector;
 
 namespace Microsoft.Extensions.Hosting;
@@ -54,6 +53,7 @@ private static void AddMySqlDataSource(IHostApplicationBuilder builder, string c
         Action<MySqlConnectorSettings>? configureSettings, string connectionName, object? serviceKey)
     {
         ArgumentNullException.ThrowIfNull(builder);
+        ArgumentException.ThrowIfNullOrEmpty(connectionName);
 
         MySqlConnectorSettings settings = new();
         builder.Configuration.GetSection(configurationSectionName).Bind(settings);

src/Components/Aspire.Npgsql.EntityFrameworkCore.PostgreSQL/AspireEFPostgreSqlExtensions.cs

@@ -49,6 +49,7 @@ public static partial class AspireEFPostgreSqlExtensions
         Action<DbContextOptionsBuilder>? configureDbContextOptions = null) where TContext : DbContext
     {
         ArgumentNullException.ThrowIfNull(builder);
+        ArgumentException.ThrowIfNullOrEmpty(connectionName);
 
         builder.EnsureDbContextNotRegistered<TContext>();
 

src/Components/Aspire.Npgsql/AspirePostgreSqlNpgsqlExtensions.cs

@@ -53,10 +53,15 @@ public static void AddKeyedNpgsqlDataSource(this IHostApplicationBuilder builder
         AddNpgsqlDataSource(builder, $"{DefaultConfigSectionName}:{name}", configureSettings, connectionName: name, serviceKey: name, configureDataSourceBuilder: configureDataSourceBuilder);
     }
 
-    private static void AddNpgsqlDataSource(IHostApplicationBuilder builder, string configurationSectionName,
-        Action<NpgsqlSettings>? configureSettings, string connectionName, object? serviceKey, Action<NpgsqlDataSourceBuilder>? configureDataSourceBuilder)
+    private static void AddNpgsqlDataSource(
+        IHostApplicationBuilder builder,
+        string configurationSectionName,
+        Action<NpgsqlSettings>? configureSettings,
+        string connectionName,
+        object? serviceKey, Action<NpgsqlDataSourceBuilder>? configureDataSourceBuilder)
     {
         ArgumentNullException.ThrowIfNull(builder);
+        ArgumentException.ThrowIfNullOrEmpty(connectionName);
 
         NpgsqlSettings settings = new();
         builder.Configuration.GetSection(configurationSectionName).Bind(settings);

src/Components/Aspire.Oracle.EntityFrameworkCore/AspireOracleEFCoreExtensions.cs

@@ -42,6 +42,7 @@ public static class AspireOracleEFCoreExtensions
         Action<DbContextOptionsBuilder>? configureDbContextOptions = null) where TContext : DbContext
     {
         ArgumentNullException.ThrowIfNull(builder);
+        ArgumentException.ThrowIfNullOrEmpty(connectionName);
 
         builder.EnsureDbContextNotRegistered<TContext>();