该单元测试中是否有太多断言？

发布于 2024-10-02 18:36:53 字数 1639 浏览 5 评论 0原文

这一单元测试中是否有太多断言？

[Fact]
public void Send_sends_an_email_message() {
    using (var server = new MockSmtpServer()) {
        server.Start();
        using (var client = new EmailClient("localhost")) {
            string from = "[email protected]";
            IEnumerable<string> to = new[] { "[email protected]" };
            string subject = "Test";
            string body = "Test.";
            client.Send(from, to, subject, body);
            var session = server.Sessions.FirstOrDefault();
            Assert.NotNull(session);
            var message = session.Messages.FirstOrDefault();
            Assert.NotNull(message);
            Assert.NotNull(message.From);
            Assert.Equal(message.From.Address, "[email protected]");
            Assert.NotNull(message.To);
            var recipient = message.To.FirstOrDefault();
            Assert.NotNull(recipient);
            Assert.Equal(recipient.Address, "[email protected]");
            Assert.Equal(message.Subject, "Test");
            Assert.Equal(message.Body, "Test.");
        }
    }
}

我认为这段代码不需要任何解释，但如果需要，请告诉我。

原文

Are there too many asserts in this one unit test?

[Fact]
public void Send_sends_an_email_message() {
    using (var server = new MockSmtpServer()) {
        server.Start();
        using (var client = new EmailClient("localhost")) {
            string from = "[email protected]";
            IEnumerable<string> to = new[] { "[email protected]" };
            string subject = "Test";
            string body = "Test.";
            client.Send(from, to, subject, body);
            var session = server.Sessions.FirstOrDefault();
            Assert.NotNull(session);
            var message = session.Messages.FirstOrDefault();
            Assert.NotNull(message);
            Assert.NotNull(message.From);
            Assert.Equal(message.From.Address, "[email protected]");
            Assert.NotNull(message.To);
            var recipient = message.To.FirstOrDefault();
            Assert.NotNull(recipient);
            Assert.Equal(recipient.Address, "[email protected]");
            Assert.Equal(message.Subject, "Test");
            Assert.Equal(message.Body, "Test.");
        }
    }
}

I don't think this code requires any explanation, but if it does please let me know.

分享到QQ

分享到微博

如果你对这篇内容有疑问，欢迎到本站社区发帖提问参与讨论，获取更多帮助，或者扫码二维码加入 Web 技术交流群。

发布评论

需要登录才能够评论，你可以免费注册一个本站的账号。

通知家属抬走 2024-10-09 18:36:53

我尝试使我的单元测试保持相当小的规模并一次测试一件事。因此，我可能会将测试不同的部分放入单独的测试中，例如

sendWillSendAnEmail、
fromContainsSenderAddress、
toContainsRecipientAddress、
mailBodyContainsMailMessage、
mailContainsSubject

回复收藏 0 原文

浮萍、无处依 2024-10-09 18:36:53

我认为它太大了。

当你有一堆较小的测试时，你可以获得“缺陷定位”——只需运行所有测试，你就可以准确地看到问题出在哪里。使用当前拥有的尽可能多的断言（并且没有断言消息），您可能必须启动调试器才能找到答案。请记住，您可能最终会进行数百甚至数千个测试，如果其中一堆测试失败，您不想调试每个测试来了解原因。

此外，任何在测试早期失败的断言都意味着后面的断言不会运行。当它们被分成单独的测试时，每个断言都会被检查。这是一种权衡；其中许多断言可能是相关的，并且会同时失败，因此您将有五个红色测试，而不是一个。但我的工作前提是信息越多越好，所以我宁愿进行这五个测试并知道所有五个断言都失败了。

回复收藏 0 原文

黑白记忆 2024-10-09 18:36:53

我认为你的断言没有特别的问题，但是如果你想清理你的代码，你可以更改

var session = server.Sessions.FirstOrDefault();
Assert.NotNull(session);

为

var session = server.Sessions.First();

First() 无论如何都会抛出异常，所以只需更改你的代码你就可以获得以下好处：断言将使您无需那么多代码。您也可以在其他地方进行类似的更改。

但作为一般规则，在单元测试中不要认为任何事情都是理所当然的——这意味着大量的断言！

I see no particular problem with your asserts, but if you want to clean up your code, you could change

var session = server.Sessions.FirstOrDefault();
Assert.NotNull(session);

into

var session = server.Sessions.First();

First() will throw an exception anyway, so by just changing your code you get the benefit that the assert would bring you without as much code. There are other places you can make similar changes too.

But as a general rule, take nothing for granted in unit tests -- and that means lots of asserts!

回复收藏 0 原文