Include extra tests for time limits

[mmusgrov]

Add a test for timelimits on LRAs. Check the behaviour for both the LRA.Type.SUPPORTS and
LRA.Type.REQUIRED annotations.

[rdebusscher]

Add it to the list of TCK tests, see #92. And then this can be closed.

[mmusgrov]

If we don't have separate issues for tests where do we discuss the behaviour of the testcase before raising PR's?

[xstefank]

@mmusgrov you can discuss it in the issue #92 or on gitter/hangout. #92 should only act as a tracker so if you really feel the need to have a separate discussion in separate issue like this one I don't see any problem with it, just please link this issue as well as PR to the particular line of the test case in #92.

[mmusgrov]

Putting every discussion on #92 will become very difficult to parse, particularly if more than one TCK test is being discussed.

[xstefank]

@mmusgrov ok, you are right, if there is need to discuss some tests then a separate issue should be created. So far the scenarios were pretty necessary and straightforward but we should keep things clear and separated. And use #92 only for tracking purposes.

[mmusgrov]

For example the first test in #92 just says timelimit and I wanted to check whether there was any overlap with #6 and there is nowhere I can go to find out other than looking at the code.

[xstefank]

I believe this particular one was migrated from the sandbox when you first moved the code base. But of course if there is an issue open just link it to the tracker.

[rdebusscher]

Some tests related to #233 to test timeout related stuff.

Test A

Service A - Timeout 1 Hour
Service B - Type.MANDATORY - TimeOut 500 ms
Service A calls Service B

Test calls service A, wait 1 sec, trigger recovery and verifies if LRA is cancelled.

Test B

Service A - Timeout 500 ms
Service B - Type.MANDATORY``
Service A calls Service B after it has waited 1 sec. Service A returns http Status from the call to Service B.

test calls A and verifies if return is status 412 (precondition failed) since LRA is not active when Service B endpoint is called.

[mmusgrov]

Also include a test that verifies that the time limit is respected after a system crash. When the system restarts the time limit should become active again. If the time limit has already passed when the system comes back up then the test should check that the compensation callbacks are triggered.

[tomjenkinson]

Also include a test that verifies that the time limit is respected after a system crash. When the system restarts the time limit should become active again. If the time limit has already passed when the system comes back up then the test should verify that the completing the transaction is not possible.

[mmusgrov]

PR merged