From 038e52627fddd331361fa73f848bd3ad3f9c7cf7 Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Sat, 23 Dec 2017 14:40:49 -0800
Subject: [PATCH] test: remove undefined function

`common.fail()` no longer exists as its functionality is now in
`assert.fail()`. Replace only two instances in the code base with
`assert.fail()`.

PR-URL: https://github.com/nodejs/node/pull/17845
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Weijia Wang <starkwang@126.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
---
 test/sequential/test-http-server-consumed-timeout.js | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/test/sequential/test-http-server-consumed-timeout.js b/test/sequential/test-http-server-consumed-timeout.js
index abe100c1907e0f..56217226503be1 100644
--- a/test/sequential/test-http-server-consumed-timeout.js
+++ b/test/sequential/test-http-server-consumed-timeout.js
@@ -1,6 +1,8 @@
 'use strict';
 
 const common = require('../common');
+
+const assert = require('assert');
 const http = require('http');
 
 let time = Date.now();
@@ -16,7 +18,7 @@ const server = http.createServer((req, res) => {
   req.setTimeout(TIMEOUT, () => {
     if (!intervalWasInvoked)
       return common.skip('interval was not invoked quickly enough for test');
-    common.fail('Request timeout should not fire');
+    assert.fail('Request timeout should not fire');
   });
 
   req.resume();