Merge remote-tracking branch 'upstream/v2' into wbinnssmith/merge-v2-…

…2022-04-22

* upstream/v2:
  Register swc error handler (#8032)
  Correctly emit warnings for unnecessary PostCSS plugins in package.json (#8024)
  Pass targets for swc/helpers to Rust and only filter them for preset env (#8020)

packages/core/integration-tests/test/integration/js-export-arrow-support/bar/index.js

@@ -0,0 +1 @@
+export {bar} from "./other.js";

packages/core/integration-tests/test/integration/js-export-arrow-support/bar/other.js

@@ -0,0 +1 @@
+export const bar = "bar";

packages/core/integration-tests/test/integration/js-export-arrow-support/index.js

@@ -0,0 +1,5 @@
+import * as helpers from "@swc/helpers";
+import * as bar from "./bar";
+import * as foo from "foo";
+
+export default [helpers, bar, foo];

packages/core/integration-tests/test/integration/js-export-arrow-support/package.json

@@ -0,0 +1,3 @@
+{
+  "browserslist": "ie 11"
+}

packages/core/integration-tests/test/integration/postcss-modules-config-package/foo.css

@@ -0,0 +1,3 @@
+.foo {
+  color: #fff;
+}

packages/core/integration-tests/test/integration/postcss-modules-config-package/foo.js

@@ -0,0 +1,5 @@
+const map = require('./foo.css');
+
+module.exports = {
+  foo: map.foo,
+};

packages/core/integration-tests/test/integration/postcss-modules-config-package/index.css

@@ -0,0 +1,3 @@
+body {
+  background: blue;
+}

packages/core/integration-tests/test/integration/postcss-modules-config-package/index.js

@@ -0,0 +1,6 @@
+require('./index.css');
+var foo = require('./foo');
+
+module.exports = function () {
+  return foo.foo;
+};

packages/core/integration-tests/test/integration/postcss-modules-config-package/package.json

@@ -0,0 +1,10 @@
+{
+  "postcss": {
+    "modules": true,
+    "plugins": {
+      "autoprefixer": {
+        "grid": true
+      }
+    }
+  }
+}

packages/core/integration-tests/test/integration/transpilation-invalid/index.js

@@ -0,0 +1,9 @@
+const Boom = () => {
+  const littleBoom = ['hello', 'world']
+  return <div>{...littleBoom.map(el => el)}</div>
+}
+class X {
+  #x(){}
+  #x(){}
+}
+console.log(Boom, X);

packages/core/integration-tests/test/integration/transpilation-invalid/package.json

@@ -0,0 +1,6 @@
+{
+  "dependencies": {
+    "react": "^18.1.0"
+  },
+  "browserslist": "ie 11"
+}

packages/core/integration-tests/test/javascript.js

@@ -5109,6 +5109,14 @@ describe('javascript', function () {
     assert.deepEqual(res, {a: 4});
   });
 
+  it('should not use arrow functions for reexport declarations unless supported', async function () {
+    let b = await bundle(
+      path.join(__dirname, 'integration/js-export-arrow-support/index.js'),
+    );
+    let content = await outputFS.readFile(b.getBundles()[0].filePath, 'utf8');
+    assert(!content.includes('=>'));
+  });
+
   it('should support import namespace declarations of other ES modules', async function () {
     let b = await bundle(
       path.join(__dirname, 'integration/js-import-namespace/a.js'),

packages/core/integration-tests/test/postcss.js

@@ -46,6 +46,37 @@ describe('postcss', () => {
     assert(css.includes(`.${cssClass}`));
   });
 
+  it('should build successfully with only postcss-modules config in package.json', async () => {
+    let b = await bundle(
+      path.join(
+        __dirname,
+        '/integration/postcss-modules-config-package/index.js',
+      ),
+    );
+
+    assertBundles(b, [
+      {
+        name: 'index.js',
+        assets: ['foo.css', 'foo.js', 'index.css', 'index.js'],
+      },
+      {
+        name: 'index.css',
+        assets: ['foo.css', 'index.css'],
+      },
+    ]);
+
+    let output = await run(b);
+    assert.equal(typeof output, 'function');
+
+    let value = output();
+    assert(/foo_[0-9a-z]/.test(value));
+
+    let cssClass = value.match(/(foo_[0-9a-z])/)[1];
+
+    let css = await outputFS.readFile(path.join(distDir, 'index.css'), 'utf8');
+    assert(css.includes(`.${cssClass}`));
+  });
+
   it('should support transforming with postcss twice with the same result', async () => {
     let b = await bundle(
       path.join(__dirname, '/integration/postcss-plugins/index.js'),

packages/core/integration-tests/test/transpilation.js

@@ -335,6 +335,65 @@ describe('transpilation', function () {
     });
   });
 
+  it('should print errors from transpilation', async function () {
+    let source = path.join(
+      __dirname,
+      '/integration/transpilation-invalid/index.js',
+    );
+    // $FlowFixMe
+    await assert.rejects(() => bundle(source), {
+      name: 'BuildError',
+      diagnostics: [
+        {
+          codeFrames: [
+            {
+              codeHighlights: [
+                {
+                  message: null,
+                  start: {
+                    column: 15,
+                    line: 3,
+                  },
+                  end: {
+                    column: 43,
+                    line: 3,
+                  },
+                },
+              ],
+              filePath: source,
+            },
+          ],
+          hints: null,
+          message: 'Spread children are not supported in React.',
+          origin: '@parcel/transformer-js',
+        },
+        {
+          codeFrames: [
+            {
+              codeHighlights: [
+                {
+                  message: null,
+                  start: {
+                    column: 4,
+                    line: 7,
+                  },
+                  end: {
+                    column: 4,
+                    line: 7,
+                  },
+                },
+              ],
+              filePath: source,
+            },
+          ],
+          hints: null,
+          message: 'duplicate private name #x.',
+          origin: '@parcel/transformer-js',
+        },
+      ],
+    });
+  });
+
   describe('tests needing the real filesystem', () => {
     afterEach(async () => {
       if (process.platform === 'win32') {