Uploaded image for project: 'Kafka'
  1. Kafka
  2. KAFKA-14346

Remove static methods from internal Connect APIs for easier testing

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 3.4.0
    • connect
    • None

    Description

      Our use of static methods for internal APIs such as the RestClient class and Plugins::compareAndSwapLoaders makes testing difficult, especially with the in-progress migration from EasyMock/PowerMock to Mockito.

      We should remove these static methods and replace them with non-static methods that can be more easily mocked.

      Attachments

        Issue Links

          causes

          Bug - A problem which impairs or prevents the functions of the product. KAFKA-14816 Connect loading SSL configs when contacting non-HTTPS URLs

          • Blocker - Blocks development and/or testing work, production could not run
          • Resolved
          links to

          Web Link GitHub Pull Request #12817

          Web Link GitHub Pull Request #12828

          Web Link GitHub Pull Request #12839

          Web Link GitHub Pull Request #12866

          Activity

            People

              gharris1727 Greg Harris
              ChrisEgerton Chris Egerton
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: