Replace Guava Files management with STL (refs #2111)

[tomasbjerre]

Removing imports on Guava Files and using Java standard library.

References

#2111

Submitter checklist

• The PR request is well described and justified, including the body and the references
• The PR title represents the desired changelog entry
• The repository's code style is followed (see the contributing guide)
• Test coverage that demonstrates that the change works as expected
• For new features, there's necessary documentation in this pull request or in a subsequent PR to wiremock.org

[oleg-nenashev]

Thanks! When you use Paths.get() please make sure to catch InvalidPathException in all usages Oracle was so kind to make it a runtime exception instead of IOException, and we had a bunch of bugs in Jenkins because of that

[tomasbjerre]

Thanks! When you use Paths.get() please make sure to catch InvalidPathException in all usages Oracle was so kind to make it a runtime exception instead of IOException, and we had a bunch of bugs in Jenkins because of that

Good to know! When I looked at the individual changes here I cannot see that catching InvalidPathException would change any behavior.

In production code AbstractFileSource was throwing RuntimeException before and will do it now.

In test cases I usually just let it crash with whatever exception is thrown, it seems to me that this is the strategy here as well.

[oleg-nenashev]

Well, you're totally right. Current handling is not what I would expect, but indeed the change does not alter the behavior. Let's merge it, and thanks!