fix tests again

Inist-CNRS · Nov 22, 2024 · da8ba14 · da8ba14
1 parent 6fe2ff5
commit da8ba14
Show file tree

Hide file tree

Showing 5 changed files with 19 additions and 6 deletions.
diff --git a/packages/basics/test/basics.js b/packages/basics/test/basics.js
@@ -115,7 +115,6 @@ describe('test', () => {
                 res.push(chunk);
             })
             .on('end', () => {
-                console.log(res);
                 assert.equal(3, res.length);
                 assert.equal('"a";"b";"c"\r\n', res[0]);
                 assert.equal('"1";"2";"3"\r\n', res[1]);

diff --git a/packages/basics/test/url-fetch.js b/packages/basics/test/url-fetch.js
@@ -43,6 +43,10 @@ httpbin
     .post('/status/400')
     .reply(400);
 
+httpbin
+    .get('/status/400')
+    .reply(400);
+
 httpbin
     .post('/status/503')
     .reply(503);

diff --git a/packages/core/src/index.js b/packages/core/src/index.js
@@ -116,11 +116,11 @@ ezs.createPipeline = (input, commands, trap) => {
     }
     return output
         .pipe(ezs.catch((e) => {
-            trap.write(e.toJSON()); // see engine.js createErrorWith
+            trap.write(JSON.parse(ezs.serializeError(e))); // see engine.js createErrorWith
             return false; // do not catch the error
         }))
         .once('error', (e) => {
-            trap.write(e.toJSON()); // see engine.js createErrorWith
+            trap.write(JSON.parse(ezs.serializeError(e))); // see engine.js createErrorWith
             trap.end();
         })
         .once('end', () => {

diff --git a/packages/core/test/expand.js b/packages/core/test/expand.js
@@ -1,4 +1,5 @@
 import fs from 'fs';
+import debug from 'debug';
 import from from 'from';
 import { PassThrough } from 'stream';
 import ezs from '../src';
@@ -513,7 +514,12 @@ test('with a buggy script', (done) => {
             output.push(chunk);
         })
         .on('error', (e) => {
-            expect(e.message).toEqual(expect.stringContaining('BADVAL is not defined'));
+            try {
+                expect(e.message).toEqual(expect.stringContaining('Lodash'));
+            }
+            catch (ee) {
+                done(ee);
+            }
             expect(output.length).toEqual(0);
             done();
         })

diff --git a/packages/core/test/overturn.js b/packages/core/test/overturn.js
@@ -205,8 +205,12 @@ test('with a buggy script', (done) => {
             output.push(chunk);
         })
         .on('error', (e) => {
-            expect(e.message).toEqual(expect.stringContaining('BADVAL is not defined'));
-            expect(output.length).toEqual(0);
+            try {
+                expect(e.message).toEqual(expect.stringContaining('Lodash'));
+                expect(output.length).toEqual(0);
+            } catch (ee) {
+                done(ee);
+            }
             done();
         })
         .on('end', () => {