Как написать unit тест для функции, которая вызывает другую функцию?

Question

[Игорь Махов]

Предположим, есть файл utils.ts такого вида (совсем утрированный пример), который надо протестировать с помощью Jest:

export function a(param: number): void {
    const data = param * 2;
    b(data);
}

export function b(param: number): void {
    console.log(`This is param: ${param}`);
}

Функция b что-то делает с данными и может что-нибудь вернуть, с ее тестированием никаких проблем не будет. Вопрос в том, что делать с функцией a, которая может делать какие-то преобразования, но которая вызывает функцию b. Нужно либо как-то мокнуть b, либо отловить ее вызов. Была идея сделать так чтобы все функции принимали в себя в качестве аргументов все другие функции, которые они вызывают (что-то вроде dependency injection), но тогда во-первых получится, что все функции будут принимать коллбэки, а это не так легко читается, во вторых функции которые вызываются самыми первыми должны будут принимать весь код остальной программы в виде гигантского списка коллбэков. И еще такая проблема, что в конкретно данным случае, поскольку a вызывает только b, можно было бы сделать, чтобы a возвращала значение и b вызывалась снаружи, но в общем случае a может вызывать много разных функций и тогда это решение не подойдет. Может быть есть другой способ все это организовать, не создавая DI?

Comments

[Антон Антон]

Нужно либо как-то мокнуть b, либо отловить ее вызов

не нужно

[Игорь Махов]

Антон Антон, Почему?

[Антон Антон]

Игорь Махов, а зачем? что ты этим добьешься? либо разбить a на три функции - pre_b, b, post_b и тестировать их по отдельности, либо тестировать целиком a. мок или перехват вызова ну вот никак не поможет.

[Игорь Махов]

Антон Антон, Давайте приведу более реальный пример, может будет понятнее:

function inviteUserByName(username: string, db: Database): void {
    const user = findUserByName(username, db);
    if (user.age > 18) {
        inviteUser(user, db);
    }
}

Функции findUserByName и inviteUser можно протестировать, а как протестировать эту функцию?

[Антон Антон]

Игорь Махов, ну вообще это очень плохая функция ))
но если прям очень надо - так и тестить - накитуть в то, что представляется БД 2 значения, больше 18 и меньше, протетсить с 3 вариантами - больше, меньше отсутствующий юзернейм. без тестирования внутренностей.

[Игорь Махов]

Антон Антон, Так а тестировать то что?) Результатом работы этой функции является вызов других функций, а его отловить не получится. Ну или я логику не правильно строю.

[Антон Антон]

Игорь Махов, результатом работы этой функции является приглашение (или не приглашение) пользователя или ошибка
например компромисс: (если все остальное тестируется):

function inviteUserByName(username: string, db: Database): void {
    const user = findUserByName(username, db);
   if (user === null) throw new Error("userNotFound")
    if (user.age > 18) {
        inviteUser(user, db);
        return true
    } else {
       return false
    }
}

но тоже, конечно, не айс. говорю - сам подход плоховат, функция делает два действия.