Re: [lucene-solr] branch master updated: SOLR-13783: fix failing tests due to NamedList.toString() change

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

Re: [lucene-solr] branch master updated: SOLR-13783: fix failing tests due to NamedList.toString() change

Tomás Fernández Löbbe
Thanks Munendra,
sorry for the noise.

On Tue, Oct 29, 2019 at 1:06 AM <[hidden email]> wrote:
This is an automated email from the ASF dual-hosted git repository.

munendrasn pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/lucene-solr.git


The following commit(s) were added to refs/heads/master by this push:
     new b82b772  SOLR-13783: fix failing tests due to NamedList.toString() change
b82b772 is described below

commit b82b7725e110cd482c7a4372dc3b1a47eee2023a
Author: Munendra S N <[hidden email]>
AuthorDate: Tue Oct 29 13:35:31 2019 +0530

    SOLR-13783: fix failing tests due to NamedList.toString() change
---
 solr/core/src/test/org/apache/solr/core/TestBadConfig.java     |  2 +-
 .../update/processor/UpdateRequestProcessorFactoryTest.java    | 10 +++++-----
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/solr/core/src/test/org/apache/solr/core/TestBadConfig.java b/solr/core/src/test/org/apache/solr/core/TestBadConfig.java
index db04152..1dfad85 100644
--- a/solr/core/src/test/org/apache/solr/core/TestBadConfig.java
+++ b/solr/core/src/test/org/apache/solr/core/TestBadConfig.java
@@ -82,7 +82,7 @@ public class TestBadConfig extends AbstractBadConfigTestBase {

   public void testSchemaMutableButNotManaged() throws Exception {
     assertConfigs("bad-solrconfig-schema-mutable-but-not-managed.xml",
-                  "schema-minimal.xml", "Unexpected arg(s): {mutable=false,managedSchemaResourceName=schema.xml}");
+                  "schema-minimal.xml", "Unexpected arg(s): {mutable=false, managedSchemaResourceName=schema.xml}");
   }

   public void testManagedSchemaCannotBeNamedSchemaDotXml() throws Exception {
diff --git a/solr/core/src/test/org/apache/solr/update/processor/UpdateRequestProcessorFactoryTest.java b/solr/core/src/test/org/apache/solr/update/processor/UpdateRequestProcessorFactoryTest.java
index 7d53b4f..66d612f 100644
--- a/solr/core/src/test/org/apache/solr/update/processor/UpdateRequestProcessorFactoryTest.java
+++ b/solr/core/src/test/org/apache/solr/update/processor/UpdateRequestProcessorFactoryTest.java
@@ -16,21 +16,21 @@
  */
 package org.apache.solr.update.processor;

-import static org.apache.solr.update.processor.DistributingUpdateProcessorFactory.DISTRIB_UPDATE_PARAM;
-
 import java.lang.invoke.MethodHandles;
-import java.util.Arrays;
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.List;

+import org.apache.solr.SolrTestCaseJ4;
 import org.apache.solr.common.params.ModifiableSolrParams;
 import org.apache.solr.core.SolrCore;
 import org.apache.solr.response.SolrQueryResponse;
-import org.apache.solr.SolrTestCaseJ4;
 import org.junit.BeforeClass;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;

+import static org.apache.solr.update.processor.DistributingUpdateProcessorFactory.DISTRIB_UPDATE_PARAM;
+
 /**
  *
  */
@@ -87,7 +87,7 @@ public class UpdateRequestProcessorFactoryTest extends SolrTestCaseJ4 {
     assertEquals( custom, core.getUpdateProcessingChain( "custom" ) );

     // Make sure the NamedListArgs got through ok
-    assertEquals( "{name={n8=88,n9=99}}", link.args.toString() );
+    assertEquals( "{name={n8=88, n9=99}}", link.args.toString() );
   }

   public void testUpdateDistribChainSkipping() throws Exception {