Fix namespace filtering

cognitect-labs / test-runner

A test runner for clojure.test

Eclipse Public License 2.0

288 stars 32 forks source link

Fix namespace filtering #11

Closed kawas44 closed 6 years ago

kawas44 commented 6 years ago

Reading the usage instruction some can assume that running test-runner with a specific namespace will run only this specific namespace, but it appears that you get a "default" namespace regex configuration as well. Pretty odd to see all your tests running.

Refactor the code to put all default configurations handling in a function.

Set a default test dir when no test dirs specified
Set a default namespace regex when no specific namespace or namespace regex specified.

Cheers

kawas44 commented 6 years ago

Hello,

No issue exists for this fix. If you agree, I can create one and edit my commit message to refer to it.

levand commented 6 years ago

Sorry for not seeing this sooner, fixed in another PR. Thanks!