PowerMockito和AspectJ给了我NullPointerException



>我有一个正在测试的类,看起来像这样:

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注释和我需要模拟的静态方法。

我的测试使用(模拟的)静态方法,并从此类创建新对象。

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确实为 null

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

当然,这不是真的,但很好:PowerMock是一个错误。当你开始使用它的那一刻,你必须准备好在这些问题上花费大量时间。

尤其是当您开始混合功率模拟时...使用另一个执行字节码操作的库。那只是在自找麻烦。

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

相反:了解 PowerMock 启用测试所需的每种情况(如静态方法调用)都想告诉您:"您的设计很糟糕,请修复它"。

如果你真的遇到不允许设计更改的情况,那么你可能仍然不愿意使用PowerMock;最坏的情况是考虑为那个角落编写一些真正的"功能"测试 - 只是为了避免你现在面临的情况。

我的代码中也有类似的问题。不幸的是,我不得不模拟内部组件的行为,如下所示:

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";
}

并将其移至某种实用程序?对于没有被 Roo<</p>

div class="one_answers" 注释的其他类>

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

相关内容

  • 没有找到相关文章

最新更新