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);
 
         new EventProcessorClientComponent()
@@ -75,6 +79,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);
@@ -95,6 +102,7 @@ public static void AddKeyedAzurePartitionReceiverClient(
         Action<AzureMessagingEventHubsPartitionReceiverSettings>? configureSettings = null,
         Action<IAzureClientBuilder<PartitionReceiver, PartitionReceiverOptions>>? configureClientBuilder = null)
     {
+        ArgumentNullException.ThrowIfNull(builder);
         ArgumentException.ThrowIfNullOrEmpty(name);
 
         new PartitionReceiverClientComponent()
@@ -117,6 +125,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);
@@ -137,6 +148,7 @@ public static void AddKeyedAzureEventHubProducerClient(
         Action<AzureMessagingEventHubsProducerSettings>? configureSettings = null,
         Action<IAzureClientBuilder<EventHubProducerClient, EventHubProducerClientOptions>>? configureClientBuilder = null)
     {
+        ArgumentNullException.ThrowIfNull(builder);
         ArgumentException.ThrowIfNullOrEmpty(name);
 
         new EventHubProducerClientComponent()
@@ -160,6 +172,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);
@@ -181,6 +196,7 @@ public static void AddKeyedAzureEventHubBufferedProducerClient(
         Action<AzureMessagingEventHubsBufferedProducerSettings>? configureSettings = null,
         Action<IAzureClientBuilder<EventHubBufferedProducerClient, EventHubBufferedProducerClientOptions>>? configureClientBuilder = null)
     {
+        ArgumentNullException.ThrowIfNull(builder);
         ArgumentException.ThrowIfNullOrEmpty(name);
 
         new EventHubBufferedProducerClientComponent()
@@ -203,6 +219,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);
@@ -224,6 +243,7 @@ public static void AddKeyedAzureEventHubConsumerClient(
         Action<AzureMessagingEventHubsConsumerSettings>? configureSettings = null,
         Action<IAzureClientBuilder<EventHubConsumerClient, EventHubConsumerClientOptions>>? configureClientBuilder = null)
     {
+        ArgumentNullException.ThrowIfNull(builder);
         ArgumentException.ThrowIfNullOrEmpty(name);
 
         new 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);
 
         new MessageBusComponent().AddClient(builder, DefaultConfigSectionName, configureSettings, configureClientBuilder, connectionName: name, serviceKey: name);

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,6 +37,8 @@ public static void AddAzureBlobClient(
         Action<AzureStorageBlobsSettings>? configureSettings = null,
         Action<IAzureClientBuilder<BlobServiceClient, BlobClientOptions>>? configureClientBuilder = null)
     {
+        ArgumentException.ThrowIfNullOrEmpty(connectionName);
+
         new BlobStorageComponent().AddClient(builder, DefaultConfigSectionName, configureSettings, configureClientBuilder, connectionName, serviceKey: null);
     }
 
@@ -64,9 +66,11 @@ public static void AddKeyedAzureBlobClient(
     private sealed class BlobStorageComponent : AzureComponent<AzureStorageBlobsSettings, BlobServiceClient, BlobClientOptions>
     {
         protected override IAzureClientBuilder<BlobServiceClient, BlobClientOptions> AddClient(
-            AzureClientFactoryBuilder azureFactoryBuilder, AzureStorageBlobsSettings settings, string connectionName,
+            AzureClientFactoryBuilder azureFactoryBuilder,
+            AzureStorageBlobsSettings settings,
+            string connectionName,
             string configurationSectionName)
-        {
+        {          
             return ((IAzureClientFactoryBuilderWithCredential)azureFactoryBuilder).RegisterClientFactory<BlobServiceClient, BlobClientOptions>((options, cred) =>
             {
                 var connectionString = settings.ConnectionString;

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

@@ -42,15 +42,19 @@ 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, configureSettings, connectionName: name, serviceKey: name);
     }
 
-    private static void AddSqlClient(IHostApplicationBuilder builder,
-        Action<MicrosoftDataSqlClientSettings>? configure, string connectionName, object? serviceKey)
+    private static void AddSqlClient(
+        IHostApplicationBuilder builder,
+        Action<MicrosoftDataSqlClientSettings>? configure,
+        string connectionName,
+        object? serviceKey)
     {
         ArgumentNullException.ThrowIfNull(builder);
+        ArgumentException.ThrowIfNullOrEmpty(connectionName);
 
         MicrosoftDataSqlClientSettings settings = new();
         var configSection = builder.Configuration.GetSection(DefaultConfigSectionName);

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,
         Action<MySqlConnectorSettings>? configureSettings, string connectionName, object? serviceKey)
     {
         ArgumentNullException.ThrowIfNull(builder);
+        ArgumentException.ThrowIfNullOrEmpty(connectionName);
 
         MySqlConnectorSettings settings = new();
         var configSection = builder.Configuration.GetSection(DefaultConfigSectionName);

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

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

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>();