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 2010/08/19 13:46:47 UTC

svn commit: r987133 - in /directory/shared/branches/shared-codec-merge/dsml-parser/src: main/java/org/apache/directory/shared/dsmlv2/reponse/SearchResponse.java test/java/org/apache/directory/shared/dsmlv2/extendedResponse/ExtendedResponseTest.java

Author: elecharny
Date: Thu Aug 19 11:46:46 2010
New Revision: 987133

URL: http://svn.apache.org/viewvc?rev=987133&view=rev
Log:
Fixed some more errors in the extendedResponse message

Modified:
    directory/shared/branches/shared-codec-merge/dsml-parser/src/main/java/org/apache/directory/shared/dsmlv2/reponse/SearchResponse.java
    directory/shared/branches/shared-codec-merge/dsml-parser/src/test/java/org/apache/directory/shared/dsmlv2/extendedResponse/ExtendedResponseTest.java

Modified: directory/shared/branches/shared-codec-merge/dsml-parser/src/main/java/org/apache/directory/shared/dsmlv2/reponse/SearchResponse.java
URL: http://svn.apache.org/viewvc/directory/shared/branches/shared-codec-merge/dsml-parser/src/main/java/org/apache/directory/shared/dsmlv2/reponse/SearchResponse.java?rev=987133&r1=987132&r2=987133&view=diff
==============================================================================
--- directory/shared/branches/shared-codec-merge/dsml-parser/src/main/java/org/apache/directory/shared/dsmlv2/reponse/SearchResponse.java (original)
+++ directory/shared/branches/shared-codec-merge/dsml-parser/src/main/java/org/apache/directory/shared/dsmlv2/reponse/SearchResponse.java Thu Aug 19 11:46:46 2010
@@ -53,7 +53,7 @@ public class SearchResponse extends Abst
      */
     public SearchResponse( int messageId, MessageTypeEnum type )
     {
-        super( -1, type );
+        super( messageId, type );
     }
 
 

Modified: directory/shared/branches/shared-codec-merge/dsml-parser/src/test/java/org/apache/directory/shared/dsmlv2/extendedResponse/ExtendedResponseTest.java
URL: http://svn.apache.org/viewvc/directory/shared/branches/shared-codec-merge/dsml-parser/src/test/java/org/apache/directory/shared/dsmlv2/extendedResponse/ExtendedResponseTest.java?rev=987133&r1=987132&r2=987133&view=diff
==============================================================================
--- directory/shared/branches/shared-codec-merge/dsml-parser/src/test/java/org/apache/directory/shared/dsmlv2/extendedResponse/ExtendedResponseTest.java (original)
+++ directory/shared/branches/shared-codec-merge/dsml-parser/src/test/java/org/apache/directory/shared/dsmlv2/extendedResponse/ExtendedResponseTest.java Thu Aug 19 11:46:46 2010
@@ -630,7 +630,7 @@ public class ExtendedResponseTest extend
 
         ExtendedResponse extendedResponse = ( ExtendedResponse ) parser.getBatchResponse().getCurrentResponse();
 
-        assertEquals( "This is a response", extendedResponse.getResponseValue() );
+        assertEquals( "This is a response", StringTools.utf8ToString( extendedResponse.getResponseValue() ) );
     }
 
 
@@ -684,7 +684,7 @@ public class ExtendedResponseTest extend
 
         ExtendedResponse extendedResponse = ( ExtendedResponse ) parser.getBatchResponse().getCurrentResponse();
 
-        assertEquals( "", extendedResponse.getResponseValue() );
+        assertEquals( "", StringTools.utf8ToString( extendedResponse.getResponseValue() ) );
     }
 
 
@@ -712,7 +712,7 @@ public class ExtendedResponseTest extend
         org.apache.directory.shared.ldap.message.ExtendedResponse extendedResponse = ( ExtendedResponse ) parser
             .getBatchResponse().getCurrentResponse();
 
-        assertEquals( "This is a response", extendedResponse.getResponseValue() );
+        assertEquals( "This is a response", StringTools.utf8ToString( extendedResponse.getResponseValue() ) );
 
         try
         {