Question

我的测试类看起来像这样：

import org.springframework.roo.addon.javabean.RooJavaBean;
import org.springframework.roo.addon.jpa.activerecord.RooJpaActiveRecord;

@RooJavaBean
@RooJpaActiveRecord(table="test_class", finders={ "findById" })
public class TestClass {
    boolean test;

    public static String returnSomething() {
        return "X";
    }
}

正如您所看到的，它具有Roo-Annotations和我需要模拟的静态方法。

我的测试使用（模拟的）静态方法，并且还从此类创建新对象。

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

@RunWith(PowerMockRunner.class)
@PrepareForTest(TestClass.class)
@PowerMockIgnore("javax.management.*")
public class PowerMockitoTest {

    @Test
    public void test() {
        PowerMockito.mockStatic(TestClass.class);
        PowerMockito.when(TestClass.returnSomething()).thenReturn("Y");

        Assert.assertEquals("Y", TestClass.returnSomething());

        TestClass x = new TestClass();
    }

}

对new TestClass()的调用给了我以下例外：

java.lang.NullPointerException
    at org.springframework.beans.factory.aspectj.AnnotationBeanConfigurerAspect.ajc$if$bb0(AnnotationBeanConfigurerAspect.aj:1)
    at ....TestClass.<init>(TestClass.java:8)

当我调试AnnotationBeanConfigurerAspect类时，我可以看到传入此方法的Configurable确实为空

public static final boolean ajc$if$bb0(Configurable c) {
        return c.preConstruction();
    }

这是Powermock中的错误吗？有没有解决方法？

使用的版本：

[INFO] +- junit:junit:jar:4.11:test
[INFO] |  \- org.hamcrest:hamcrest-core:jar:1.3:test
[INFO] +- org.mockito:mockito-all:jar:1.9.5:test
[INFO] +- org.powermock:powermock-module-junit4:jar:1.5.3:test
[INFO] |  \- org.powermock:powermock-module-junit4-common:jar:1.5.3:test
[INFO] |     \- org.powermock:powermock-reflect:jar:1.5.3:test
[INFO] +- org.powermock:powermock-api-mockito:jar:1.5.3:test
[INFO] |  \- org.powermock:powermock-api-support:jar:1.5.3:test
[INFO] +- org.powermock:powermock-module-junit4-rule:jar:1.5.1:test
[INFO] |  +- org.powermock:powermock-classloading-base:jar:1.5.1:test
[INFO] |  \- org.powermock:powermock-core:jar:1.5.1:test
[INFO] \- org.powermock:powermock-classloading-xstream:jar:1.5.1:test

Answer 1

当然，事实并非如此，但是：PowerMock 是一个错误。第二个你开始使用它你必须准备好花很多时间在这些问题上。

特别是当您开始将power mock ...与另一个执行字节代码操作的库混合时。这简直就是在寻找麻烦。

因此，从长远来看，您将从中受益的唯一答案是：不要这样做。不要使用PowerMock。

相反：要了解PowerMock启用测试所需的每一种情况（如静态方法调用）都想告诉您：＆＃34;您的设计很糟糕，请修复它＆＃34;。

如果您确实遇到了不允许进行设计更改的情况，那么您可能仍然不想使用PowerMock;最坏的情况考虑写一些真正的＆＃34;功能＆＃34;测试那个角落 - 只是为了避免你现在正面临的情况。

Answer 2

我的代码中遇到了类似的问题。不幸的是，我不得不模仿内部组件的行为，如下所示：

public class LockWithRequestTimeout implements ReadWriteLock
//...
@TracePerformance // ----> this annotation somehow influence PowerMockito
@Override
public void lock()
{
  try
  {
    while (!l.tryLock(1, TimeUnit.SECONDS))
    {
      PerformanceAdvice.checkRequestTimeout(() -> toString()); // ----> this is what i wanted to mock, PerformanceAdvice is AOP that handles TracePerformance annotation
    }
  }
  catch (final InterruptedException e)
  {
    final String msg = "interrupted while waiting for " + this;
    LogFactory.getLog(LockWithRequestTimeout.class).warn(msg);
    Thread.currentThread().interrupt();
    throw new RuntimeException(msg, e);
  }
}
//...

我的第一个想法是模拟PerformanceAdvice checkRequestTimeout方法，但它的行为与你的例子一样，我得到了NullPointerException。

我的解决方案是进行小型重构，我提取了新方法：

protected static void checkRequestTimeout(final Supplier<String> msg)
{
  PerformanceAdvice.checkRequestTimeout(msg);
}

而不是模拟PerformanceAdvice我在类LockWithRequestTimeout中模拟了这个新方法。

我的主张是提取方法

public static String returnSomething() {
  return "X";
}

并将其移至某种类型的Utils？其他未经Roo注释的课程

Answer 3

我提交了一个错误并提供了修复： https://github.com/jayway/powermock/issues/676

PowerMockito和AspectJ给了我NullPointerException

3 个答案: