Small nit pick stuff... can you delete these imports and use directly `assertThat(condition).isFalse()` or `assertThat(condition).isTrue()`?. No point in adding a dependency to `org.junit.Assert` if the class already imports `org.assertj.core.api.Assertions.assertThat` (I also think the general consensus is to entirely replace `org.junit.Assert` with `assertj`).
[ Full content available at: https://github.com/apache/geode/pull/4032 ] This message was relayed via gitbox.apache.org for [email protected]
