Add unit tests for StringExtensions.

Behavioral/DesignPatterns.Visitor.UnitTests/DesignPatterns.Visitor.UnitTests.csproj

@@ -11,6 +11,7 @@
 
     <ItemGroup>
         <PackageReference Include="AutoFixture" Version="4.18.1" />
+        <PackageReference Include="FluentAssertions" Version="6.12.0" />
         <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
         <PackageReference Include="Moq" Version="4.20.70" />
         <PackageReference Include="Shouldly" Version="4.2.1" />

Behavioral/DesignPatterns.Visitor.UnitTests/StringExtensionsTests.cs

@@ -0,0 +1,35 @@
+using DesignPatterns.Visitor.Models;
+using FluentAssertions;
+using FluentAssertions.Execution;
+using Moq;
+using Shouldly;
+using Xunit;
+
+namespace DesignPatterns.Visitor.UnitTests;
+
+public class StringExtensionsTests
+{
+    [Theory]
+    [MemberData(nameof(TestCases))]
+    public void GivenSize_WhenCallVisit_ThenResultAsExpected(
+        string value, ISize expectedSizeInKiloBytes, ISize expectedSizeInBytes)
+    {
+        var visitorMock = new Mock<IVisitor<string, ISize>>();
+        visitorMock.Setup(x => x.Visit(It.IsNotNull<string>())).Returns(expectedSizeInKiloBytes).Verifiable();
+
+        ISize? actualResult = null;
+        var action = () => actualResult = value.Accept(visitorMock.Object);
+
+        using (new AssertionScope())
+        {
+            action.ShouldNotThrow();
+            actualResult.ShouldBeEquivalentTo(expectedSizeInKiloBytes);
+            visitorMock.Verify(x => x.Visit(value), Times.Once);
+        }
+    }
+
+    public static IEnumerable<object[]> TestCases()
+    {
+        yield return [ "1KB", new SizeInKiloBytes { Value = 1 }, new SizeInBytes { Value = 102400 }];
+    }
+}

Behavioral/DesignPatterns.Visitor/Context.cs

@@ -1,30 +1,54 @@
-// using AutoFixture;
-//
-// namespace DesignPatterns.Visitor;
-//
-// public class Context
-// {
-//     private readonly IFixture fixture = new Fixture();
-//     private readonly Random random = new Random();
-//
-//     public (ICollection<IAmount> itemsInBytes, ICollection<IAmount> itemsInGB) GetData()
-//     {
-//         var inBytes = GetSizesInBytes().ToList();
-//         var inGB = GetSizesInGB().ToList();
-//         return (inBytes, inGB);
-//     }
-//
-//     private static IEnumerable<IAmount> GetSizesInBytes()
-//     {
-//         yield return new AmountInBytes { Value = 1073741824 };
-//         yield return new AmountInBytes { Value = 2147483648 };
-//         yield return new AmountInBytes { Value = 3221225472 };
-//     }
-//
-//     private static IEnumerable<IAmount> GetSizesInGB()
-//     {
-//         yield return new AmountInGB { Value = 10 };
-//         yield return new AmountInGB { Value = 20 };
-//         yield return new AmountInGB { Value = 30 };
-//     }
-// }
+using DesignPatterns.Visitor.Converters;
+using DesignPatterns.Visitor.Models;
+
+namespace DesignPatterns.Visitor;
+
+public class Context
+{
+    public IVisitor<string, ISize> GetStringVisitor() => new StringVisitor();
+
+    public IVisitor<ISize, ISize> GetSizeVisitor() => new SizeVisitor(
+        new List<ISizeConverter<ISize>>
+        {
+            new KiloBytesToBytesConverter(),
+            new MegaBytesToBytesConverter(),
+            new GigaBytesToBytesConverter(),
+            new TeraBytesToBytesConverter(),
+            new PetaBytesToBytesConverter(),
+        });
+
+    public (ICollection<string> strings, ICollection<ISize> sizes) GetData()
+    {
+        var strings = GetStringData().ToList();
+        var sizes = GetSizes().ToList();
+        return (strings, sizes);
+    }
+
+    private static IEnumerable<string> GetStringData()
+    {
+        yield return "1KB";
+        yield return "2 KB";
+        yield return "4MB";
+        yield return "8 MB";
+        yield return "16GB";
+        yield return "32 GB";
+        yield return "64TB";
+        yield return "128 TB";
+        yield return "256PB";
+        yield return "512 PB";
+    }
+
+    private static IEnumerable<ISize> GetSizes()
+    {
+        yield return new SizeInKiloBytes { Value = 1 };
+        yield return new SizeInKiloBytes { Value = 2 };
+        yield return new SizeInMegaBytes { Value = 4 };
+        yield return new SizeInMegaBytes { Value = 8 };
+        yield return new SizeInGigaBytes { Value = 16 };
+        yield return new SizeInGigaBytes { Value = 32 };
+        yield return new SizeInTeraBytes { Value = 64 };
+        yield return new SizeInTeraBytes { Value = 128 };
+        yield return new SizeInPetaBytes { Value = 256 };
+        yield return new SizeInPetaBytes { Value = 512 };
+    }
+}

