UnnecessaryThrows still ends up with `Exception 'java.lang.Exception' is never thrown in the method`

[Pankraz76]

Exception 'java.lang.Exception' is never thrown in the method

• https://docs.openrewrite.org/recipes/staticanalysis/unnecessarythrows
• https://github.com/apache/maven/blob/3706aa1e5c7b96632eb0b69e1210f7648df0e981/impl/maven-executor/src/test/java/org/apache/maven/cling/executor/MavenExecutorTestSupport.java#L322

[timtebeek]

Just so we're clear: You're saying we missed out on removing that throws Exception ?

[Pankraz76]

exactly, much was updated but this one not.

[greg-at-moderne]

@Pankraz76 If that's not a problem for you, I suggest to post the context information (like expected vs. observed, or code to reproduce) as text, not screenshots. It's easier if this reporting structure is followed to: understand what's the issue, and decide on the actual expectations, validate the assumptions, etc.

[timtebeek]

Since that method is protected we should not remove the throws declaration as per:

• UnnecessaryThrows should not change API for protected methods #553