DATAREST-765 - Polishing.

Removed solrconfig.xml. Minor formatting in solr.xml. Reduced nesting in shutdown handler in SolrInfrastructureConfig.

Original pull request: #209.

spring-data-rest-webmvc/src/test/java/org/springframework/data/rest/webmvc/solr/SolrInfrastructureConfig.java

@@ -58,8 +58,8 @@ public class SolrInfrastructureConfig {
 	private static final Resource SOLR_XML = new ClassPathResource("solr.xml", SolrInfrastructureConfig.class);
 
 	@Bean
-	public SolrClientFactory solrClientFactory(final String solrHomeDir) throws ParserConfigurationException,
-			IOException, SAXException {
+	public SolrClientFactory solrClientFactory(final String solrHomeDir)
+			throws ParserConfigurationException, IOException, SAXException {
 
 		prepareConfiguration(solrHomeDir);
 		return new EmbeddedSolrServerFactory(solrHomeDir);
@@ -104,7 +104,8 @@ public class SolrInfrastructureConfig {
 
 	/**
 	 * {@link SpringJUnit4ClassRunner} executes {@link ClassRule}s before the actual shutdown of the
-	 * {@link ApplicationContext}. This causes the {@link TemporaryFolder} to vanish before Solr can gracefully shutdown.<br />
+	 * {@link ApplicationContext}. This causes the {@link TemporaryFolder} to vanish before Solr can gracefully shutdown.
+	 * <br />
 	 * To prevent error messages popping up we register a {@link CloseHook} re adding the index directory and removing it
 	 * after {@link SolrCore#close()}.
 	 * 
@@ -125,37 +126,42 @@ public class SolrInfrastructureConfig {
 
 					CoreDescriptor cd = core.getCoreDescriptor();
 
-					if (cd != null) {
+					if (cd == null) {
+						return;
+					}
 
-						File tmp = new File(core.getIndexDir()).getParentFile();
+					File tmp = new File(core.getIndexDir()).getParentFile();
 
-						if (!tmp.exists()) {
-							try {
+					if (tmp.exists()) {
+						return;
+					}
 
-								File indexFile = new File(tmp, "index");
-								indexFile.mkdirs();
+					try {
 
-								this.path = indexFile.getPath();
-							} catch (Exception e) {
-								e.printStackTrace();
-							}
-						}
+						File indexFile = new File(tmp, "index");
+						indexFile.mkdirs();
+
+						this.path = indexFile.getPath();
+					} catch (Exception e) {
+						e.printStackTrace();
 					}
 				}
 
 				@Override
 				public void postClose(SolrCore core) {
 
-					if (StringUtils.hasText(this.path)) {
+					if (!StringUtils.hasText(this.path)) {
+						return;
+					}
 
-						File tmp = new File(this.path);
-						if (tmp.exists() && tmp.getPath().startsWith(FileUtils.getTempDirectoryPath())) {
+					File tmp = new File(this.path);
 
-							try {
-								FileUtils.deleteDirectory(tmp);
-							} catch (IOException e) {
-								e.printStackTrace();
-							}
+					if (tmp.exists() && tmp.getPath().startsWith(FileUtils.getTempDirectoryPath())) {
+
+						try {
+							FileUtils.deleteDirectory(tmp);
+						} catch (IOException e) {
+							e.printStackTrace();
 						}
 					}
 				}

spring-data-rest-webmvc/src/test/resources/org/springframework/data/rest/webmvc/solr/solr.xml

@@ -8,8 +8,7 @@
     <bool name="genericCoreNodeNames">${genericCoreNodeNames:true}</bool>
   </solrcloud>
 
-  <shardHandlerFactory name="shardHandlerFactory"
-    class="HttpShardHandlerFactory">
+  <shardHandlerFactory name="shardHandlerFactory" class="HttpShardHandlerFactory">
     <int name="socketTimeout">${socketTimeout:0}</int>
     <int name="connTimeout">${connTimeout:0}</int>
   </shardHandlerFactory>