next {dev,build} command should ignore __tests__ dir

In jest example, if we move ./__tests__ to ./pages/__tests__ and then run next build then an error occurs.

The solution is not trying to solve this error, but make next ignore some patterns, fe *.{spec,test}.js or some dirs like __{mocks,tests}__

Otherwise, those dev files would be available in public dir, WDYT ?

Issues #988 #1545 #1689 talks about the same problem (sadly without any solution).

BTW next.js rocks!

[scsper]

agreed with @sboudrias-aa and @ericnograles . @arunoda please reconsider this decision. writing tests next to the module also helps a lot with refactoring because you can delete / move entire directories and still have things work.

[bensalilijames]

Design suggestion: add an excludeFile option to next.config.js which accepts a function to exclude certain files from being webpack’d.

This option would be passed straight to emit-file-loader’s exclude option (and merged with the normal node_modules exclude) to prevent it from building it as a page. For example:

// next.config.js
module.exports = {
  excludeFile: (str) => /\*.{spec,test}.js/.test(str)
}

If that sounds good I can make a PR for this!