Fixing render test console.logs in browser (mapbox#10937)

Fixing console.log() not working on successful render tests when using watch-render.

Author: SnailBones

build/test/build-tape.js

@@ -1,18 +1,20 @@
 /* eslint-disable flowtype/require-valid-file-annotation */
 import browserify from 'browserify';
+import envify from 'envify';
 import fs from 'fs';
 import {fileURLToPath} from 'url';
 
 export default function() {
     return new Promise((resolve, reject) => {
-        browserify(fileURLToPath(new URL('../../test/util/tape_config.js', import.meta.url)), { standalone: 'tape' })
+        browserify(fileURLToPath(new URL('../../test/util/tape_config.js', import.meta.url)), {standalone: 'tape'})
+            .transform(envify) // Makes env available in tape_config.js
             .bundle((err, buff) => {
                 if (err) { throw err; }
 
-                fs.writeFile('test/integration/dist/tape.js', buff, { encoding: 'utf8'}, (err) => {
+                fs.writeFile('test/integration/dist/tape.js', buff, {encoding: 'utf8'}, (err) => {
                     if (err) { reject(err); }
                     resolve();
                 });
             });
     });
-};
+}

package.json

@@ -63,6 +63,7 @@
     "diff": "^5.0.0",
     "documentation": "~13.1.1",
     "ejs": "^3.1.6",
+    "envify": "^4.1.0",
     "eslint": "^7.30.0",
     "eslint-config-mourner": "^3.0.0",
     "eslint-plugin-flowtype": "^5.2.0",

test/integration/testem/testem.js

@@ -108,6 +108,7 @@ const ciTestemConfig = {
     "reporter": "xunit",
     "report_file": ciOutputFile,
     "xunit_intermediate_output": true,
+    "tap_quiet_logs": true,
     "browser_args": {
         "Chrome": {
             "ci": [ "--disable-backgrounding-occluded-windows", "--ignore-gpu-blocklist", "--use-gl=desktop" ]

test/util/tape_config.js

@@ -29,8 +29,8 @@ function getQueryVariable(variable) {
 Testem.handleConsoleMessage = function(msg) {
     // Send output over ws to testem server
     Testem.emit('tap', msg);
-
-    return false;
+    // Return true and log output only when not in CI mode. (yarn run watch-render).
+    return !process.env.CI;
 };
 
 // Persist the current html on the page as an artifact once tests finish