[node:test] `concurrency` type and default value

[Semigradsky]

Affected URL(s)

https://nodejs.org/docs/latest-v18.x/api/test.html

Description of the problem

By https://nodejs.org/docs/latest-v18.x/api/test.html#testname-options-fn we have concurrency field in options which can be number or boolean with defaut value false.

---

By https://nodejs.org/docs/latest-v18.x/api/test.html#contexttestname-options-fn we have concurrency field in options which can be only number with defaut value 1.

But looks like options should be the same:
https://github.com/nodejs/node/blob/v18.x/lib/internal/test_runner/test.js#L126-L131

---

By https://nodejs.org/docs/latest-v18.x/api/test.html#runoptions we have concurrency field in options which can be number or boolean with defaut value true.

Should the default value be different from that in test?

[MoLow]

this should accept boolean as well

[Semigradsky]

Pay attention to different default values also.

[WonJo-Jung]

I'd like to get breakthrough about this issue. @MoLow could you assign this issue to me? :)

[WonJo-Jung]

I checked context.test() and run() are configuring option values like test(), except default value of concurrency field is 1, not false. So I think next step would be revising document about test_runner. Is my suggestion correct? Or other steps remain?

[MoLow]

fixed by 7d68b7b

[MoLow]

thanks @richiemccoll 🎉