fix: Fix unit test issues

zhenlei520 · zhenlei520 · commit 60aec79d6a9e · 2022-11-10T09:58:56.000+08:00
diff --git a/src/BuildingBlocks/Exception/Masa.BuildingBlocks.Exceptions.Tests/MasaArgumentExceptionTest.cs b/src/BuildingBlocks/Exception/Masa.BuildingBlocks.Exceptions.Tests/MasaArgumentExceptionTest.cs
@@ -19,8 +19,8 @@ public void TestThrowIfNull()
         }
         catch (MasaArgumentException ex)
         {
-            Assert.AreEqual(Masa.BuildingBlocks.Data.Constants.ErrorCode.NULL_VALIDATOR, ex.ErrorCode);
-            Assert.AreEqual("Value cannot be null. (Parameter '{0}')", ex.ErrorMessage);
+            Assert.AreEqual(Data.Constants.ErrorCode.NOT_NULL_VALIDATOR, ex.ErrorCode);
+            Assert.AreEqual("'{0}' must not be empty.", ex.ErrorMessage);
         }
     }
 }
diff --git a/src/Contrib/Caching/Distributed/Tests/Masa.Contrib.Caching.Distributed.StackExchangeRedis.Tests/DistributedCacheClientTest.cs b/src/Contrib/Caching/Distributed/Tests/Masa.Contrib.Caching.Distributed.StackExchangeRedis.Tests/DistributedCacheClientTest.cs
@@ -528,7 +528,7 @@ public async Task TestHashIncrementAndValueLessThan1Async(string key, int value)
     {
         await _distributedCacheClient.RemoveAsync(key);
 
-        await Assert.ThrowsExceptionAsync<ArgumentOutOfRangeException>(async ()
+        await Assert.ThrowsExceptionAsync<MasaArgumentException>(async ()
             => await _distributedCacheClient.HashIncrementAsync(key, value));
     }
 
@@ -555,7 +555,7 @@ public async Task TestHashDecrementAndValueLessThan1Async(string key, long value
     {
         await _distributedCacheClient.RemoveAsync(key);
 
-        await Assert.ThrowsExceptionAsync<ArgumentOutOfRangeException>(async ()
+        await Assert.ThrowsExceptionAsync<MasaArgumentException>(async ()
             => await _distributedCacheClient.HashDecrementAsync(key, value));
     }
 
diff --git a/src/Contrib/Caching/Distributed/Tests/Masa.Contrib.Caching.Distributed.StackExchangeRedis.Tests/TestBase.cs b/src/Contrib/Caching/Distributed/Tests/Masa.Contrib.Caching.Distributed.StackExchangeRedis.Tests/TestBase.cs
@@ -13,7 +13,7 @@ protected static RedisConfigurationOptions GetConfigurationOptions()
         {
             GlobalCacheOptions = new CacheOptions()
             {
-                CacheKeyType = CacheKeyType.TypeName
+                CacheKeyType = CacheKeyType.None
             }
         };
         redisConfigurationOptions.Servers.Add(new RedisServerOptions());
diff --git a/src/Contrib/Configuration/ConfigurationApi/Tests/Masa.Contrib.Configuration.ConfigurationApi.Dcc.Tests/DccTest.cs b/src/Contrib/Configuration/ConfigurationApi/Tests/Masa.Contrib.Configuration.ConfigurationApi.Dcc.Tests/DccTest.cs
@@ -524,7 +524,7 @@ public void TestComplementAndCheckDccConfigurationOptionByCustomerDccOptions()
     public void TestComplementAndCheckDccConfigurationOptionByManageServiceAddressIsEmpty()
     {
         DccOptions dccOptions = new DccOptions();
-        Assert.ThrowsException<ArgumentNullException>(() =>
+        Assert.ThrowsException<MasaArgumentException>(() =>
         {
             MasaConfigurationExtensions.ComplementAndCheckDccConfigurationOption(_masaConfigurationBuilder.Object, dccOptions);
         });
@@ -537,7 +537,7 @@ public void TestComplementAndCheckDccConfigurationOptionByRedisServersIsEmpty()
         {
             ManageServiceAddress = nameof(DccOptions.ManageServiceAddress),
         };
-        Assert.ThrowsException<ArgumentException>(() =>
+        Assert.ThrowsException<MasaArgumentException>(() =>
         {
             MasaConfigurationExtensions.ComplementAndCheckDccConfigurationOption(_masaConfigurationBuilder.Object, dccOptions);
         });
@@ -599,7 +599,7 @@ public void TestComplementAndCheckDccConfigurationOptionByAppIdIsEmpty()
                 }
             }
         };
-        Assert.ThrowsException<ArgumentException>(() =>
+        Assert.ThrowsException<MasaArgumentException>(() =>
         {
             MasaConfigurationExtensions.ComplementAndCheckDccConfigurationOption(_masaConfigurationBuilder.Object, dccOptions);
         });
diff --git a/src/Contrib/Configuration/Masa.Contrib.Configuration/Extensions/ServiceCollectionExtensions.cs b/src/Contrib/Configuration/Masa.Contrib.Configuration/Extensions/ServiceCollectionExtensions.cs
@@ -2,6 +2,7 @@
 // Licensed under the MIT License. See LICENSE.txt in the project root for license information.
 
 // ReSharper disable once CheckNamespace
+
 namespace Microsoft.Extensions.DependencyInjection;
 
 public static class ServiceCollectionExtensions
@@ -135,12 +136,7 @@ public static IServiceCollection AddMasaConfiguration(
         configurationBuilder.Sources.Clear();
         configurationBuilder.AddConfiguration(masaConfiguration);
 
-        if (sourceConfiguration == null)
-        {
-            var configurationManager = new ConfigurationManager();
-            configurationBuilder.AddConfiguration(configurationBuilder.Build());
-            services.AddSingleton<IConfiguration>(_ => configurationManager);
-        }
+        if (sourceConfiguration == null) services.AddSingleton<IConfiguration>(_ => configurationBuilder.Build());
 
         return services;
     }
diff --git a/src/Contrib/Dispatcher/IntegrationEvents/Tests/Masa.Contrib.Dispatcher.IntegrationEvents.Dapr.Tests/IntegrationEventBusTest.cs b/src/Contrib/Dispatcher/IntegrationEvents/Tests/Masa.Contrib.Dispatcher.IntegrationEvents.Dapr.Tests/IntegrationEventBusTest.cs
@@ -44,11 +44,11 @@ public void TestDispatcherOption()
         var services = new ServiceCollection();
         DispatcherOptions options;
 
-        Assert.ThrowsException<ArgumentException>(() =>
+        Assert.ThrowsException<MasaArgumentException>(() =>
         {
             options = new DispatcherOptions(services, null!);
         });
-        Assert.ThrowsException<ArgumentException>(() =>
+        Assert.ThrowsException<MasaArgumentException>(() =>
         {
             options = new DispatcherOptions(services, Array.Empty<Assembly>());
         });
diff --git a/src/Contrib/Dispatcher/IntegrationEvents/Tests/Masa.Contrib.Dispatcher.IntegrationEvents.Tests/DispatcherOptionTest.cs b/src/Contrib/Dispatcher/IntegrationEvents/Tests/Masa.Contrib.Dispatcher.IntegrationEvents.Tests/DispatcherOptionTest.cs
@@ -22,7 +22,7 @@ public void TestSetLocalRetryTimes()
         _options.LocalRetryTimes = 5;
         Assert.IsTrue(_options.LocalRetryTimes == 5);
 
-        Assert.ThrowsException<ArgumentException>(() => _options.LocalRetryTimes = 0);
+        Assert.ThrowsException<MasaArgumentException>(() => _options.LocalRetryTimes = 0);
     }
 
     [TestMethod]
@@ -32,7 +32,7 @@ public void TestSetMaxRetryTimes()
         _options.MaxRetryTimes = 5;
         Assert.IsTrue(_options.MaxRetryTimes == 5);
 
-        Assert.ThrowsException<ArgumentException>(() => _options.MaxRetryTimes = 0);
+        Assert.ThrowsException<MasaArgumentException>(() => _options.MaxRetryTimes = 0);
     }
 
     [TestMethod]
@@ -42,7 +42,7 @@ public void TestSetFailedRetryInterval()
         _options.FailedRetryInterval = 5;
         Assert.IsTrue(_options.FailedRetryInterval == 5);
 
-        Assert.ThrowsException<ArgumentException>(() => _options.FailedRetryInterval = 0);
+        Assert.ThrowsException<MasaArgumentException>(() => _options.FailedRetryInterval = 0);
     }
 
     [TestMethod]
@@ -52,7 +52,7 @@ public void TestSetMinimumRetryInterval()
         _options.MinimumRetryInterval = 5;
         Assert.IsTrue(_options.MinimumRetryInterval == 5);
 
-        Assert.ThrowsException<ArgumentException>(() => _options.MinimumRetryInterval = 0);
+        Assert.ThrowsException<MasaArgumentException>(() => _options.MinimumRetryInterval = 0);
     }
 
     [TestMethod]
@@ -62,7 +62,7 @@ public void TestSetLocalFailedRetryInterval()
         _options.LocalFailedRetryInterval = 5;
         Assert.IsTrue(_options.LocalFailedRetryInterval == 5);
 
-        Assert.ThrowsException<ArgumentException>(() => _options.LocalFailedRetryInterval = -1);
+        Assert.ThrowsException<MasaArgumentException>(() => _options.LocalFailedRetryInterval = -1);
     }
 
     [TestMethod]
@@ -72,7 +72,7 @@ public void TestSetRetryBatchSize()
         _options.RetryBatchSize = 5;
         Assert.IsTrue(_options.RetryBatchSize == 5);
 
-        Assert.ThrowsException<ArgumentException>(() => _options.RetryBatchSize = -1);
+        Assert.ThrowsException<MasaArgumentException>(() => _options.RetryBatchSize = -1);
     }
 
     [TestMethod]
