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]