Fix #708 (#1495)

Author: cowtowncoder

release-notes/VERSION-2.x

@@ -18,8 +18,11 @@ a pure JSON library.
 
 #363: UTF-8 decoding should fail on Surrogate characters (0xD800 - 0xDFFF)
  (fix by @cowtowncoder, w/ Claude code)
-#1180: `JsonLocation` off for unrecognized tokens
+#708: `FilteringParserDelegate` cannot deserialize empty list
+ (reported by @xiazuojie)
  (fix by @cowtowncoder, w/ Claude code)
+#1180: `JsonLocation` off for unrecognized tokens
+(fix by @cowtowncoder, w/ Claude code)
 #1470: Add method `copyCurrentStructureExact()` to `JsonGenerator`
  (contributed by Lars H)
 #1477: Add `JsonGenerator.has(StreamWriteCapability)` convenience method

src/main/java/com/fasterxml/jackson/core/filter/FilteringParserDelegate.java

@@ -671,7 +671,7 @@ protected final JsonToken _nextToken2() throws IOException
                     boolean returnEnd = _headContext.isStartHandled();
                     f = _headContext.getFilter();
                     if ((f != null) && (f != TokenFilter.INCLUDE_ALL)) {
-                        boolean includeEmpty = f.includeEmptyArray(_headContext.hasCurrentName());
+                        boolean includeEmpty = f.includeEmptyObject(_headContext.hasCurrentName());
                         f.filterFinishObject();
                         if (includeEmpty) {
                             return _nextBuffered(_headContext);

src/test/java/com/fasterxml/jackson/core/tofix/ParserFilterEmpty708Test.java renamed to src/test/java/com/fasterxml/jackson/core/filter/ParserFilterEmpty708Test.java

@@ -1,12 +1,9 @@
-package com.fasterxml.jackson.core.tofix;
+package com.fasterxml.jackson.core.filter;
 
 import org.junit.jupiter.api.Test;
 
 import com.fasterxml.jackson.core.*;
-import com.fasterxml.jackson.core.filter.FilteringParserDelegate;
-import com.fasterxml.jackson.core.filter.TokenFilter;
 import com.fasterxml.jackson.core.filter.TokenFilter.Inclusion;
-import com.fasterxml.jackson.core.testutil.failure.JacksonTestFailureExpected;
 
 import static org.junit.jupiter.api.Assertions.assertNull;
 
@@ -17,7 +14,17 @@ class ParserFilterEmpty708Test extends JUnit5TestBase
     static class IncludeAllFilter extends TokenFilter {
         @Override
         public TokenFilter includeProperty(String name) {
-            return this;
+            return TokenFilter.INCLUDE_ALL;
+        }
+
+        @Override
+        public boolean includeEmptyArray(boolean contentsFiltered) {
+            return true;
+        }
+
+        @Override
+        public boolean includeEmptyObject(boolean contentsFiltered) {
+            return true;
         }
     }
 
@@ -30,12 +37,10 @@ public TokenFilter includeProperty(String name) {
     private final JsonFactory JSON_F = newStreamFactory();
 
     // [core#708]
-    @JacksonTestFailureExpected
     @Test
     void emptyArray() throws Exception
     {
         final String json = "[ ]";
-        // should become: {"value":12}
         JsonParser p0 = _createParser(JSON_F, json);
         JsonParser p = new FilteringParserDelegate(p0,
                 new IncludeAllFilter(),
@@ -50,12 +55,10 @@ void emptyArray() throws Exception
     }
 
     // [core#708]
-    @JacksonTestFailureExpected
     @Test
     void emptyObject() throws Exception
     {
         final String json = "{ }";
-        // should become: {"value":12}
         JsonParser p0 = _createParser(JSON_F, json);
         JsonParser p = new FilteringParserDelegate(p0,
                 new IncludeAllFilter(),