QuarkusTransaction.isActive() should be compared with Status.STATUS_ACTIVE

According to @mmusgrov:

> QuarkusTransaction.isActive() is, in my opinion, incorrectly coded since it returns true if getStatus() returns anything other than Status.STATUS_NO_TRANSACTION but is should be comparing with Status.STATUS_ACTIVE

Also included a `getStatus()` method for fine-grained use cases

Author: gastaldi

extensions/narayana-jta/runtime/src/main/java/io/quarkus/narayana/jta/QuarkusTransaction.java

@@ -69,12 +69,23 @@ static void rollback() {
 
     /**
      * If a transaction is active.
+     * A transaction is considered active after it has been started and prior to a Coordinator issuing any prepares, unless the
+     * transaction has been marked for rollback
      *
-     * @return {@code true} if the transaction is active.
+     * @return {@code true} if the transaction is in the {@link Status#STATUS_ACTIVE} state.
      */
     static boolean isActive() {
+        return getStatus() == Status.STATUS_ACTIVE;
+    }
+
+    /**
+     * Returns the status of the current transaction.
+     *
+     * @return The status of the current transaction based on the {@link Status} constants.
+     */
+    static int getStatus() {
         try {
-            return UserTransaction.userTransaction().getStatus() != Status.STATUS_NO_TRANSACTION;
+            return UserTransaction.userTransaction().getStatus();
         } catch (SystemException e) {
             throw new QuarkusTransactionException(e);
         }
@@ -86,11 +97,7 @@ static boolean isActive() {
      * @return If the transaction has been marked for rollback
      */
     static boolean isRollbackOnly() {
-        try {
-            return UserTransaction.userTransaction().getStatus() == Status.STATUS_MARKED_ROLLBACK;
-        } catch (SystemException e) {
-            throw new QuarkusTransactionException(e);
-        }
+        return getStatus() == Status.STATUS_MARKED_ROLLBACK;
     }
 
     /**