fixed formatting bug with CompressionRatio

Author: santisq

module/PSCompression.psd1

@@ -11,7 +11,7 @@
     RootModule         = 'bin/netstandard2.0/PSCompression.dll'
 
     # Version number of this module.
-    ModuleVersion      = '2.0.4'
+    ModuleVersion      = '2.0.5'
 
     # Supported PSEditions
     # CompatiblePSEditions = @()

src/PSCompression/ZipEntryFile.cs

@@ -25,7 +25,7 @@ private static string GetRatio(long size, long compressedSize)
 
         if (float.IsNaN(compressedRatio))
         {
-            return "0.00%";
+            compressedRatio = 0;
         }
 
         return string.Format("{0:F2}%", 100 - (compressedRatio * 100));

tests/ZipEntryBase.tests.ps1

@@ -47,6 +47,7 @@ Describe 'ZipEntryBase Class' {
     }
 
     It 'Has a CompressionRatio Property' {
+        New-ZipEntry $zip.FullName -EntryPath empty.txt
         ($zip | Get-ZipEntry).CompressionRatio | Should -BeOfType ([string])
     }
 

tests/ZipEntryCmdlets.tests.ps1

@@ -157,6 +157,11 @@ Describe 'ZipEntry Cmdlets' {
     }
 
     Context 'Remove-ZipEntry' -Tag 'Remove-ZipEntry' {
+        It 'No entry is removed with -WhatIf' {
+            $zip | Get-ZipEntry | Remove-ZipEntry -WhatIf
+            $zip | Get-ZipEntry | Should -Not -BeNullOrEmpty
+        }
+
         It 'Can remove file entries' {
             { $zip | Get-ZipEntry -Type Archive | Remove-ZipEntry } |
                 Should -Not -Throw
@@ -238,13 +243,21 @@ Describe 'ZipEntry Cmdlets' {
             $structure, $content | Out-Null
         }
 
+        It 'No entry is renamed with -WhatIf' {
+            $zip | Get-ZipEntry |
+                Rename-ZipEntry -NewName { 'test' + $_.Name } -WhatIf
+
+            $zip | Get-ZipEntry |
+                Should -Not -Match '^testtest'
+        }
+
         It 'Can rename file entries using a delay-bind ScriptBlock' {
             { $zip | Get-ZipEntry -Type Archive | Rename-ZipEntry -NewName { 'test' + $_.Name } } |
                 Should -Not -Throw
 
             $zip | Get-ZipEntry -Type Archive |
                 ForEach-Object Name |
-                Should -Match '^test'
+                Should -Match '^testtest'
         }
 
         It 'Produces output with -PassThru' {