Behavioral/DesignPatterns.Visitor/DesignPatterns.Visitor.csproj

@@ -9,4 +9,8 @@
         <RootNamespace>DesignPatterns.Visitor</RootNamespace>
     </PropertyGroup>
 
+    <ItemGroup>
+        <PackageReference Include="Shouldly" Version="4.2.1" />
+    </ItemGroup>
+
 </Project>

Behavioral/DesignPatterns.Visitor/Handler.cs

@@ -1,15 +1,30 @@
-// namespace DesignPatterns.Visitor;
-//
-// public class Handler(IEnumerable<IVisitor> visitors)
-// {
-//     public IEnumerable<IAmount> Process(ICollection<IAmount> sizes)
-//     {
-//         var result = new List<IAmount>();
-//         foreach (var visitor in visitors)
-//         {
-//             result.AddRange(sizes.Select(size => size.Accept(visitor)));
-//         }
-//
-//         return result;
-//     }
-// }
+namespace DesignPatterns.Visitor;
+
+public class Handler
+{
+    public IEnumerable<ISize> ProcessStrings(
+        ICollection<string> sizes,
+        IEnumerable<IVisitor<string, ISize>> visitors)
+    {
+        var result = new List<ISize>();
+        foreach (var visitor in visitors)
+        {
+            result.AddRange(sizes.Select(x => x.Accept(visitor)));
+        }
+
+        return result;
+    }
+
+    public IEnumerable<ISize> ProcessSizes(
+        ICollection<ISize> sizes,
+        IEnumerable<IVisitor<ISize, ISize>> visitors)
+    {
+        var result = new List<ISize>();
+        foreach (var visitor in visitors)
+        {
+            result.AddRange(sizes.Select(x => x.Accept(visitor)));
+        }
+
+        return result;
+    }
+}

Behavioral/DesignPatterns.Visitor/Models/Size.cs

@@ -9,4 +9,6 @@ public ISize Accept(IVisitor<ISize, ISize> visitor)
     {
         return visitor.Visit(this);
     }
+
+    public sealed override string ToString() => $"{this.Value.ToString()} {this.Unit}";
 }

Behavioral/DesignPatterns.Visitor/Program.cs

@@ -1,41 +1,15 @@
 using DesignPatterns.Visitor;
-using DesignPatterns.Visitor.Converters;
 
-// var context = new Context();
-// var discountVisitor = new SizeVisitor();
-// var handler = new Handler(new[] { discountVisitor });
-// var data = context.GetData();
-// var convertedInGB = handler.Process(data.itemsInBytes);
-// var convertedInBytes = handler.Process(data.itemsInGB);
-//
-// Console.WriteLine();
+var context = new Context();
+var handler = new Handler();
+var data = context.GetData();
+var sizes = handler.ProcessStrings(data.strings, [context.GetStringVisitor()]).ToList();
+var finalSizes = handler.ProcessSizes(sizes, [context.GetSizeVisitor()]).ToList();
 
-var strings = new List<string>
-{
-    "100KB",
-    "200 KB",
-    "10 MB",
-    "200MB",
-    "1GB",
-    "20 GB"
-};
-
-var visitor = new StringVisitor();
-
-var result = new List<ISize>();
-result.AddRange(strings.Select(x => x.Accept(visitor)));
+Console.WriteLine("| String Data | Sizes       | Final Sizes              |");
+Console.WriteLine("|-------------|-------------|--------------------------|");
 
-var amountVisitor = new SizeVisitor(
-    new List<ISizeConverter<ISize>>
-    {
-        new KiloBytesToBytesConverter(),
-        new MegaBytesToBytesConverter(),
-        new GigaBytesToBytesConverter(),
-        new TeraBytesToBytesConverter(),
-        new PetaBytesToBytesConverter(),
-    });
-
-var newResult = new List<ISize>();
-newResult.AddRange(result.Select(x => x.Accept(amountVisitor)));
-
-Console.WriteLine();
+for (var i = 0; i < data.strings.Count; i++)
+{
+    Console.WriteLine($"| {data.strings.ElementAt(i),-11} | {sizes.ElementAt(i),-11} | {finalSizes.ElementAt(i),-24} |");
+}