Anonymous | Login | Signup for a new account | 2021-01-25 01:46 UTC | ![]() |
Main | My View | View Issues | Roadmap | Repositories |
View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0000683 | OLV | General Features | public | 2012-09-09 11:22 | 2012-09-11 13:10 | ||||
Reporter | Fred | ||||||||
Assigned To | BenFenner | ||||||||
Priority | low | Severity | tweak | Reproducibility | sometimes | ||||
Status | closed | Resolution | fixed | ||||||
Platform | All | OS | All | OS Version | All | ||||
Product Version | 0.0.3-SNAPSHOT | ||||||||
Target Version | 0.0.3 | Fixed in Version | 0.0.3 | ||||||
Summary | 0000683: Unit tests not robust across borders | ||||||||
Description | International borders, that is. Ideally you should use a system provided separator, however it might be overkill for a set of tests ;-) (13:01:26) struberg: Failed tests: testRoundDecimalPlaces(org.diyefi.openlogviewer.utils.MathUtilsTest): expected:<.......> but was:<...,...> | ||||||||
Steps To Reproduce | Set locale to one that uses , not . to separate decimals then run mvn test | ||||||||
Tags | No tags attached. | ||||||||
Attached Files | |||||||||
![]() |
|
![]() Fred (administrator) 2012-09-10 20:09 |
Just realised that you can probably set the locale to US-EN before running the test, and that'll solve it. |
![]() BenFenner (developer) 2012-09-10 20:16 |
Changing target version from "Future" to "0.0.3" since this should be easy to fix and it causes issues with building when unit tests fail. |
![]() Fred (administrator) 2012-09-11 12:19 |
Was feeling impatient, fixed in: fca0346 close if happy. Be happy by changing to FRANCE to cause failure. |
![]() BenFenner (developer) 2012-09-11 13:10 |
Switching to Locale.FRANCE caused the tests to fail. So that's good. Switching default locale (without switching it back) in the tests doesn't affect the main app either in Eclipse or in Maven since they don't re-use JVM instances between the tests and the main app. |
Copyright © 2000 - 2011 MantisBT Group |