[
https://jira.codehaus.org/browse/JBEHAVE-739?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=293927#comment-293927
]
Alexander Lehmann commented on JBEHAVE-739:
-------------------------------------------
I'm not quite sure what the reason for the canonicalize method is, either
BigDecimal is not Locale capable and the method tries to fix that or it tries
to fix a problem with the String parsing in BigDecimal, git blame gives this
commit
https://github.com/jbehave/jbehave-core/commit/13c03225d597c412060280e4912ead14c01605ba
the reason why this is not working is probably that replace doesn't process
regexp and replaceAll should be used instead, but I have to do a few more tests
to actually be able to fix it.
(if the regex is applied correctly, I think the method will drop a minus sign)
> NumberFormatException when trying to convert parameters to BigDecimal with
> Locale
> ---------------------------------------------------------------------------------
>
> Key: JBEHAVE-739
> URL: https://jira.codehaus.org/browse/JBEHAVE-739
> Project: JBehave
> Issue Type: Bug
> Components: Core
> Affects Versions: 3.6
> Reporter: Daniel Schneller
>
> When using a German localized number with group separators and decimals, a
> conversion to BigDecimal fails with a {{NumberFormatException}}. For German,
> group separators are dots, decimal separator is a comma (just the other way
> round of the US version).
> In the following test class, the first case is green, the second fails with
> {noformat}
> org.jbehave.core.steps.ParameterConverters$ParameterConvertionFailed:
> 1.000.000,01
> at
> org.jbehave.core.steps.ParameterConverters$NumberConverter.convertValue(ParameterConverters.java:256)
> at
> de.westlb.jets.xce.jbehave.tools.NumberFormatTest.jbehaveConverter(NumberFormatTest.java:39)
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> at
> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
> at
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
> at java.lang.reflect.Method.invoke(Method.java:597)
> at
> org.junit.internal.runners.TestMethodRunner.executeMethodBody(TestMethodRunner.java:99)
> ...
> at
> org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.main(RemoteTestRunner.java:197)
> Caused by: java.lang.NumberFormatException
> at java.math.BigDecimal.<init>(BigDecimal.java:453)
> at java.math.BigDecimal.<init>(BigDecimal.java:728)
> at
> org.jbehave.core.steps.ParameterConverters$NumberConverter.convertValue(ParameterConverters.java:247)
> ... 21 more
> {noformat}
> Putting a breakpoint in the BigDecimal constructor reveals it is fed a String
> "1.000.000.000.00" which is obviously wrong. I suspect, the
> "canonicalization" of the original String breaks it.
> {code:title=Test.java}
> public class NumberFormatTest {
> private static final BigDecimal EXPECTED = new BigDecimal("1000000.01");
> private final String localizedString = "1.000.000,01";
> private DecimalFormat germanNumbers;
> @Before
> public void setup() {
> germanNumbers = (DecimalFormat) DecimalFormat
> .getNumberInstance(Locale.GERMAN);
> germanNumbers.setParseBigDecimal(true);
> }
> @Test
> public void plainConverter() throws Exception {
> Number parse = germanNumbers.parse(localizedString);
> assertEquals(EXPECTED, parse);
> }
> @Test
> public void jbehaveConverter() throws Exception {
> ParameterConverters.NumberConverter numberConverter = new
> ParameterConverters.NumberConverter(
> germanNumbers);
> Object v = numberConverter.convertValue(localizedString,
> BigDecimal.class);
> assertEquals(EXPECTED, v);
> }
> {code}
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://jira.codehaus.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe from this list, please visit:
http://xircles.codehaus.org/manage_email
