False Positive for Rule java:S2259: “A NullPointerException could be thrown; map() can return null.”

[ganncamp (G Ann Campbell)]

Hi,

You haven’t provided the rule ID, so I can’t be sure, but I suspect this is the same issue…?

[java:S3655] Checking isPresent() after Optional.map() should imply that the original is present

Reproduced this in SonarLint for Eclipse 10.8.0.82289. There are cases where the rule “S3655: Optional value should only be accessed after calling isPresent()” produces a false-positive when we could logically reason that the accessed value is present at that point. Take the following simplified example: private String test() { Optional<String> opt = getOptString(); // Get an Optional from somewhere. if ( opt.map( String::toLowerCase ).filter( s -> s.equals( "test" ) ).isPresent()…

 
Ann