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

Posted to notifications@geode.apache.org by "jdeppe-pivotal (GitHub)" <gi...@apache.org> on 2020/03/19 21:03:34 UTC

[GitHub] [geode] jdeppe-pivotal commented on pull request #4821: GEODE-7886 Update strings tests to cover sad as well as happy paths

There are a couple of these comments. Since all of these good changes are already being applied to this class, is there any reason not to validate these remaining few tests? Or is this more work than what it appears on the surface?

[ Full content available at: https://github.com/apache/geode/pull/4821 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org