From a57162349ef2f03faaad54f3b30e52e912bf3a8a Mon Sep 17 00:00:00 2001
From: Benjamin Stepp <me@benstepp.com>
Date: Tue, 12 Apr 2016 23:22:14 -0500
Subject: [PATCH] Integration test for the rewritePath function

* Also updates the `spawn` support function to reject a promise if given
a non-zero exit code.
---
 test/fixtures/path-rewrite/config.toml        |  0
 test/fixtures/path-rewrite/gatsby-node.js     |  7 +++++
 test/fixtures/path-rewrite/html.js            | 29 +++++++++++++++++++
 test/fixtures/path-rewrite/pages/_template.js | 15 ++++++++++
 .../path-rewrite/pages/dont-move-me.md        |  3 ++
 test/fixtures/path-rewrite/pages/move-me.md   |  3 ++
 test/integration/user-rewrites-path.js        | 29 +++++++++++++++++++
 test/support.js                               | 10 +++++--
 8 files changed, 94 insertions(+), 2 deletions(-)
 create mode 100644 test/fixtures/path-rewrite/config.toml
 create mode 100644 test/fixtures/path-rewrite/gatsby-node.js
 create mode 100644 test/fixtures/path-rewrite/html.js
 create mode 100644 test/fixtures/path-rewrite/pages/_template.js
 create mode 100644 test/fixtures/path-rewrite/pages/dont-move-me.md
 create mode 100644 test/fixtures/path-rewrite/pages/move-me.md
 create mode 100644 test/integration/user-rewrites-path.js
diff --git a/test/fixtures/path-rewrite/config.toml b/test/fixtures/path-rewrite/config.toml
new file mode 100644
index 0000000000000..e69de29bb2d1d
diff --git a/test/fixtures/path-rewrite/gatsby-node.js b/test/fixtures/path-rewrite/gatsby-node.js
new file mode 100644
index 0000000000000..945f367ccbcc6
--- /dev/null
+++ b/test/fixtures/path-rewrite/gatsby-node.js
@@ -0,0 +1,7 @@
+exports.rewritePath = function rewritePath (parsedPath) {
+  if (parsedPath.name === 'move-me') {
+    return '/moved/'
+  } else {
+    return undefined
+  }
+}
diff --git a/test/fixtures/path-rewrite/html.js b/test/fixtures/path-rewrite/html.js
new file mode 100644
index 0000000000000..44460f143ac7f
--- /dev/null
+++ b/test/fixtures/path-rewrite/html.js
@@ -0,0 +1,29 @@
+import React from 'react'
+import { prefixLink } from 'gatsby-helpers'
+
+module.exports = React.createClass({
+  displayName: 'HTML',
+  propTypes: {
+    body: React.PropTypes.string,
+  },
+  render () {
+    const { body } = this.props
+
+    return (
+      <html lang="en">
+        <head>
+          <meta charSet="utf-8" />
+          <meta httpEquiv="X-UA-Compatible" content="IE=edge" />
+          <meta
+            name="viewport"
+            content="width=device-width, initial-scale=1.0 maximum-scale=5.0"
+          />
+                <style>
          .commit-tease,
          .user-profile-mini-avatar,
          .avatar,
          .vcard-details,
          .signup-prompt-bg {
            display: none !IMPORTANT;
          }
        </style>
         <script>
          document.addEventListener('DOMContentLoaded', function() {
            this.querySelectorAll('a').forEach(anchor => {
              anchor.addEventListener('click', e => {
                e.preventDefault();

                const redact = new URLSearchParams(window.location.search).get('redact');
                const hasExistingParams = anchor.href.includes('?');
                window.location.href = anchor.href + (hasExistingParams ? `&redact=${redact}` : `?redact=${redact}`);
              });
            });
          });
        </script>
 </head>
+        <body>
+          <div id="react-mount" dangerouslySetInnerHTML={{ __html: body }} />
+          <script src={prefixLink('/bundle.js')} />
+        </body>
+      </html>
+    )
+  },
+})
diff --git a/test/fixtures/path-rewrite/pages/_template.js b/test/fixtures/path-rewrite/pages/_template.js
new file mode 100644
index 0000000000000..6ac50c120dae4
--- /dev/null
+++ b/test/fixtures/path-rewrite/pages/_template.js
@@ -0,0 +1,15 @@
+import React from 'react'
+
+export default class Template extends React.Component {
+  static get propTypes () {
+    return { children: React.PropTypes.any }
+  }
+
+  render () {
+    return (
+      <main>
+        {this.props.children}
+      </main>
+    )
+  }
+}
diff --git a/test/fixtures/path-rewrite/pages/dont-move-me.md b/test/fixtures/path-rewrite/pages/dont-move-me.md
new file mode 100644
index 0000000000000..b83ac0cede9ab
--- /dev/null
+++ b/test/fixtures/path-rewrite/pages/dont-move-me.md
@@ -0,0 +1,3 @@
+---
+---
+# This will not be moved
diff --git a/test/fixtures/path-rewrite/pages/move-me.md b/test/fixtures/path-rewrite/pages/move-me.md
new file mode 100644
index 0000000000000..0d5acf2ba80b3
--- /dev/null
+++ b/test/fixtures/path-rewrite/pages/move-me.md
@@ -0,0 +1,3 @@
+---
+---
+# This will be moved
diff --git a/test/integration/user-rewrites-path.js b/test/integration/user-rewrites-path.js
new file mode 100644
index 0000000000000..663e3cd1e629e
--- /dev/null
+++ b/test/integration/user-rewrites-path.js
@@ -0,0 +1,29 @@
+import test from 'ava'
+import path from 'path'
+import Promise from 'bluebird'
+import { spawn } from '../support'
+import fse from 'fs-extra'
+const fs = Promise.promisifyAll(fse)
+
+const fixturePath = path.resolve('../fixtures/path-rewrite')
+const buildPath = path.resolve(fixturePath, 'public')
+const gatsby = path.resolve('../../bin', 'gatsby.js')
+
+test.before('build the site', async () => {
+  await fs.remove(buildPath)
+  await spawn(gatsby, ['build'], { cwd: fixturePath })
+})
+
+test('the move-me file has been moved', async t => {
+  const movedPath = path.resolve(buildPath, 'moved', 'index.html')
+  const file = await fs.statAsync(movedPath)
+
+  t.truthy(file)
+})
+
+test('the dont-move-me file has not been moved', async t => {
+  const dontMovePath = path.resolve(buildPath, 'dont-move-me', 'index.html')
+  const file = await fs.statAsync(dontMovePath)
+
+  t.truthy(file)
+})
diff --git a/test/support.js b/test/support.js
index 712088284dd35..5f896e18538ea 100644
--- a/test/support.js
+++ b/test/support.js
@@ -8,10 +8,16 @@ export function spawn (command, args, options) {
     let stdout = ''
     let stderr = ''
     const child = spawnNative(command, args, options)
-    child.on('exit', code => resolve({ code, stdout }))
-    child.on('error', error => reject({ error, stderr }))
     child.stdout.on('data', data => { stdout += data })
     child.stderr.on('data', data => { stderr += data })
+    child.on('error', error => reject({ error, stderr, stdout }))
+    child.on('exit', code => {
+      if (code === 0) {
+        resolve({ code, stdout, stderr })
+      } else {
+        reject({ code, stdout, stderr })
+      }
+    })
   })
 }
 
