fix(test): Avoid EPIPE error in "web-ext sign" test

mozilla · Jul 16, 2018 · bc4eb1c · bc4eb1c
1 parent 4134812
commit bc4eb1c
Showing 1 changed file with 11 additions and 3 deletions.
diff --git a/tests/functional/fake-amo-server.js b/tests/functional/fake-amo-server.js
@@ -29,9 +29,17 @@ http.createServer(function(req, res) {
   const reply = FAKE_REPLIES[replyIndex++];
 
   if (reply) {
-    res.writeHead(200, {'content-type': 'application/json'});
-    res.write(JSON.stringify(reply));
-    res.end();
+    req.on('data', function() {
+      // Ignore request body.
+    });
+    // Wait for the transfer of the request body to finish before sending a response.
+    // Otherwise the client could experience an EPIPE error:
+    // https://github.com/nodejs/node/issues/12339
+    req.once('end', function() {
+      res.writeHead(200, {'content-type': 'application/json'});
+      res.write(JSON.stringify(reply));
+      res.end();
+    });
   } else {
     process.exit(1);
   }