fix: jsx pragma with automatic runtime

astroturfcss · Oct 21, 2020 · c18f201 · c18f201
1 parent 3d4ad74
commit c18f201
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 11 deletions.
diff --git a/src/features/css-prop.js b/src/features/css-prop.js
@@ -153,8 +153,9 @@ export default {
       addNamed(path, 'F', 'astroturf', { nameHint: jsxFrag.name });
 
       state.file.get(STYLES).changeset.unshift(
-        { code: `/*${jsxPrgama}*/\n` },
-        { code: `/*${jsxFragPrgama}*/\n\n` },
+        { code: `/* @jsxRuntime classic */\n` },
+        { code: `/${jsxPrgama}/\n` },
+        { code: `/${jsxFragPrgama}/\n\n` },
         {
           code: `const { jsx: ${jsx.name}, F: ${jsxFrag.name} } = require('astroturf');\n`,
         },

diff --git a/test/__file_snapshots__/integration-js.js b/test/__file_snapshots__/integration-js.js
@@ -279,9 +279,11 @@ __webpack_require__.r(__webpack_exports__);
 /* harmony import */ var react__WEBPACK_IMPORTED_MODULE_1___default = /*#__PURE__*/__webpack_require__.n(react__WEBPACK_IMPORTED_MODULE_1__);
 /* harmony import */ var widget__WEBPACK_IMPORTED_MODULE_2__ = __webpack_require__(/*! widget */ "./test/integration/shared/widget/index.js");
 /* harmony import */ var _Button__WEBPACK_IMPORTED_MODULE_3__ = __webpack_require__(/*! ./Button */ "./test/integration/Button.js");
-/** @jsx _j **/
+/* @jsxRuntime classic */
 
-/** @jsxFrag _f **/
+/* @jsx _j */
+
+/* @jsxFrag _f */
 const {
   jsx: _j,
   F: _f

diff --git a/test/__file_snapshots__/loader__css-prop.js b/test/__file_snapshots__/loader__css-prop.js
@@ -1,5 +1,6 @@
-/** @jsx _j **/
-/** @jsxFrag _f **/
+/* @jsxRuntime classic */
+/* @jsx _j */
+/* @jsxFrag _f */
 
 const { jsx: _j, F: _f } = require("astroturf");
 

diff --git a/test/__file_snapshots__/loader__typescript.tsx b/test/__file_snapshots__/loader__typescript.tsx
@@ -1,5 +1,6 @@
-/** @jsx _j **/
-/** @jsxFrag _f **/
+/* @jsxRuntime classic */
+/* @jsx _j */
+/* @jsxFrag _f */
 
 const { jsx: _j, F: _f } = require("astroturf");
 import styled from "astroturf";

diff --git a/test/loader.test.js b/test/loader.test.js
@@ -22,9 +22,8 @@ describe('webpack loader', () => {
     `,
       { enableCssProp: true },
     );
-
-    expect(code.includes('/** @jsx _j **/')).toBe(true);
-    expect(code.includes('/** @jsxFrag _f **/')).toBe(true);
+    expect(code.includes('/* @jsx _j */')).toBe(true);
+    expect(code.includes('/* @jsxFrag _f */')).toBe(true);
 
     expect(
       code.includes('[require("./MyStyleFile-CssProp1_button.css")'),