fixup

Author: ronag

test/parallel/test-http-outgoing-finish-writable.js

@@ -3,9 +3,6 @@ const common = require('../common');
 const assert = require('assert');
 const http = require('http');
 
-// Verify that after calling end() on an `OutgoingMessage` (or a type that
-// inherits from `OutgoingMessage`), its `writable` property is not set to false
-
 const server = http.createServer(common.mustCall(function(req, res) {
   assert.strictEqual(res.writable, true);
   assert.strictEqual(res.finished, false);
@@ -14,7 +11,7 @@ const server = http.createServer(common.mustCall(function(req, res) {
 
   // res.writable is set to false after it has finished sending
   // Ref: https://github.com/nodejs/node/issues/15029
-  assert.strictEqual(res.writable, true);
+  assert.strictEqual(res.writable, false);
   assert.strictEqual(res.finished, true);
   assert.strictEqual(res.writableEnded, true);
 
@@ -32,9 +29,5 @@ server.on('listening', common.mustCall(function() {
 
   assert.strictEqual(clientRequest.writable, true);
   clientRequest.end();
-
-  // Writable is still true when close
-  // THIS IS LEGACY, we cannot change it
-  // unless we break error detection
-  assert.strictEqual(clientRequest.writable, true);
+  assert.strictEqual(clientRequest.writable, false);
 }));

test/parallel/test-http-writable-true-after-close.js

@@ -4,16 +4,13 @@ const common = require('../common');
 const assert = require('assert');
 const { get, createServer } = require('http');
 
-// res.writable should not be set to false after it has finished sending
-// Ref: https://github.com/nodejs/node/issues/15029
-
 let internal;
 let external;
 
 // Proxy server
 const server = createServer(common.mustCall((req, res) => {
   const listener = common.mustCall(() => {
-    assert.strictEqual(res.writable, true);
+    assert.strictEqual(res.writable, false);
   });
 
   // on CentOS 5, 'finish' is emitted