Fix generating names when outside of the entry directory (#1206)

Author: fathyb

src/Bundle.js

@@ -142,7 +142,7 @@ class Bundle {
           Path.dirname(entryAsset.name)
         ),
         name + ext
-      );
+      ).replace(/\.\.(\/|\\)/g, '__$1');
     }
 
     // If this is an index file, use the parent directory name instead

test/integration/service-worker/a/index.js

@@ -0,0 +1,3 @@
+import '../b/nested'
+
+navigator.serviceWorker.register('../b/worker-outside.js');

test/integration/service-worker/b/nested/index.js

@@ -0,0 +1 @@
+navigator.serviceWorker.register('worker-nested.js', { scope: './' });

test/integration/service-worker/worker.js …service-worker/b/nested/worker-nested.jstest/integration/service-worker/worker.js renamed to test/integration/service-worker/b/nested/worker-nested.js test/integration/service-worker/worker.js renamed to test/integration/service-worker/b/nested/worker-nested.js

@@ -0,0 +1 @@
+self.addEventListener('message', () => {});

test/integration/service-worker/b/worker-outside.js

@@ -220,6 +220,36 @@ describe('javascript', function() {
     });
   });
 
+  it('should support bundling service-workers', async function() {
+    let b = await bundle(__dirname + '/integration/service-worker/a/index.js');
+
+    assertBundleTree(b, {
+      name: 'index.js',
+      assets: ['index.js', 'index.js'],
+      childBundles: [
+        {
+          type: 'map'
+        },
+        {
+          assets: ['worker-nested.js'],
+          childBundles: [
+            {
+              type: 'map'
+            }
+          ]
+        },
+        {
+          assets: ['worker-outside.js'],
+          childBundles: [
+            {
+              type: 'map'
+            }
+          ]
+        }
+      ]
+    });
+  });
+
   it('should dynamic import files which import raw files', async function() {
     let b = await bundle(
       __dirname + '/integration/dynamic-references-raw/index.js'