From e78d8189e31dfa98281baabc3b1693bc1d095583 Mon Sep 17 00:00:00 2001 From: sakno Date: Sat, 18 Jan 2025 11:32:39 +0200 Subject: [PATCH] Fixed method names --- src/DotNext.Tests/IO/FileUriTests.cs | 14 ++++++++------ src/DotNext/IO/FileUri.cs | 18 +++++++++--------- 2 files changed, 17 insertions(+), 15 deletions(-) diff --git a/src/DotNext.Tests/IO/FileUriTests.cs b/src/DotNext.Tests/IO/FileUriTests.cs index ca4e3cca0..a50856f9f 100644 --- a/src/DotNext.Tests/IO/FileUriTests.cs +++ b/src/DotNext.Tests/IO/FileUriTests.cs @@ -30,15 +30,15 @@ public sealed class FileUriTests : Test [Theory] [MemberData(nameof(GetPaths))] - public static void EncodeAsUri(string fileName, string expected) + public static void FromFileNameToUri(string fileName, string expected) { - var uri = new Uri(FileUri.Encode(fileName), UriKind.Absolute); + var uri = new Uri(FileUri.CreateFromFileName(fileName), UriKind.Absolute); True(uri.IsFile); Equal(expected, uri.LocalPath); } [Fact] - public static void GetUriExtension() + public static void GetUriExtensionMethod() { var fileName = OperatingSystem.IsWindows() ? "C:\\some\\path" : "/some/path"; var uri = new FileInfo(fileName).GetUri(); @@ -47,10 +47,10 @@ public static void GetUriExtension() [Theory] [MemberData(nameof(GetPaths))] - public static void EncodeAsUriChars(string fileName, string expected) + public static void TryCreateFromFileName(string fileName, string expected) { Span buffer = stackalloc char[512]; - True(FileUri.TryEncode(fileName, UrlEncoder.Default, buffer, out var charsWritten)); + True(FileUri.TryCreateFromFileName(fileName, UrlEncoder.Default, buffer, out var charsWritten)); var uri = new Uri(buffer.Slice(0, charsWritten).ToString(), UriKind.Absolute); Equal(expected, uri.LocalPath); @@ -66,9 +66,11 @@ public static void MaxEncodedLength() [Theory] [InlineData("~/path/name")] [InlineData("C:path\\name")] + [InlineData("../path")] + [InlineData("./path")] [InlineData("")] public static void CheckFullyQualifiedPath(string path) { - Throws(() => FileUri.Encode(path)); + Throws(() => FileUri.CreateFromFileName(path)); } } \ No newline at end of file diff --git a/src/DotNext/IO/FileUri.cs b/src/DotNext/IO/FileUri.cs index 82b25b8ba..59b6dd7eb 100644 --- a/src/DotNext/IO/FileUri.cs +++ b/src/DotNext/IO/FileUri.cs @@ -28,28 +28,28 @@ public static class FileUri /// The encoding settings. /// as URI. The return value can be passed to constructor. /// is not fully-qualified. - public static string Encode(ReadOnlySpan fileName, TextEncoderSettings? settings = null) + public static string CreateFromFileName(ReadOnlySpan fileName, TextEncoderSettings? settings = null) { if (fileName.IsEmpty) throw new ArgumentException(ExceptionMessages.FullyQualifiedPathExpected, nameof(fileName)); - return EncodeCore(fileName, settings is null ? UrlEncoder.Default : UrlEncoder.Create(settings)); + return CreateFromFileNameCore(fileName, settings is null ? UrlEncoder.Default : UrlEncoder.Create(settings)); } - private static string EncodeCore(ReadOnlySpan fileName, UrlEncoder encoder) + private static string CreateFromFileNameCore(ReadOnlySpan fileName, UrlEncoder encoder) { var maxLength = GetMaxEncodedLengthCore(fileName, encoder); using var buffer = (uint)maxLength <= (uint)SpanOwner.StackallocThreshold ? stackalloc char[maxLength] : new SpanOwner(maxLength); - return TryEncodeCore(fileName, encoder, buffer.Span, out var writtenCount) + return TryCreateFromFileNameCore(fileName, encoder, buffer.Span, out var writtenCount) ? new(buffer.Span.Slice(0, writtenCount)) : string.Empty; } /// - /// Gets the maximum number of characters that can be produced by method. + /// Gets the maximum number of characters that can be produced by method. /// /// The file name to be encoded. /// The encoder. @@ -71,15 +71,15 @@ private static int GetMaxEncodedLengthCore(ReadOnlySpan fileName, UrlEncod /// The number of characters written to . /// if is encoded successfully; otherwise, . /// is not fully-qualified. - public static bool TryEncode(ReadOnlySpan fileName, UrlEncoder? encoder, Span output, out int charsWritten) + public static bool TryCreateFromFileName(ReadOnlySpan fileName, UrlEncoder? encoder, Span output, out int charsWritten) { if (fileName.IsEmpty) throw new ArgumentException(ExceptionMessages.FullyQualifiedPathExpected, nameof(fileName)); - return TryEncodeCore(fileName, encoder ?? UrlEncoder.Default, output, out charsWritten); + return TryCreateFromFileNameCore(fileName, encoder ?? UrlEncoder.Default, output, out charsWritten); } - private static bool TryEncodeCore(ReadOnlySpan fileName, UrlEncoder encoder, Span output, out int charsWritten) + private static bool TryCreateFromFileNameCore(ReadOnlySpan fileName, UrlEncoder encoder, Span output, out int charsWritten) { var writer = new SpanWriter(output); writer.Write(FileScheme); @@ -151,6 +151,6 @@ public static Uri GetUri(this FileSystemInfo fileInfo, TextEncoderSettings? sett { ArgumentNullException.ThrowIfNull(fileInfo); - return new(EncodeCore(fileInfo.FullName, settings is null ? UrlEncoder.Default : UrlEncoder.Create(settings)), UriKind.Absolute); + return new(CreateFromFileNameCore(fileInfo.FullName, settings is null ? UrlEncoder.Default : UrlEncoder.Create(settings)), UriKind.Absolute); } } \ No newline at end of file