摆脱单元测试对 StreamReader/FileStream 的依赖的好方法是什么？

Question

场景如下：

我有一个通过 .NET 中的 FileStream 和 StreamReader 读取文件的方法。我想对这个方法进行单元测试，并以某种方式删除对 StreamReader 对象的依赖。

理想情况下，我希望能够提供自己的测试数据字符串，而不是使用真实的文件。现在，该方法始终使用 StreamReader.ReadLine 方法。为了使这个测试成为可能，修改我现在的设计的方法是什么？

Answer 1

而是依赖于 Stream 和 TextReader。然后您的单元测试可以使用 MemoryStream 和 StringReader。 （或者如有必要，从测试程序集中加载资源。）

请注意ReadLine 最初是由 TextReader 声明的，不是 StreamReader 声明的。

Answer 2

最简单的解决方案是让该方法接受 Stream 作为参数，而不是打开自己的 FileStream。您的实际代码可以像往常一样传入 FileStream，而您的测试方法可以使用不同的 FileStream 来测试数据，或者使用您想要测试的内容填充 MemoryStream（不需要文件）。

Answer 3

我突然想到，这是一个很好的机会来研究依赖注入的优点。

您可能需要考虑重新设计您的方法，以便它采用返回文件内容的委托。一个委托（生产委托）可能使用 System.IO 中的类，而第二个委托（用于单元测试）则直接以字符串形式返回内容。

Answer 4

我认为这个想法是依赖注入 TextReader 并模拟它进行单元测试。我认为你只能模拟 TextReader 因为它是一个抽象类。

public class FileParser
{
    private readonly TextReader _textReader;

    public FileParser(TextReader reader)
    {
        _textReader = reader;
    }

    public List<TradeInfo> ProcessFile()
    {
        var rows = _textReader.ReadLine().Split(new[] { ',' }).Take(4);
        return FeedMapper(rows.ToList());
    }

    private List<TradeInfo> FeedMapper(List<String> rows)
    {
        var row = rows.Take(4).ToList();
        var trades = new List<TradeInfo>();
        trades.Add(new TradeInfo { TradeId = row[0], FutureValue = Convert.ToInt32(row[1]), NotionalValue = Convert.ToInt32(row[3]), PresentValue = Convert.ToInt32(row[2]) });
        return trades;
    } 
}

然后使用Rhino Mock进行Mock，

public class UnitTest1
{
    [Test]
    public void Test_Extract_First_Row_Mocked()
    {            
        //Arrange
        List<TradeInfo> listExpected = new List<TradeInfo>();
        var tradeInfo = new TradeInfo() { TradeId = "0453", FutureValue = 2000000, PresentValue = 3000000, NotionalValue = 400000 };
        listExpected.Add(tradeInfo);
        var textReader = MockRepository.GenerateMock<TextReader>();
        textReader.Expect(tr => tr.ReadLine()).Return("0453, 2000000, 3000000, 400000");
        var fileParser = new FileParser(textReader);
        var list = fileParser.ProcessFile();           
        listExpected.ShouldAllBeEquivalentTo(list);         

    }
}

但问题在于从客户端代码传递这样的对象是否是一个好的做法，而不是我觉得它应该在负责处理的类中使用来管理。我同意使用 sep 委托来执行实际代码，并使用 sep 委托来进行单元测试，但这又是生产中的一些额外代码。我可能对依赖注入和模拟甚至文件 IO 打开/读取的想法有点迷失，这实际上不是单元测试的候选者，但文件处理逻辑可以通过仅传递文件的字符串内容来测试（ AAA23^YKL890^300000^TTRFGYUBARC)。

任何想法请！谢谢