Could not parse duration with ','

[cbsuh]

version: 0.10.0.777

[ OK ] Redis.ConnectDockerizedServer (4,656 ms)
Warning: Could not parse duration in line '[ OK ] Redis.ConnectDockerizedServer (4,656 ms)'

I guess that the cause is ',' among duration.

[csoltenborn]

Thanks for your report! Weird - we haven't seen such values up to now. Two questions:

• How long does this test take? Is it 4656ms (=4.656s) or 4.656ms (=0.004656s)?
• What is your build environment? This is just out of curiosity - we'd like to know where that value comes from...

We are going to release version 0.10.1 pretty soon - if you answer within the next couple of hours, the fix for this will go into that version.

[jgefele]

In google test internally test times are represented by TimeInMillis:

  // Returns the elapsed time, in milliseconds.
  TimeInMillis elapsed_time() const { return elapsed_time_; }

which is just a 64-bit integer:

typedef TypeWithSize<8>::Int TimeInMillis;  // Represents time in milliseconds.

PrettyUnitTestResultPrinter::OnTestCaseEnd just printfs a string it's gotten from internal::StreamableToString(test_case.elapsed_time())

StreamableToString streams this integer into class Message which streams it into std::stringstream.

The output will always be an integer, no decimals.

So what @cbsuh sees is the number fourthousandsixhundredfiftysix in the current locale, i.e. with comma as thousands separator.