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

Posted to issues@geode.apache.org by "Patrick Rhomberg (JIRA)" <ji...@apache.org> on 2017/09/19 20:30:02 UTC

[jira] [Created] (GEODE-3652) Remove test hooks from production code.

Patrick Rhomberg created GEODE-3652:
---------------------------------------

             Summary: Remove test hooks from production code.
                 Key: GEODE-3652
                 URL: https://issues.apache.org/jira/browse/GEODE-3652
             Project: Geode
          Issue Type: Bug
            Reporter: Patrick Rhomberg


Test hooks represent suboptimal design at best and significant security, stability, and performance concerns in practice.  These test hooks should be removed.  Tests relying on them should, for instance, use Mockito spys to inject test code in the desired location.  This will likely require nontrivial refactoring of the class in question.

Subtasks will be added for the 21 classes matching the regex
{noformat}
public.* static.*hook
{noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)