test: add comments explaining _setSimultaneousAccepts deprecation tests

PR-URL: #41307 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
nodejs · Feb 1, 2022 · 8d15bac · 8d15bac
1 parent ffca1a7
commit 8d15bac
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/test/parallel/test-net-deprecated-setsimultaneousaccepts.js b/test/parallel/test-net-deprecated-setsimultaneousaccepts.js
@@ -1,6 +1,8 @@
 // Flags: --no-warnings
 'use strict';
 
+// Test that DEP0121 is emitted on the first call of _setSimultaneousAccepts().
+
 const {
   expectWarning
 } = require('../common');

diff --git a/test/parallel/test-net-server-simultaneous-accepts-produce-warning-once.js b/test/parallel/test-net-server-simultaneous-accepts-produce-warning-once.js
@@ -1,12 +1,19 @@
 'use strict';
 
+// Test that DEP0121 is emitted only once if _setSimultaneousAccepts() is called
+// more than once. This test is similar to
+// test-net-deprecated-setsimultaneousaccepts.js, but that test calls
+// _setSimultaneousAccepts() only once. Unlike this test, that will confirm
+// that the warning is emitted on the first call. This test doesn't check which
+// call caused the warning to be emitted.
+
 const { expectWarning } = require('../common');
-const net = require('net');
+const { _setSimultaneousAccepts } = require('net');
 
 expectWarning(
   'DeprecationWarning',
   'net._setSimultaneousAccepts() is deprecated and will be removed.',
   'DEP0121');
 
-net._setSimultaneousAccepts();
-net._setSimultaneousAccepts();
+_setSimultaneousAccepts();
+_setSimultaneousAccepts();