You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by cz...@apache.org on 2016/11/05 16:04:01 UTC

svn commit: r1768234 - /sling/trunk/installer/factories/configuration/src/test/java/org/apache/sling/installer/factories/configuration/impl/ConfigUtilTest.java

Author: cziegeler
Date: Sat Nov  5 16:04:01 2016
New Revision: 1768234

URL: http://svn.apache.org/viewvc?rev=1768234&view=rev
Log:
SLING-6247 : Potential class cast exception for primitive arrays

Modified:
    sling/trunk/installer/factories/configuration/src/test/java/org/apache/sling/installer/factories/configuration/impl/ConfigUtilTest.java

Modified: sling/trunk/installer/factories/configuration/src/test/java/org/apache/sling/installer/factories/configuration/impl/ConfigUtilTest.java
URL: http://svn.apache.org/viewvc/sling/trunk/installer/factories/configuration/src/test/java/org/apache/sling/installer/factories/configuration/impl/ConfigUtilTest.java?rev=1768234&r1=1768233&r2=1768234&view=diff
==============================================================================
--- sling/trunk/installer/factories/configuration/src/test/java/org/apache/sling/installer/factories/configuration/impl/ConfigUtilTest.java (original)
+++ sling/trunk/installer/factories/configuration/src/test/java/org/apache/sling/installer/factories/configuration/impl/ConfigUtilTest.java Sat Nov  5 16:04:01 2016
@@ -88,4 +88,21 @@ public class ConfigUtilTest {
         assertTrue(ConfigUtil.isSameData(a, b));
         assertTrue(ConfigUtil.isSameData(b, a));
     }
+
+    @Test public void testIsSameDataWithPrimitiveArrays() throws Exception {
+        final Dictionary<String, Object> a = new Hashtable<String, Object>();
+        final Dictionary<String, Object> b = new Hashtable<String, Object>();
+
+        a.put("b", new int[] {1,2,3});
+        b.put("b", a.get("b"));
+
+        a.put("c", new long[] {1L,2L,3L});
+        b.put("c", a.get("c"));
+
+        a.put("d", new int[] {1,2,3});
+        b.put("d", new String[] {"1", "2", "3"});
+
+        assertTrue(ConfigUtil.isSameData(a, b));
+        assertTrue(ConfigUtil.isSameData(b, a));
+    }
 }