pkolaczk commented on code in PR #281:
URL: https://github.com/apache/cassandra-dtest/pull/281#discussion_r2270040761
##########
cqlsh_tests/test_cqlsh.py:
##########
@@ -599,7 +599,7 @@ def test_unicode_invalid_request_error(self):
err = node1.run_cqlsh(cmds=cmd, cqlsh_options=options).stderr
if self.cluster.version() >= LooseVersion('4.0'):
- assert "Keyspace name must not be empty, more than 48 characters
long, or contain non-alphanumeric-underscore characters (got 'ä')" in err
+ assert 'Keyspace name must not be empty and must contain
alphanumeric or underscore characters (got "ä")' in err
Review Comment:
This is much better.
Minor nit: add "only" at the end:
`Keyspace name must not be empty and must contain alphanumeric or underscore
characters only`
--
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]
