console - make warn/error log to stderr properly again

Author: wesgarland

python/pythonmonkey/builtin_modules/console.js

@@ -54,8 +54,8 @@ class Console {
     this.log   = (...args) => this.#writeToStdout(this.#formatToStr(...args));
     this.debug = (...args) => this.#writeToStdout(this.#formatToStr(...args));
     this.info  = (...args) => this.#writeToStdout(this.#formatToStr(...args));
-    this.warn  = (...args) => this.#writeToStdout(this.#formatToStr(...args));
-    this.error = (...args) => this.#writeToStdout(this.#formatToStr(...args));
+    this.warn  = (...args) => this.#writeToStderr(this.#formatToStr(...args));
+    this.error = (...args) => this.#writeToStderr(this.#formatToStr(...args));
   }
 
   /**

tests/js/console-stdio.bash

@@ -0,0 +1,45 @@
+#! /bin/bash
+#
+# @file         console-stdio.bash
+#               A peter-jr test which ensures that the console object uses the right file descriptors.
+#
+# @author       Wes Garland, [email protected]
+# @date         July 2023
+
+set -u
+set -o pipefail
+
+panic()
+{
+  echo "FAIL: $*" >&2
+  exit 2
+}
+
+cd `dirname "$0"` || panic "could not change to test directory"
+
+"${PMJS:-../../pmjs}" \
+-e 'console.log("stdout")' \
+-e 'console.debug("stdout")' \
+-e 'console.info("stdout")' \
+< /dev/null \
+| grep -c '^stdout$' \
+| while read qty
+  do
+    echo "stdout: $qty"
+    [ "$qty" != "3" ] && panic qty should not be $qty
+    break
+  done || exit $?
+
+"${PMJS:-../../pmjs}" \
+-e 'console.error("stderr")' \
+-e 'console.warn("stderr")' \
+< /dev/null 2>&1 \
+| grep -c '^stderr$' \
+| while read qty
+  do
+    echo "stderr: $qty"
+    [ "$qty" != "2" ] && panic qty should not be $qty
+    break
+  done || exit $?
+
+echo "done"