Skip to content
This repository has been archived by the owner on Nov 10, 2023. It is now read-only.

SuperConsoleEventBusListenerTest fails because of OS decimal separator setting #58

Closed
saper opened this issue Oct 6, 2013 · 3 comments
Labels

Comments

@saper
Copy link

saper commented Oct 6, 2013

I was running "ant test" under FreeBSD with LANG set to pl_PL.UTF-8 (which has comma as the decimal separator), and this fails:

    [junit] Testcase: testSimpleBuild took 0,008 sec
    [junit]     FAILED
    [junit] expected:<...D FILES...FINISHED 0[.]4s
    [junit] > but was:<...D FILES...FINISHED 0[,]4s
    [junit] >
    [junit] junit.framework.AssertionFailedError: expected:<...D FILES...FINISHED 0[.]4s
    [junit] > but was:<...D FILES...FINISHED 0[,]4s
    [junit] >
    [junit]     at com.facebook.buck.event.listener.SimpleConsoleEventBusListenerTest.testSimpleBuild(SimpleConsoleEventBusListenerTest.java:89)
    [junit] 
    [junit] TEST com.facebook.buck.event.listener.SimpleConsoleEventBusListenerTest FAILED
    [junit] Testsuite: com.facebook.buck.event.listener.SuperConsoleEventBusListenerTest
    [junit] Tests run: 1, Failures: 1, Errors: 0, Time elapsed: 0,022 sec
    [junit] 
    [junit] Testcase: testSimpleBuild took 0,012 sec
    [junit]     FAILED
    [junit] expected:<[[+] PARSING BUILD FILES...0.3s]> but was:<[[+] PARSING BUILD FILES...0,3s]>
    [junit] junit.framework.AssertionFailedError: expected:<[[+] PARSING BUILD FILES...0.3s]> but was:<[[+] PARSING BUILD FILES...0,3s]>
    [junit]     at com.facebook.buck.event.listener.SuperConsoleEventBusListenerTest.validateConsole(SuperConsoleEventBusListenerTest.java:199)
    [junit]     at com.facebook.buck.event.listener.SuperConsoleEventBusListenerTest.testSimpleBuild(SuperConsoleEventBusListenerTest.java:85)
    [junit] 
    [junit] TEST com.facebook.buck.event.listener.SuperConsoleEventBusListenerTest FAILED
@ghost ghost assigned rowillia Oct 6, 2013
@rowillia
Copy link
Contributor

rowillia commented Oct 6, 2013

Thanks!

@bolinfest
Copy link
Contributor

Roy fixed this in c5a95f5.

@dreiss dreiss reopened this Jan 22, 2014
@natthu natthu closed this as completed May 2, 2014
@davido
Copy link
Contributor

davido commented Dec 7, 2015

@sdwilsh Looks like we have a regression here: #557.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

6 participants