test: remove assert.doesNotThrow()

There is actually no reason to use `assert.doesNotThrow()` in the
tests. If a test throws, just let the error bubble up right away
instead of first catching it and then rethrowing it.

PR-URL: https://github.com/nodejs/node/pull/18669
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>

1 files changed, 9 insertions, 8 deletions

diff --git a/test/parallel/test-stream-writable-null.js b/test/parallel/test-stream-writable-null.js
index f1b91dee21..706141821b 100644
--- a/test/parallel/test-stream-writable-null.js
+++ b/test/parallel/test-stream-writable-null.js
@@ -27,14 +27,14 @@ common.expectsError(
   }
 );
 
-assert.doesNotThrow(() => {
+{
   const m = new MyWritable({ objectMode: true }).on('error', (e) => {
     assert.ok(e);
   });
   m.write(null, (err) => {
     assert.ok(err);
   });
-});
+}
 
 common.expectsError(
   () => {
@@ -47,24 +47,25 @@ common.expectsError(
   }
 );
 
-assert.doesNotThrow(() => {
+{
   const m = new MyWritable().on('error', (e) => {
     assert.ok(e);
   });
   m.write(false, (err) => {
     assert.ok(err);
   });
-});
+}
 
-assert.doesNotThrow(() => {
+{
   const m = new MyWritable({ objectMode: true });
   m.write(false, (err) => assert.ifError(err));
-});
-assert.doesNotThrow(() => {
+}
+
+{
   const m = new MyWritable({ objectMode: true }).on('error', (e) => {
     assert.ifError(e || new Error('should not get here'));
   });
   m.write(false, (err) => {
     assert.ifError(err);
   });
-});
+}