fix: rendering of substituted list with spacce (#841)

Author: johanandren

config/src/main/java/com/typesafe/config/impl/ConfigString.java

@@ -86,6 +86,8 @@ protected void render(StringBuilder sb, int indent, boolean atRoot, ConfigRender
             String rendered;
             if (options.getJson())
                 rendered = ConfigImplUtil.renderJsonString(value);
+            else if (!wasQuoted())
+                rendered = value;
             else
                 rendered = ConfigImplUtil.renderStringUnquotedIfPossible(value);
             sb.append(rendered);

config/src/test/scala/com/typesafe/config/impl/ConcatenationTest.scala

@@ -8,8 +8,10 @@ import org.junit._
 import com.typesafe.config.ConfigValue
 import com.typesafe.config.ConfigException
 import com.typesafe.config.ConfigResolveOptions
+import com.typesafe.config.ConfigRenderOptions
 import com.typesafe.config.Config
 import com.typesafe.config.ConfigFactory
+import com.typesafe.config.ConfigParseOptions
 import scala.collection.JavaConverters._
 
 class ConcatenationTest extends TestUtils {
@@ -488,6 +490,32 @@ class ConcatenationTest extends TestUtils {
         assertEquals(List(1, 2), conf.getIntList("x").asScala)
     }
 
+    // https://github.com/lightbend/config/issues/829
+    @Test
+    def renderArrayConcatenationRoundTrip() {
+        val renderOptions = ConfigRenderOptions.defaults
+            .setJson(false)
+            .setOriginComments(false)
+            .setComments(true)
+            .setFormatted(true)
+        val parseOptions = ConfigParseOptions.defaults.setAllowMissing(true)
+
+        val original = """ex1 = [1, 2]
+                         |except = ${ex1} ${ex1}
+                         |""".stripMargin
+        val rendered = ConfigFactory
+            .parseString(original, parseOptions)
+            .root
+            .render(renderOptions)
+
+        // the rendered config should be parseable and resolvable
+        val resolved = ConfigFactory
+            .parseString(rendered, parseOptions.setAllowMissing(false))
+            .resolve()
+
+        assertEquals(List(1, 2, 1, 2), resolved.getIntList("except").asScala.toList)
+    }
+
     // but quoted whitespace should be an error
     @Test
     def concatSubstitutionsThatAreObjectsWithQuotedSpace() {