Stefano Cordio
Stefano Cordio
The problem is at ```java .allSatisfy(elt -> assertThat(elt) ``` where [`assertThat(Future)`](https://www.javadoc.io/doc/org.assertj/assertj-core/latest/org/assertj/core/api/Assertions.html#assertThat(java.util.concurrent.Future)) is selected as `AType` indirectly implements `Future`. The workaround is to use [`assertThatObject`](https://www.javadoc.io/doc/org.assertj/assertj-core/latest/org/assertj/core/api/Assertions.html#assertThatObject(T)) instead. However, this doesn't solve your...
We should probably have better support for `Future` subtypes, like what we have for the `Comparable` ones: https://github.com/assertj/assertj/blob/a6ed772e8afd74727125fd3e82e7c80a6f81de1e/assertj-core/src/main/java/org/assertj/core/api/Assertions.java#L3629-L3631 However, we must carefully evaluate potential breaking changes and see if a...
BTW thanks a lot, @NathanEckert, for the accurate reproducer!
Thanks, @neilwangweili! We'll review it shortly. Two initial thoughts: * I'd avoid additions to the internal package and I'd rather favor package-private classes in the api package, if they are...
No worries, take your time of course, and let's also wait for some thoughts from @joel-costigliola.
Thanks for the offer, @nicokosi! We haven't started development on version 4 so this change will have to wait a little longer.
Hi @etrandafir93, this cannot be picked up until we start work on version 4 which will increase the Java version to 17. I suggest keeping an eye on issues labeled...
Sorry for the slow feedback, @Shreya-7 – yes, it is!
The related change has been reverted in #3318 and the topic should be evaluated again together with #1652 and #2519.
No problem at all, @findepi! Unfortunately, we missed that potential breaking change but now we learned 🙂