dcapwell commented on code in PR #3865:
URL: https://github.com/apache/cassandra/pull/3865#discussion_r1940273211
##########
test/unit/org/apache/cassandra/tcm/ClusterMetadataSerializerTest.java:
##########
@@ -47,16 +47,16 @@ public class ClusterMetadataSerializerTest
public void serdeLatest()
{
DataOutputBuffer output = new DataOutputBuffer();
- qt().forAll(new
ClusterMetadataBuilder().build()).checkAssert(orFail(cm -> {
+ qt().forAll(Generators.toGen(new
ClusterMetadataBuilder().build())).check(cm -> {
Review Comment:
accord `qt` will show the seed that failed when generators fail, but
QuickTheories doesn't; making it hard to reproduce these type of errors.
I plan to migrate all tests off of QuickTheories *after* we merge to trunk,
and these are cep-15-accord only tests....
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
