test: refactor test-accessor-properties

himself65 · BethGriggs · commit 0dae8feefd4f · 2020-02-06T02:49:07.000Z
PR-URL: #29943 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
diff --git a/test/parallel/test-accessor-properties.js b/test/parallel/test-accessor-properties.js
@@ -16,18 +16,6 @@ const UDP = internalBinding('udp_wrap').UDP;
 
 {
   // Should throw instead of raise assertions
-  assert.throws(() => {
-    TTY.prototype.bytesRead;
-  }, TypeError);
-
-  assert.throws(() => {
-    TTY.prototype.fd;
-  }, TypeError);
-
-  assert.throws(() => {
-    TTY.prototype._externalStream;
-  }, TypeError);
-
   assert.throws(() => {
     UDP.prototype.fd;
   }, TypeError);
@@ -36,6 +24,11 @@ const UDP = internalBinding('udp_wrap').UDP;
   const properties = ['bytesRead', 'fd', '_externalStream'];
 
   properties.forEach((property) => {
+    // Should throw instead of raise assertions
+    assert.throws(() => {
+      TTY.prototype[property];
+    }, TypeError, `Missing expected TypeError for TTY.prototype.${property}`);
+
     // Should not throw for Object.getOwnPropertyDescriptor
     assert.strictEqual(
       typeof Object.getOwnPropertyDescriptor(StreamWrapProto, property),
