Mockito - Инъекция списка издевается

Question

У меня есть следующий код:

@Component 
public class Wrapper 
{ 
    @Resource 
    private List<Strategy> strategies; 

    public String getName(String id) 
    { 
    // the revelant part of this statement is that I would like to iterate over "strategies" 
     return strategies.stream() 
      .filter(strategy -> strategy.isApplicable(id)) 
      .findFirst().get().getAmount(id); 
    } 
} 

---

@Component 
public class StrategyA implements Strategy{...} 

@Component 
public class StrategyB implements Strategy{...} 

Я хотел бы создать тест для его использования Mockito. я написал тест следующим образом:

@InjectMocks 
private Wrapper testedObject = new Wrapper(); 

// I was hoping that this list will contain both strategies: strategyA and strategyB 
@Mock 
private List<Strategy> strategies; 

@Mock 
StrategyA strategyA; 

@Mock 
StrategyB strategyB; 

@Test 
public void shouldReturnNameForGivenId() 
{ // irrevelant code... 
    //when 
    testedObject.getName(ID); 
} 

Я получаю NullPointerException на линии:

filter(strategy -> strategy.isApplicable(id)) 

, в котором говорится, что список «стратегии» инициализируется, но пуста. Есть ли способ, которым Мохито будет вести себя так же, как Весна? Автоматическое добавление всех экземпляров, реализующих интерфейс «Стратегия» к списку?

Btw У меня нет никаких сеттеров в классе Wrapper, и я хотел бы оставить его таким образом, если это возможно.

Answer 1

Mockito не может знать, что вы хотите поставить Somthing в список стратегий.

Вы должны пересмотреть ли это сделать что-то вроде этого

@InjectMocks 
private Wrapper testedObject = new Wrapper(); 

private List<Strategy> mockedStrategies; 

@Mock 
StrategyA strategyA; 

@Mock 
StrategyB strategyB; 

@Before 
public void setup() throws Exception { 
    mockedStrategies = Arrays.asList(strategyA, strategyB); 
    wrapper.setStrategies(mockedStrategies); 
} 

Answer 2

Вы не должны издеваться над коллекциями.

Создание издевается вам нужно, и поместить их в список:

private List<Strategy> strategies; // not mocked! 

@Mock 
StrategyA strategyA; 

@Mock 
StrategyB strategyB; 

@Before 
public void setup(){ 
    strategies= Arrays.asList(strategyA,strategyB); 
    testedObject.strategies= strategies; 
} 

@Test 
public void shouldReturnNameForGivenId() 
{ // irrevelant code... 
    //when 
    testedObject.getName(ID); 
} 

Answer 3

Почему не просто издеваются свой призыв к toStream()?

@InjectMocks 
private Wrapper testedObject = new Wrapper(); 

private List<Strategy> mockedStrategies; 

@Mock 
StrategyA strategyA; 

@Mock 
StrategyB strategyB; 

@Before 
public void setup() { 
    when(strategies.stream()).thenReturn(Stream.of(strategyA, strategyB)); 
} 

Для меня это гораздо более элегантно, как это не требует, чтобы добавить вспомогательный метод, который имеет отношение только к тестированию в код.

Answer 4

Аннотировать его @Spy вместо @Mock. Поскольку Mockito не может отслеживать интерфейс, используйте конкретную реализацию, например ArrayList. Во время тестовой настройки добавьте в список шпионов. Таким образом, вам не нужно изменять ваш испытуемый только в целях тестирования.

@InjectMocks 
private Wrapper testedObject = new Wrapper(); 

@Spy 
private ArrayList<Strategy> mockedStrategies; 

@Mock 
private StrategyA strategyA; 

@Mock 
private StrategyB strategyB; 

@Before 
public void setup() throws Exception { 
    mockedStrategies.add(strategyA); 
    mockedStrategies.add(strategyB); 
}