fix(graphql): handle parentheses in fragment import file paths. (#1746)

fix(graphql): handle parentheses in fragment import file paths

Author: grxy

packages/graphql/src/toESModules.js

@@ -29,15 +29,13 @@ function replaceRequires(source) {
   let index = 0;
 
   // replace a require statement with a variable
-  const replaceSource = source.replace(/require\(([^)]+)\)/gi, (match, path) => {
-    const replacePath = path.replace(/["']+/g, '');
-
-    if (!imports[replacePath]) {
+  const replaceSource = source.replace(/require\(["']([^"']+)["']\)/gi, (match, path) => {
+    if (!imports[path]) {
       index += 1;
-      imports[replacePath] = `frgmt${index}`;
+      imports[path] = `frgmt${index}`;
     }
 
-    return imports[replacePath];
+    return imports[path];
   });
 
   // prepare import statements

packages/graphql/test/fixtures/fragments-with-special-characters/(parentheses)/fragment.graphql

@@ -0,0 +1,3 @@
+fragment ParenthesesFragment on Parentheses {
+  id
+}

packages/graphql/test/fixtures/fragments-with-special-characters/[brackets]/fragment.graphql

@@ -0,0 +1,3 @@
+fragment BracketsFragment on Brackets {
+  id
+}

packages/graphql/test/fixtures/fragments-with-special-characters/index.js

@@ -0,0 +1,2 @@
+// eslint-disable-next-line import/prefer-default-export
+export { default as doc } from './query.graphql';

packages/graphql/test/fixtures/fragments-with-special-characters/query.graphql

@@ -0,0 +1,7 @@
+#import "./(parentheses)/fragment.graphql"
+#import "./[brackets]/fragment.graphql"
+
+query Query {
+  ...ParenthesesFragment
+  ...BracketsFragment
+}

packages/graphql/test/test.js

@@ -62,3 +62,15 @@ test('should support graphqls schema files', async (t) => {
   t.truthy('doc' in module.exports);
   t.is(module.exports.doc.kind, 'Document');
 });
+
+test('should support fragment imports with brackets and parentheses in file paths', async (t) => {
+  const bundle = await rollup({
+    input: 'fixtures/fragments-with-special-characters/index.js',
+    plugins: [graphql()]
+  });
+
+  const { module } = await testBundle(t, bundle);
+
+  t.truthy('doc' in module.exports);
+  t.is(module.exports.doc.kind, 'Document');
+});