net: migrate errors to internal/errors

doc/api/errors.md

@@ -1355,6 +1355,12 @@ The [`server.listen()`][] method was called while a `net.Server` was already
 listening. This applies to all instances of `net.Server`, including HTTP, HTTPS,
 and HTTP/2 Server instances.
 
+<a id="ERR_SERVER_NOT_RUNNING"></a>
+### ERR_SERVER_NOT_RUNNING
+
+The [`server.close()`][] method was called when a `net.Server` was not running. This applies to all instances of `net.Server`, including HTTP, HTTPS,
+and HTTP/2 Server instances.
+
 <a id="ERR_SOCKET_ALREADY_BOUND"></a>
 ### ERR_SOCKET_ALREADY_BOUND
 

lib/internal/errors.js

@@ -447,6 +447,7 @@ E('ERR_PARSE_HISTORY_DATA', 'Could not parse history data in %s');
 E('ERR_REQUIRE_ESM', 'Must use import to load ES Module: %s');
 E('ERR_SERVER_ALREADY_LISTEN',
   'Listen method has been called more than once without closing.');
+E('ERR_SERVER_NOT_RUNNING', 'Server is not running.');
 E('ERR_SOCKET_ALREADY_BOUND', 'Socket is already bound');
 E('ERR_SOCKET_BAD_BUFFER_SIZE', 'Buffer size must be a positive integer');
 E('ERR_SOCKET_BAD_PORT', 'Port should be > 0 and < 65536. Received %s.');

lib/net.js

@@ -724,7 +724,7 @@ Socket.prototype._writeGeneric = function(writev, data, encoding, cb) {
   this._unrefTimer();
 
   if (!this._handle) {
-    this.destroy(new Error('This socket is closed'), cb);
+    this.destroy(new errors.Error('ERR_SOCKET_CLOSED'), cb);
     return false;
   }
 
@@ -1624,7 +1624,7 @@ Server.prototype.close = function(cb) {
   if (typeof cb === 'function') {
     if (!this._handle) {
       this.once('close', function close() {
-        cb(new Error('Not running'));
+        cb(new errors.Error('ERR_SERVER_NOT_RUNNING'));
       });
     } else {
       this.once('close', cb);

test/parallel/test-http-unix-socket.js

@@ -59,7 +59,7 @@ server.listen(common.PIPE, common.mustCall(function() {
       server.close(common.mustCall(function(error) {
         assert.strictEqual(error, undefined);
         server.close(common.mustCall(function(error) {
-          assert.strictEqual(error && error.message, 'Not running');
+          assert.strictEqual(error && error.code, 'ERR_SERVER_NOT_RUNNING');
         }));
       }));
     }));

test/parallel/test-net-socket-destroy-send.js

@@ -13,10 +13,10 @@ server.listen(0, common.mustCall(function() {
     // Test destroy returns this, even on multiple calls when it short-circuits.
     assert.strictEqual(conn, conn.destroy().destroy());
     conn.on('error', common.mustCall(function(err) {
-      assert.strictEqual(err.message, 'This socket is closed');
+      assert.strictEqual(err.code, 'ERR_SOCKET_CLOSED');
     }));
     conn.write(Buffer.from('kaboom'), common.mustCall(function(err) {
-      assert.strictEqual(err.message, 'This socket is closed');
+      assert.strictEqual(err.code, 'ERR_SOCKET_CLOSED');
     }));
     server.close();
   }));

test/parallel/test-net-socket-write-after-close.js

@@ -28,8 +28,7 @@ const net = require('net');
     const client = net.connect({ port }, common.mustCall(() => {
       client.on('error', common.mustCall((err) => {
         server.close();
-        assert.strictEqual(err.constructor, Error);
-        assert.strictEqual(err.message, 'This socket is closed');
+        assert.strictEqual(err.code, 'ERR_SOCKET_CLOSED');
       }));
       client._handle.close();
       client._handle = null;