#275 Fix tests that wrongly assume callbacks are delivered with "exac…

[xstefank]

…tly once" semantics

resolves #275

Changes:

• Complete/Compensate to "at least once" where there is no Status method in the resource
  • otherwise keep exactly-once semantics, we probably need to cover how the implementation should behave if there is the Status method and Complete/Compensate call fails (this will also require spec text changes so I will open a new issue - https://github.com/eclipse/microprofile-lra/issues/283)
• AfterLRA, Forget, Status to "at least once"

Signed-off-by: xstefank xstefank122@gmail.com

[xstefank]

@ochaloup the answer is already in the PR description - Complete/Compensate to "at least once" where there is no Status method in the resource -- all of the tests that you mentioned invoke resources which have a Status method.

[ochaloup]

@xstefank oh... I see. Thanks!

[xstefank]

@mmusgrov This PR has two approvals so I will merge it later today. This is just a notification in case you would still like to take a look.

[xstefank]

@mmusgrov added a comment and a message to the test that requires only a single complete/compensate call.