Contribution Date
Technology
Contribution Project
Contribution Details
> is it worth defaulting $expectedDestination to [web-root] as this is where the bulk of the items live and are likely to in the future
I am not very sure about this. Since this is a test method and not an API, I think it's worth being explicit rather than making it the default. I am not very strongly opinionated about this though.
Meanwhile, I am adding the param docblock for the new parameter in that method.
Contribution Issue Link
Contribution Link
Files count
2
Patches count
1