Revert #2805, to fix #2878

Author: cowtowncoder

release-notes/VERSION-2.x

@@ -4,7 +4,13 @@ Project: jackson-databind
 === Releases === 
 ------------------------------------------------------------------------
 
-2.12.0 (not yet released)
+
+2.12.0-rc2 (not yet released)
+
+#2878: Revert change initially made to fix #2805: change in signature
+  of `ObjectMapper.treeToValue()` regarding exceptions
+
+2.12.0-rc1 (12-Oct-2020)
 
 #43: Add option to resolve type from multiple existing properties,
   `@JsonTypeInfo(use=DEDUCTION)`
@@ -65,7 +71,6 @@ Project: jackson-databind
  (reported by mjustin@github)
 #2800: Extract getter/setter/field name mangling from `BeanUtil` into
   pluggable `AccessorNamingStrategy`
-#2805: Remove `JsonProcessingException` from `ObjectMapper.treeToValue()`
 #2873: `MapperFeature.ACCEPT_CASE_INSENSITIVE_ENUMS` should work for enum as keys
  (fix contributed by Ilya G)
 - Add `BeanDeserializerBase.isCaseInsensitive()`

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

@@ -3206,11 +3206,17 @@ public JsonParser treeAsTokens(TreeNode n) {
      *<pre>
      *   objectMapper.convertValue(n, valueClass);
      *</pre>
+     *<p>
+     * Note: erroneously includes {@code throws JsonProcessingException} even
+     * though this exception is never thrown, only {@link IllegalArgumentException}
+     * is thrown. But since changing exception part of method signature is
+     * source-incompatible change, needs to be retained until 3.0.
      */
     @SuppressWarnings("unchecked")
     @Override
     public <T> T treeToValue(TreeNode n, Class<T> valueType)
-        throws IllegalArgumentException
+        throws IllegalArgumentException,
+            JsonProcessingException // is NOT actually thrown but retained as per #2878
     {
         if (n == null) {
             return null;

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

@@ -316,7 +316,7 @@ public void testBufferedLongViaCoercion() throws Exception {
         assertEquals(Long.valueOf(EXP), obj.longObj);
     }
 
-    public void testConversionsOfNull()
+    public void testConversionsOfNull() throws Exception
     {
         // First: `null` value should become `NullNode`
         JsonNode n = MAPPER.valueToTree(null);