From 6cac9794fd325b6431ba208b8985859fdb6e0282 Mon Sep 17 00:00:00 2001
From: Kaito Sugimoto <hellorusk.github@gmail.com>
Date: Thu, 16 Jul 2020 04:37:17 +0900
Subject: [PATCH] fix: quote component name in error message (#1136)

* quote component name in error message

* revert unnecessary changes in readme
---
 docs/advanced/components.mdx                        | 4 ++--
 packages/mdx/mdx-hast-to-jsx.js                     | 2 +-
 packages/mdx/test/index.test.js                     | 6 +++---
 packages/remark-mdx/test/__snapshots__/test.js.snap | 2 +-
 4 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/docs/advanced/components.mdx b/docs/advanced/components.mdx
index a6f475919..2b8696d87 100644
--- a/docs/advanced/components.mdx
+++ b/docs/advanced/components.mdx
@@ -68,9 +68,9 @@ stubbed `Button`:
 const makeShortcode = name =>
   function MDXDefaultShortcode(props) {
     console.warn(
-      'Component ' +
+      "Component '" +
         name +
-        ' was not imported, exported, or provided by MDXProvider as global scope'
+        "' was not imported, exported, or provided by MDXProvider as global scope"
     )
     return <div {...props} />
   }
diff --git a/packages/mdx/mdx-hast-to-jsx.js b/packages/mdx/mdx-hast-to-jsx.js
index 7b8d75979..2b936d52e 100644
--- a/packages/mdx/mdx-hast-to-jsx.js
+++ b/packages/mdx/mdx-hast-to-jsx.js
@@ -163,7 +163,7 @@ MDXContent.isMDXComponent = true`
     const importExportNames = importNames.concat(exportNames)
     const fakedModulesForGlobalScope =
       `const makeShortcode = name => function MDXDefaultShortcode(props) {
-  console.warn("Component " + name + " was not imported, exported, or provided by MDXProvider as global scope")
+  console.warn("Component '" + name + "' was not imported, exported, or provided by MDXProvider as global scope")
   return <div {...props}/>
 };
 ` +
diff --git a/packages/mdx/test/index.test.js b/packages/mdx/test/index.test.js
index a5a1241d1..9aeea5a45 100644
--- a/packages/mdx/test/index.test.js
+++ b/packages/mdx/test/index.test.js
@@ -47,9 +47,9 @@ it('Should match sample blog post snapshot', async () => {
     const makeShortcode = (name) =>
       function MDXDefaultShortcode(props) {
         console.warn(
-          \\"Component \\" +
+          \\"Component '\\" +
             name +
-            \\" was not imported, exported, or provided by MDXProvider as global scope\\"
+            \\"' was not imported, exported, or provided by MDXProvider as global scope\\"
         );
         return <div {...props} />;
       };
@@ -333,7 +333,7 @@ test('Should handle layout props', () => {
       authors: ['fred', 'sally']
     };
     const makeShortcode = name => function MDXDefaultShortcode(props) {
-      console.warn(\\"Component \\" + name + \\" was not imported, exported, or provided by MDXProvider as global scope\\")
+      console.warn(\\"Component '\\" + name + \\"' was not imported, exported, or provided by MDXProvider as global scope\\")
       return <div {...props}/>
     };
     const Foo = makeShortcode(\\"Foo\\");
diff --git a/packages/remark-mdx/test/__snapshots__/test.js.snap b/packages/remark-mdx/test/__snapshots__/test.js.snap
index 83736e754..3dd0b2012 100644
--- a/packages/remark-mdx/test/__snapshots__/test.js.snap
+++ b/packages/remark-mdx/test/__snapshots__/test.js.snap
@@ -5,7 +5,7 @@ exports[`correctly transpiles 1`] = `
 
 export { Baz } from './foo';
 const makeShortcode = name => function MDXDefaultShortcode(props) {
-  console.warn(\\"Component \\" + name + \\" was not imported, exported, or provided by MDXProvider as global scope\\")
+  console.warn(\\"Component '\\" + name + \\"' was not imported, exported, or provided by MDXProvider as global scope\\")
   return <div {...props}/>
 };
 const Baz = makeShortcode(\\"Baz\\");