You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by co...@apache.org on 2012/11/21 15:09:02 UTC

svn commit: r1412113 - /cxf/branches/2.6.x-fixes/systests/ws-security/src/test/java/org/apache/cxf/systest/ws/gcm/GCMTest.java

Author: coheigea
Date: Wed Nov 21 14:09:02 2012
New Revision: 1412113

URL: http://svn.apache.org/viewvc?rev=1412113&view=rev
Log:
Merged revisions 1412109 via  git cherry-pick from
https://svn.apache.org/repos/asf/cxf/trunk

........
  r1412109 | coheigea | 2012-11-21 14:04:20 +0000 (Wed, 21 Nov 2012) | 2 lines

  Updated disabled tests with a bit more info on the cause

........

Modified:
    cxf/branches/2.6.x-fixes/systests/ws-security/src/test/java/org/apache/cxf/systest/ws/gcm/GCMTest.java

Modified: cxf/branches/2.6.x-fixes/systests/ws-security/src/test/java/org/apache/cxf/systest/ws/gcm/GCMTest.java
URL: http://svn.apache.org/viewvc/cxf/branches/2.6.x-fixes/systests/ws-security/src/test/java/org/apache/cxf/systest/ws/gcm/GCMTest.java?rev=1412113&r1=1412112&r2=1412113&view=diff
==============================================================================
--- cxf/branches/2.6.x-fixes/systests/ws-security/src/test/java/org/apache/cxf/systest/ws/gcm/GCMTest.java (original)
+++ cxf/branches/2.6.x-fixes/systests/ws-security/src/test/java/org/apache/cxf/systest/ws/gcm/GCMTest.java Wed Nov 21 14:09:02 2012
@@ -64,6 +64,9 @@ public class GCMTest extends AbstractBus
     public void testAESGCM128() throws Exception {
         //
         // This test fails with the IBM JDK 7
+        // IBM JDK 7 appears to require a GCMParameter class to be used, which
+        // only exists in JDK 7. The Sun JDK appears to be more lenient and 
+        // allows us to use the existing IVParameterSpec class.
         //
         if ("IBM Corporation".equals(System.getProperty("java.vendor"))
             && System.getProperty("java.version") != null
@@ -98,6 +101,9 @@ public class GCMTest extends AbstractBus
         
         //
         // This test fails with the IBM JDK 7
+        // IBM JDK 7 appears to require a GCMParameter class to be used, which
+        // only exists in JDK 7. The Sun JDK appears to be more lenient and 
+        // allows us to use the existing IVParameterSpec class.
         //
         if ("IBM Corporation".equals(System.getProperty("java.vendor"))
             && System.getProperty("java.version") != null
@@ -132,6 +138,9 @@ public class GCMTest extends AbstractBus
         
         //
         // This test fails with the IBM JDK 7
+        // IBM JDK 7 appears to require a GCMParameter class to be used, which
+        // only exists in JDK 7. The Sun JDK appears to be more lenient and 
+        // allows us to use the existing IVParameterSpec class.
         //
         if ("IBM Corporation".equals(System.getProperty("java.vendor"))
             && System.getProperty("java.version") != null