reexporting + renaming when scopehoisting (#2491)

packages/core/integration-tests/test/integration/scope-hoisting/es6/re-export-renamed/a.js

@@ -0,0 +1,3 @@
+import { x } from './b.js';
+
+output = x;

packages/core/integration-tests/test/integration/scope-hoisting/es6/re-export-renamed/b.js

@@ -0,0 +1,3 @@
+export const x = 'foobar';
+
+export { x as y } from './c.js'

packages/core/integration-tests/test/integration/scope-hoisting/es6/re-export-renamed/c.js

@@ -0,0 +1 @@
+export const x = 'xyz';

packages/core/integration-tests/test/scope-hoisting.js

@@ -321,6 +321,18 @@ describe('scope hoisting', function() {
       assert.deepEqual(output, 'foobar');
     });
 
+    it('supports requiring a re-exported and renamed ES6 import', async function() {
+      let b = await bundle(
+        path.join(
+          __dirname,
+          '/integration/scope-hoisting/es6/re-export-renamed/a.js'
+        )
+      );
+
+      let output = await run(b);
+      assert.deepEqual(output, 'foobar');
+    });
+
     it('keeps side effects by default', async function() {
       let b = await bundle(
         path.join(

packages/core/parcel-bundler/src/scope-hoisting/renamer.js

@@ -1,3 +1,5 @@
+const t = require('@babel/types');
+
 function rename(scope, oldName, newName) {
   if (oldName === newName) {
     return;
@@ -19,6 +21,9 @@ function rename(scope, oldName, newName) {
 
   // Rename all references
   for (let path of binding.referencePaths) {
+    if (t.isExportSpecifier(path.parent) && path.parentPath.parent.source) {
+      continue;
+    }
     if (path.node.name === oldName) {
       path.node.name = newName;
     }