Regression: unexpected globbing warning

[nicojs]

In Stryker 4.0.0-beta.4:

nicojs@nicoj03:~/stryker/perf/test/express$ npx stryker run --mutate lib/middleware/init.js
13:44:00 (31266) INFO ConfigReader Using stryker.conf.json
13:44:00 (31266) WARN InputFileResolver Globbing expression "{src,lib}/**/__tests__/**/*.+(cjs|mjs|js|ts|jsx|tsx|html|vue)" did not result in any files.

If you run with --mutate, then one wouldn't expect this warning.

[simondel]

@nicojs Can you still reproduce this? I get:

simon@SimonL03 ~/dev/stryker/perf/test/express (2352-deprecate-react-ts)$ npx stryker run --mutate lib/middleware/init.js
13:23:18 (9796) INFO ConfigReader Using stryker.conf.json
13:23:18 (9796) WARN InputFileResolver Globbing expression "lib/middleware/init.js" did not result in any files.
13:23:18 (9796) WARN InputFileResolver Globbing expression "{src,lib}/**/__tests__/**/*.js?(x)" did not result in any files.
13:23:18 (9796) WARN InputFileResolver Globbing expression "{src,lib}/**/?(*.)+(spec|test).js?(x)" did not result in any files.
13:23:18 (9796) WARN InputFileResolver Globbing expression "{src,lib}/**/*+(Spec|Test).js?(x)" did not result in any files.
13:23:18 (9796) WARN InputFileResolver No files selected. Please make sure you either
 (1) Run Stryker inside a Git repository; or
 (2) Specify the `files` property in your Stryker configuration (`--files via command line`).

[simondel]

Alright, reproduction is now possible

[simondel]

@nicojs It doesn't add the cli arguments to the default list:

11:03:14 (3037) DEBUG InputFileResolver Files to mutate: [
  "/home/simon/dev/stryker/perf/test/express/lib/middleware/init.js"
]

So it seems to just be the input files which do not have the support to hide the glob warnings when it could not be found.

[simondel]

My local setting was out of date. The deepMerge sees an array as an object so it will deep merge the settings. This causes the two arrays to merge together, thus causing the error that you're seeing.

If I add detailed logging this, I get:

the type of [
  '{src,lib}/**/!(*.+(s|S)pec|*.+(t|T)est).+(cjs|mjs|js|ts|jsx|tsx|html|vue)',
  '!{src,lib}/**/__tests__/**/*.+(cjs|mjs|js|ts|jsx|tsx|html|vue)'
] is object

deep merging the properties of [
  '{src,lib}/**/!(*.+(s|S)pec|*.+(t|T)est).+(cjs|mjs|js|ts|jsx|tsx|html|vue)',
  '!{src,lib}/**/__tests__/**/*.+(cjs|mjs|js|ts|jsx|tsx|html|vue)'
] with [ 'lib/middleware/init.js' ]

key '0' will be set to value 'lib/middleware/init.js'