Rename addChildNodeToParentNode to addNodeToParent

gatsbyjs · May 25, 2017 · 62bbd99 · 62bbd99
1 parent b8317cf
commit 62bbd99
Show file tree

Hide file tree

Showing 11 changed files with 36 additions and 36 deletions.
diff --git a/packages/gatsby-transformer-json/src/__tests__/gatsby-node.js b/packages/gatsby-transformer-json/src/__tests__/gatsby-node.js
@@ -26,18 +26,18 @@ describe(`Process JSON nodes correctly`, () => {
     node.content = JSON.stringify(data)
 
     const createNode = jest.fn()
-    const addChildNodeToParentNode = jest.fn()
-    const boundActionCreators = { createNode, addChildNodeToParentNode }
+    const addNodeToParent = jest.fn()
+    const boundActionCreators = { createNode, addNodeToParent }
 
     await onNodeCreate({
       node,
       loadNodeContent,
       boundActionCreators,
     }).then(() => {
       expect(createNode.mock.calls).toMatchSnapshot()
-      expect(addChildNodeToParentNode.mock.calls).toMatchSnapshot()
+      expect(addNodeToParent.mock.calls).toMatchSnapshot()
       expect(createNode).toHaveBeenCalledTimes(2)
-      expect(addChildNodeToParentNode).toHaveBeenCalledTimes(1)
+      expect(addNodeToParent).toHaveBeenCalledTimes(1)
     })
   })
 
