♻️ Use createContentDigest helper

charlyx · Mar 31, 2019 · 8b10d38 · 8b10d38
1 parent 7b5a6b8
commit 8b10d38
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 8 deletions.
diff --git a/packages/gatsby-transformer-toml/src/__tests__/__snapshots__/gatsby-node.js.snap b/packages/gatsby-transformer-toml/src/__tests__/__snapshots__/gatsby-node.js.snap
@@ -7,7 +7,7 @@ Array [
       "children": Array [],
       "id": "uuid-from-gatsby",
       "internal": Object {
-        "contentDigest": "9ce7919cb3f607a0542bac4aa46136b6",
+        "contentDigest": "contentDigest",
         "type": "",
       },
       "parent": "whatever",
@@ -45,7 +45,7 @@ Array [
         "children": Array [],
         "id": "uuid-from-gatsby",
         "internal": Object {
-          "contentDigest": "9ce7919cb3f607a0542bac4aa46136b6",
+          "contentDigest": "contentDigest",
           "type": "",
         },
         "parent": "whatever",

diff --git a/packages/gatsby-transformer-toml/src/__tests__/gatsby-node.js b/packages/gatsby-transformer-toml/src/__tests__/gatsby-node.js
@@ -42,17 +42,20 @@ describe(`Process TOML nodes correctly`, () => {
     const actions = { createNode, createParentChildLink }
     const createNodeId = jest.fn()
     createNodeId.mockReturnValue(`uuid-from-gatsby`)
+    const createDigestContent = jest.fn().mockReturnValue(`contentDigest`)
 
     await onCreateNode({
       node,
       loadNodeContent,
       actions,
       createNodeId,
+      createDigestContent,
     }).then(() => {
       expect(createNode.mock.calls).toMatchSnapshot()
       expect(createParentChildLink.mock.calls).toMatchSnapshot()
       expect(createNode).toHaveBeenCalledTimes(1)
       expect(createParentChildLink).toHaveBeenCalledTimes(1)
+      expect(createDigestContent).toHaveBeenCalledTimes(1)
     })
   })
 

diff --git a/packages/gatsby-transformer-toml/src/gatsby-node.js b/packages/gatsby-transformer-toml/src/gatsby-node.js
@@ -1,8 +1,13 @@
 const toml = require(`toml`)
 const _ = require(`lodash`)
-const crypto = require(`crypto`)
 
-async function onCreateNode({ node, actions, loadNodeContent, createNodeId }) {
+async function onCreateNode({
+  node,
+  actions,
+  loadNodeContent,
+  createNodeId,
+  createDigestContent,
+}) {
   const { createNode, createParentChildLink } = actions
   // Filter out non-toml content
   // Currently TOML files are considered null in 'mime-db'
@@ -19,10 +24,7 @@ async function onCreateNode({ node, actions, loadNodeContent, createNodeId }) {
   // 1) More TOML files -> more types
   // 2) Different files with the same name creating conflicts
   const parsedContentStr = JSON.stringify(parsedContent)
-  const contentDigest = crypto
-    .createHash(`md5`)
-    .update(parsedContentStr)
-    .digest(`hex`)
+  const contentDigest = createDigestContent(parsedContentStr)
 
   const newNode = {
     ...parsedContent,