Merge pull request #275 from java-operator-sdk/fix-uncheck-warnings

Fix uncheck warnings

Author: s-soroosh

operator-framework/src/main/java/io/javaoperatorsdk/operator/config/runtime/ControllerAnnotationProcessor.java

@@ -4,6 +4,7 @@
 import static io.javaoperatorsdk.operator.config.runtime.RuntimeControllerMetadata.DONEABLES_RESOURCE_PATH;
 
 import com.google.auto.service.AutoService;
+import com.squareup.javapoet.AnnotationSpec;
 import com.squareup.javapoet.ClassName;
 import com.squareup.javapoet.JavaFile;
 import com.squareup.javapoet.MethodSpec;
@@ -35,7 +36,7 @@
 import javax.tools.JavaFileObject;
 
 @SupportedAnnotationTypes("io.javaoperatorsdk.operator.api.Controller")
-@SupportedSourceVersion(SourceVersion.RELEASE_8)
+@SupportedSourceVersion(SourceVersion.RELEASE_11)
 @AutoService(Processor.class)
 public class ControllerAnnotationProcessor extends AbstractProcessor {
 
@@ -106,12 +107,8 @@ private void generateDoneableClass(TypeElement controllerClassSymbol) {
         controllersResourceWriter.add(
             controllerClassSymbol.getQualifiedName().toString(),
             CustomResource.class.getCanonicalName());
-        return;
-      }
-
-      if (resourceType == null) {
         System.out.println(
-            "No defined a resource type for '"
+            "No defined resource type for '"
                 + controllerClassSymbol.getQualifiedName()
                 + "': ignoring!");
         return;
@@ -149,6 +146,10 @@ private void generateDoneableClass(TypeElement controllerClassSymbol) {
                 .addParameter(customResourceType, "resource")
                 .addParameter(Function.class, "function")
                 .addStatement("super(resource,function)")
+                .addAnnotation(
+                    AnnotationSpec.builder(SuppressWarnings.class)
+                        .addMember("value", "$S", "unchecked")
+                        .build())
                 .build();
 
         final TypeSpec typeSpec =

operator-framework/src/test/resources/compile-fixtures/ControllerImplemented2InterfacesExpected.java

@@ -2,8 +2,10 @@
 
 import io.fabric8.kubernetes.api.builder.Function;
 import io.fabric8.kubernetes.client.CustomResourceDoneable;
+import java.lang.SuppressWarnings;
 
 public class MyCustomResourceDoneable extends CustomResourceDoneable<ControllerImplemented2Interfaces.MyCustomResource> {
+    @SuppressWarnings("unchecked")
     public MyCustomResourceDoneable(ControllerImplemented2Interfaces.MyCustomResource resource, Function function) {
         super(resource, function);
     }

operator-framework/src/test/resources/compile-fixtures/ControllerImplementedIntermediateAbstractClassExpected.java

@@ -2,8 +2,10 @@
 
 import io.fabric8.kubernetes.api.builder.Function;
 import io.fabric8.kubernetes.client.CustomResourceDoneable;
+import java.lang.SuppressWarnings;
 
 public class MyCustomResourceDoneable extends CustomResourceDoneable<AbstractController.MyCustomResource> {
+    @SuppressWarnings("unchecked")
     public MyCustomResourceDoneable(AbstractController.MyCustomResource resource, Function function) {
         super(resource, function);
     }

operator-framework/src/test/resources/compile-fixtures/MultilevelControllerExpected.java

@@ -2,8 +2,10 @@
 
 import io.fabric8.kubernetes.api.builder.Function;
 import io.fabric8.kubernetes.client.CustomResourceDoneable;
+import java.lang.SuppressWarnings;
 
 public class MyCustomResourceDoneable extends CustomResourceDoneable<MultilevelController.MyCustomResource> {
+    @SuppressWarnings("unchecked")
     public MyCustomResourceDoneable(MultilevelController.MyCustomResource resource, Function function) {
         super(resource, function);
     }