You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@directory.apache.org by Emmanuel Lécharny <el...@gmail.com> on 2015/01/12 15:49:30 UTC

Re: [3/4] directory-kerberos git commit: Refactored the decryption test codes some bit and disabled DES related encryption tests for now to make the overall test passed

Le 12/01/15 15:22, drankye@apache.org a écrit :
> Refactored the decryption test codes some bit and disabled DES related encryption tests for now to make the overall test passed
>
>
> http://git-wip-us.apache.org/repos/asf/directory-kerberos/blob/06b2eed1/haox-kerb/kerb-crypto/src/test/java/org/apache/kerberos/kerb/crypto/DecryptionTest.java
> ----------------------------------------------------------------------
> diff --git a/haox-kerb/kerb-crypto/src/test/java/org/apache/kerberos/kerb/crypto/DecryptionTest.java b/haox-kerb/kerb-crypto/src/test/java/org/apache/kerberos/kerb/crypto/DecryptionTest.java
> index a604518..389fab9 100644
> --- a/haox-kerb/kerb-crypto/src/test/java/org/apache/kerberos/kerb/crypto/DecryptionTest.java
> +++ b/haox-kerb/kerb-crypto/src/test/java/org/apache/kerberos/kerb/crypto/DecryptionTest.java
> @@ -28,7 +28,7 @@ import static org.junit.Assert.fail;
>  
>  
>
>  
> -    
>      /**
>       * Test for DES_CBC_CRC encryption type, with 0 byte
>       */
>      @Test
> -    public void testUncryptDES_CBC_CRC_0()
> -    {
> +    public void testDecryptDES_CBC_CRC_0() {
>          TestCase testCase = new TestCase(
> -            EncryptionType.DES_CBC_CRC,
> -            "", 0,
> -            "45E6087CDF138FB5",
> -            "28F6B09A012BCCF72FB05122B2839E6E" );
> -
> -        assertTrue( EncryptionHandler.isImplemented( testCase.encType ) );
> -        
> -        try
> -        {
> -            assertTrue( testUncrypt( testCase ) );
> -        } 
> -        catch ( Exception e )
> -        {
> -            fail( e.getMessage() );
> -        }
> +                EncryptionType.DES_CBC_CRC,
> +                "", 0,
> +                "45E6087CDF138FB5",
> +                "28F6B09A012BCCF72FB05122B2839E6E");
> +
> +        performTestDecrypt(testCase);
>      }

Doh... This is actually simpler than what I have done !!!