beobal commented on a change in pull request #1208:
URL: https://github.com/apache/cassandra/pull/1208#discussion_r732053985
##########
File path:
test/unit/org/apache/cassandra/auth/CassandraNetworkAuthorizerTest.java
##########
@@ -63,13 +62,12 @@ public static void setupSuperUser()
public static void defineSchema() throws ConfigurationException
{
SchemaLoader.prepareServer();
- SchemaLoader.setupAuth(new LocalCassandraRoleManager(),
- new PasswordAuthenticator(),
- new AuthTestUtils.LocalCassandraAuthorizer(),
- new
AuthTestUtils.LocalCassandraNetworkAuthorizer());
+ SchemaLoader.setupAuth(new AuthTestUtils.LocalCassandraRoleManager(),
Review comment:
There is some documentation on this
[here](https://cassandra.apache.org/_/development/code_style.html), although
when I went to check it, it appears to have gotten a bit mangled over the
course of a few site updates.
The alignment as illustrated in the style guide looks more like
yours/Intellij's, but previously this was specified in the way it was initially
[(see
history)](https://web.archive.org/web/20191020015442/http://cassandra.apache.org/doc/latest/development/code_style.html)
The code style page does include links to style configs for Eclipse and
Intellij, but I don't know whether these are compatible with the latest
versions (the link to the Intellij jar is mostly broken anyway, requiring a few
extra clicks). I'll open a Jira / PR to update the docs.
--
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]
