Author: sebb
Date: Wed Jun 18 18:26:51 2008
New Revision: 669350

URL: http://svn.apache.org/viewvc?rev=669350&view=rev
Log:
Include retrieval of search items in sample time
Check if search was successful even if not retrieving any data.
Close enumerations just in case

Modified:
    
jakarta/jmeter/trunk/src/protocol/ldap/org/apache/jmeter/protocol/ldap/sampler/LDAPExtSampler.java

Modified: 
jakarta/jmeter/trunk/src/protocol/ldap/org/apache/jmeter/protocol/ldap/sampler/LDAPExtSampler.java
URL: 
http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/protocol/ldap/org/apache/jmeter/protocol/ldap/sampler/LDAPExtSampler.java?rev=669350&r1=669349&r2=669350&view=diff
==============================================================================
--- 
jakarta/jmeter/trunk/src/protocol/ldap/org/apache/jmeter/protocol/ldap/sampler/LDAPExtSampler.java
 (original)
+++ 
jakarta/jmeter/trunk/src/protocol/ldap/org/apache/jmeter/protocol/ldap/sampler/LDAPExtSampler.java
 Wed Jun 18 18:26:51 2008
@@ -716,6 +716,7 @@
                res.setResponseData("successfull".getBytes());
                res.setResponseMessage("Success"); // $NON-NLS-1$
                res.setResponseCode("0"); // $NON-NLS-1$
+               res.setContentType("text/xml");// $NON-NLS-1$
                boolean isSuccessful = true;
                res.setSampleLabel(getName());
                LdapExtClient temp_client = (LdapExtClient) 
ldapConnections.get(getThreadName());
@@ -757,19 +758,22 @@
                                                                + 
getPropertyAsString(COMPAREDN));
                                
xmlBuffer.tag("comparedn",getPropertyAsString(COMPAREDN)); // $NON-NLS-1$
                                
xmlBuffer.tag("comparefilter",getPropertyAsString(COMPAREFILT)); // $NON-NLS-1$
-                NamingEnumeration cmp;
+                NamingEnumeration cmp=null;
                                try {
                                        res.sampleStart();
                                        cmp = temp_client.compare(dirContext, 
getPropertyAsString(COMPAREFILT),
                                                        
getPropertyAsString(COMPAREDN));
+                       if (!cmp.hasMore()) {
+                           res.setResponseCode("5"); // $NON-NLS-1$
+                           res.setResponseMessage("compareFalse");
+                           isSuccessful = false;
+                       }
                                } finally {
                                        res.sampleEnd();
+                    if (cmp != null) {
+                        cmp.close();
+                    }
                                }                               
-                               if (!cmp.hasMore()) {
-                                       res.setResponseCode("5"); // $NON-NLS-1$
-                                       res.setResponseMessage("compareFalse");
-                                       isSuccessful = false;
-                               }
                        } else if (testType.equals(ADD)) {
                                res.setSamplerData("Add object " + 
getBaseEntryDN());
                                
xmlBuffer.tag("attributes",getArguments().toString()); // $NON-NLS-1$
@@ -805,7 +809,7 @@
                                xmlBuffer.tag("countlimit",countLimit); // 
$NON-NLS-1$
                                xmlBuffer.tag("timelimit",timeLimit); // 
$NON-NLS-1$
 
-                NamingEnumeration srch;
+                NamingEnumeration srch=null;
                                try {
                                        res.sampleStart();
                                        srch = temp_client.searchTest(
@@ -815,18 +819,24 @@
                                                        
getRequestAttributes(getAttrs()),
                                                        isRetobj(),
                                                        isDeref());
+                       if (isParseFlag()) {
+                           try {
+                               xmlBuffer.openTag("searchresults"); // 
$NON-NLS-1$
+                               writeSearchResults(xmlBuffer, srch);
+                           } finally {
+                               xmlBuffer.closeTag("searchresults"); // 
$NON-NLS-1$
+                           }                   
+                       } else {
+                        xmlBuffer.tag("searchresults", // $NON-NLS-1$
+                                "hasElements="+srch.hasMoreElements()); // 
$NON-NLS-1$
+                       }
                                } finally {
+                                   if (srch != null){
+                                       srch.close();
+                                   }
                                        res.sampleEnd();
                                }                               
 
-                if (isParseFlag()) {
-                                       try {
-                                               
xmlBuffer.openTag("searchresults"); // $NON-NLS-1$
-                                               writeSearchResults(xmlBuffer, 
srch);
-                                       } finally {
-                                               
xmlBuffer.closeTag("searchresults"); // $NON-NLS-1$
-                                       }                                       
-                }
                        }
 
                } catch (NamingException ex) {



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to