Как покрыть тестом try/catch при помощи JUNIT4?

Question

[Kentaro23]

Добрый день! Idea подсвечивает этот кусок как не покрытый тестом

} catch (Throwable e) {
                        System.out.println("An exception has occurred while trying to get logger. Stack trace: "
                                        + ExceptionUtils.getStackTrace(e));
                    }

Тест сделал но idea всё равно показывает его как не покрытым тестом вот кусок теста, просьба помочь может я не правильно делаю

Assert.fail("An exception has occurred while trying to get logger. Stack trace: ");

        } catch (Throwable e) {
            assertTrue(e instanceof Throwable);
            System.out.println("An exception has occurred while trying to get logger. Stack trace: "
                    + ExceptionUtils.getStackTrace(e));
            Assert.assertNotEquals("", "An exception has occurred while trying to get logger. Stack trace: "
                    + ExceptionUtils.getStackTrace(e));

        }

Comments

[MaxLich]

Либо лыжи не едут, либо я.... Но тесты пишут под каждые входные и выходные параметры метода. Выходными параметрами в том числе являются и вылетаемые экспешены. Соответственно, вы на вход методу подаёте данные, которые заведо вызовут нужный эксепнш, а на выходе проверяет, вылетел ли эксепшн, и если вылетел, то тот или нет.
Если в трае у вас тестируемый метод, то вроде бы всё нормально, разве что я бы не тестировал месседж эксепшн, потому что важнее сам класс, чем месседж.
ЗЫ Посмотрим, что ещё скажут более опытные товарищи.

Answer 1

[aol-nnov]

мусье документацию на жэ-юнит четыре читали?

https://junit.org/junit4/faq.html#atests_7

@Test(expected = MyMegaException.class)
public void testIfItBroke() {
    MyClass mega;
    mega.methodThatThrows(wrong params);
}

и, да, я бы использовал что-то более специфичное, чем Throwable.

Comments

[Kentaro23]

Пробывал и через анотацию Евгений Руденко тоже пробывал. idea помечает как не покрытый тестом кусок } catch (Throwable e) {

[MaxLich]

Kentaro23, Что это за такой странный код!? Буквально сегодня вернулся в JUnit, начал его подробно изучать. И даже в примерах типа хеллоу ворлд показано, как работать с исключениями. Эксепнш должен вылетать из метода. Его не надо ловить (или ловить, но бросать заново). Пример из моего тестового проекта:

@Test(expected = NoSuchFileException.class)
    public void testReadLinesWithNonExistenFile() throws IOException {
        Path filePath = Paths.get("testTask_not_exists.csv");
        FileLineReader fileLineReader = new FileLineReader(filePath);
        fileLineReader.readLines().isEmpty();
    }

ЗЫ И это плохой тон - работать напрямую с Throwable или Exception. Нужно обрабатывать конкретный эксепшн (эксепшн конкретного класса), или брать его непосредственного родителя (то есть идти снизу иерархии, а не сверху).

[Kentaro23]

MaxLich, Код не мой так-что я не могу изменять его,я должен тока тесты написать к нему. Взял другой класс

@Test(expected = UnsupportedEncodingException.class)
    public void trunc() throws UnsupportedEncodingException {

        final String str = "aaaaaaa";

        assertEquals(JournalTrace.trunc(str, 5).length(), 5);

        throw new UnsupportedEncodingException();

    }

Всё равно обозначается этот кусок как не покрытый тестом

} catch (UnsupportedEncodingException e) {
                return str.substring(0, i);
            }

[aol-nnov]

Kentaro23, тест, проверяющий, что в тестовой функции брошено нужное исключение?! нет пути.

Не пиши тесты.

[Kentaro23]

aol-nnov, А как сгенерировать исключение UnsupportedEncodingException в функции которую проверяю?

[aol-nnov]

Kentaro23, как-как! подать такие входные значения, которые и приведут к этому исключению!

Иначе как ты заставишь тот код выполниться?!

[MaxLich]

Kentaro23, ЧТо-то ничего не понимаю. Объясните понятнее.

Answer 2

[Yurii Nekrasov]

Стесняюсь спросить: зачем тестировать данный блок кода? Ради 100% или чтобы Идея не подсвечивала?
Бытует мнение, что тестировать нужно бизнес логику и скользкие моменты в частности.
Более того, ошибка вашего примера теста в том, что
ExceptionUtils.getStackTrace(e)
в коде и тесте скорее всего будут разные, поэтому тест не выполниться. Это на будущее