ok for me Felix On Saturday, November 25, 2017, Felix Schumacher < [email protected]> wrote:
> Am 23.11.2017 um 23:27 schrieb [email protected]: > >> Author: pmouawad >> Date: Thu Nov 23 22:27:35 2017 >> New Revision: 1816194 >> >> URL: http://svn.apache.org/viewvc?rev=1816194&view=rev >> Log: >> Refactored TestChangeCase >> Contributed by Graham Russell (@ham1) >> This closes #339 >> >> Modified: >> jmeter/trunk/test/src/org/apache/jmeter/functions/TestChang >> eCase.java >> >> Modified: jmeter/trunk/test/src/org/apache/jmeter/functions/TestChange >> Case.java >> URL: http://svn.apache.org/viewvc/jmeter/trunk/test/src/org/apach >> e/jmeter/functions/TestChangeCase.java?rev=1816194&r1= >> 1816193&r2=1816194&view=diff >> ============================================================ >> ================== >> --- jmeter/trunk/test/src/org/apache/jmeter/functions/TestChangeCase.java >> (original) >> +++ jmeter/trunk/test/src/org/apache/jmeter/functions/TestChangeCase.java >> Thu Nov 23 22:27:35 2017 >> @@ -20,8 +20,10 @@ package org.apache.jmeter.functions; >> >> > > ... > > @Test >> public void testChangeCaseCamelCaseFirstLowerWithFirstUpperCaseChar() >> throws Exception { >> - params.add(new CompoundVariable("Ab-CD eF")); >> - params.add(new CompoundVariable("lower_CAMEL_CASE")); >> - changeCase.setParameters(params); >> - String returnValue = changeCase.execute(result, null); >> + String returnValue = execute("Ab-CD eF", "lower_CAMEL_CASE"); >> assertEquals("abCdEf", returnValue); >> - >> - params.clear(); >> - params.add(new CompoundVariable(" zadad")); >> - params.add(new CompoundVariable("lower_CAMEL_CASE")); >> - changeCase.setParameters(params); >> - returnValue = changeCase.execute(result, null); >> + >> + returnValue = execute(" zadad", "lower_CAMEL_CASE"); >> assertEquals("Zadad", returnValue); >> > > Is "Zadad" really the expected result for "[SPACE]zadad" in lower camel > case? > > I would prefer to trim the input before conversion, so that we would get > "zadad" in this case. > > Felix > > >> > > -- Cordialement. Philippe Mouawad.
