Uploaded image for project: 'Causeway'
  1. Causeway
  2. CAUSEWAY-1508

The integration test framework should be more resilient to badly behaved tests that leave the transaction incomplete

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • 1.13.0
    • 1.13.1
    • Core
    • None

    Description

      Most notably, it's possible for a test to fail an assertion, meaning that the test method is exited, while there are still pending SQL statements to be flushed.

      The current clean up processing (in the IsisTransactionRule) just does a close session, and the first thing that does is close the current transaction if any, by (attempting to commit). However, if that commit in turn throws an exception, then remaining close tran/close session functionality is skipped. The net result is that the transaction is left incomplete (transactionLevel=1) with the currentTransaction in a state of MUST_ABORT.

      Every subsequent test, when it attempts to start its own transaction, immediately finds there's already a MUST_ABORT transaction in place, and fails immediately.

      The net effect is that, once one test has failed in this way, then all other tests in the test suite will also fail.

      The fix is just to do the clean up more robustly in IsisTransactionRule.

      Attachments

        Activity

          People

            danhaywood Daniel Keir Haywood
            danhaywood Daniel Keir Haywood
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: