Merge pull request #139 from jamezp/issue138

[138] Check if the stack trace is empty before attempting to copy it.

Author: jamezp

processor/pom.xml

@@ -196,6 +196,19 @@
                         <expressionPropertiesPath>${expression.properties.path}</expressionPropertiesPath>
                     </systemPropertyVariables>
                 </configuration>
+                <executions>
+                    <execution>
+                        <id>no-stack-trace</id>
+                        <goals>
+                            <goal>test</goal>
+                        </goals>
+                        <phase>test</phase>
+                        <configuration>
+                            <argLine>-XX:-StackTraceInThrowable</argLine>
+                            <groups>NoStackTrace</groups>
+                        </configuration>
+                    </execution>
+                </executions>
             </plugin>
         </plugins>
     </build>

processor/src/main/java/org/jboss/logging/processor/generator/model/ImplementationClassModel.java

@@ -679,7 +679,8 @@ private JCall getCopyStackMethod(final JClassDef classDef) {
             sourceFile._import(arrays);
             final JExpr e = $v(param);
             final JVarDeclaration st = body.var(FINAL, $t(StackTraceElement.class).array(), "st", e.call("getStackTrace"));
-            body.add(e.call("setStackTrace")
+            // Check if the stack trace is an empty array, if not copy the stack trace minus 1
+            body._if($v(st).field("length").gt(JExpr.ZERO)).add(e.call("setStackTrace")
                     .arg(arrays.call("copyOfRange").arg($v(st)).arg(JExpr.ONE).arg($v(st).field("length"))));
         }
         return JExprs.call(methodName);

processor/src/test/java/org/jboss/logging/processor/generated/tests/ThrowableSignatureTest.java

@@ -24,16 +24,17 @@
 import org.jboss.logging.processor.generated.SignatureMessages.RedirectException;
 import org.jboss.logging.processor.generated.SignatureMessages.TestException;
 import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.Tag;
 import org.junit.jupiter.api.Test;
 
 /**
  * @author <a href="mailto:[email protected]">James R. Perkins</a>
  */
+@Tag("NoStackTrace")
 public class ThrowableSignatureTest {
 
     @Test
     public void testSignatures() {
-        @SuppressWarnings("RedundantStringFormatCall")
         final String formattedMessage = String.format(SignatureMessages.TEST_MSG);
         final RuntimeException cause = new RuntimeException("This was the cause");