You are viewing a plain text version of this content. The canonical link for it is here.

Posted to gitbox@activemq.apache.org by GitBox <gi...@apache.org> on 2020/06/12 04:26:27 UTC

[GitHub] [activemq-artemis] jsmucr commented on pull request #3177: ARTEMIS-2801 Fix ByteUtil.getHumanReadableByteCount() giving inconsistent results

jsmucr commented on pull request #3177:
URL: https://github.com/apache/activemq-artemis/pull/3177#issuecomment-643057629


   @jbertram I too thought about that but for me it seemed a bit tough and not very transparent to cover various locales in a test. What I mean is that you'd either have to enumerate the possible values, or compare the method return value to something you cannot really see at the moment of testing.
   Maybe I'm not fully awake yet but wouldn't that mean to pretty much copy-paste the method functionality to the test?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org