Skip to content

Commit ec016d9

Browse files
committed
fix: update ConfiginfoTests for optional InstallPath validation
Validate_InstallPathNullOrWhitespace_ThrowsArgumentException → Validate_InstallPathNullOrWhitespace_NoException InstallPath is no longer required in Validate(); it defaults to AppDomain.CurrentDomain.BaseDirectory via BaseConfigInfo.
1 parent 9a92293 commit ec016d9

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

tests/CoreTest/Configuration/ConfiginfoTests.cs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -167,7 +167,7 @@ public void Validate_ClientVersionNullOrWhitespace_ThrowsArgumentException(strin
167167
[InlineData(null)]
168168
[InlineData("")]
169169
[InlineData(" ")]
170-
public void Validate_InstallPathNullOrWhitespace_ThrowsArgumentException(string installPath)
170+
public void Validate_InstallPathNullOrWhitespace_NoException(string installPath)
171171
{
172172
var config = new Configinfo
173173
{
@@ -178,7 +178,8 @@ public void Validate_InstallPathNullOrWhitespace_ThrowsArgumentException(string
178178
ClientVersion = "1.0.0",
179179
InstallPath = installPath
180180
};
181-
Assert.Throws<ArgumentException>(() => config.Validate());
181+
var exception = Record.Exception(() => config.Validate());
182+
Assert.Null(exception);
182183
}
183184

184185
[Fact]

0 commit comments

Comments
 (0)