Uploaded image for project: 'Geode'
  1. Geode
  2. GEODE-3652

Remove test hooks from production code.

Agile BoardRank to TopRank to BottomAttach filesAttach ScreenshotBulk Copy AttachmentsBulk Move AttachmentsAdd voteVotersWatch issueWatchersCreate sub-taskLinkCloneLabelsUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Open
    • Major
    • Resolution: Unresolved
    • None
    • None
    • general, tests
    • None

    Description

      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.

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            Unassigned Unassigned
            prhomberg Patrick Rhomberg

            Dates

              Created:
              Updated:

              Time Tracking

              Estimated:
              Original Estimate - Not Specified
              Not Specified
              Remaining:
              Remaining Estimate - 0h
              0h
              Logged:
              Time Spent - 2h
              2h

              Slack

                Issue deployment