Статический анализ выдает предупреждение в классе испытаний, когда объект находится в процессе тестирования.

Question

У меня есть много классов тестов, подобных этому.

[TestClass] 
[System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Design", "CA1001:TypesThatOwnDisposableFieldsShouldBeDisposable")] 
public class TestClass 
{ 
    private IDisposable _disposable; 

    [TestInitialize] 
    public void TestInitialize() 
    { 
     _disposable = //new disposable object...; 
    } 

    [TestCleanup] 
    public void TestCleanup() 
    { 
     _disposable.Dispose(); 
    } 

    [TestMethod] 
    public void Test1() 
    { 
     //Uses _disposable 
    } 

    [TestMethod] 
    public void Test2() 
    { 
     //Uses _disposable 
    } 

    [TestMethod] 
    public void TestN() 
    { 
     //Uses _disposable 
    } 
} 

Статический анализ с результатами FXCop в следующем предупреждении, потому что я не реализую Dispose рисунка на моем тестовом классе.

«CA1001: Виды, которые владеют одноразовые поля должны быть одноразовыми»

Прямо сейчас я просто подавить сообщение в источнике, но я чувствую, что должен быть лучший способ, чем загромождать все мои тесты с SuppressMessageAttribute. Похоже, что это будет распространенный шаблон при тестировании - создайте объект для теста, а затем удалите его после теста. Я не могу просто реализовать IDisposable в тестовом классе, потому что для всех методов тестирования создается только один тестовый объект. Я хочу уничтожить этот объект между каждым методом тестирования.

Я знаю, что я мог бы создать объект в каждом тесте и утилизировать его в тесте, но я предпочел бы продолжать использовать SuppressMessageAttribute для копирования и вставки одного и того же кода в каждый тестовый метод. Это похоже на меньшее из двух зол. Есть ли лучший способ создать одноразовый объект перед каждым тестом и удалить его после каждого теста, который не приведет к предупреждению CA1001?

Благодарим за помощь заранее.

Answer 1

Лучшим способом я нашел, чтобы реализовать IDisposable в тестовом классе и отметьте Dispose метод с атрибутом TestCleanup.

[TestClass] 
public class TestClass : IDisposable 
{ 
    private IDisposable _disposable; 

    [TestInitialize] 
    public void TestInitialize() 
    { 
     _disposable = //new disposable object...; 
    } 

    [TestCleanup] 
    public void Dispose() 
    { 
     _disposable.Dispose(); 
    } 

Answer 2

Ошибка обращается к интерфейсу IDisposable на вашем TestClass. (Here - хорошее обсуждение, почему это необходимо). Его реализация должна быть как:

public class TestClass :IDisposable 
{ 

    public void Dispose() 
    { 
     if (!IsDisposed) //Check if _disposable is not disposed 
      _disposable.Dispose(); 
    } 

См: Implement IDisposable correctly