dsmiley commented on code in PR #2378:
URL: https://github.com/apache/solr/pull/2378#discussion_r1551972946
##########
solr/core/src/test/org/apache/solr/cli/TestExportTool.java:
##########
@@ -218,6 +231,66 @@ public void testVeryLargeCluster() throws Exception {
}
}
+ @Test
+ public void testWithBasicAuth() throws Exception {
+ String COLLECTION_NAME = "secureCollection";
+ String USER = "solr";
+ String PASS = "SolrRocksAgain";
+ final String SECURITY_JSON =
+ Utils.toJSONString(
+ Map.of(
+ "authorization",
+ Map.of(
+ "class",
+ RuleBasedAuthorizationPlugin.class.getName(),
+ "user-role",
+ singletonMap(USER, "admin"),
+ "permissions",
+ singletonList(Map.of("name", "all", "role", "admin"))),
+ "authentication",
+ Map.of(
+ "class",
+ BasicAuthPlugin.class.getName(),
+ "blockUnknown",
+ true,
+ "credentials",
+ singletonMap(USER, getSaltedHashedValue(PASS)))));
+
+ configureCluster(2)
+ .addConfig("conf", configset("cloud-minimal"))
+ .withSecurityJson(SECURITY_JSON)
+ .configure();
+
+ CloudSolrClient cloudSolrClient = cluster.getSolrClient();
+
+ CollectionAdminRequest.createCollection(COLLECTION_NAME, "conf", 8, 1)
Review Comment:
8 shards?!
To tame our tests, tests should have a comment justifying >1 nodes or >1
shards. If there is a reason; justify it.
--
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: issues-unsubscr...@solr.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org
