为什么这个 CollectionAssert.AreEquivalent() 测试会失败?

5

此问题所述,我正在开发一个方法,返回一个List<FileInfo>中存在而另一个List<FileInfo>中不存在的元素。我已经按照Nawfal的解决方案进行了实现:

public List<FileInfo> SourceNotInDest(List<FileInfo> SourceFiles, List<FileInfo> DestFiles)
{
 var notInDest = SourceFiles.Where(c => !DestFiles.Any(p => p.Name == c.Name)).ToList();
 return notInDest;
}

我的SourceFiles数据集如下:

u:\folder1\a.txt
u:\folder1\b.txt
u:\folder1\c.txt
u:\folder1\d.txt

DestFiles是:

u:\folder2\a.txt
u:\folder2\b.txt
u:\folder2\c.txt

当我逐步执行代码并检查列表的值时,这似乎返回了预期的结果。但是单元测试失败,并显示以下代码:

public void SourceNotInDestTest()
    {
        //arrange
        FileListComparer flc = new FileListComparer(); //class that has the list compare method
        FolderReader fr = new FolderReader(); //class for getting FileInfo from folder
        List<FileInfo> expectedResult = new List<FileInfo>();
        expectedResult.Add(new FileInfo(@"U:\folder1\d.txt"));
        List<FileInfo> SourceFiles = fr.fileList(@"U:\folder1");  //gets the FileInfo for each file in the folder
        List<FileInfo> DestFiles = fr.fileList(@"U:\folder2");


        //act
        List<FileInfo> result = flc.SourceNotInDest(FTPFiles, LocalFiles);

        //assert
        CollectionAssert.AreEquivalent(result, expectedResult);
    }

尽管resultexpectedResult具有相同的内容——两个列表都有一个元素,具有相同的文件路径和其他属性——但测试失败并显示以下消息:

CollectionAssert.AreEquivalent failed. The expected collection contains 1 occurrence(s)
of <U:\folder1\d.txt>. The actual collection contains 0 occurrence(s).

expectedResult确实存在U:\folder1\d.txt的一个出现,但我在想,也许问题在于我正在比较两个对象的内存地址,而不是这些对象的内容,但我认为AreEquivalent()正在比较属性。那不是这种情况吗?

编辑:根据比较属性而不是地址的建议,我使用了这个Assert来代替,这样测试就可以通过了:

foreach (FileInfo fi1 in result)
    {
     Assert.IsNotNull(expectedResult.Find(fi2 => fi2.FullName == fi1.FullName));
    }
foreach (FileInfo fi1 in expectedResult)
    {
     Assert.IsNotNull(result.Find(fi2 => fi2.FullName == fi1.FullName));
    }
3个回答

7

可能是因为FileInfo是引用类型,而默认的比较器只检查两个元素的地址是否相等。由于FileInfo是密封的,您无法从中派生并覆盖相等比较器。在我看来,最好的选择是编写自己的集合比较器方法(因为您无法将IEqualityComparer实例传递给CollectionAssert.AreEquivalent)。


4
测试失败是因为您的集合中包含不同的对象。如果您有两个指向同一文件的FileInfo类实例,并且调用instanceA.Equals(instanceB),则结果为false
如果您可以更改代码以使用字符串而不是FileInfo,则它将按照您的预期工作。

1
我可以提出两种方法,我认为比你的更好 :)
  1. 选择2个文件名集合并比较这些集合:

    CollectionAssert.AreEquivalent(
        result.Select(fi => fi.FullName).ToArray(),
        expectedResult.Select(fi => fi.FullName).ToArray()
    );
    // 添加ToArray()只是为了在测试失败时更好地输出。
    
  2. 使用用户定义的比较器并比较FileInfo列表:

    Assert.That(
        result,
        Is
            .EquivalentTo(expectedResult)
            .Using((Comparison<FileInfo>)((fi1, fi2) => fi1.FullName.CompareTo(fi2.FullName)))
    );
    

你目前使用的两个foreach循环实现在以下情况下不会失败:

result =
    u:\folder1\a.txt
    u:\folder1\a.txt

expectedResult =
    u:\folder1\a.txt

是的,对于文件列表来说似乎不是真实情况,但通常用两个循环替换AreEquivalent()/Is.EquivalentTo()并不是一个好主意。


网页内容由stack overflow 提供, 点击上面的
可以查看英文原文,
原文链接