Fix #4047 (#4050)

Author: JooHyukKim

src/main/java/com/fasterxml/jackson/databind/ObjectMapper.java

@@ -3521,7 +3521,8 @@ public <T extends JsonNode> T valueToTree(Object fromValue)
 
         // inlined 'writeValue' with minor changes:
         // first: disable wrapping when writing
-        final SerializationConfig config = getSerializationConfig().without(SerializationFeature.WRAP_ROOT_VALUE);
+        // [databind#4047] ObjectMapper.valueToTree will ignore the configuration SerializationFeature.WRAP_ROOT_VALUE
+        final SerializationConfig config = getSerializationConfig();
         final DefaultSerializerProvider context = _serializerProvider(config);
 
         // Then create TokenBuffer to use as JsonGenerator

src/test/java/com/fasterxml/jackson/databind/node/TestConversions.java

@@ -1,5 +1,6 @@
 package com.fasterxml.jackson.databind.node;
 
+import com.fasterxml.jackson.annotation.JsonRootName;
 import java.io.IOException;
 import java.math.BigDecimal;
 import java.util.*;
@@ -108,6 +109,13 @@ public void serializeWithType(JsonGenerator g,
         }
     }
 
+    // [databind#4047]
+    @JsonRootName("event")
+    static class Event {
+        public Long id;
+        public String name;
+    }
+
     /*
     /**********************************************************
     /* Unit tests
@@ -363,4 +371,22 @@ public void testNodeConvert() throws Exception
         result = MAPPER.treeToValue(node, MAPPER.constructType(ObjectNode.class));
         assertSame(src, result);
     }
+
+    // [databind#4047] : ObjectMapper.valueToTree will ignore the configuration SerializationFeature.WRAP_ROOT_VALUE
+    public void testValueToTree() throws Exception
+    {
+        // Arrange
+        Event value = new Event();
+        value.id = 1L;
+        value.name = "foo";
+
+        ObjectMapper wrapRootMapper = jsonMapperBuilder()
+                .enable(SerializationFeature.WRAP_ROOT_VALUE)
+                .build();
+
+        // Act & Assert
+        String expected = "{\"event\":{\"id\":1,\"name\":\"foo\"}}";
+        assertEquals(expected, wrapRootMapper.writeValueAsString(value));
+        assertEquals(expected, wrapRootMapper.valueToTree(value).toString());
+    }
 }