Merge pull request #47 from matchews/codex/add-xstream-alias-for-response-class

Alias Response element for XStream parsing in OmniLogic binding

Author: matchews

bundles/org.openhab.binding.haywardomnilogiclocal/src/main/java/org/openhab/binding/haywardomnilogiclocal/internal/net/UdpClient.java

@@ -52,6 +52,7 @@ public class UdpClient {
 
     static {
         XSTREAM.allowTypes(new Class[] { Message.class, Parameter.class });
+        XSTREAM.alias("Response", Message.class);
         XSTREAM.setClassLoader(UdpClient.class.getClassLoader());
         XSTREAM.ignoreUnknownElements();
         XSTREAM.processAnnotations(Message.class);

bundles/org.openhab.binding.haywardomnilogiclocal/src/test/java/org/openhab/binding/haywardomnilogiclocal/internal/net/UdpClientTest.java

@@ -14,6 +14,7 @@
 import java.util.Arrays;
 import java.util.concurrent.atomic.AtomicBoolean;
 import java.util.zip.DeflaterOutputStream;
+import java.lang.reflect.Method;
 
 import org.eclipse.jdt.annotation.NonNullByDefault;
 import org.junit.jupiter.api.Test;
@@ -105,6 +106,15 @@ public void sendShouldIgnoreAckAndProcessSubsequentData() throws Exception {
         assertTrue(ackAfterLead.get());
     }
 
+    @Test
+    public void parseIntParameterShouldHandleResponseRoot() throws Exception {
+        Method method = UdpClient.class.getDeclaredMethod("parseIntParameter", String.class, String.class);
+        method.setAccessible(true);
+        String xml = "<Response><Parameter name=\"Test\">42</Parameter></Response>";
+        int value = (int) method.invoke(null, xml, "Test");
+        assertEquals(42, value);
+    }
+
     @Test
     public void sendShouldHandleUncompressedMultiBlockResponse() throws Exception {
         final DatagramSocket server = new DatagramSocket(0);