2

У меня есть класс:

public abstract class SomeClass {
    private readonly ISomeInterface _interface;

    protected SomeClass(ISomeInterface interface) {
        this._interface = interface;
    }

    protected virtual void SomeMethod(string someProp) {
        if (_interface == null) throw new ArgumentNullException();
        if (String.IsNullOrEmpty(someProp)) throw new ArgumentNullException();
    }
}

Пытаюсь написать тест, который проверяет будут ли вылетать Exception при разных условиях (на каждый exception свой метод теста). Как вызывать методы я, вроде, понял:

[TestMethod]
public void SomeClass_WhenISomeInterfaceIsNull_ShouldThrowArgumentNullException
{
   var someClassMock = new Mock<SomeClass>(null);
   var result = site.Protected().Setup<string>("SomeMethod", "1111");
}

Метод, вроде, вызван, но как проверить было ли в нем выброшено исключение?

PS:Ответ предложенный sp7, здесь не походит. Мой ISomeInterface работает с сетью. Возвращаемый им результат, для данного приложения не походит, например, он возвращает json строку, а конечному классу(наследнику SomeClass) нежен он в виде класса. Реализовывать преобразование из Json в class в ISomeInterface нет смысла, так как:

  1. Может потянуться за собой не нужные зависимости(например, Newtonsoft.Json)
  2. Будет использоваться в других приложениях, где может быть уже не Json.

Использовать дополнительный класс, в который будет инжектиться ISomeInterface, который в свою очередь будет инжектиться в наследников SomeClass, тоже плохой вариант так как:

  1. В методе SomeMethod класса SomeClass выполняются действия зависящие от настроек наследников. Например, количество запросов, которое
    должно быть совершено если не поступил ответ или другим причинам.
  2. Выносить настройки SomeClass в ISomeInterface не правильно, так как это приведет к множеству проинициализированных классов реализующих ISomeInterface (что является недопустимым), либо к добавлению сохранения настроек в реализации ISomeInterface (что тоже является недопустимым, так как это не его задача).
10
  • 1
    Что это такое у вас тут написано protected abstract SomeMethod(string someProp) {.....}?
    – sp7
    19 июл 2017 в 5:52
  • Это метод. Ошибся забыл void написать. Исправил
    – Vyacheslav
    19 июл 2017 в 6:03
  • Вы все-равно не правильно исправили. Уберите abstract в protected abstract void SomeMethod(string someProp) {.....}
    – sp7
    19 июл 2017 в 6:31
  • 1
    Дак а суть то это как меняет. Вопрос стоит относительно тестирования protected методов, иерархия классов здесь уже второстепенна или я не понимаю что-то?
    – sp7
    19 июл 2017 в 7:10
  • 2
    Если вам нужно тестировать protected или private методы - это говорит о том, что надо менять структуру классов.
    – MihailPw
    19 июл 2017 в 7:10

1 ответ 1

1

Есть рассматривать тот код, который представлен в примере, то для тестов вы просто можете создать еще один класс для тестирования, который будет наследовать SomeClass и открывать нужные вам protected члены класса. Не вижу криминала в подобном подходе.

public SomeClassNew : SomeClass {

  public SomeClassNew(ISomeInterface interface) : base(interface)

  public void SomeMethodTest(string someProp) {
    base.SomeMethod(someProp);
  }

}

Но лучше сделать конструктор для класса открытым.

Для вызова private методов класса вы можете использовать рефлексию:

SomeClass someClass = new SomeClass(someInterface);
MethodInfo methodInfo = typeof(SomeClass).GetMethod("SomeMethod", 
    BindingFlags.NonPublic | BindingFlags.Instance);
methodInfo.Invoke(someClass, new object[] { someProp });

"Правильность" данного подхода каждый для себя определяет сам, я не вижу в этом ничего дурного. По крайней мере для тестирования.

7
  • А не поясните почему конструктор лучше сделать открытым? С protected тоже подумал об этом, но решил задать вопрос, так не нашел в инете похожего. А что касается pivate это получается уже нарушение принципов инкапсуляции?
    – Vyacheslav
    19 июл 2017 в 7:34
  • По поводу private модификатора и инкапсуляции. Тут можно рассматривать вопрос с нескольких точек зрения, разработчика который пользуется кодом и который его разрабатывает/сопровождает. В первом случае ничего не нарушается. Во втором случае иногда приходится прибегать к такому, т.к. надо оттестировать надо именно логику закрытого метода.
    – artwolf
    19 июл 2017 в 7:53
  • По поводу конструкторов. Лично я вижу всего две причины закрывать его или объявлять его protected. В первом случае, если нужно на этапе получения нового экземпляра произвести какие то действия с уже созданным экземпляром, но еще до присвоения переменной. В таком случае делаем открытый статический конструктор или применяем фабричный метод. Второй случай применим, если мы хотим передать право на создание новых экземпляров класса исключительно потомкам. Иначе говоря принудительно заставить наследовать наш класс. В данном случае можно так же использовать абстрактные классы.
    – artwolf
    19 июл 2017 в 7:56
  • Этот подход мне кажется неправильным. Задайте себе вопрос: что должен тестировать тест? Правильный ответ: соответствие поведения класса спецификации. А какое отношение имеют приватные методы к спецификации? Никакого, это деталь имплементации.
    – VladD
    19 июл 2017 в 7:57
  • @VladD, к privat методам это ещё можно отнести(хотя а как точно знать что они работают не протестировав их?). Но это не относиться к protected методам, которые используются только в наследниках, для которых(грубо говоря) они являются public методами, которые уже надо тестировать. В этом и вопрос.
    – Vyacheslav
    19 июл 2017 в 8:07

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