Zip helper adds directories (#1525)

* Zip helper adds directories

* ZipHelpSpec rm obsolete comment

---------

Co-authored-by: Muki Seiler <muuki88@users.noreply.github.com>

Author: tsurovec

src/main/scala/com/typesafe/sbt/packager/universal/ZipHelper.scala

@@ -109,17 +109,19 @@ object ZipHelper {
       val outputDir = outputFile.getParentFile
       IO createDirectory outputDir
       withZipOutput(outputFile) { output =>
-        for (FileMapping(file, name, mode) <- sources; if !file.isDirectory) {
+        for (FileMapping(file, name, mode) <- sources) {
           val entry = new ZipArchiveEntry(file, normalizePath(name))
           // Now check to see if we have permissions for this sucker.
           mode foreach (entry.setUnixMode)
           output putArchiveEntry entry
-          val fis = new java.io.FileInputStream(file)
-          try try
-          // TODO - Write file into output?
-          IOUtils.copy(fis, output)
-          finally output.closeArchiveEntry()
-          finally fis.close()
+
+          try if (file.isFile) {
+            val fis = new java.io.FileInputStream(file)
+            try
+            // TODO - Write file into output?
+            IOUtils.copy(fis, output)
+            finally fis.close()
+          } finally output.closeArchiveEntry()
         }
       }
     }

src/test/scala/com/typesafe/sbt/packager/universal/ZipHelperSpec.scala

@@ -31,8 +31,7 @@ class ZipHelperSpec extends WordSpec with Matchers with BeforeAndAfterEach with
       zipSingleFile(ZipHelper.zip)
     }
 
-    // ignores empty directories
-    "create a zip with nested directories" taggedAs (LinuxTag, WindowsTag) ignore {
+    "create a zip with nested directories" taggedAs (LinuxTag, WindowsTag) in {
       zipNestedFile(ZipHelper.zip)
     }