Output phantom.js errors to stderr.

Use .contain
uncss · Mar 9, 2015 · 62145b3 · 62145b3
1 parent cd46c50
commit 62145b3
Show file tree

Hide file tree

Showing 3 changed files with 35 additions and 0 deletions.
diff --git a/src/phantom.js b/src/phantom.js
@@ -26,6 +26,9 @@ function init(instance) {
             sslProtocol: 'any'
         }));
     }).then(function (ph) {
+        /* Phridge outputs everything to stdout by default */
+        ph.childProcess.cleanStdout.unpipe();
+        ph.childProcess.cleanStdout.pipe(process.stderr);
         phantom = ph;
     }).disposer(phridge.disposeAll);
 }

diff --git a/tests/errors.js b/tests/errors.js
@@ -66,6 +66,24 @@ describe('Error reporting', function () {
         });
     });
 
+    it('PhantomJS errors to stderr', function (done) {
+        var stderrBuffer = '';
+        var oldWrite = process.stderr.write;
+        process.stderr.write = function (data) {
+            stderrBuffer += data;
+        };
+
+        uncss(
+            ['tests/phantomjs/throw.html'],
+            { raw: '' },
+            function (error) {
+                expect(error).to.equal(null);
+                expect(stderrBuffer).to.contain('Exception');
+                process.stderr.write = oldWrite;
+                done();
+            });
+    });
+
     it('css-parse errors', function (done) {
         uncss(
             ['tests/selectors/index.html'],

diff --git a/tests/phantomjs/throw.html b/tests/phantomjs/throw.html
@@ -0,0 +1,14 @@
+<!DOCTYPE html>
+<html lang="en">
+    <head>
+        <meta charset="utf-8">
+        <title>PhantomJS basic test</title>
+        <link rel="stylesheet" href="phantomjs.css">
+    </head>
+    <body>
+        <script>
+            /* Append an element */
+            throw 'Exception';
+        </script>
+    </body>
+</html>