[GitHub] [geode] kirklund commented on pull request #2820: Feature/geode 6016

["kirklund (GitHub)" <gi...@apache.org>]

You should either add a "fail" statement in before the catch-block or change this to use AssertJ's catchThrowable:
```
import static org.assertj.core.api.Assertions.catchThrowable;

Throwable thrown = catchThrowable(() -> txManager.rollback());
assertThat(thrown).isInstanceOf(FunctionException.class);
assertThat(thrown.getCause()).isInstanceOf(ServerOperationException.class);
assertThat(thrown.getCause().getCause()).isInstanceOf(FunctionException.class);
assertThat(thrown.getCause().getCause().getCause()).isInstanceOf(TransactionDataRebalancedException.class);
etc
```

[ Full content available at: https://github.com/apache/geode/pull/2820 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org