@@ -49,8 +49,8 @@ describe(`Process JSON nodes correctly`, () => {
     node.content = JSON.stringify(data)
 
     const createNode = jest.fn()
-    const addChildNodeToParentNode = jest.fn()
-    const boundActionCreators = { createNode, addChildNodeToParentNode }
+    const addNodeToParent = jest.fn()
+    const boundActionCreators = { createNode, addNodeToParent }
 
     await onNodeCreate({
       node,
@@ -71,8 +71,8 @@ describe(`Process JSON nodes correctly`, () => {
     node.content = JSON.stringify(data)
 
     const createNode = jest.fn()
-    const addChildNodeToParentNode = jest.fn()
-    const boundActionCreators = { createNode, addChildNodeToParentNode }
+    const addNodeToParent = jest.fn()
+    const boundActionCreators = { createNode, addNodeToParent }
 
     await onNodeCreate({
       node,

diff --git a/packages/gatsby-transformer-json/src/gatsby-node.js b/packages/gatsby-transformer-json/src/gatsby-node.js
@@ -5,7 +5,7 @@ const _ = require(`lodash`)
 const crypto = require(`crypto`)
 
 async function onNodeCreate({ node, boundActionCreators, loadNodeContent }) {
-  const { createNode, addChildNodeToParentNode } = boundActionCreators
+  const { createNode, addNodeToParent } = boundActionCreators
 
   // Don't reprocess our own nodes!  (note: this doesn't normally happen
   // but since this transformer creates new nodes with the same media-type
@@ -51,7 +51,7 @@ async function onNodeCreate({ node, boundActionCreators, loadNodeContent }) {
 
     _.each(JSONArray, j => {
       createNode(j)
-      addChildNodeToParentNode({ parent: node, child: j })
+      addNodeToParent({ parent: node, child: j })
     })
   }
 

diff --git a/packages/gatsby-transformer-react-docgen/src/__tests__/on-node-create.js b/packages/gatsby-transformer-react-docgen/src/__tests__/on-node-create.js
@@ -38,7 +38,7 @@ describe(`transformer-react-doc-gen: onNodeCreate`, () => {
     loadNodeContent = jest.fn(node => readFile(node.__fixture))
     boundActionCreators = {
       createNode: jest.fn(n => createdNodes.push(n)),
-      addChildNodeToParentNode: jest.fn(n => updatedNodes.push(n)),
+      addNodeToParent: jest.fn(n => updatedNodes.push(n)),
     }
   })
 

diff --git a/packages/gatsby-transformer-react-docgen/src/on-node-create.js b/packages/gatsby-transformer-react-docgen/src/on-node-create.js
@@ -65,7 +65,7 @@ export default function onNodeCreate(
   { node, loadNodeContent, boundActionCreators },
   pluginOptions
 ) {
-  const { createNode, addChildNodeToParentNode } = boundActionCreators
+  const { createNode, addNodeToParent } = boundActionCreators
 
   if (node.internal.mediaType !== `application/javascript`) return null
 
@@ -92,7 +92,7 @@ export default function onNodeCreate(
           },
         }
 
-        addChildNodeToParentNode({ parent: node, child: metadataNode })
+        addNodeToParent({ parent: node, child: metadataNode })
         metadataNode = createPropNodes(
           metadataNode,
           component,

diff --git a/packages/gatsby-transformer-remark/src/__tests__/gatsby-node.js b/packages/gatsby-transformer-remark/src/__tests__/gatsby-node.js
@@ -26,18 +26,18 @@ Where oh where is my little pony?
     node.content = content
 
     const createNode = jest.fn()
-    const addChildNodeToParentNode = jest.fn()
-    const boundActionCreators = { createNode, addChildNodeToParentNode }
+    const addNodeToParent = jest.fn()
+    const boundActionCreators = { createNode, addNodeToParent }
 
     await onNodeCreate({
       node,
       loadNodeContent,
       boundActionCreators,
     }).then(() => {
       expect(createNode.mock.calls).toMatchSnapshot()
-      expect(addChildNodeToParentNode.mock.calls).toMatchSnapshot()
+      expect(addNodeToParent.mock.calls).toMatchSnapshot()
       expect(createNode).toHaveBeenCalledTimes(1)
-      expect(addChildNodeToParentNode).toHaveBeenCalledTimes(1)
+      expect(addNodeToParent).toHaveBeenCalledTimes(1)
     })
   })
 })
diff --git a/packages/gatsby-transformer-remark/src/on-node-create.js b/packages/gatsby-transformer-remark/src/on-node-create.js
@@ -7,7 +7,7 @@ module.exports = async function onNodeCreate({
   loadNodeContent,
   boundActionCreators,
 }) {
-  const { createNode, addChildNodeToParentNode } = boundActionCreators
+  const { createNode, addNodeToParent } = boundActionCreators
 
   // Don't reprocess our own nodes!  (note: this doesn't normally happen
   // but since this transformer creates new nodes with the same media-type
@@ -51,5 +51,5 @@ module.exports = async function onNodeCreate({
   }
 
   createNode(markdownNode)
-  addChildNodeToParentNode({ parent: node, child: markdownNode })
+  addNodeToParent({ parent: node, child: markdownNode })
 }
diff --git a/packages/gatsby-transformer-sharp/src/__tests__/gatsby-node.js b/packages/gatsby-transformer-sharp/src/__tests__/gatsby-node.js
@@ -13,17 +13,17 @@ describe(`Process image nodes correctly`, () => {
       },
     }
     const createNode = jest.fn()
-    const addChildNodeToParentNode = jest.fn()
-    const boundActionCreators = { createNode, addChildNodeToParentNode }
+    const addNodeToParent = jest.fn()
+    const boundActionCreators = { createNode, addNodeToParent }
 
     await onNodeCreate({
       node,
       boundActionCreators,
     }).then(() => {
       expect(createNode.mock.calls).toMatchSnapshot()
-      expect(addChildNodeToParentNode.mock.calls).toMatchSnapshot()
+      expect(addNodeToParent.mock.calls).toMatchSnapshot()
       expect(createNode).toHaveBeenCalledTimes(1)
-      expect(addChildNodeToParentNode).toHaveBeenCalledTimes(1)
+      expect(addNodeToParent).toHaveBeenCalledTimes(1)
     })
   })
 })
diff --git a/packages/gatsby-transformer-sharp/src/on-node-create.js b/packages/gatsby-transformer-sharp/src/on-node-create.js
@@ -1,7 +1,7 @@
 const _ = require(`lodash`)
 
 module.exports = async function onNodeCreate({ node, boundActionCreators }) {
-  const { createNode, addChildNodeToParentNode } = boundActionCreators
+  const { createNode, addNodeToParent } = boundActionCreators
 
   const extensions = [`jpeg`, `jpg`, `png`, `webp`, `tif`, `tiff`, `svg`]
   if (!_.includes(extensions, node.extension)) {
@@ -20,7 +20,7 @@ module.exports = async function onNodeCreate({ node, boundActionCreators }) {
   }
 
   createNode(imageNode)
-  addChildNodeToParentNode({ parent: node, child: imageNode })
+  addNodeToParent({ parent: node, child: imageNode })
 
   return
 }
diff --git a/packages/gatsby-transformer-yaml/src/__tests__/gatsby-node.js b/packages/gatsby-transformer-yaml/src/__tests__/gatsby-node.js
@@ -24,18 +24,18 @@ describe(`Process YAML nodes correctly`, () => {
     node.content = yaml.safeDump(data)
 
     const createNode = jest.fn()
-    const addChildNodeToParentNode = jest.fn()
-    const boundActionCreators = { createNode, addChildNodeToParentNode }
+    const addNodeToParent = jest.fn()
+    const boundActionCreators = { createNode, addNodeToParent }
 
     await onNodeCreate({
       node,
       loadNodeContent,
       boundActionCreators,
     }).then(() => {
       expect(createNode.mock.calls).toMatchSnapshot()
-      expect(addChildNodeToParentNode.mock.calls).toMatchSnapshot()
+      expect(addNodeToParent.mock.calls).toMatchSnapshot()
       expect(createNode).toHaveBeenCalledTimes(2)
-      expect(addChildNodeToParentNode).toHaveBeenCalledTimes(1)
+      expect(addNodeToParent).toHaveBeenCalledTimes(1)
     })
   })
 
@@ -47,8 +47,8 @@ describe(`Process YAML nodes correctly`, () => {
     node.content = yaml.safeDump(data)
 
     const createNode = jest.fn()
-    const addChildNodeToParentNode = jest.fn()
-    const boundActionCreators = { createNode, addChildNodeToParentNode }
+    const addNodeToParent = jest.fn()
+    const boundActionCreators = { createNode, addNodeToParent }
 
     await onNodeCreate({
       node,
@@ -69,8 +69,8 @@ describe(`Process YAML nodes correctly`, () => {
     node.content = yaml.safeDump(data)
 
     const createNode = jest.fn()
-    const addChildNodeToParentNode = jest.fn()
-    const boundActionCreators = { createNode, addChildNodeToParentNode }
+    const addNodeToParent = jest.fn()
+    const boundActionCreators = { createNode, addNodeToParent }
 
     await onNodeCreate({
       node,

diff --git a/packages/gatsby-transformer-yaml/src/gatsby-node.js b/packages/gatsby-transformer-yaml/src/gatsby-node.js
@@ -6,7 +6,7 @@ const _ = require(`lodash`)
 const crypto = require(`crypto`)
 
 async function onNodeCreate({ node, boundActionCreators, loadNodeContent }) {
-  const { createNode, addChildNodeToParentNode } = boundActionCreators
+  const { createNode, addNodeToParent } = boundActionCreators
   if (node.internal.mediaType !== `text/yaml`) {
     return
   }
@@ -42,7 +42,7 @@ async function onNodeCreate({ node, boundActionCreators, loadNodeContent }) {
 
     _.each(yamlArray, y => {
       createNode(y)
-      addChildNodeToParentNode({ parent: node, child: y })
+      addNodeToParent({ parent: node, child: y })
     })
   }
 

diff --git a/packages/gatsby/src/redux/actions.js b/packages/gatsby/src/redux/actions.js
@@ -226,7 +226,7 @@ actions.addFieldToNode = ({ node, fieldName, fieldValue }, plugin) => {
   }
 }
 
-actions.addChildNodeToParentNode = ({ parent, child }, plugin) => {
+actions.addNodeToParent = ({ parent, child }, plugin) => {
   // Update parent
   parent.children.push(child.id)
   parent.children = _.uniq(parent.children)