You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by el...@apache.org on 2007/04/11 18:45:37 UTC

svn commit: r527575 - /directory/shared/trunk/ldap/src/test/java/org/apache/directory/shared/ldap/codec/bind/BindRequestPerfTest.java

Author: elecharny
Date: Wed Apr 11 09:45:36 2007
New Revision: 527575

URL: http://svn.apache.org/viewvc?view=rev&rev=527575
Log:
Fixed a potential bug in a test.

Modified:
    directory/shared/trunk/ldap/src/test/java/org/apache/directory/shared/ldap/codec/bind/BindRequestPerfTest.java

Modified: directory/shared/trunk/ldap/src/test/java/org/apache/directory/shared/ldap/codec/bind/BindRequestPerfTest.java
URL: http://svn.apache.org/viewvc/directory/shared/trunk/ldap/src/test/java/org/apache/directory/shared/ldap/codec/bind/BindRequestPerfTest.java?view=diff&rev=527575&r1=527574&r2=527575
==============================================================================
--- directory/shared/trunk/ldap/src/test/java/org/apache/directory/shared/ldap/codec/bind/BindRequestPerfTest.java (original)
+++ directory/shared/trunk/ldap/src/test/java/org/apache/directory/shared/ldap/codec/bind/BindRequestPerfTest.java Wed Apr 11 09:45:36 2007
@@ -168,7 +168,6 @@
             message.setMessageId( 1 );
             
             BindRequest br = new BindRequest();
-            br.setMessageId( 1 );
             br.setName( name );
             
             Control control = new Control();
@@ -177,7 +176,7 @@
             LdapAuthentication authentication = new SimpleAuthentication();
             ((SimpleAuthentication)authentication).setSimple( StringTools.getBytesUtf8( "password" ) );
 
-            br.addControl( control );
+            message.addControl( control );
             br.setAuthentication( authentication );
             message.setProtocolOP( br );