@@ -82,7 +82,7 @@ public void TestSetCleaningLocalQueueExpireInterval()
         _options.CleaningLocalQueueExpireInterval = 5;
         Assert.IsTrue(_options.CleaningLocalQueueExpireInterval == 5);
 
-        Assert.ThrowsException<ArgumentException>(() => _options.CleaningLocalQueueExpireInterval = 0);
+        Assert.ThrowsException<MasaArgumentException>(() => _options.CleaningLocalQueueExpireInterval = 0);
     }
 
     [TestMethod]
@@ -92,7 +92,7 @@ public void TestSetCleaningExpireInterval()
         _options.CleaningExpireInterval = 5;
         Assert.IsTrue(_options.CleaningExpireInterval == 5);
 
-        Assert.ThrowsException<ArgumentException>(() => _options.CleaningExpireInterval = 0);
+        Assert.ThrowsException<MasaArgumentException>(() => _options.CleaningExpireInterval = 0);
     }
 
     [TestMethod]
@@ -102,7 +102,7 @@ public void TestSetPublishedExpireTime()
         _options.PublishedExpireTime = 24 * 3 * 3600;
         Assert.IsTrue(_options.PublishedExpireTime == 24 * 3 * 3600);
 
-        Assert.ThrowsException<ArgumentException>(() => _options.PublishedExpireTime = 0);
+        Assert.ThrowsException<MasaArgumentException>(() => _options.PublishedExpireTime = 0);
     }
 
     [TestMethod]
