二、静态方法对修改开放，对“测试”关闭
使用静态方法非常简单，无需实例化并传递对象，在哪里都可以直接使用，但如果过度依赖它们，在测试中就不好办了。
Java代码
public boolean preHandle(HttpServletRequest request, HttpServletResponse response) {
if (SessionUtils.isUnexpired(request)) {
return true;
} else {
//在response中输出alert语句
}
} |
由于在静态方法中对保存在session中的属性做了计算和校验，要对其进行测试就需要事先在session中填充很多信息。
Java代码
|
@Test
public void returnTrueWhenUserSessionIsNotExpired() throws Exception {
MockHttpServletRequest request = new MockHttpServletRequest();
MockHttpSession session = new MockHttpSession();
request.setSession(session);
session.set......blablabla
assertTrue(target.preHandle(request));
}
@Test
public void printAlertAndreturnFalseWhenUserSessionIsExpired() throws Exception {
MockHttpServletRequest request = new MockHttpServletRequest();
MockHttpSession session = new MockHttpSession();
request.setSession(session);
session.set......blablabla
assertFalse(target.preHandle(request, response));
assertEquals(ALERT, response.getContentAsString());
} |
由于静态方法属于类，无法通过子类化并覆写的方式在测试时替换，即使测试重点并非SessionUtils.isUnexpired(req)本身，但由于它是一个必经步骤，你不得不为其准备完整的测试数据（哪怕非常麻烦）。让我们对它改造一下吧，不过我也并不想再实现一次isUnexpired(req)，耍个小花招吧
再来看测试，就简单多了，我们可以用Stub/mock来替换SessionGateway的实现。
Java代码
|
@Test
public void returnTrueWhenUserSessionIsNotExpired() throws Exception {
context.checking(new Expectations() {
{
allowing(sessionGateway).isUnexpired(request)；
will(returnValue(true));
}
});
assertTrue(target.preHandle(request, response));
}
@Test
public void printAlertAndreturnFalseWhenUserSessionIsExpired() throws Exception {
context.checking(new Expectations() {
{
allowing(sessionGateway).isUnexpired(request)；
will(returnValue(false));
}
});
assertFalse(target.preHandle(request, response));
assertEquals(ALERT, response.getContentAsString());
} |
