ReflectionSupport#tryToGetResources never returns a failed Try

[ptomli]

Version: github/main (e1f503f)

Not sure if this is a documentation issue, or a genuine bug. I implemented some code based on the docs and it didn't work for me, so I went digging.

I wanted to be able to do something like

ReflectionSupport.tryToGetResources("blah").toOptional().isPresent();

which would appear to be supported by the documented API, but not the implementation.

Steps to reproduce

Run the attached test

Context

• Build Tool/IDE: IntelliJ

[ptomli]

I fail at GitHub, please see the test commit here

ptomli@9305cd8

[marcphilipp]

I can see that the Javadoc is confusing in this case. The Try will contain an empty set if no resources were found. It will only contain a failure if an exception occurred while trying to list and convert resources from the ClassLoader.

[ptomli]

Thanks for the quick response.

So, the intended pattern (for what I was trying to do) would be to check the size of the (never null) Set is > 0?

[marcphilipp]

So, the intended pattern (for what I was trying to do) would be to check the size of the (never null) Set is > 0?

Exactly! 👍