[GitHub] flink pull request #5048: [Flink-7871] [flip6] SlotPool should release unuse...

[tillrohrmann]

Github user tillrohrmann commented on a diff in the pull request:

    https://github.com/apache/flink/pull/5048#discussion_r154499440
  
    --- Diff: 
flink-runtime/src/test/java/org/apache/flink/runtime/instance/AvailableSlotsTest.java
 ---
    @@ -104,6 +111,35 @@ public void testPollFreeSlot() {
                assertFalse(availableSlots.containsTaskManager(resource1));
        }
     
    +   @Test
    +   public void testAddAndRemoveWithTimerService() {
    +           @SuppressWarnings("unchecked")
    +           final TimerService<AllocatedSlot> timerService = 
Mockito.mock(TimerService.class);
    --- End diff --
    
    Let's try to not use mockito but instead a `TimerService` implementation.


---