如何销毁静态对象?- 测试静态字段时出现问题



我想测试这个类,不想将id设置为公共或允许它从外部设置它。

public class A {
    private int id;
    private static int prevId = 0;
    private static nextId(){
        prevId++;
    }
    public A(){
        id = nextId()
    }
    public int getId(){
       return id;
    }
    ...
}

如果我测试它并在设置中生成 A 的对象:

A foo;
@Before
public void setUp(){
    foo = new A();
}
@Test
public void test1(){
   assertEquals(1, foo.getId());
}
@Test
public void test2(){
    assertEquals(1, foo.getId());
}

测试失败,因为 setUp 被调用了两次,因此静态字段 ID 为 2

断言失败不等于 预期 1 实际 2

如上所述,我不想让id带有公共访问修饰符,否则我可以自己将其设置为 0。

有没有办法销毁静态对象拆解法 ?

@After
public void tearDown(){
    //TODO finalize static object A 
}

很简单 - 你不能,没有提供一些方法来重置prevId值。可测试性降低是反对可变全局状态的(许多)论据之一。

您可以提供一个工厂类来创建A实例,而不是像这样依赖可变的全局状态:

class AFactory {
  private int nextId;
  A createA() {
    return new A(nextId++);
  }
}

您需要在当前需要创建A实例的任何位置注入单个AFactory实例。现在,在测试中,您只需为每个测试用例创建一个新AFactory,每次nextId值都将从零开始。


或者,可以通过创建某种唯一值提供程序类来反转控制:

final class UniqueValueProvider {
  private int nextId;
  int nextId() {
    return nextId++;
  }
}

然后将其作为A构造函数的参数:

class A {
  final int id;
  A(UniqueValueProvider uvp) {
    this.id = uvp.nextId();
  }
}

如果你想任意子类化A,这使得它更具可扩展性:

class ChildOfA extends A {
  ChildOfA(UniqueValueProvider uvp) {
    super(uvp);
  }
}

同样,在你的测试中,你可以简单地创建一个新的UniqueValueProvider实例;你需要在需要创建A(或其子类之一)的地方注入一个实例。


作为旁注:您还可以更改正在测试的内容。您可以创建两个实例,并断言 ID 不同/大于另一个实例,而不是每次都断言 ID 为 1,等等:

Foo first = new Foo();
Foo second = new Foo();
assertNotEqual(first.getId(), second.getId());

如果它真的只是关于您描述的测试场景,只需使用反射来重置 @Before 方法中静态成员变量的值。如果您使用的是 Mockito,您可以使用他们的 Whitebox 助手在一行中实现这一点,如下所示:

import org.mockito.internal.util.reflection.Whitebox;
Foo foo = new Foo();
@Before
public void setup() {
    Whitebox.setInternalState(foo, "STATIC_MEMBER", 0);
}
@Test
public void staticMemberTest() {
     assertEquals(0, foo.next());
     assertEquals(1, foo.next());
 }
@Test
public void anotherStaticMemberTest() {
     assertEquals(0, foo.next());
     assertEquals(1, foo.next());
 }  
private static class Foo {
    private static int STATIC_MEMBER = 0;
    public int next() {
        return STATIC_MEMBER++;
    }
}

最新更新