Создание тестируемого объекта в конструкторе класса unit-тестов?

Question

[F1000]

Здравствуйте. Есть метод, создающий коллекцию объектов из входящей строки. Если при unit-тестировании получить коллекцию и сохранить её в конструкторе, а в тест-методах только вызывать assert'ы, проверяющие объекты в коллекции, то это будет считаться плохой практикой?

public class Test
  {
    private List<T> collection;

    public Test()
    {
      collection = Foo("const"); // тестируемая функция
    }
        

    [Fact]
    public void CollectionIsNotNull()
    {
      Assert.NotNull(collection);
    }    
    // остальные тесты
  }

Comments

[Василий Банников]

Сначала хотел резко сказать, что так делать не надо, тк тестовые методы должны следовать структуре AAA, но потом мне этот подход наоборот показался более удобным, хотя всё равно что-то не нравится.
Подписался, может кто-то даст более полный ответ.

[сергей кузьмин]

инверсия зависимостей
sergeyteplyakov.blogspot.com/2014/09/the-dependenc...
объясняет почему
так как вы сделали делать не надо

[Роман]

сергей кузьмин, речь о тестах, где тестируются конкретные реализации, причем тут DI?

[Роман]

Василий Банников, оно то удобно, но смысл в том, что состояние объекта перед каждым тестом должно быть "чистым", как после инициализации в конструкторе, а после тестов может быть не так, но это в общем. В NUnit можно метод пометить атрибутом StartUp и он будет перед каждым тестом запускаться, туда и можно вынести создание объекта (почему в xUnit такого нет?)
Опять же, для этого конкретного случая, если тестируется только содержание коллекции, тогда лучше коллекцию делать ReadOnly.

[Василий Банников]

Роман, это прямо в FAQ xunit и описано.
В нём конструктор - это StartUp
Dispose - после тестов
При этом на каждый ран он будет создавать новый объект

[Роман]

Василий Банников, он на каждый тест заново конструктор вызывает?

[Василий Банников]

Роман, да