dcapwell commented on code in PR #3785:
URL: https://github.com/apache/cassandra/pull/3785#discussion_r1913645183
##########
test/unit/org/apache/cassandra/utils/GeneratorsTest.java:
##########
@@ -45,4 +55,42 @@ public void dnsDomainName()
{
qt().forAll(Generators.DNS_DOMAIN_NAME).checkAssert(InternetDomainName::from);
}
+
+ @Test
+ public void asciiDeterministic()
Review Comment:
I thought I found a deterministic issue with text but it ended up being an
issue with IntelliJ displaying texts... these tests didn't solve any problem
but did tell me that could be the issue...
by default text types now display a descriptor rather than the raw text, and
lets you override and show the text with a flag... one issue is that the text
displayed by IntelliJ might not be the exact string which can make repos
harder....
--
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]
