Fixing a bug which caused wrong directory names in archive

Because the asset paths were not normalized, in some cases directories
called '.' were inside the WAR archive

Author: DevWurm

src/WebpackWarPlugin.spec.ts

@@ -144,6 +144,36 @@ describe('WebpackWarPlugin', function () {
         assert.calledWith(spyAppend, resolve((compiler as any).options.output.path, asset), { name: asset }));
     });
 
+    it('Should normalize asset paths', function () {
+      const ArchiverDummy = {
+        append: (() => null),
+        pipe: (() => null),
+        on: (() => null),
+        finalize: (() => null)
+      };
+      const spyAppend = spy(ArchiverDummy, 'append');
+
+      const assets = {
+        './asset1.txt': {},
+        './assets/./asset2.txt': {}
+      };
+
+      compilation.assets = assets;
+
+      const plugin = new WebpackWarPlugin({ archiveName: 'Test' });
+      (plugin as any).archiver = (() => ArchiverDummy);
+      plugin.apply(compiler);
+
+      assert.calledWith(spyAppend,
+        resolve((compiler as any).options.output.path, Object.getOwnPropertyNames(assets)[0]),
+        { name: 'asset1.txt' }
+      );
+      assert.calledWith(spyAppend,
+        resolve((compiler as any).options.output.path, Object.getOwnPropertyNames(assets)[1]),
+        { name: 'assets/asset2.txt' }
+      );
+    });
+
     it('Should add the WEB-INF folder to the archive', function () {
       const ArchiverStub = {
         append: (() => null),

src/WebpackWarPlugin.ts

@@ -42,7 +42,7 @@ export class WebpackWarPlugin implements Plugin {
 
       Object.getOwnPropertyNames(compilation.assets).forEach(asset => {
         const srcPath = resolve(outputPath, asset);
-        archive.append(srcPath, { name: asset });
+        archive.append(srcPath, { name: normalize(asset) });
       });
 
       additionalElements.forEach(({ path, destPath }) => {