[ 
https://issues.apache.org/jira/browse/CASSANDRA-16612?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17395279#comment-17395279
 ] 

Ekaterina Dimitrova commented on CASSANDRA-16612:
-------------------------------------------------

The test class was cleaned as part of CASSANDRA-16629. 

So after a rebase I applied only the relevant change. [~adelapena], do you mind 
to look at this again, please? :)
||Patch||CI||
|[trunk|https://github.com/ekaterinadimitrova2/cassandra/pull/161]|[CircleCI 
|https://app.circleci.com/pipelines/github/ekaterinadimitrova2/cassandra?branch=16612-trunk]|
I think it almost qualifies for a ninja fix :-) 

> RingTest has inconsistent assertion
> -----------------------------------
>
>                 Key: CASSANDRA-16612
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-16612
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Test/unit
>            Reporter: Jacek Lewandowski
>            Assignee: Ekaterina Dimitrova
>            Priority: Normal
>             Fix For: 4.0.x
>
>
> There is an assertion for the node current load like:
> {noformat}
> assertThat(hostRing, matchesPattern(".*\\d+\\.\\d+ KiB.*"));
> {noformat}
> while we are formatting that value with {{#.##}}. Therefore if we say have 46 
> KiB, it will be formatted as 46 KiB rather than 46.00 KiB which is expected 
> by the test. We need to either change this assertion or change the format 
> string to {{#0.00}}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to