Question

我有一个CSV类和一个CSV解析器类，以及它们的单元测试。我想知道是否必须删除以下单元测试的一部分：

public class CSV{

public string[] columns GetColumns()
        {                       
           var columns = null;

            if (!string.IsNullOrEmpty(this.textReader.ReadLine()))
            {
               var columns = this.csvParser.GetColumns(line);              
            }

            return columns;
        }



  }

   [Test]
        public void GetColumns_ReturnsCorrectLine()
        {            
            reader.Setup(r => r.ReadLine()).Returns("a\tb\tc");


            //Act
            var columns = csvReader.GetColumns();

            //Assert
            Assert.IsTrue(columns!=null);
            Assert.AreEqual(3, columns.Length);

            Assert.AreEqual("a", columns[0]);
            Assert.AreEqual("b", columns[1]);
            Assert.AreEqual("c", columns[2]);
      }

问题

1）必须删除断言列（a，b，c）的三行代码吗？

2）删除最后三个断言后，GetColumns_ReturnsCorrectLine测试方法是否执行了正确的单元测试

请注意，已有代码单元测试CSV解析器的GetColumns()方法。 GetColumns的功能是解析制表符分隔的字符串并将其转换为列。

有什么想法吗？

Answer 1

应该检查这些值，但不要像你一样。

使用NUnit，有CollectionAssert：

CollectionAssert.AreEqual(new [] {"a", "b", "c"}, columns);

Answer 2

是否必须删除断言列（a，b，c）的三行代码？

tl; dr：否

如果你想确保“a \ tb \ tc”导致[“a”，“b”，“c”]而不是[“b”，“a”，“c”]，那么你应该留下它们。前两行只检查它是否为空，长度，内容可以是任何内容。

请注意，已经有单元测试CSV的代码解析器的GetColumns（）方法。

那些测试是什么？如果没有看到它们就很难分辨它们。

Answer 3

看起来您正在尝试测试CSV.GetColumns（）。

GetColumns（）有三条路径

1）textReader.ReadLine（）返回null =＆gt; return null
2）textReader.ReadLine（）返回“”=＆gt; return null
3）testReader.ReadLine（）返回！string.NullOrEmpty（）=＆gt;返回CsvParser的输出。

应该是三个测试

TextReader返回null =＆gt;结果为空 TextReader返回“”=＆gt;结果为空 TestReader返回=＆gt;结果是CsvParser的返回值。

CsvParser返回的值并不重要 - 正如您所提到的，CsvParser正常工作在其他地方进行了测试 - 我们只想说我们返回解析器的输出。

public interface ITextReader {
    string ReadLine();
}

public interface ICsvParser {
    string[] GetColumns(string line);
}

public class CSV {
    private readonly ITextReader textReader;
    private readonly ICsvParser csvParser;

    public CSV(ITextReader textReader, ICsvParser csvParser) {
         this.textReader = textReader;
         this.csvParser = csvParser;
    }

    public string[] GetColumns() {                       
        string[] columns = null;
        var line = this.textReader.ReadLine();          

        if (!string.IsNullOrEmpty(line)){
           columns = this.csvParser.GetColumns(line);              
        }

        return columns;
    }

}

[TestClass]
public class CSVFixture {
    private Mock<ITextReader> mockTextReader;
    private Mock<ICsvParser> mockCsvParser;
    private CSV csv;

    private readonly static string [] Columns = new string[]{};

    [TestInitialize]
    public void Setup() {
        mockTextReader = new Mock<ITextReader>();
        mockCsvParser = new Mock<ICsvParser>();
        csv = new CSV(mockTextReader.Object, mockCsvParser.Object);
    }


    [TestMethod]
    public void NullLine() {
        Execute(null);
    }

    [TestMethod]
    public void EmptyLine() {
        Execute("");
    }

    [TestMethod]
    public void PopulatedLine() {
        Execute("SomeLineValue", Columns);
    }

    private void Execute(string line, string[] expected = null) {
        mockTextReader.Setup(mk => mk.ReadLine()).Returns(line);
        mockCsvParser.Setup(mk => mk.GetColumns(line)).Returns(Columns);

        var actual = csv.GetColumns();

        Assert.AreEqual(actual, expected);

    }
}

其他要点：只要从此处调用CsvParser，它只会移动代码，但您可能会更改CsvParser，以便在输入为null或为空时返回null。这意味着每次调用CsvParser时都不必检查它。

单元测试或不单元测试此部分

3 个答案: