Pass name as a named parameter in test-each

Author: ehmicky

test/echo.js

@@ -7,7 +7,7 @@ import { METHODS } from './helpers/methods.js'
 testEach(
   METHODS,
   [{ opts: { echo: false } }, { opts: { echo: true } }],
-  (title, methodProps, data) =>
-    test(`'echo' option | ${title}`, t =>
+  ({ name }, methodProps, data) =>
+    test(`'echo' option | ${name}`, t =>
       snapshotTest({ t, methodProps, data })),
 )

test/exec.js

@@ -4,6 +4,6 @@ import { testEach } from './helpers/test_each/main.js'
 import { snapshotTest } from './helpers/snapshot.js'
 import { EXEC_METHODS } from './helpers/methods.js'
 
-testEach(EXEC_METHODS, [{}], (title, methodProps, data) =>
-  test(`exec() | ${title}`, t => snapshotTest({ t, methodProps, data })),
+testEach(EXEC_METHODS, [{}], ({ name }, methodProps, data) =>
+  test(`exec() | ${name}`, t => snapshotTest({ t, methodProps, data })),
 )

test/helpers/test_each/main.js

@@ -11,7 +11,10 @@ export const testEach = function(...inputArgs) {
 
   const names = getNames(args)
 
-  const results = args.map((values, index) => func(names[index], ...values))
+  const results = args.map((values, index) => {
+    const name = names[index]
+    return func({ name }, ...values)
+  })
 
   // Can use `Promise.all(results)` if `func` is async
   return { args, results }

test/helpers/test_each/serialize.js

@@ -2,7 +2,7 @@ import { inspect } from 'util'
 
 import { isPlainObject } from './utils.js'
 
-// Serialize an argument so it can be used in the title
+// Serialize an argument so it can be used in the names
 export const serializeArg = function(arg) {
   if (hasName(arg)) {
     return arg.name
@@ -34,7 +34,7 @@ const serializeFunction = function(func) {
   return func.name
 }
 
-// Make title short and on a single line
+// Make names short and on a single line
 const INSPECT_OPTS = {
   breakLength: Infinity,
   depth: 1,

test/input.js

@@ -7,15 +7,15 @@ import { METHODS } from './helpers/methods.js'
 testEach(
   METHODS,
   [{ command: true }, { command: ' ' }],
-  (title, methodProps, data) =>
-    test(`Invalid command | ${title}`, t =>
+  ({ name }, methodProps, data) =>
+    test(`Invalid command | ${name}`, t =>
       snapshotTest({ t, methodProps, data })),
 )
 
 testEach(
   METHODS,
   [{ opts: { uid: 0.5 } }, { command: 'invalid', read: false }],
-  (title, methodProps, data) =>
-    test(`Errored command | ${title}`, t =>
+  ({ name }, methodProps, data) =>
+    test(`Errored command | ${name}`, t =>
       snapshotTest({ t, methodProps, data })),
 )

test/options.js

@@ -37,11 +37,11 @@ testEach(
     { opts: { result: 'invalid' } },
     { opts: { from: 'invalid' } },
   ],
-  (title, methodProps, data) =>
-    test(`Invalid options | ${title}`, t =>
+  ({ name }, methodProps, data) =>
+    test(`Invalid options | ${name}`, t =>
       snapshotTest({ t, methodProps, data })),
 )
 
-testEach(METHODS, [{}, { opts: {} }], (title, methodProps, data) =>
-  test(`No options | ${title}`, t => snapshotTest({ t, methodProps, data })),
+testEach(METHODS, [{}, { opts: {} }], ({ name }, methodProps, data) =>
+  test(`No options | ${name}`, t => snapshotTest({ t, methodProps, data })),
 )

test/stream.js

@@ -20,6 +20,6 @@ testEach(
     { opts: { encoding: 'utf8' } },
     { opts: { stripFinalNewline: true } },
   ],
-  (title, methodProps, data) =>
-    test(`stream() | ${title}`, t => snapshotTest({ t, methodProps, data })),
+  ({ name }, methodProps, data) =>
+    test(`stream() | ${name}`, t => snapshotTest({ t, methodProps, data })),
 )

test/task.js

@@ -4,6 +4,9 @@ import { testEach } from './helpers/test_each/main.js'
 import { snapshotTest } from './helpers/snapshot.js'
 import { TASK_METHODS } from './helpers/methods.js'
 
-testEach(TASK_METHODS, [{}, { task: 'nested' }], (title, methodProps, data) =>
-  test(`task() | ${title}`, t => snapshotTest({ t, methodProps, data })),
+testEach(
+  TASK_METHODS,
+  [{}, { task: 'nested' }],
+  ({ name }, methodProps, data) =>
+    test(`task() | ${name}`, t => snapshotTest({ t, methodProps, data })),
 )

test/verbose.js

@@ -17,7 +17,7 @@ testEach(
     { opts: { verbose: true, stdio: 'pipe' } },
     { opts: { verbose: true, stdio: 'pipe', stdout: 'pipe', stderr: 'pipe' } },
   ],
-  (title, methodProps, data) =>
-    test(`'verbose' option | ${title}`, t =>
+  ({ name }, methodProps, data) =>
+    test(`'verbose' option | ${name}`, t =>
       snapshotTest({ t, methodProps, data })),
 )