Uploaded image for project: 'Flink'
  1. Flink
  2. FLINK-15626 Remove legacy scheduler
  3. FLINK-16300

Rework SchedulerTestUtils with testing classes to replace mockito usages

    XMLWordPrintableJSON

Details

    Description

      Mockito is used in SchedulerTestUtils to mock ExecutionVertex and Execution for testing. It fails to mock every getter so that other tests use it may encounter NPE issues, e.g. ExecutionVertex#getID().
      Mockito is also discouraged to be used in Flink tests. So I'd propose to rework the utils with testing classes.

      Attachments

        Issue Links

          Activity

            People

              zhuzh Zhu Zhu
              zhuzh Zhu Zhu
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

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