[SOLR-13794] Delete solr/core/src/test-files/solr/configsets/_default

• Type: Test
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 8.5, 9.0
• Assignee: Chris M. Hostetter
• Reporter: Chris M. Hostetter
• Created: 26/Sep/19 00:37
• Updated: 02/Jul/20 16:45
• Resolved: 14/Feb/20 22:43

Description

For as long as we've had a _default configset in solr, we've also had a copy of that default in core/src/test-files/ - as well as a unit test that confirms they are identical.

It's never really been clear to me why we have this duplication, instead of just having the test-framework take the necessary steps to ensure that server/solr/configsets/_default is properly used when running tests.

I'd like to propose we eliminate the duplication since it only ever seems to cause problems (notably spurious test failures when people modify the _default configset w/o remembering that they need to make identical edits to the test-files clone) and instead have SolrTestCase set the (already existing & supported) solr.default.confdir system property to point to the (already existing) ExternalPaths.DEFAULT_CONFIGSET

Issue Links

• contains: SOLR-13308 Move SystemPropertiesRestoreRule to SolrTestCase