Merge pull request #149 from jamesramsay/json-stdout

Fix: json reporting behaviour should match docs (fixes #148)
jamesramsay · Jan 4, 2016 · c31f390 · c31f390
2 parents b7a632e + 7415d42
commit c31f390
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/src/config.js b/src/config.js
@@ -29,6 +29,12 @@ export const BUNYAN_DEFAULTS = {
     }],
   },
   json: {
+    name: APP_NAME,
+    streams: [{
+      stream: process.stdout,
+    }],
+  },
+  ['json-err']: {
     name: APP_NAME,
     streams: [{
       stream: process.stderr,

diff --git a/src/hercule.js b/src/hercule.js
@@ -74,7 +74,7 @@ export function transcludeFileSync(input, { relativePath }, log) {
     cwd: __dirname,
     timeout: SYNC_TIMEOUT,
   };
-  const args = [input, '--reporter', 'json'];
+  const args = [input, '--reporter', 'json-err'];
   const result = childProcess.spawnSync('../bin/hercule', args, options);
   const outputContent = result.stdout.toString();
   const outputLogs = result.stderr.toString().split('\n');
@@ -96,7 +96,7 @@ export function transcludeStringSync(input, { relativePath }, log) {
     cwd: __dirname,
     timeout: SYNC_TIMEOUT,
   };
-  const args = ['--relative', relativePath, '--reporter', 'json'];
+  const args = ['--relative', relativePath, '--reporter', 'json-err'];
   const result = childProcess.spawnSync('../bin/hercule', args, options);
   const outputContent = result.stdout.toString();
   const outputLogs = result.stderr.toString().split('\n');