kotman12 commented on PR #4279: URL: https://github.com/apache/solr/pull/4279#issuecomment-4230616318
Btw @dsmiley I got Claude to factor this as an EmbeddedSolrServerTest [here](https://github.com/kotman12/solr/commit/2f851b94da5d3562bdee0eaef216bbfeba2fb8d9) but I am not sure what to think about it. Part of me thinks the collection-centric language, i.e. `.newCollection()` is a bit weird to put in a test of a feature that is explicitly not supported in cloud mode. That being said it is just a naming thing and the test does work. So its perhaps more of a problem with the solrTestRule. Perhaps I have some trouble grasping the finer details as I've only ever interacted with Solr in Cloud mode. -- 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]
