Разыскивается, но не вызывается: Mockito PrintWriter

Привет, я работаю над проектом и использую класс PrintWriter для открытия и записи в файл. Но когда я пишу тестовый пример для того же, он выдает следующую ошибку в строке 153.

Wanted but not invoked:
  mockPrintWriter.println("ID    url1
  ");
 -> at x.y.z.verify(ProcessImageDataTest.java:153)
 Actually, there were zero interactions with this mock.

Код: (использует библиотеку Ломбока)

ProcessImageData.java

@Setter
@RequiredArgsConstructor
public class ProcessImageData implements T {
    private final File newImageDataTextFile;

   @Override
    public void execute() {
       LineIterator inputFileIterator = null;
       try {
        File filteredImageDataTextFile = new File(filteredImageDataTextFilepath);

          PrintWriter writer = new PrintWriter(newImageDataTextFile);
          inputFileIterator = FileUtils.lineIterator(filteredImageDataTextFile, StandardCharsets.UTF_8.displayName());
          while (inputFileIterator.hasNext()) {
                    if(someCondition)
           **Line51**           writer.println(imageDataFileLine);
                    //FileUtils.writeStringToFile(newImageDataTextFile, imageDataFileLine + NEWLINE, true);
                }
            }          

      } catch (Exception e) {

      } finally {
          LineIterator.closeQuietly(inputFileIterator);
 **LINE63**         writer.close();
      }
  }

ProcessImageDataTest.java

@RunWith(PowerMockRunner.class)
@PrepareForTest({ ProcessImageData.class, FileUtils.class, Printwriter.class })
public class ProcessImageDataTest {

private ProcessImageData processImageData;

private static final String FILTERED_IMAGE_DATA_TEXT_FILE_PATH = "filteredFilepath";
private File FILTEREDFILE = new File(FILTERED_PATH);
private static final File IMAGE__FILE = new File("imageFilePath");

private LineIterator lineIterator;
@Mock
private PrintWriter mockPrintWriter;

@Before
public void init() throws Exception {
    MockitoAnnotations.initMocks(this);

    processImageData = new ProcessImageData(Palettes_file, FILTERED_PATH, IMAGE_FILE);

    PowerMockito.mockStatic(FileUtils.class);
    PowerMockito.whenNew(PrintWriter.class).withArguments(IMAGE_FILE).thenReturn(mockPrintWriter);

    PowerMockito.when(FileUtils.lineIterator(FILTERED_FILE, StandardCharsets.UTF_8.displayName())).thenReturn(lineIterator);
    PowerMockito.when(lineIterator.hasNext()).thenReturn(true, true, false);

}

@Test
public void testTaskWhenIDInDBAndStale() throws IOException {
    PowerMockito.when(lineIterator.nextLine()).thenReturn(ID2 + SPACE + URL1, ID1 + SPACE + URL2);

    processImageData.execute();

    List<String> exepctedFileContentOutput = Arrays.asList(ID2 + SPACE + URL1 + NEWLINE);
    verify(exepctedFileContentOutput, 1, 1);
}


@Test
public void testTaskWhenIDNotInDB() throws IOException {
    PowerMockito.when(lineIterator.nextLine()).thenReturn(ID2 + SPACE + URL1, ID3 + SPACE + URL2);

    processImageData.execute();

    List<String> exepctedFileContentOutput = Arrays.asList(ID3 + SPACE + URL2 + NEWLINE);
    verify(exepctedFileContentOutput, 1, 1);
}

private void verify(List<String> exepctedFileContentOutput, int fileWriteTimes, int fileReadTimes) throws IOException {
    for (String line : exepctedFileContentOutput){
**Line153**                     Mockito.verify(mockPrintWriter,  Mockito.times(fileWriteTimes)).print(line);  
    }

    PowerMockito.verifyStatic(Mockito.times(fileReadTimes));
    FileUtils.lineIterator(FILTERED_IMAGE_DATA_TEXT_FILE, StandardCharsets.UTF_8.displayName());
}

}

Я также издеваюсь над новым оператором для PrintWriter, вводя с помощью bean-компонентов. Какую ошибку я делаю?? Я застрял на нем долгое время и не получаю ошибку? Любая помощь приветствуется.

Обновлено :

Я внес изменения, предложенные ниже, и обновил код, но теперь получаю сообщение об ошибке:

Wanted but not invoked: mockPrintWriter.print("ASIN2 url1 "); -> 
at softlines.ctl.ruleExecutor.tasks.ProcessImageDataTest.verify‌​(ProcessImageDataTes‌​t.java:153) 
However, there were other interactions with this mock: -> at softlines.ctl.ruleExecutor.tasks.ProcessImageData.execute(Pr‌​ocessImageData.java:‌​51) ->  
at  softlines.ctl.ruleExecutor.tasks.ProcessImageData.execute(Pr‌​ocessImageData.java:‌​51) -> 
at softlines.ctl.ruleExecutor.tasks.ProcessImageData.execute(Pr‌​ocessImageData.java:‌​58) –

java junit4 mockito printwriter powermockito
person user2696258    schedule 21.10.2016    source источник
comment
постарайтесь не скрывать никаких исключений, оставив пункт catch пустым. Не могли бы вы включить печать или что-то там?   -  person SomeJavaGuy    schedule 21.10.2016
comment
обратите внимание, что @Mock отсутствует на lineIterator. Кроме того, вместо проверки print с новой строкой попробуйте проверить println без новой строки, это гораздо менее подвержено ошибкам, и скажите мне, помогает ли это   -  person Nicolas Filotto    schedule 21.10.2016

Ответы (2)


arrow_upward
1
arrow_downward

Я вижу 3 проблемы в вашем тесте:

  1. Вы не пытаетесь имитировать правильный конструктор, на самом деле в методе execute вы создаете свой PrintWriter с только одним аргументом типа File, пока пытаетесь имитировать конструктор с двумя аргументами, один из которых имеет тип File, а другой — тип String.

Поэтому код должен быть таким:

PowerMockito.whenNew(PrintWriter.class)
    .withArguments(IMAGE_FILE)
    .thenReturn(mockPrintWriter);

  1. Чтобы иметь возможность издеваться над конструктором, вам нужно подготовить класс, создающий экземпляр, который в данном случае ProcessImageData, поэтому вам нужно добавить ProcessImageData.class в аннотацию @PrepareForTest. (я не уверен, что там нужно ProcessImageDataTest.class)

  2. Поле lineIterator должно быть аннотировано @Mock.

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

Я упростил ваш код, чтобы показать идею.

Предположим, что ProcessImageData:

public class ProcessImageData {

    private final File newImageDataTextFile;

    public ProcessImageData(final File newImageDataTextFile) {
        this.newImageDataTextFile = newImageDataTextFile;
    }

    public void execute() throws Exception{
        try (PrintWriter writer = new PrintWriter(newImageDataTextFile)) {
            LineIterator inputFileIterator = FileUtils.lineIterator(
                newImageDataTextFile, StandardCharsets.UTF_8.displayName()
             );
            while (inputFileIterator.hasNext()) {
                writer.println(inputFileIterator.nextLine());
            }
        }
    }
}

Тогда мой модульный тест будет таким:

@RunWith(PowerMockRunner.class)
@PrepareForTest({ProcessImageData.class, FileUtils.class})
public class ProcessImageDataTest {

    private File file = new File("imageFilePath");

    private ProcessImageData processImageData;
    @Mock
    private PrintWriter mockPrintWriter;
    @Mock
    private LineIterator lineIterator;

    @Before
    public void init() throws Exception {
        MockitoAnnotations.initMocks(this);

        processImageData = new ProcessImageData(file);
        PowerMockito.whenNew(PrintWriter.class)
            .withArguments(file)
            .thenReturn(mockPrintWriter);
        PowerMockito.mockStatic(FileUtils.class);
        PowerMockito.when(
            FileUtils.lineIterator(file, StandardCharsets.UTF_8.displayName())
        ).thenReturn(lineIterator);
        PowerMockito.when(lineIterator.hasNext()).thenReturn(true, true, false);
    }

    @Test
    public void testExecute() throws Exception {
        PowerMockito.when(lineIterator.nextLine()).thenReturn("Foo", "Bar");
        processImageData.execute();
        Mockito.verify(mockPrintWriter,  Mockito.times(1)).println("Foo");
        Mockito.verify(mockPrintWriter,  Mockito.times(1)).println("Bar");
    }
}

Для получения более подробной информации см. Как имитировать строительство новых объектов.

как я могу добавить проверку в модульном тесте для write.close?

Один из способов может состоять в том, чтобы просто проверить, что close() вызывается один раз, добавив следующую строку в ваш модульный тест:

Mockito.verify(mockPrintWriter,  Mockito.times(1)).close();
person Nicolas Filotto    schedule 21.10.2016
comment
Эй, спасибо. Это сработало. Но он показывает другую ошибку: Разыскивается, но не вызывается: mockPrintWriter.print(ASIN2 url1); -› at softlines.ctl.ruleExecutor.tasks.ProcessImageDataTest.verify(ProcessImageDataTest.java:153) Однако были и другие взаимодействия с этим макетом: -› at softlines.ctl.ruleExecutor.tasks.ProcessImageData.execute(ProcessImageData.java: 51) -> at softlines.ctl.ruleExecutor.tasks.ProcessImageData.execute(ProcessImageData.java:51) -> at softlines.ctl.ruleExecutor.tasks.ProcessImageData.execute(ProcessImageData.java:58) - person user2696258; 21.10.2016
comment
Я обновил вопрос. можете ли вы сказать, почему конкретное фиктивное взаимодействие не вызывается. - person user2696258; 21.10.2016
comment
Сделанный!! Еще одна вещь. Где я могу закрыть средство записи печати в коде ProcessImageDataTest?? - person user2696258; 23.10.2016
comment
Или он будет закрыт автоматически при выполнении функции execute() и при запуске блока finally в строке 63 кода? - person user2696258; 23.10.2016
comment
да, вам не нужно close() вашего писателя в тесте, так как это уже сделано в методе execute() - person Nicolas Filotto; 23.10.2016
comment
Большое спасибо! Еще одна вещь, как я могу добавить проверку в модульном тесте для write.close ?? - person user2696258; 25.10.2016

arrow_upward
0
arrow_downward

Ваша конструкция PrintWriter не соответствует макету. Вы сказали PowerMockito вернуть ваш макет следующим образом:

PowerMockito.whenNew(PrintWriter.class).withArguments(IMAGE_FILE , StandardCharsets.UTF_8.name()).thenReturn(mockPrintWriter);

Итак, вы должны были бы сказать:

new PrintWriter(IMAGE_FILE, "UTF-8"); // 2 arguments

Но вместо этого в вашем методе execute в тестируемом коде вы делаете:

PrintWriter writer = new PrintWriter(newImageDataTextFile); // only 1 argument

Поэтому вам нужно либо изменить предложение PowerMockito withArguments, либо вам нужно добавить "UTF-8" к вызову конструктора в методе execute.

person Erwin Bolwidt    schedule 21.10.2016