Convert root to array if root is a string #152

[rohmanhm]

fix issue #152,

[codecov]

Codecov Report

Merging #153 into master will not change coverage.
The diff coverage is 100%.

Impacted Files	Coverage Δ
src/index.js	100% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 566c777...38c11c7. Read the comment docs.

[tleunen]

Thank for the contribution @rohmanhm. Most users will probably only have one custom root directory, so supporting a string is nice.

cc @fatfisz I believe you also wanted the feature.

[fatfisz]

Hey, thanks for the contribution!

This is looking ok, though the tests could be improved a bit. The test that you've added does not actually involve resolving the path, so the only thing that's being tested is the lack of an error when trying to run reduce over a string.

My suggestion is to replace arrays in the whole test suite with a string root (I did that in my fork too), and add one test that shows that two roots are working (currently there is no test like that).

Would you be willing to do that @rohmanhm?

[rohmanhm]

My suggestion is to replace arrays in the whole test suite with a string root

I'm done.

[fatfisz]

Ok, I think this can be merged!