Fixed #3 when trying to output 0 bytes as human readable.

- Renamed ToSizeSuffix() to ToSizeWithSuffix()

Author: bounav

Source/SqlPrune.Test/Int64ExetensionsTest.cs

@@ -0,0 +1,40 @@
+using NUnit.Framework;
+
+namespace Comsec.SqlPrune
+{
+    [TestFixture]
+    public class Int64ExtensionsTest
+    {
+        [Test]
+        public void TestToSizeWithSuffixWhenZero()
+        {
+            const long value = 0;
+
+            Assert.AreEqual("0", value.ToSizeWithSuffix());
+        }
+
+        [Test]
+        public void TestToSizeWithSuffixWhenTwoDigitsLong()
+        {
+            const long value = 12;
+
+            Assert.AreEqual("12.0 bytes", value.ToSizeWithSuffix());
+        }
+
+        [Test]
+        public void TestToSizeWithSuffixWhenThreeDigitsLong()
+        {
+            const long value = 354;
+
+            Assert.AreEqual("354.0 bytes", value.ToSizeWithSuffix());
+        }
+
+        [Test]
+        public void TestToSizeWithSuffixWhenFourDigitsLong()
+        {
+            const long value = 2400;
+
+            Assert.AreEqual("2.3 KB", value.ToSizeWithSuffix());
+        }
+    }
+}

Source/SqlPrune.Test/SqlPrune.Test.csproj

@@ -57,6 +57,7 @@
     <Compile Include="AutoMockingTest.cs" />
     <Compile Include="BakFilenameExtractorTest.cs" />
     <Compile Include="DateTimeExtensions.cs" />
+    <Compile Include="Int64ExetensionsTest.cs" />
     <Compile Include="Models\CalendarModel.cs" />
     <Compile Include="Commands\PruneCommandTest.cs" />
     <Compile Include="Factories\BakFileListFactory.cs" />

Source/SqlPrune/Commands/PruneCommand.cs

@@ -237,12 +237,12 @@ public override int Execute(Options options)
             if (totalBytes > 0)
             {
                 ColorConsole.Write(ConsoleColor.DarkGreen, "              Kept");
-                Console.WriteLine(": {0,19:N0} ({1})", totalKept, totalKept.ToSizeSuffix());
+                Console.WriteLine(": {0,19:N0} ({1})", totalKept, totalKept.ToSizeWithSuffix());
 
                 ColorConsole.Write(ConsoleColor.DarkRed, "            Pruned");
-                Console.WriteLine(": {0,19:N0} ({1})", totalPruned, totalPruned.ToSizeSuffix());
+                Console.WriteLine(": {0,19:N0} ({1})", totalPruned, totalPruned.ToSizeWithSuffix());
 
-                Console.WriteLine("             Total: {0,19:N0} ({1})", totalBytes, totalBytes.ToSizeSuffix());
+                Console.WriteLine("             Total: {0,19:N0} ({1})", totalBytes, totalBytes.ToSizeWithSuffix());
             }
 
             return (int) ExitCode.Success;

Source/SqlPrune/Int64Extensions.cs

@@ -17,8 +17,13 @@ public static class Int64Extensions
         /// <remarks>
         /// JLRishe's implementation from http://stackoverflow.com/questions/14488796/does-net-provide-an-easy-way-convert-bytes-to-kb-mb-gb-etc
         /// </remarks>
-        public static string ToSizeSuffix(this Int64 value)
+        public static string ToSizeWithSuffix(this Int64 value)
         {
+            if (value == 0)
+            {
+                return "0";
+            }
+
             var mag = (int) Math.Log(value, 1024);
 
             var adjustedSize = (decimal) value/(1L << (mag*10));