WW-5363 Add test coverage

plugins/velocity/src/main/java/org/apache/struts2/views/velocity/StrutsVelocityContext.java

@@ -65,7 +65,7 @@ public boolean internalContainsKey(String key) {
     }
 
     public Object internalGet(String key) {
-        return chainedGets(key, super::internalGet, this::chainedContextGet, this::stackGet, this::stackContextGet);
+        return chainedGets(key, this::chainedContextGet, this::stackGet, this::stackContextGet);
     }
 
     @SafeVarargs

plugins/velocity/src/test/java/org/apache/struts2/views/velocity/StrutsVelocityContextTest.java

@@ -0,0 +1,70 @@
+package org.apache.struts2.views.velocity;
+
+import com.opensymphony.xwork2.util.ValueStack;
+import org.apache.velocity.VelocityContext;
+import org.junit.Before;
+import org.junit.Rule;
+import org.junit.Test;
+import org.mockito.Mock;
+import org.mockito.junit.MockitoJUnit;
+import org.mockito.junit.MockitoRule;
+
+import java.util.HashMap;
+import java.util.Map;
+
+import static java.util.Collections.singletonList;
+import static org.junit.Assert.assertEquals;
+import static org.mockito.Mockito.when;
+
+public class StrutsVelocityContextTest {
+
+    @Rule
+    public MockitoRule mockitoRule = MockitoJUnit.rule();
+
+    private StrutsVelocityContext strutsVelocityContext;
+
+    @Mock
+    private VelocityContext chainedContext;
+
+    @Mock
+    private ValueStack stack;
+
+    private Map<String, Object> stackContext;
+
+    @Before
+    public void setUp() throws Exception {
+        stackContext = new HashMap<>();
+        when(stack.getContext()).thenReturn(stackContext);
+        strutsVelocityContext = new StrutsVelocityContext(singletonList(chainedContext), stack);
+    }
+
+    @Test
+    public void getChainedValue() {
+        when(chainedContext.internalGet("foo")).thenReturn("bar");
+        assertEquals("bar", strutsVelocityContext.internalGet("foo"));
+    }
+
+    @Test
+    public void getStackValue() {
+        when(stack.findValue("foo")).thenReturn("bar");
+        assertEquals("bar", strutsVelocityContext.internalGet("foo"));
+    }
+
+    @Test
+    public void getStackContextValue() {
+        stackContext.put("foo", "bar");
+        assertEquals("bar", strutsVelocityContext.internalGet("foo"));
+    }
+
+    @Test
+    public void getValuePrecedence() {
+        when(chainedContext.internalGet("foo")).thenReturn("bar");
+        assertEquals("bar", strutsVelocityContext.internalGet("foo"));
+
+        when(stack.findValue("foo")).thenReturn("baz");
+        assertEquals("bar", strutsVelocityContext.internalGet("foo"));
+
+        stackContext.put("foo", "qux");
+        assertEquals("bar", strutsVelocityContext.internalGet("foo"));
+    }
+}