@@ -112,7 +112,7 @@ public void TestSetDeleteBatchCount()
         _options.DeleteBatchCount = 100;
         Assert.IsTrue(_options.DeleteBatchCount == 100);
 
-        Assert.ThrowsException<ArgumentException>(() => _options.DeleteBatchCount = 0);
+        Assert.ThrowsException<MasaArgumentException>(() => _options.DeleteBatchCount = 0);
     }
 
     [TestMethod]
diff --git a/src/Contrib/Dispatcher/IntegrationEvents/Tests/Masa.Contrib.Dispatcher.IntegrationEvents.Tests/IntegrationEventBusTest.cs b/src/Contrib/Dispatcher/IntegrationEvents/Tests/Masa.Contrib.Dispatcher.IntegrationEvents.Tests/IntegrationEventBusTest.cs
@@ -49,11 +49,11 @@ public void TestDispatcherOption()
         var services = new ServiceCollection();
         DispatcherOptions options;
 
-        Assert.ThrowsException<ArgumentException>(() =>
+        Assert.ThrowsException<MasaArgumentException>(() =>
         {
             options = new DispatcherOptions(services, null!);
         });
-        Assert.ThrowsException<ArgumentException>(() =>
+        Assert.ThrowsException<MasaArgumentException>(() =>
         {
             options = new DispatcherOptions(services, Array.Empty<Assembly>());
         });
diff --git a/src/Contrib/Dispatcher/Tests/Masa.Contrib.Dispatcher.Events.Tests/AssemblyResolutionTests.cs b/src/Contrib/Dispatcher/Tests/Masa.Contrib.Dispatcher.Events.Tests/AssemblyResolutionTests.cs
@@ -24,7 +24,7 @@ public void TestAddNullAssembly()
     {
         var services = new ServiceCollection();
         services.AddLogging(loggingBuilder => loggingBuilder.AddConsole());
-        Assert.ThrowsException<ArgumentException>(() =>
+        Assert.ThrowsException<MasaArgumentException>(() =>
         {
             Assembly[] assemblies = null!;
             services.AddEventBus(assemblies!);
@@ -36,7 +36,7 @@ public void TestAddEmptyAssembly()
     {
         var services = new ServiceCollection();
         services.AddLogging(loggingBuilder => loggingBuilder.AddConsole());
-        Assert.ThrowsException<ArgumentException>(() =>
+        Assert.ThrowsException<MasaArgumentException>(() =>
         {
             services.AddEventBus(Array.Empty<Assembly>());
         });
@@ -47,7 +47,7 @@ public void TestEventBusByAddNullAssembly()
     {
         var services = new ServiceCollection();
         services.AddLogging(loggingBuilder => loggingBuilder.AddConsole());
-        Assert.ThrowsException<ArgumentException>(() =>
+        Assert.ThrowsException<MasaArgumentException>(() =>
         {
             services.AddTestEventBus(null!, ServiceLifetime.Scoped);
         });
diff --git a/src/Contrib/Storage/ObjectStorage/Tests/Masa.Contrib.Storage.ObjectStorage.Aliyun.Tests/StorageTest.cs b/src/Contrib/Storage/ObjectStorage/Tests/Masa.Contrib.Storage.ObjectStorage.Aliyun.Tests/StorageTest.cs
@@ -19,7 +19,7 @@ public void TestAddAliyunStorageAndNotAddConfigurationReturnClientIsNotNull()
     public void TestAddAliyunStorageByEmptySectionReturnThrowArgumentException()
     {
         var services = new ServiceCollection();
-        Assert.ThrowsException<ArgumentException>(() => services.AddAliyunStorage(string.Empty));
+        Assert.ThrowsException<MasaArgumentException>(() => services.AddAliyunStorage(string.Empty));
     }
 
     [TestMethod]
@@ -226,7 +226,7 @@ public void TestAddAliyunStorageAndNullALiYunStorageOptionsReturnThrowArgumentNu
     [TestMethod]
     public void TestAddAliyunStorageByEmptyAccessKeyIdReturnThrowArgumentNullException()
     {
-        Assert.ThrowsException<ArgumentException>(() => new AliyunStorageOptions(null!, null!, null!));
+        Assert.ThrowsException<MasaArgumentException>(() => new AliyunStorageOptions(null!, null!, null!));
     }
 
     [TestMethod]
diff --git a/src/Contrib/Storage/ObjectStorage/Tests/Masa.Contrib.Storage.ObjectStorage.Aliyun.Tests/TestALiYunStorageOptions.cs b/src/Contrib/Storage/ObjectStorage/Tests/Masa.Contrib.Storage.ObjectStorage.Aliyun.Tests/TestALiYunStorageOptions.cs
@@ -24,7 +24,7 @@ public void TestErrorParameterThrowArgumentException(
         string roleSessionName,
         string parameterName)
     {
-        Assert.ThrowsException<ArgumentException>(() =>
+        Assert.ThrowsException<MasaArgumentException>(() =>
                 new AliyunStorageOptions(accessKeyId, accessKeySecret, regionId, roleArn, roleSessionName),
             $"{parameterName} cannot be empty");
     }
@@ -73,7 +73,7 @@ public void TestErrorTemporaryCredentialsCacheKeyReturnThrowArgumentException(
         string temporaryCredentialsCacheKeyName)
     {
         var options = new AliyunStorageOptions("AccessKeyId", "AccessKeySecret", HANG_ZHOUE_PUBLIC_ENDPOINT, "RoleArn", "RoleSessionName");
-        Assert.ThrowsException<ArgumentException>(() =>
+        Assert.ThrowsException<MasaArgumentException>(() =>
                 options.SetTemporaryCredentialsCacheKey(temporaryCredentialsCacheKey),
             $"{temporaryCredentialsCacheKeyName} cannot be empty");
     }

Original file line number	Diff line number	Diff line change
`@@ -19,8 +19,8 @@ public void TestThrowIfNull()`
`19`	`19`	`}`
`20`	`20`	`catch (MasaArgumentException ex)`
`21`	`21`	`{`
`22`		`- Assert.AreEqual(Masa.BuildingBlocks.Data.Constants.ErrorCode.NULL_VALIDATOR, ex.ErrorCode);`
`23`		`- Assert.AreEqual("Value cannot be null. (Parameter '{0}')", ex.ErrorMessage);`
	`22`	`+ Assert.AreEqual(Data.Constants.ErrorCode.NOT_NULL_VALIDATOR, ex.ErrorCode);`
	`23`	`+ Assert.AreEqual("'{0}' must not be empty.", ex.ErrorMessage);`
`24`	`24`	`}`
`25`	`25`	`}`
`26`	`26`	`}`
Original file line number	Diff line number	Diff line change
`@@ -528,7 +528,7 @@ public async Task TestHashIncrementAndValueLessThan1Async(string key, int value)`
`528`	`528`	`{`
`529`	`529`	`await _distributedCacheClient.RemoveAsync(key);`
`530`	`530`
`531`		`- await Assert.ThrowsExceptionAsync<ArgumentOutOfRangeException>(async ()`
	`531`	`+ await Assert.ThrowsExceptionAsync<MasaArgumentException>(async ()`
`532`	`532`	`=> await _distributedCacheClient.HashIncrementAsync(key, value));`
`533`	`533`	`}`
`534`	`534`
`@@ -555,7 +555,7 @@ public async Task TestHashDecrementAndValueLessThan1Async(string key, long value`
`555`	`555`	`{`
`556`	`556`	`await _distributedCacheClient.RemoveAsync(key);`
`557`	`557`
`558`		`- await Assert.ThrowsExceptionAsync<ArgumentOutOfRangeException>(async ()`
	`558`	`+ await Assert.ThrowsExceptionAsync<MasaArgumentException>(async ()`
`559`	`559`	`=> await _distributedCacheClient.HashDecrementAsync(key, value));`
`560`	`560`	`}`
`561`	`561`
Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@ protected static RedisConfigurationOptions GetConfigurationOptions()`
`13`	`13`	`{`
`14`	`14`	`GlobalCacheOptions = new CacheOptions()`
`15`	`15`	`{`
`16`		`- CacheKeyType = CacheKeyType.TypeName`
	`16`	`+ CacheKeyType = CacheKeyType.None`
`17`	`17`	`}`
`18`	`18`	`};`
`19`	`19`	`redisConfigurationOptions.Servers.Add(new RedisServerOptions());`
Original file line number	Diff line number	Diff line change
`@@ -524,7 +524,7 @@ public void TestComplementAndCheckDccConfigurationOptionByCustomerDccOptions()`
`524`	`524`	`public void TestComplementAndCheckDccConfigurationOptionByManageServiceAddressIsEmpty()`
`525`	`525`	`{`
`526`	`526`	`DccOptions dccOptions = new DccOptions();`
`527`		`- Assert.ThrowsException<ArgumentNullException>(() =>`
	`527`	`+ Assert.ThrowsException<MasaArgumentException>(() =>`
`528`	`528`	`{`
`529`	`529`	`MasaConfigurationExtensions.ComplementAndCheckDccConfigurationOption(_masaConfigurationBuilder.Object, dccOptions);`
`530`	`530`	`});`
`@@ -537,7 +537,7 @@ public void TestComplementAndCheckDccConfigurationOptionByRedisServersIsEmpty()`
`537`	`537`	`{`
`538`	`538`	`ManageServiceAddress = nameof(DccOptions.ManageServiceAddress),`
`539`	`539`	`};`
`540`		`- Assert.ThrowsException<ArgumentException>(() =>`
	`540`	`+ Assert.ThrowsException<MasaArgumentException>(() =>`
`541`	`541`	`{`
`542`	`542`	`MasaConfigurationExtensions.ComplementAndCheckDccConfigurationOption(_masaConfigurationBuilder.Object, dccOptions);`
`543`	`543`	`});`
`@@ -599,7 +599,7 @@ public void TestComplementAndCheckDccConfigurationOptionByAppIdIsEmpty()`
`599`	`599`	`}`
`600`	`600`	`}`
`601`	`601`	`};`
`602`		`- Assert.ThrowsException<ArgumentException>(() =>`
	`602`	`+ Assert.ThrowsException<MasaArgumentException>(() =>`
`603`	`603`	`{`
`604`	`604`	`MasaConfigurationExtensions.ComplementAndCheckDccConfigurationOption(_masaConfigurationBuilder.Object, dccOptions);`
`605`	`605`	`});`
Original file line number	Diff line number	Diff line change
`@@ -44,11 +44,11 @@ public void TestDispatcherOption()`
`44`	`44`	`var services = new ServiceCollection();`
`45`	`45`	`DispatcherOptions options;`
`46`	`46`
`47`		`- Assert.ThrowsException<ArgumentException>(() =>`
	`47`	`+ Assert.ThrowsException<MasaArgumentException>(() =>`
`48`	`48`	`{`
`49`	`49`	`options = new DispatcherOptions(services, null!);`
`50`	`50`	`});`
`51`		`- Assert.ThrowsException<ArgumentException>(() =>`
	`51`	`+ Assert.ThrowsException<MasaArgumentException>(() =>`
`52`	`52`	`{`
`53`	`53`	`options = new DispatcherOptions(services, Array.Empty<Assembly>());`
`54`	`54`	`});`
Original file line number	Diff line number	Diff line change
`@@ -49,11 +49,11 @@ public void TestDispatcherOption()`
`49`	`49`	`var services = new ServiceCollection();`
`50`	`50`	`DispatcherOptions options;`
`51`	`51`
`52`		`- Assert.ThrowsException<ArgumentException>(() =>`
	`52`	`+ Assert.ThrowsException<MasaArgumentException>(() =>`
`53`	`53`	`{`
`54`	`54`	`options = new DispatcherOptions(services, null!);`
`55`	`55`	`});`
`56`		`- Assert.ThrowsException<ArgumentException>(() =>`
	`56`	`+ Assert.ThrowsException<MasaArgumentException>(() =>`
`57`	`57`	`{`
`58`	`58`	`options = new DispatcherOptions(services, Array.Empty<Assembly>());`
`59`	`59`	`});`
Original file line number	Diff line number	Diff line change
`@@ -24,7 +24,7 @@ public void TestAddNullAssembly()`
`24`	`24`	`{`
`25`	`25`	`var services = new ServiceCollection();`
`26`	`26`	`services.AddLogging(loggingBuilder => loggingBuilder.AddConsole());`
`27`		`- Assert.ThrowsException<ArgumentException>(() =>`
	`27`	`+ Assert.ThrowsException<MasaArgumentException>(() =>`
`28`	`28`	`{`
`29`	`29`	`Assembly[] assemblies = null!;`
`30`	`30`	`services.AddEventBus(assemblies!);`
`@@ -36,7 +36,7 @@ public void TestAddEmptyAssembly()`
`36`	`36`	`{`
`37`	`37`	`var services = new ServiceCollection();`
`38`	`38`	`services.AddLogging(loggingBuilder => loggingBuilder.AddConsole());`
`39`		`- Assert.ThrowsException<ArgumentException>(() =>`
	`39`	`+ Assert.ThrowsException<MasaArgumentException>(() =>`
`40`	`40`	`{`
`41`	`41`	`services.AddEventBus(Array.Empty<Assembly>());`
`42`	`42`	`});`
`@@ -47,7 +47,7 @@ public void TestEventBusByAddNullAssembly()`
`47`	`47`	`{`
`48`	`48`	`var services = new ServiceCollection();`
`49`	`49`	`services.AddLogging(loggingBuilder => loggingBuilder.AddConsole());`
`50`		`- Assert.ThrowsException<ArgumentException>(() =>`
	`50`	`+ Assert.ThrowsException<MasaArgumentException>(() =>`
`51`	`51`	`{`
`52`	`52`	`services.AddTestEventBus(null!, ServiceLifetime.Scoped);`
`53`	`53`	`});`
Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,7 @@ public void TestAddAliyunStorageAndNotAddConfigurationReturnClientIsNotNull()`
`19`	`19`	`public void TestAddAliyunStorageByEmptySectionReturnThrowArgumentException()`
`20`	`20`	`{`
`21`	`21`	`var services = new ServiceCollection();`
`22`		`- Assert.ThrowsException<ArgumentException>(() => services.AddAliyunStorage(string.Empty));`
	`22`	`+ Assert.ThrowsException<MasaArgumentException>(() => services.AddAliyunStorage(string.Empty));`
`23`	`23`	`}`
`24`	`24`
`25`	`25`	`[TestMethod]`
`@@ -226,7 +226,7 @@ public void TestAddAliyunStorageAndNullALiYunStorageOptionsReturnThrowArgumentNu`
`226`	`226`	`[TestMethod]`
`227`	`227`	`public void TestAddAliyunStorageByEmptyAccessKeyIdReturnThrowArgumentNullException()`
`228`	`228`	`{`
`229`		`- Assert.ThrowsException<ArgumentException>(() => new AliyunStorageOptions(null!, null!, null!));`
	`229`	`+ Assert.ThrowsException<MasaArgumentException>(() => new AliyunStorageOptions(null!, null!, null!));`
`230`	`230`	`}`
`231`	`231`
`232`	`232`	`[TestMethod]`