refactor: move annotation presence check to core

Author: Vitor Hugo Salgado

puma4j-core/src/main/java/io/github/vitorsalgado/puma4j/core/Puma4j.java

@@ -7,11 +7,13 @@
 import com.fasterxml.jackson.datatype.jsr310.JavaTimeModule;
 import com.fasterxml.jackson.module.paramnames.ParameterNamesModule;
 import com.google.gson.Gson;
+import io.github.vitorsalgado.puma4j.annotations.Res;
 import io.github.vitorsalgado.puma4j.core.unmarshallers.BinaryUnmarshaller;
 import io.github.vitorsalgado.puma4j.core.unmarshallers.JsonUnmarshaller;
 import io.github.vitorsalgado.puma4j.core.unmarshallers.PropertiesUnmarshaller;
 import io.github.vitorsalgado.puma4j.core.unmarshallers.TextUnmarshaller;
 import io.github.vitorsalgado.puma4j.core.unmarshallers.YamlUnmarshaller;
+import java.lang.reflect.AnnotatedElement;
 import java.util.Arrays;
 import java.util.Optional;
 
@@ -62,6 +64,10 @@ public static Puma4j instance() {
     return Puma4jSingleton.INSTANCE;
   }
 
+  public static boolean isResourceAnnotationPresent(final AnnotatedElement annotatedElement) {
+    return annotatedElement.isAnnotationPresent(Res.class);
+  }
+
   public Provider resourceProvider() {
     return new DefaultResourceProvider();
   }
@@ -71,27 +77,28 @@ public void registerMarshallerForExtension(
     READER_REGISTRY.registerMarshallerForExtension(extension, unmarshaller);
   }
 
+  @SuppressWarnings("squid:S1452")
   public Optional<Unmarshaller<?>> getMarshallerByExtension(final String extension) {
     return READER_REGISTRY.getMarshallerByExtension(extension);
   }
 
-  public Unmarshaller<?> jsonMarshaller() {
+  public JsonUnmarshaller jsonMarshaller() {
     return this.jsonMarshaller;
   }
 
-  public Unmarshaller<?> yamlMarshaller() {
+  public YamlUnmarshaller yamlMarshaller() {
     return this.yamlMarshaller;
   }
 
-  public Unmarshaller<?> propertiesMarshaller() {
+  public PropertiesUnmarshaller propertiesMarshaller() {
     return this.propertiesMarshaller;
   }
 
-  public Unmarshaller<?> textMarshaller() {
+  public TextUnmarshaller textMarshaller() {
     return this.textMarshaller;
   }
 
-  public Unmarshaller<?> binaryMarshaller() {
+  public BinaryUnmarshaller binaryMarshaller() {
     return this.binaryMarshaller;
   }
 

puma4j-junit5-extension/src/main/java/io/github/vitorsalgado/puma4j/junit5/Puma4jBeforeAllCallback.java

@@ -1,5 +1,7 @@
 package io.github.vitorsalgado.puma4j.junit5;
 
+import static io.github.vitorsalgado.puma4j.core.Puma4j.isResourceAnnotationPresent;
+
 import io.github.vitorsalgado.puma4j.annotations.Res;
 import io.github.vitorsalgado.puma4j.core.Context;
 import io.github.vitorsalgado.puma4j.core.Provider;
@@ -23,7 +25,7 @@ public void beforeAll(final ExtensionContext extensionContext) throws Exception
 
     for (final Field field : extensionContext.getRequiredTestClass().getDeclaredFields()) {
       if (Modifier.isStatic(field.getModifiers())
-          && field.isAnnotationPresent(Res.class)
+          && isResourceAnnotationPresent(field)
           && !Modifier.isFinal(field.getModifiers())
           && !field.isEnumConstant()
           && !field.isSynthetic()) {

puma4j-junit5-extension/src/main/java/io/github/vitorsalgado/puma4j/junit5/Puma4jBeforeEachCallback.java

@@ -1,5 +1,7 @@
 package io.github.vitorsalgado.puma4j.junit5;
 
+import static io.github.vitorsalgado.puma4j.core.Puma4j.isResourceAnnotationPresent;
+
 import io.github.vitorsalgado.puma4j.annotations.Res;
 import io.github.vitorsalgado.puma4j.core.Context;
 import io.github.vitorsalgado.puma4j.core.Provider;
@@ -23,7 +25,7 @@ public void beforeEach(final ExtensionContext extensionContext) throws Exception
 
     for (final Field field : extensionContext.getRequiredTestClass().getDeclaredFields()) {
       if (!Modifier.isStatic(field.getModifiers())
-          && field.isAnnotationPresent(Res.class)
+          && isResourceAnnotationPresent(field)
           && !Modifier.isFinal(field.getModifiers())
           && !field.isEnumConstant()
           && !field.isSynthetic()) {

puma4j-junit5-extension/src/main/java/io/github/vitorsalgado/puma4j/junit5/Puma4jResourceParameterResolver.java

@@ -1,5 +1,7 @@
 package io.github.vitorsalgado.puma4j.junit5;
 
+import static io.github.vitorsalgado.puma4j.core.Puma4j.isResourceAnnotationPresent;
+
 import io.github.vitorsalgado.puma4j.annotations.Res;
 import io.github.vitorsalgado.puma4j.core.Context;
 import io.github.vitorsalgado.puma4j.core.Provider;
@@ -18,7 +20,7 @@ public class Puma4jResourceParameterResolver implements ParameterResolver {
   @Override
   public boolean supportsParameter(
       final ParameterContext parameterContext, final ExtensionContext extensionContext) {
-    return parameterContext.getParameter().isAnnotationPresent(Res.class);
+    return isResourceAnnotationPresent(parameterContext.getParameter());
   }
 
   @Override