parent
cc85a28ff7
commit
9127a91dfc
|
@ -380,8 +380,17 @@ namespace NzbDrone.Common.Test
|
||||||
[TestCase(@" C:\Test\TV\")]
|
[TestCase(@" C:\Test\TV\")]
|
||||||
[TestCase(@" C:\Test\TV")]
|
[TestCase(@" C:\Test\TV")]
|
||||||
|
|
||||||
public void IsPathValid_should_be_false(string path)
|
public void IsPathValid_should_be_false_on_windows(string path)
|
||||||
{
|
{
|
||||||
|
WindowsOnly();
|
||||||
|
path.IsPathValid(PathValidationType.CurrentOs).Should().BeFalse();
|
||||||
|
}
|
||||||
|
|
||||||
|
[TestCase(@"")]
|
||||||
|
[TestCase(@"relative/path")]
|
||||||
|
public void IsPathValid_should_be_false_on_unix(string path)
|
||||||
|
{
|
||||||
|
PosixOnly();
|
||||||
path.AsOsAgnostic().IsPathValid(PathValidationType.CurrentOs).Should().BeFalse();
|
path.AsOsAgnostic().IsPathValid(PathValidationType.CurrentOs).Should().BeFalse();
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -152,16 +152,20 @@ namespace NzbDrone.Common.Extensions
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
var directoryInfo = new DirectoryInfo(path);
|
// Only check for leading or trailing spaces for path when running on Windows.
|
||||||
|
if (OsInfo.IsWindows)
|
||||||
while (directoryInfo != null)
|
|
||||||
{
|
{
|
||||||
if (directoryInfo.Name.Trim() != directoryInfo.Name)
|
var directoryInfo = new DirectoryInfo(path);
|
||||||
{
|
|
||||||
return false;
|
|
||||||
}
|
|
||||||
|
|
||||||
directoryInfo = directoryInfo.Parent;
|
while (directoryInfo != null)
|
||||||
|
{
|
||||||
|
if (directoryInfo.Name.Trim() != directoryInfo.Name)
|
||||||
|
{
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
directoryInfo = directoryInfo.Parent;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if (validationType == PathValidationType.AnyOs)
|
if (validationType == PathValidationType.AnyOs)
|
||||||
|
|
Loading…
Reference in New Issue