2

Есть модульный тест проверяющий что при определенных условиях будет брошен exception, таких exception'в теоретически может быть несколько, отличаются они значением свойства message. Появилась идея проверять значения message. Хорошая ли это идея? Не приведет ли в дальнейшем к каким нибудь непредсказуемым поворотам, оказывающим влияние на поддерживаемость тестов и т.д. Делают ли так вообще?

Пример тестируемого метода:

public void SomeTestMethod(string input)
{
    if (string.IsNullOrEmpty(input))
    {
        throw new BadRequestException("не указано значение параметра input");
    }
       
    if (input.Contains("drop"))
    {
        throw new BadRequestException("параметр input содержит недопустимый аргумент");
    }
}

Тест:

[Fact]
public void SomeTestMethod_EmptyInput_ThrowException()
{
    // Act & Assert
    
    var exception = Assert.Throws<BadRequestException>(() => Sut.SomeTestMethod(string.Empty));
    
    Assert.Equal("не указано значение параметра input", exception.Message);
}

[Fact]
public void SomeTestMethod_EmptyInput_ThrowException()
{
    // Act & Assert
    
    var exception = Assert.Throws<BadRequestException>(() => Sut.SomeTestMethod("drop"));
    
    Assert.Equal("параметр input содержит недопустимый аргумент", exception.Message);
}
1
  • В книге The Art of Unit Testing, автор Roy Osherove предлагает проверять сообщения исключений не на точное совпадение, а на вхождение: Contains. 5 авг 2020 в 8:25

2 ответа 2

3

Я не сторонник проверять exception.Message в тестах. Как минимум потому, что в некоторых стандартных исключениях, если явно не указывать message, значение свойства Message будет зависеть от Culture процесса, на которых запускается тест (для английской - будет сообщение на английском языке, для русской - на русском и т.д). Иногда прибегаю к такому подходу (проверка exception.Message), но все же стараюсь избегать его.

В приведенном вами коде я вижу одну логику - параметр input - невалидный, поэтому тест будет валиться с BadRequestException. С этой точки зрения тестирующий тест может выглядеть так:

[Theory]
[InlineData("")]
[InlineData("drop")]
public void SomeTestMethod_InvalidInput_ThrowsBadRequestException(string invalidInput)
{
    // Act & Assert
    
    var exception = Assert.Throws<BadRequestException>(() => Sut.SomeTestMethod(invalidInput));
}

Если же вам все же нужно четко разделить эти два кейса ("пустое значение" и "недопустимое значение"), то можно создать отдельныеexceptions для каждого кейса, и в тестах уже отлавливать два разных типа exception.

3

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

2
  • Спасибо за ответ) Смущает не исключение а такая связанность, где то есть строка к которой я привязываюсь в тестовом методе. Хотя это похоже больше с непривычки. 5 авг 2020 в 5:54
  • 1
    По сути конечно нужно текст загонять в константы и их проверять. Это так между прочим. Думаю понятно зачем. 5 авг 2020 в 5:56

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.