From 7a70dce2512501e8578de3be0e6a21dd3d5d8541 Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Mon, 3 Sep 2018 05:42:37 -0700 Subject: [PATCH] test: fix test-trace-events-dns Test is using `common.fileExists()` which has been removed. There is no need to check that the file exists as the attempt to read the file in the next line will fail if the file does not exist. Remove existence check. PR-URL: https://github.com/nodejs/node/pull/22674 Reviewed-By: Daniel Bevenius Reviewed-By: Anna Henningsen Reviewed-By: Colin Ihrig Reviewed-By: Refael Ackermann --- test/internet/test-trace-events-dns.js | 2 -- 1 file changed, 2 deletions(-) diff --git a/test/internet/test-trace-events-dns.js b/test/internet/test-trace-events-dns.js index 3eb787a2caf70e..e1cfd6a597acca 100644 --- a/test/internet/test-trace-events-dns.js +++ b/test/internet/test-trace-events-dns.js @@ -53,8 +53,6 @@ for (const tr in tests) { const file = path.join(tmpdir.path, traceFile); - // Confirm that trace log file is created. - assert(common.fileExists(file)); const data = fs.readFileSync(file); const traces = JSON.parse(data.toString()).traceEvents .filter((trace) => trace.cat !== '__metadata');