fixup! wrap t.log in causal console

endojs · Feb 29, 2024 · caf11d6 · caf11d6
1 parent a1abaa6
commit caf11d6
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 9 deletions.
diff --git a/packages/ses-ava/src/ses-ava-test.js b/packages/ses-ava/src/ses-ava-test.js
@@ -1,4 +1,5 @@
 import 'ses';
+import { loggedErrorHandler, makeCausalConsole } from 'ses/console-tools.js';
 
 const { stringify } = JSON;
 const { defineProperty, freeze } = Object;
@@ -91,8 +92,11 @@ const getLogger = (tlogger, loggerOrOptions) => {
   } else if (typeof loggerOrOptions === 'object') {
     const { tlog } = loggerOrOptions;
     if (tlog) {
-      // TODO wrap with SES causal console
-      return tlogger;
+      const baseConsole = harden({
+        error: tlogger,
+      });
+      const causalConsole = makeCausalConsole(baseConsole, loggedErrorHandler);
+      return causalConsole.error;
     } else {
       // Without causal console wrapper
       return tlogger;

diff --git a/packages/ses-ava/test/test-ses-ava-reject.js b/packages/ses-ava/test/test-ses-ava-reject.js
@@ -25,13 +25,13 @@ test('ses-ava reject console output', t => {
   t.assert(true);
   // Uncomment this to see something like the text in the extended comment below
 
-  // return Promise.resolve(null)
-  //   .then(v => v)
-  //   .then(v => v)
-  //   .then(_ => {
-  //     assert.typeof(88, 'string', assert.details`msg ${'NOTICE ME'}`);
-  //   });
-});
+//   return Promise.resolve(null)
+//     .then(v => v)
+//     .then(v => v)
+//     .then(_ => {
+//       assert.typeof(88, 'string', assert.details`msg ${'NOTICE ME'}`);
+//     });
+// });
 
 /*
 Uncommenting the test code above should produce something like the following.