#15 added support for transactional interfaces

src/main/resources/META-INF/jqassistant-rules/spring-transaction.xml

@@ -53,12 +53,12 @@
 
     <concept id="spring-transaction:TransactionalClass">
         <description>
-            Label all classes annotated with "@org.springframework.transaction.annotation.Transactional", "jakarta.transaction.Transactional",
+            Label all classes directly or indirectly annotated with "@org.springframework.transaction.annotation.Transactional", "jakarta.transaction.Transactional",
             or "javax.transaction.Transactional", and their declared methods with "Spring" and "Transactional".
         </description>
         <cypher><![CDATA[
            MATCH
-             (transactionalClass:Class)-[:EXTENDS*0..]->(:Class)-[:ANNOTATED_BY]->()-[:OF_TYPE]->(annotationType:Type)
+             (transactionalClass:Type)-[:EXTENDS|IMPLEMENTS*0..]->(:Type)-[:ANNOTATED_BY]->()-[:OF_TYPE]->(annotationType:Type)
            WHERE
              annotationType.fqn in [
                "org.springframework.transaction.annotation.Transactional",

src/test/java/org/jqassistant/plugin/spring/test/concept/TransactionalMethodIT.java

@@ -11,7 +11,6 @@
 import static org.hamcrest.CoreMatchers.*;
 import static org.hamcrest.MatcherAssert.assertThat;
 
-
 class TransactionalMethodIT extends AbstractSpringIT {
 
     @Test
@@ -30,19 +29,30 @@ void transactionalMethod() throws Exception {
 
     @Test
     void transactionalClass() throws Exception {
-        scanClasses(SpringTransactionalClass.class, SpringTransactionalSubClass.class, JtaTransactionalClass.class, JtaJakartaTransactionalClass.class);
+        scanClasses(SpringTransactionalClass.class, SpringTransactionalSubClass.class,
+            SpringTransactionalInterface.class, SpringTransactionalImplementingClass.class,
+            JtaTransactionalClass.class, JtaJakartaTransactionalClass.class);
         assertThat(applyConcept("spring-transaction:TransactionalClass").getStatus(), equalTo(SUCCESS));
         store.beginTransaction();
-        List<Object> classes = query("MATCH (c:Spring:Class:Transactional) RETURN c").getColumn("c");
-        assertThat(classes.size(), equalTo(4));
+        List<Object> classes = query("MATCH (c:Spring:Type:Transactional) RETURN c").getColumn("c");
+        assertThat(classes.size(), equalTo(6));
         assertThat(classes, hasItem(typeDescriptor(SpringTransactionalClass.class)));
         assertThat(classes, hasItem(typeDescriptor(SpringTransactionalSubClass.class)));
+        assertThat(classes, hasItem(typeDescriptor(SpringTransactionalInterface.class)));
+        assertThat(classes,
+            hasItem(typeDescriptor(SpringTransactionalImplementingClass.class)));
         assertThat(classes, hasItem(typeDescriptor(JtaTransactionalClass.class)));
         assertThat(classes, hasItem(typeDescriptor(JtaJakartaTransactionalClass.class)));
         List<Object> methods = query("MATCH (m:Spring:Method:Transactional) RETURN m").getColumn("m");
-        assertThat(methods.size(), equalTo(4));
+        assertThat(methods.size(), equalTo(7));
         assertThat(methods, hasItem(methodDescriptor(SpringTransactionalClass.class, "transactionalMethod")));
-        assertThat(methods, hasItem(methodDescriptor(SpringTransactionalSubClass.class, "transactionalSubClassMethod")));
+        assertThat(methods,
+            hasItem(methodDescriptor(SpringTransactionalSubClass.class, "transactionalSubClassMethod")));
+        assertThat(methods, hasItem(methodDescriptor(SpringTransactionalInterface.class, "transactionalMethod")));
+        assertThat(methods,
+            hasItem(methodDescriptor(SpringTransactionalImplementingClass.class, "transactionalMethod")));
+        assertThat(methods,
+            hasItem(methodDescriptor(SpringTransactionalImplementingClass.class, "transactionalImplementingClassMethod")));
         assertThat(methods, hasItem(methodDescriptor(JtaTransactionalClass.class, "transactionalMethod")));
         assertThat(methods, hasItem(methodDescriptor(JtaJakartaTransactionalClass.class, "transactionalMethod")));
         store.commitTransaction();

src/test/java/org/jqassistant/plugin/spring/test/set/transaction/SpringTransactionalImplementingClass.java

@@ -0,0 +1,20 @@
+package org.jqassistant.plugin.spring.test.set.transaction;
+
+import org.springframework.stereotype.Service;
+
+@Service
+public class SpringTransactionalImplementingClass implements SpringTransactionalInterface {
+
+    public static void staticMethod() {
+        // intentionally left blank
+    }
+
+    public void transactionalImplementingClassMethod() {
+        // intentionally left blank
+    }
+
+    @Override
+    public void transactionalMethod() {
+        // intentionally left blank
+    }
+}

src/test/java/org/jqassistant/plugin/spring/test/set/transaction/SpringTransactionalInterface.java

@@ -0,0 +1,10 @@
+package org.jqassistant.plugin.spring.test.set.transaction;
+
+import org.springframework.transaction.annotation.Transactional;
+
+@Transactional
+public interface SpringTransactionalInterface {
+
+    void transactionalMethod();
+
+}