test: refactor test-dgram-oob-buffer

* Change common.noop to common.mustNotCall() to verify callback is not invoked. * Add destructuring assignment for clarity. Yeah, clarity. That's why. PR-URL: #13443 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
nodejs · Aug 14, 2017 · 7c2befc · 7c2befc
1 parent 30201e5
commit 7c2befc
Showing 1 changed file with 6 additions and 7 deletions.
diff --git a/test/parallel/test-dgram-oob-buffer.js b/test/parallel/test-dgram-oob-buffer.js
@@ -10,15 +10,14 @@ const socket = dgram.createSocket('udp4');
 const buf = Buffer.from([1, 2, 3, 4]);
 const portGetter = dgram.createSocket('udp4')
   .bind(0, 'localhost', common.mustCall(() => {
-    const address = portGetter.address();
+    const { address, port } = portGetter.address();
     portGetter.close(common.mustCall(() => {
-      function ok() {}
-      socket.send(buf, 0, 0, address.port, address.address, ok);
-      socket.send(buf, 0, 4, address.port, address.address, ok);
-      socket.send(buf, 1, 3, address.port, address.address, ok);
-      socket.send(buf, 3, 1, address.port, address.address, ok);
+      socket.send(buf, 0, 0, port, address, common.mustNotCall());
+      socket.send(buf, 0, 4, port, address, common.mustNotCall());
+      socket.send(buf, 1, 3, port, address, common.mustNotCall());
+      socket.send(buf, 3, 1, port, address, common.mustNotCall());
       // Since length of zero means nothing, don't error despite OOB.
-      socket.send(buf, 4, 0, address.port, address.address, ok);
+      socket.send(buf, 4, 0, port, address, common.mustNotCall());
 
       socket.close();
     }));