10

我正在尝试测试一个使用具有许多静态方法的计算器类的类。我已经以类似的方式成功地嘲笑了另一个班级,但事实证明这个班级更加顽固。

似乎如果模拟方法包含对传入参数之一的方法调用,则不会模拟静态方法(并且测试中断)。删除内部调用显然不是一种选择。我在这里有什么明显的遗漏吗?

这是一个压缩版本,其行为方式相同......

public class SmallCalculator {

    public static int getLength(String string){

        int length = 0;

        //length = string.length(); // Uncomment this line and the mocking no longer works... 

        return length;
    }
}

这是测试...

import static org.junit.Assert.assertEquals;
import static org.mockito.BDDMockito.given;
import static org.mockito.Matchers.any;

import org.junit.Test;
import org.junit.runner.RunWith;
import org.powermock.api.mockito.PowerMockito;
import org.powermock.core.classloader.annotations.PrepareForTest;
import org.powermock.modules.junit4.PowerMockRunner;

import com.solveit.aps.transport.model.impl.SmallCalculator;

@RunWith(PowerMockRunner.class)
@PrepareForTest({ SmallCalculator.class})
public class SmallTester {

    @Test
    public void smallTest(){

        PowerMockito.spy(SmallCalculator.class);
        given(SmallCalculator.getLength(any(String.class))).willReturn(5);

        assertEquals(5, SmallCalculator.getLength(""));
    }
}

这个问题似乎有些混乱,所以我设计了一个更“现实”的例子。这增加了一层间接性,因此看起来我没有直接测试模拟方法。SmallCalculator 类没有改​​变:

public class BigCalculator {

    public int getLength(){

        int length  = SmallCalculator.getLength("random string");

        // ... other logic

        return length;
    }

    public static void main(String... args){

        new BigCalculator();
    }
}

这是新的测试类...

import static org.junit.Assert.assertEquals;
import static org.mockito.BDDMockito.given;
import static org.mockito.Matchers.any;

import org.junit.Test;
import org.junit.runner.RunWith;
import org.powermock.api.mockito.PowerMockito;
import org.powermock.core.classloader.annotations.PrepareForTest;
import org.powermock.modules.junit4.PowerMockRunner;

import com.solveit.aps.transport.model.impl.BigCalculator;
import com.solveit.aps.transport.model.impl.SmallCalculator;

@RunWith(PowerMockRunner.class)
@PrepareForTest({ SmallCalculator.class})
public class BigTester {

    @Test
    public void bigTest(){

        PowerMockito.spy(SmallCalculator.class);
        given(SmallCalculator.getLength(any(String.class))).willReturn(5);

        BigCalculator bigCalculator = new BigCalculator();
        assertEquals(5, bigCalculator.getLength());
    }
}
4

4 回答 4

22

我在这里找到了答案https://blog.codecentric.de/en/2011/11/testing-and-mocking-of-static-methods-in-java/

这是有效的最终代码。我已经在原始代码(以及人为的示例)中测试了这种方法,并且效果很好。简单...

import static org.junit.Assert.assertEquals;
import static org.mockito.Matchers.any;

import org.junit.Test;
import org.junit.runner.RunWith;
import org.powermock.api.mockito.PowerMockito;
import org.powermock.core.classloader.annotations.PrepareForTest;
import org.powermock.modules.junit4.PowerMockRunner;

@RunWith(PowerMockRunner.class)
@PrepareForTest({ SmallCalculator.class})
public class BigTester {

    @Test
    public void bigTest(){

        PowerMockito.mockStatic(SmallCalculator.class);
        PowerMockito.when(SmallCalculator.getLength(any(String.class))).thenReturn(5);

        BigCalculator bigCalculator = new BigCalculator();
        assertEquals(5, bigCalculator.getLength());
    }
}
于 2015-07-23T13:33:25.577 回答
1

使用anyString()而不是any(String.class).

当使用any(String.class)传递的参数时null,因为 Mockito 将返回引用类型的默认值,即null. 结果你得到一个例外。

使用 时anyString(),传递的参数将为空字符串。

请注意,这解释了为什么会出现异常,但是您需要查看测试方法的方式,如其他评论和答案中所述。

于 2015-07-23T12:51:35.197 回答
1

首先,删除该行:

given(SmallCalculator.getLength(any(String.class))).willReturn(5);

由于您正在测试相同的方法。您不想模拟您正在测试的方法。

其次,将您的注释修改为:

@PrepareForTest({ SmallCalculator.class, String.class})

最后,为长度()添加模拟;像这样:

given(String.length()).willReturn(5);

我认为它会做;)

于 2015-07-23T12:52:33.960 回答
0

如果您不希望调用该实际方法。代替

when(myMethodcall()).thenReturn(myResult);

利用

doReturn(myResult).when(myMethodCall());

这是一种嘲弄魔法,很难解释为什么它真的有效。

你忘记的另一件事是mockStatic(SmallCalculator.class)

你不需要PowerMockito.spy(SmallCalculator.class);

于 2015-07-23T13:55:02.263 回答