From fbc332bdfe564e5a48579570b673156ffef14f19 Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Tue, 18 Dec 2018 16:05:06 -0800 Subject: [PATCH] test: refactor test-esm-namespace.mjs Remove unused `name` argument that is different from a subsequent `name` argument in a different but nearby function. This was mildly confusing to me at first, so hopefully this change clarifies things for others reading the test. PR-URL: https://github.com/nodejs/node/pull/25117 Reviewed-By: James M Snell --- test/es-module/test-esm-namespace.mjs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/es-module/test-esm-namespace.mjs b/test/es-module/test-esm-namespace.mjs index dcd159f6c8729a..da1286d0f40e67 100644 --- a/test/es-module/test-esm-namespace.mjs +++ b/test/es-module/test-esm-namespace.mjs @@ -6,7 +6,7 @@ import Module from 'module'; const keys = Object.entries( Object.getOwnPropertyDescriptors(new Module().require('fs'))) - .filter(([name, d]) => d.enumerable) + .filter(([ , d]) => d.enumerable) .map(([name]) => name) .concat('default') .sort();