From a54e819a6272830098cb50ec1abd75f2501d4993 Mon Sep 17 00:00:00 2001 From: anshum Date: Thu, 18 Feb 2016 16:03:58 -0800 Subject: [PATCH] SOLR-8677: Fix broken build --- .../apache/solr/client/solrj/request/CoreAdminRequest.java | 4 ++-- .../org/apache/solr/client/solrj/request/TestCoreAdmin.java | 6 +++--- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/solr/solrj/src/java/org/apache/solr/client/solrj/request/CoreAdminRequest.java b/solr/solrj/src/java/org/apache/solr/client/solrj/request/CoreAdminRequest.java index 2a2d4ae48b9..c1f986e04e9 100644 --- a/solr/solrj/src/java/org/apache/solr/client/solrj/request/CoreAdminRequest.java +++ b/solr/solrj/src/java/org/apache/solr/client/solrj/request/CoreAdminRequest.java @@ -111,7 +111,7 @@ public class CoreAdminRequest extends SolrRequest { @Override public void setCoreName(String coreName) { if (!SolrIdentifierValidator.validateCoreName(coreName)) { - throw new IllegalArgumentException("Invalid collection: " + coreName + throw new IllegalArgumentException("Invalid core: " + coreName + ". Core names must consist entirely of periods, underscores, and alphanumerics"); } this.core = coreName; @@ -560,7 +560,7 @@ public class CoreAdminRequest extends SolrRequest { public static CoreAdminResponse renameCore(String coreName, String newName, SolrClient client ) throws SolrServerException, IOException { if (!SolrIdentifierValidator.validateCoreName(newName)) { - throw new IllegalArgumentException("Invalid collection: " + newName + throw new IllegalArgumentException("Invalid core: " + newName + ". Core names must consist entirely of periods, underscores, and alphanumerics"); } diff --git a/solr/solrj/src/test/org/apache/solr/client/solrj/request/TestCoreAdmin.java b/solr/solrj/src/test/org/apache/solr/client/solrj/request/TestCoreAdmin.java index 8275ff1270e..143d2c32c3b 100644 --- a/solr/solrj/src/test/org/apache/solr/client/solrj/request/TestCoreAdmin.java +++ b/solr/solrj/src/test/org/apache/solr/client/solrj/request/TestCoreAdmin.java @@ -171,7 +171,7 @@ public class TestCoreAdmin extends AbstractEmbeddedSolrServerTestCase { final String exceptionMessage = e.getMessage(); assertTrue(exceptionMessage.contains("Invalid core")); assertTrue(exceptionMessage.contains("invalid$core@name")); - assertTrue(exceptionMessage.contains("must consist entirely of periods, underscores and alphanumerics")); + assertTrue(exceptionMessage.contains("must consist entirely of periods, underscores, and alphanumerics")); } } @@ -182,9 +182,9 @@ public class TestCoreAdmin extends AbstractEmbeddedSolrServerTestCase { fail(); } catch (IllegalArgumentException e) { final String exceptionMessage = e.getMessage(); - assertTrue(exceptionMessage.contains("Invalid core")); + assertTrue(e.getMessage(), exceptionMessage.contains("Invalid core")); assertTrue(exceptionMessage.contains("invalid$core@name")); - assertTrue(exceptionMessage.contains("must consist entirely of periods, underscores and alphanumerics")); + assertTrue(exceptionMessage.contains("must consist entirely of periods, underscores, and alphanumerics")); } }