Skip to content
Permalink
Browse files

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: #25117
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information...
Trott authored and targos committed Dec 19, 2018
1 parent 2c50bcd commit 91d1aea311fd40ad791370fe381d1083e00f6b58
Showing with 1 addition and 1 deletion.
  1. +1 −1 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();

0 comments on commit 91d1aea

Please sign in to comment.
You can’t perform that action at this time.