search

pkgjs / parseargs

Polyfill of `util.parseArgs()`
Apache License 2.0
121 stars 9 forks source link

Refactor tests? #105

Closed shadowspawn closed 2 years ago

shadowspawn commented 2 years ago

Will we be copying tests from this repo again?

Based on the PR to nodejs/node, it looks like it would make sense to have all the tests in a single file. And .mjs. And switch test library or add a wrapper so can cut-and-paste:

// node
assert.deepStrictEqual(args, expected);
// here
t.deepEqual(args, expected);
t.end();
bcoe commented 2 years ago

👍 would be nice to eliminate the manual replacing I'm doing to match Node's test runner.