fixed @transactional discovery

Author: sterlp

core/src/main/java/org/sterl/spring/persistent_tasks/task/util/ReflectionUtil.java

@@ -12,13 +12,14 @@ public abstract class ReflectionUtil {
 
     public static <A extends Annotation> A getAnnotation(PersistentTask<? extends Serializable> inTask, Class<A> searchFor) {
         var task = AopProxyUtils.ultimateTargetClass(inTask);
-        A result = AnnotationUtils.findAnnotation(task, searchFor);
-        if (result != null) return result;
-
         var targetMethod = ReflectionUtils.findMethod(task, "accept", Serializable.class);
-        if (targetMethod == null) return null;
+        
+        A result = null;
+        // check the method first
+        if (targetMethod != null) result = AnnotationUtils.findAnnotation(targetMethod, searchFor);
+        // check the class if no method annotation was found
+        if (result == null) result = AnnotationUtils.findAnnotation(task, searchFor);
 
-        result = AnnotationUtils.findAnnotation(targetMethod, searchFor);
         return result;
     }
 }

core/src/test/java/org/sterl/spring/persistent_tasks/task/TaskTransactionTest.java

@@ -36,7 +36,9 @@ public void accept(String name) {
             personRepository.save(new PersonBE(name));
         }
     }
+
     @Component("transactionalMethod")
+    @Transactional(timeout = 76, propagation = Propagation.REQUIRED, isolation = Isolation.REPEATABLE_READ)
     @RequiredArgsConstructor
     static class TransactionalMethod implements PersistentTask<String> {
         private final PersonRepository personRepository;
@@ -96,7 +98,7 @@ void testFindTransactionAnnotation() {
         assertThat(a).isNotNull();
         assertThat(a.timeout()).isEqualTo(7);
     }
-    
+
     @Test
     void testGetTransactionTemplate() {
         var a = subject.getTransactionTemplate(transactionalClass);