testem 3.12.0 → 3.14.0

package/README.md

@@ -187,6 +187,14 @@ For improved ergonomics, TAP reporter does not actually strictly adhere to the S
 }
 ```
 
+By default, the TAP reporter outputs the result of `JSON.stringify()` for any log content that is not a `String`. You can override this behavior by specifying a function for `tap_log_processor`.
+
+```js
+{
+  "tap_log_processor": function(log) { return log.toString(); }
+}
+```
+
 ## Other Test Reporters
 
 Testem has other test reporters besides TAP: `dot`, `xunit` and `teamcity`. You can use the `-R` to specify them

package/lib/reporters/tap_reporter.js

@@ -18,6 +18,7 @@ module.exports = class TapReporter {
     this.results = [];
     this.errors = [];
     this.logs = [];
+    this.logProcessor = config.get('tap_log_processor');
   }
 
   report(prefix, data) {
@@ -56,7 +57,7 @@ module.exports = class TapReporter {
    */
   display(prefix, result) {
     if (this.willDisplay(result)) {
-      this.out.write(displayutils.resultString(this.id++, prefix, result, this.quietLogs, this.strictSpecCompliance));
+      this.out.write(displayutils.resultString(this.id++, prefix, result, this.quietLogs, this.strictSpecCompliance, this.logProcessor));
     }
   }
 

package/lib/server/index.js

@@ -138,6 +138,8 @@ class Server extends EventEmitter {
 
     this.configureProxy(app);
 
+    app.use('/testem', express.static(`${__dirname}/../../public/testem`));
+
     app.get('/', (req, res) => {
       res.redirect(`/${String(Math.floor(Math.random() * 10000))}`);
     });
@@ -177,7 +179,6 @@ class Server extends EventEmitter {
       }
       next();
     });
-    app.use('/testem', express.static(`${__dirname}/../../public/testem`));
   }
 
   injectMiddleware(app) {

package/lib/utils/displayutils.js

@@ -57,7 +57,7 @@ function resultDisplay(id, prefix, result, strictSpecCompliance) {
   return output;
 }
 
-function yamlDisplay(err, logs) {
+function yamlDisplay(err, logs, logProcessor) {
   let testLogs;
   let failed = Object.keys(err || {})
     .filter(key => key !== 'passed')
@@ -68,7 +68,11 @@ function yamlDisplay(err, logs) {
         let logLine;
         if (strutils.isString(log)) {
           logLine = log;
-        } else {
+        }
+        else if (logProcessor) {
+          logLine = logProcessor(log);
+        }
+        else {
           logLine = JSON.stringify(log);
         }
         return strutils.indent(logLine);
@@ -84,10 +88,10 @@ function yamlDisplay(err, logs) {
   ].join('\n'));
 }
 
-function resultString(id, prefix, result, quietLogs, strictSpecCompliance) {
+function resultString(id, prefix, result, quietLogs, strictSpecCompliance, logProcessor) {
   let string = resultDisplay(id, prefix, result, strictSpecCompliance) + '\n';
   if (result.error || (!quietLogs && result.logs && result.logs.length)) {
-    string += yamlDisplay(result.error, result.logs) + '\n';
+    string += yamlDisplay(result.error, result.logs, logProcessor) + '\n';
   }
   return string;
 }

package/package.json

@@ -7,7 +7,7 @@
     "unittest",
     "browser"
   ],
-  "version": "3.12.0",
+  "version": "3.14.0",
   "author": "Toby Ho <airportyh@gmail.com>",
   "repository": {
     "type": "git",
@@ -80,7 +80,6 @@
     "eslint-plugin-chai-expect": "^3.0.0",
     "eslint-plugin-mocha": "^10.0.3",
     "mocha": "^9.2.0",
-    "phantomjs": "^2.1.7",
     "request": "^2.51.0",
     "saucie": "^3.3.3",
     "shelljs": "^0.8.2",