自从我使用 java 遗留代码以来已经有几个月了，这是我正在处理的一些事情:

• 0% 的测试覆盖率。
• 有时我什至看到一些功能超过 300 行的巨大功能。
• 很多私有(private)方法，有时是静态方法。
• 高度紧密耦合的代码。

一开始我很迷茫，发现遗留的TDD很难用。在练习了几周的 katas 并练习了我的单元测试和模拟技巧之后，我的恐惧感减少了，我感到更加自信了。最近我发现了一本书叫:working effectivelly with legacy ，我没有读它，我只是看了一下目录，我发现了一些对我来说是新的东西，The Seams。显然，这在处理遗留问题时非常重要。

我认为这个 Seams 可以帮助我打破依赖关系并使我的代码可测试，这样我就可以增加代码覆盖率并使我的单元测试更加精确。

但是我有很多疑惑:

• 谁能给我解释一下 seam 和 mock 之间的区别？
• 在测试之前，在不接触生产代码的情况下，Seams 是否违反了 TDD 规则？
• 你能给我一些比较 Seam 和 Mock 的简单例子吗？

下面我想粘贴一个我今天做的例子，我试图打破依赖关系，目的是使代码可测试并最终增加测试覆盖率。如果您看到一些错误，能否发表评论，我将不胜感激？

这是遗留代码在开始时的样子:

public class ABitOfLegacy
{
    private String sampleTitle;
    String output; 

    public void doSomeProcessing(HttpServletRequest request) {
    String [] values = request.getParameterValues(sampleTitle);
        if (values != null && values.length > 0)
        {
            output = sampleTitle + new Date().toString() + values[0];
        }

    }   
}

如果我只是添加一个调用该方法并断言该变量输出的单元测试，在调用后具有特定值，那么我会犯错误，因为我不是单元测试，我会进行集成测试。所以我需要做的是摆脱我对参数的依赖。为此，我将参数替换为接口(interface):

public class ABitOfLegacy
{
    private String sampleTitle;
    String output; 

    public void doSomeProcessing(ParameterSource request) {
    String [] values = request.getParameters(sampleTitle);
        if (values != null && values.length > 0)
        {
            output = sampleTitle + new Date().toString() + values[0];
        }
    }

}

界面是这样的:

public interface ParameterSource {
    String[] getParameters(String name);
}

接下来我要做的是创建我自己的接口(interface)实现，但我将 HttpServletRequest 作为全局变量包含在内，并使用 HttpServletRequest 的方法实现接口(interface)的方法:

public class HttpServletRequestParameterSource implements ParameterSource {

    private HttpServletRequest request;

    public HttpServletRequestParameterSource(HttpServletRequest request) {
        this.request = request;
    }

    public String[] getParameters(String name) {
        return request.getParameterValues(name);
    }

}

到目前为止，我认为对生产代码的所有修改都是安全的。 现在我在我的测试包中创建了 Seam。如果我理解得很好，现在我可以安全地改变 Seam 的行为。我是这样做的:

public class FakeParameterSource implements ParameterSource {

    public String[] values = {"ParamA","ParamB","ParamC"};

    public String[] getParameters(String name) {
        return values;
    }
}

最后一步是获得 Seam 的支持，以测试该方法的原始行为。

import org.junit.Before;
import org.junit.Test;
import static org.junit.Assert.*;
import static org.mockito.Mockito.*;
import code.ABitOfLegacyRefactored;
import static org.hamcrest.Matchers.*;

public class ABitOfLegacySpecification {

    private ABitOfLegacy aBitOfLegacy;
    private String EMPTY = null;

    @Before
    public void initialize() {
        aBitOfLegacy = new ABitOfLegacy();
    }

    @Test
    public void
    the_output_gets_populated_when_the_request_is_not_empty
    () {
        FakeParameterSource fakeParameterSource = new FakeParameterSource();
        aBitOfLegacy.doSomeProcessing(fakeParameterSource);
        assertThat(aBitOfLegacy.output,not(EMPTY));
    }

    @Test(expected=NullPointerException.class)
    public void
    should_throw_an_exception_if_the_request_is_null
    () {
        aBitOfLegacy.doSomeProcessing(null);
    }   
}

这将为我提供 100% 的测试覆盖率。 我很欣赏你的想法:

• 我是否正确地打破了依赖关系？
• 单元测试是否遗漏了什么？
• 哪些方面可以做得更好？
• 这个例子是否足以帮助我理解 Seam 和 Mock 之间的区别？
• 如果我不使用 Seam，模拟怎么能帮到我呢？

最佳答案

接缝是代码中可以插入行为修改的地方。您在设置依赖项注入(inject)时创建了接缝。

利用接缝的一种方法是插入某种假的。假货可以像您的示例一样手动滚动，也可以使用 Mockito 等工具创建。

因此，mock 是一种假货，假货通常是利用 Seam 来使用的。

至于你的测试和你打破依赖的方式，这几乎就是我会做的。

关于java - Seam 和 Mock 有什么区别？，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/15327724/