Do not remove type arguments within ternary

Fixes #164

Author: timtebeek

src/main/java/org/openrewrite/staticanalysis/UnnecessaryExplicitTypeArguments.java

@@ -48,6 +48,11 @@ public J.MethodInvocation visitMethodInvocation(J.MethodInvocation method, Execu
                 }
 
                 Object enclosing = getCursor().getParentTreeCursor().getValue();
+
+                if (enclosing instanceof J.Ternary) {
+                    return m; // may be necessary for type inference
+                }
+
                 JavaType inferredType = null;
                 if (enclosing instanceof J.MethodInvocation) {
                     if (shouldRetainOnStaticMethod(methodType)) {

src/test/java/org/openrewrite/staticanalysis/UnnecessaryExplicitTypeArgumentsTest.java

@@ -139,7 +139,6 @@ void test() {
         );
     }
 
-    @ExpectedToFail
     @Issue("https://github.com/openrewrite/rewrite-static-analysis/issues/164")
     @Test
     void doesNotRemoveNecessaryTypeArguments() {
@@ -150,8 +149,8 @@ void doesNotRemoveNecessaryTypeArguments() {
               import java.util.Optional;
               import java.util.stream.Stream;
               public class Test {
-                  void test() {
-                      Stream.of("hi")
+                  void test(String s) {
+                      Stream.of(s)
                               .map(it -> it == null ? Optional.<String>empty() : Optional.of(it))
                               .flatMap(Optional::stream)
                               .map(this::mapper); //this requires the type information