You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by fs...@apache.org on 2019/05/29 07:08:04 UTC

svn commit: r1860318 - /jmeter/trunk/test/src/org/apache/jmeter/report/core/ConvertersSpec.groovy

Author: fschumacher
Date: Wed May 29 07:08:04 2019
New Revision: 1860318

URL: http://svn.apache.org/viewvc?rev=1860318&view=rev
Log:
StringConverters used for report generation should ignore white space around booleans.

Even if that is not the direct bug reported here, it makes sense to test for spaces
and upper/lowercase madness for booleans as well. No change of code was needed, this
is a test case change, only.

Bugzilla Id: 63471

Modified:
    jmeter/trunk/test/src/org/apache/jmeter/report/core/ConvertersSpec.groovy

Modified: jmeter/trunk/test/src/org/apache/jmeter/report/core/ConvertersSpec.groovy
URL: http://svn.apache.org/viewvc/jmeter/trunk/test/src/org/apache/jmeter/report/core/ConvertersSpec.groovy?rev=1860318&r1=1860317&r2=1860318&view=diff
==============================================================================
--- jmeter/trunk/test/src/org/apache/jmeter/report/core/ConvertersSpec.groovy (original)
+++ jmeter/trunk/test/src/org/apache/jmeter/report/core/ConvertersSpec.groovy Wed May 29 07:08:04 2019
@@ -54,6 +54,8 @@ class ConvertersSpec extends Specificati
      input       | conversionDestClass | expectedResult
      "FALSE"     | Boolean.class       | Boolean.FALSE
      "true"      | boolean.class       | Boolean.TRUE
+     " true"     | boolean.class       | Boolean.FALSE
+     "fAlSe "    | boolean.class       | Boolean.FALSE
      "a"         | Character.class     | 'a'
      "ä"         | char.class          | 'ä'
     }