added testcase for WW-4312 / apache#69

vbhvrn16vaib · Jan 5, 2016 · 4f29d88 · 4f29d88
1 parent f457c77
commit 4f29d88
Showing 1 changed file with 38 additions and 0 deletions.
diff --git a/core/src/test/java/org/apache/struts2/components/IteratorComponentTest.java b/core/src/test/java/org/apache/struts2/components/IteratorComponentTest.java
@@ -76,6 +76,44 @@ public void testIteratorWithBegin() throws Exception {
         assertEquals("item2 item3 item4 ", out.getBuffer().toString());
     }
 
+    public void testIteratorWithNulls() throws Exception {
+        // given
+        final ValueStack stack = ActionContext.getContext().getValueStack();
+        stack.push(new FooAction() {
+            private List items  = Arrays.asList("1", "2", null, "4");
+
+            public List getItems() {
+                return items;
+            }
+        });
+
+        StringWriter out = new StringWriter();
+
+        IteratorComponent ic = new IteratorComponent(stack);
+        ic.setValue("items");
+        ic.setVar("val");
+        Property prop = new Property(stack);
+
+        ic.getComponentStack().push(prop);
+        ic.getComponentStack().push(prop);
+        ic.getComponentStack().push(prop);
+        ic.getComponentStack().push(prop);
+
+        String body = ", ";
+
+        // when
+        assertTrue(ic.start(out));
+
+        for (int i = 0; i < 4; i++) {
+            prop.start(out);
+            prop.end(out, body);
+            ic.end(out, null);
+        }
+
+        // then
+        assertEquals("1, 2, , 4, ", out.getBuffer().toString());
+    }
+
     static class FooAction {
 
         private List items;