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

ASF subversion and git services commented on GEODE-9400:
--------------------------------------------------------

Commit bc08aa01492e4a228228c267582d4b970d6ba7a7 in geode's branch 
refs/heads/develop from Donal Evans
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=bc08aa0 ]

GEODE-9400: Use positive bytes to create random strings (#6652)

Authored-by: Donal Evans <[email protected]>

> Do not use Coder methods in Redis tests
> ---------------------------------------
>
>                 Key: GEODE-9400
>                 URL: https://issues.apache.org/jira/browse/GEODE-9400
>             Project: Geode
>          Issue Type: Improvement
>          Components: redis
>    Affects Versions: 1.15.0
>            Reporter: Donal Evans
>            Assignee: Donal Evans
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.15.0
>
>
> Following a recent refactor, the {{Coder.stringToBytes()}} and 
> {{Coder.bytesToString()}} methods are used throughout the Radish module. 
> However, in integration and dunit tests, which emulate how a client interacts 
> with a Radish server, it is not appropriate to use this internal class and 
> its methods. Uses of Coder methods in tests other than unit tests should be 
> restored to {{new String(byte[])}} and {{String.getBytes()}}.



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

Reply via email to