3

我发现自己经常陷入这种模式,我编写了一个由非常小的方法组成的类,这些方法完全由我的单元测试执行。然后我发现我需要构建另一个调用这些方法的方法,我必须为此编写一个更复杂的单元测试——一个简单的例子就可以说明问题:

namespace FooRequest
{
    static public class Verifier
    {
        static public bool IsValid(string request)
        {
            return (!IsAllCaps(request) && !ContainsTheLetterB(request));
        }

        static internal bool IsAllCaps(string request)
        {
            return (request.Equals(request.ToUpper()));
        }

        static internal bool ContainsTheLetterB(string request)
        {
            return request.ToLower().Contains("b");
        }
    }
}

对于代码,我会编写单元测试来涵盖两个内部方法,如下所示:

namespace UnitTest
{
    using Microsoft.VisualStudio.TestTools.UnitTesting;
    using FooRequest;

    public class VerifierTest
    {
        [TestClass]
        public class ContainsTheLetterB
        {
            [TestMethod]
            public void ShouldReturnTrueForStringContainsB()
            {
                Assert.IsTrue(Verifier.ContainsTheLetterB("burns"));
            }

            [TestMethod]
            public void ShouldReturnFakseForStringDoesNotContainB()
            {
                Assert.IsFalse(Verifier.ContainsTheLetterB("urns"));
            }
        }

        [TestClass]
        public class IsAllCaps
        {
            [TestMethod]
            public void ShouldReturnTrueForStringIsAllCaps()
            {
                Assert.IsTrue(Verifier.IsAllCaps("IAMALLCAPS"));
            }

            [TestMethod]
            public void ShouldReturnFakseForStringDoesNotContainB()
            {
                Assert.IsFalse(Verifier.IsAllCaps("IAMnotALLCAPS"));
            }
        }
    }
}

对于公共方法,我真的只想测试“如果你调用的方法返回 false,则返回 false”——令人讨厌的是,我必须以强制我的内部方法返回 true 或 false 的方式设置输入——我的测试这个方法不应该关心它调用的内部方法(对吗?)

    [TestClass]
    public class IsValid
    {
        [TestMethod]
        public void ShouldReturnFalseForInvalidStringBecauseContainsB()
        {
            Assert.IsFalse(Verifier.IsValid("b"));
        }

        [TestMethod]
        public void ShouldReturnFalseForInvalidStringBecauseIsAllCaps()
        {
            Assert.IsFalse(Verifier.IsValid("CAPS"));
        }

        [TestMethod]
        public void ShouldReturnTrueForValidString()
        {
            Assert.IsTrue(Verifier.IsValid("Hello"));
        }
    }

显然,对于这个例子来说,这并不算太糟糕,但是当有很多内部方法并且输入的配置并不简单时,测试我的公共“此输入是否有效”方法会变得复杂。

我应该为所有内部方法创建一个接口,然后将其存根以进行测试,还是有更简洁的方法?

4

2 回答 2

4

我正在输入评论,但它太大了。我认为您处于违反 SRP 的边缘,但您肯定违反了开放/封闭原则。如果您需要更改验证字符串的方式,则需要修改您的验证程序类。

我会与@seldary 有点不同,但差别不大......

    public interface IStringRule
    {
        bool Matches(string request);
    }

    public class AllCapsRule : IStringRule
    {
        public bool Matches(string request)
        {
            //implement
        }
    }

    public class IsContainingBRule : IStringRule
    {
        public bool Matches(string request)
        {
            //implement
        }
    }

    public class Verifier
    {
        private List<IStringRule> Rules;

        public Verifier(List<IStringRule> rules)
        {
            Rules = rules;
        }

        public bool IsValid(string request)
        {
            return (!Rules.Any(x=>x.Matches(request) == false));
        }
    }

现在您的验证者可以扩展,但不能修改。您可以根据需要添加任意数量的新规则,并且实施不会改变。测试验证器就像传入一些返回任意真假值的模拟字符串规则一样简单,并确保验证器返回适当的结果。

每个 IStringRule 都会像您一样单独进行测试。

于 2012-10-12T20:25:46.927 回答
0

更整洁的方法如下:

  1. 将您的类重构Verifier为三个类,在这种情况下,每个方法一个类:Verifier, AllCapsChecker, LetterBChecker.
  2. 相应地重构您的测试类 - 现在应该有三个测试类。
  3. Verifier使用您最喜欢的 DI 方法将两个内部逻辑类注入。
  4. 该类VerifierTests应该将两个依赖项安排并注入到Verifier中,并且只测试Verifier逻辑(在这种情况下只测试逻辑运算符)。

Verifier在这里你可以找到和类的改编VerifierTests,只是为了了解这个想法(我在这里使用了 Moq):

namespace FooRequest
{
    public interface IAllCapsChecker
    {
        bool IsAllCaps(string request);
    }

    public interface ILetterBChecker
    {
        bool IsContainingB(string request);
    }

    public class Verifier
    {
        private readonly IAllCapsChecker m_AllCapsChecker;
        private readonly ILetterBChecker m_LetterBChecker;

        public Verifier(IAllCapsChecker allCapsChecker, ILetterBChecker letterBChecker)
        {
            m_AllCapsChecker = allCapsChecker;
            m_LetterBChecker = letterBChecker;
        }

        public bool IsValid(string request)
        {
            return (!m_AllCapsChecker.IsAllCaps(request) && !m_LetterBChecker.IsContainingB(request));
        }
    }

    [TestClass]
    public class IsValid
    {
        [TestMethod]
        public void ShouldReturnFalseForInvalidStringBecauseContainsB()
        {
            var allCapsMock = new Mock<IAllCapsChecker>();
            allCapsMock.Setup(checker => checker.IsAllCaps("example")).Returns(true);

            var letterBChecker = new Mock<ILetterBChecker>();
            letterBChecker.Setup(checker => checker.IsContainingB("example")).Returns(true);

            var verifier = new Verifier(allCapsMock.Object, letterBChecker.Object);

            Assert.IsFalse(verifier.IsValid("example"));
        }
    }
}
于 2012-10-09T07:36:26.300 回答