search

Test-More / Test2-Harness

Alternative to Test::Harness
Other
23 stars 26 forks source link

[major_refactor] Make sure excluded files are excluded #131

Closed atoomic closed 4 years ago

atoomic commented 4 years ago

Also add one integration test to confirm excluded files are excluded as expected.

It's common to do something like this yath test --exclude-file t/boom.t t/*.t

exodist commented 4 years ago

This logic change is not correct. The @$input check was intended to make sure any file listed on the command line bypasses the 'include_file' check.

hmm, reading 'include_file' I think the real fix is to have include_file return an empty list when the file SHOULD be included, and return a reason a file is not included when it should nto be included. Then if @$input is true it should print a warning saying that the file was ont he command line, but was excluded because . if @$input is empty then it jst does not include the file and moves on no warnings.

exodist commented 4 years ago

Fixed in 6277bd6b