2017-03-16 30 views
0

我有一个Spring Boot应用程序,它使用JMS连接到队列并侦听传入消息。在应用程序中,我有一个集成测试,它将一些消息发送到一个队列中,然后确保在听众拾取新消息时应该发生的事情发生。EntityManagerFactory在使用@DirtiesContext重新加载上下文后关闭

我已经用@DirtiesContext(classMode=ClassMode.AFTER_EACH_TEST_METHOD) 注释了我的测试类,以确保每次测试后我的数据库都是干净的。每个测试在隔离运行时都会通过。然而运行他们都在一起时的第一个测试通过后成功下一个测试失败的异常,当被测代码试图将实体保存到数据库以下:

org.springframework.transaction.CannotCreateTransactionException: Could not open JPA EntityManager for transaction; nested exception is java.lang.IllegalStateException: EntityManagerFactory is closed 
    at org.springframework.orm.jpa.JpaTransactionManager.doBegin(JpaTransactionManager.java:431) ~[spring-orm-4.3.6.RELEASE.jar:4.3.6.RELEASE] 
    at org.springframework.transaction.support.AbstractPlatformTransactionManager.getTransaction(AbstractPlatformTransactionManager.java:373) ~[spring-tx-4.3.6.RELEASE.jar:4.3.6.RELEASE] 
    at org.springframework.transaction.interceptor.TransactionAspectSupport.createTransactionIfNecessary(TransactionAspectSupport.java:447) ~[spring-tx-4.3.6.RELEASE.jar:4.3.6.RELEASE] 
    at org.springframework.transaction.interceptor.TransactionAspectSupport.invokeWithinTransaction(TransactionAspectSupport.java:277) ~[spring-tx-4.3.6.RELEASE.jar:4.3.6.RELEASE] 
    at org.springframework.transaction.interceptor.TransactionInterceptor.invoke(TransactionInterceptor.java:96) ~[spring-tx-4.3.6.RELEASE.jar:4.3.6.RELEASE] 
    at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:179) ~[spring-aop-4.3.6.RELEASE.jar:4.3.6.RELEASE] 
    at org.springframework.aop.framework.JdkDynamicAopProxy.invoke(JdkDynamicAopProxy.java:213) ~[spring-aop-4.3.6.RELEASE.jar:4.3.6.RELEASE] 
    at com.sun.proxy.$Proxy95.handleWorkflowEvent(Unknown Source) ~[na:na] 
    at com.mottmac.processflow.infra.jms.EventListener.onWorkflowEvent(EventListener.java:51) ~[classes/:na] 
    at com.mottmac.processflow.infra.jms.EventListener.onMessage(EventListener.java:61) ~[classes/:na] 
    at org.apache.activemq.ActiveMQMessageConsumer.dispatch(ActiveMQMessageConsumer.java:1401) [activemq-client-5.14.3.jar:5.14.3] 
    at org.apache.activemq.ActiveMQSessionExecutor.dispatch(ActiveMQSessionExecutor.java:131) [activemq-client-5.14.3.jar:5.14.3] 
    at org.apache.activemq.ActiveMQSessionExecutor.iterate(ActiveMQSessionExecutor.java:202) [activemq-client-5.14.3.jar:5.14.3] 
    at org.apache.activemq.thread.PooledTaskRunner.runTask(PooledTaskRunner.java:133) [activemq-client-5.14.3.jar:5.14.3] 
    at org.apache.activemq.thread.PooledTaskRunner$1.run(PooledTaskRunner.java:48) [activemq-client-5.14.3.jar:5.14.3] 
    at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source) [na:1.8.0_77] 
    at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source) [na:1.8.0_77] 
    at java.lang.Thread.run(Unknown Source) [na:1.8.0_77] 
Caused by: java.lang.IllegalStateException: EntityManagerFactory is closed 
    at org.hibernate.jpa.internal.EntityManagerFactoryImpl.validateNotClosed(EntityManagerFactoryImpl.java:367) ~[hibernate-entitymanager-5.0.11.Final.jar:5.0.11.Final] 
    at org.hibernate.jpa.internal.EntityManagerFactoryImpl.internalCreateEntityManager(EntityManagerFactoryImpl.java:316) ~[hibernate-entitymanager-5.0.11.Final.jar:5.0.11.Final] 
    at org.hibernate.jpa.internal.EntityManagerFactoryImpl.createEntityManager(EntityManagerFactoryImpl.java:286) ~[hibernate-entitymanager-5.0.11.Final.jar:5.0.11.Final] 
    at org.springframework.orm.jpa.JpaTransactionManager.createEntityManagerForTransaction(JpaTransactionManager.java:449) ~[spring-orm-4.3.6.RELEASE.jar:4.3.6.RELEASE] 
    at org.springframework.orm.jpa.JpaTransactionManager.doBegin(JpaTransactionManager.java:369) ~[spring-orm-4.3.6.RELEASE.jar:4.3.6.RELEASE] 
    ... 17 common frames omitted 

我的测试类:

@RunWith(SpringRunner.class) 
@SpringBootTest(classes = { TestGovernance.class }) 
@DirtiesContext(classMode=ClassMode.AFTER_EACH_TEST_METHOD) 
public class ActivitiIntegrationTest 
{ 
    private static final String TEST_PROCESS_KEY = "oneTaskProcess"; 
    private static final String FIRST_TASK_KEY = "theTask"; 
    private static final String NEXT_TASK_KEY = "nextTask"; 

    @Autowired 
    private JmsTemplate jms; 

    @Autowired 
    private WorkflowEventRepository eventRepository; 

    @Autowired 
    private TaskService taskService; 

    @Test 
    public void workFlowEventForRunningTaskMovesItToTheNextStage() throws InterruptedException 
    { 
     sendMessageToCreateNewInstanceOfProcess(TEST_PROCESS_KEY); 

     Task activeTask = getActiveTask();   
     assertThat(activeTask.getTaskDefinitionKey(), is(FIRST_TASK_KEY)); 

     sendMessageToUpdateExistingTask(activeTask.getProcessInstanceId(), FIRST_TASK_KEY); 

     Task nextTask = getActiveTask();   
     assertThat(nextTask.getTaskDefinitionKey(), is(NEXT_TASK_KEY)); 
    } 

    @Test 
    public void newWorkflowEventIsSavedToDatabaseAndKicksOffTask() throws InterruptedException 
    { 
     sendMessageToCreateNewInstanceOfProcess(TEST_PROCESS_KEY); 

     assertThat(eventRepository.findAll(), hasSize(1)); 
    } 

    @Test 
    public void newWorkflowEventKicksOffTask() throws InterruptedException 
    { 
     sendMessageToCreateNewInstanceOfProcess(TEST_PROCESS_KEY); 

     Task activeTask = getActiveTask();   
     assertThat(activeTask.getTaskDefinitionKey(), is(FIRST_TASK_KEY)); 
    } 


    private void sendMessageToUpdateExistingTask(String processId, String event) throws InterruptedException 
    { 
     WorkflowEvent message = new WorkflowEvent(); 
     message.setRaisedDt(ZonedDateTime.now()); 
     message.setEvent(event); 
     // Existing 
     message.setIdWorkflowInstance(processId); 
     jms.convertAndSend("workflow", message); 
     Thread.sleep(5000); 
    } 

    private void sendMessageToCreateNewInstanceOfProcess(String event) throws InterruptedException 
    { 
     WorkflowEvent message = new WorkflowEvent(); 
     message.setRaisedDt(ZonedDateTime.now()); 
     message.setEvent(event); 
     jms.convertAndSend("workflow", message); 
     Thread.sleep(5000); 
    } 

    private Task getActiveTask() 
    { 
     // For some reason the tasks in the task service are hanging around even 
     // though the context is being reloaded. This means we have to get the 
     // ID of the only task in the database (since it has been cleaned 
     // properly) and use it to look up the task. 
     WorkflowEvent workflowEvent = eventRepository.findAll().get(0); 
     Task activeTask = taskService.createTaskQuery().processInstanceId(workflowEvent.getIdWorkflowInstance().toString()).singleResult(); 
     return activeTask; 
    } 

} 

抛出异常在应用程序的方法(repository仅仅是一个标准的Spring数据CrudRepository):

@Override 
    @Transactional 
    public void handleWorkflowEvent(WorkflowEvent event) 
    { 
     try 
     { 
      logger.info("Handling workflow event[{}]", event); 

      // Exception is thrown here: 
      repository.save(event); 

      logger.info("Saved event to the database [{}]", event); 
      if(event.getIdWorkflowInstance() == null) 
      { 
       String newWorkflow = engine.newWorkflow(event.getEvent(), event.getVariables()); 
       event.setIdWorkflowInstance(newWorkflow); 
      } 
      else 
      { 
       engine.moveToNextStage(event.getIdWorkflowInstance(), event.getEvent(), event.getVariables()); 
      } 
     } 
     catch (Exception e) 
     { 
      logger.error("Error while handling workflow event:" , e); 
     } 
    } 

我的测试配置类:

@SpringBootApplication 
@EnableJms 
@TestConfiguration 
public class TestGovernance 
{ 
    private static final String WORKFLOW_QUEUE_NAME = "workflow"; 

    @Bean 
    public ConnectionFactory connectionFactory() 
    { 
     ConnectionFactory connectionFactory = new ActiveMQConnectionFactory("vm://localhost?broker.persistent=false"); 
     return connectionFactory; 
    } 

    @Bean 
    public EventListenerJmsConnection connection(ConnectionFactory connectionFactory) throws NamingException, JMSException 
    { 
     // Look up ConnectionFactory and Queue 
     Destination destination = new ActiveMQQueue(WORKFLOW_QUEUE_NAME); 

     // Create Connection 
     Connection connection = connectionFactory.createConnection(); 

     Session listenerSession = connection.createSession(false, Session.CLIENT_ACKNOWLEDGE); 
     MessageConsumer receiver = listenerSession.createConsumer(destination); 

     EventListenerJmsConnection eventListenerConfig = new EventListenerJmsConnection(receiver, connection); 
     return eventListenerConfig; 
    } 
} 

的JMS消息监听器(不知道这会帮助):

/** 
* Provides an endpoint which will listen for new JMS messages carrying 
* {@link WorkflowEvent} objects. 
*/ 
@Service 
public class EventListener implements MessageListener 
{ 
    Logger logger = LoggerFactory.getLogger(EventListener.class); 

    private WorkflowEventHandler eventHandler; 

    private MessageConverter messageConverter; 

    private EventListenerJmsConnection listenerConnection; 

    @Autowired 
    public EventListener(EventListenerJmsConnection listenerConnection, WorkflowEventHandler eventHandler, MessageConverter messageConverter) 
    { 
     this.eventHandler = eventHandler; 
     this.messageConverter = messageConverter; 
     this.listenerConnection = listenerConnection; 
    } 

    @PostConstruct 
    public void setUpConnection() throws NamingException, JMSException 
    { 
     listenerConnection.setMessageListener(this); 
     listenerConnection.start(); 
    } 

    private void onWorkflowEvent(WorkflowEvent event) 
    { 
     logger.info("Recieved new workflow event [{}]", event); 
     eventHandler.handleWorkflowEvent(event); 
    } 

    @Override 
    public void onMessage(Message message) 
    { 
     try 
     { 
      message.acknowledge(); 
      WorkflowEvent fromMessage = (WorkflowEvent) messageConverter.fromMessage(message); 
      onWorkflowEvent((WorkflowEvent) fromMessage); 
     } 
     catch (Exception e) 
     { 
      logger.error("Error: ", e); 
     } 
    } 
} 

我试着加入@Transactional' to the test methods and removing it from the code under test and various combinations with no success. I've also tried adding various test execution listeners and I still can't get it to work. If I remove the @ DirtiesContext`然后异常消失,所有的测试毫无例外地运行(然而,他们的确发生了断言错误,正如我所期望的那样)。

任何帮助将不胜感激。我迄今为止的搜索没有发现任何东西,所有内容都表明@DirtiesContext应该可以工作。

+0

这是一个非常糟糕的理由使用dirties上下文。不要这么做,因为它太慢了,当你的测试套件增长时,它的数量会更慢。所以不要。进行测试'@ Transactional',默认情况下数据将在测试后回滚。它们可能会失败,因为没有任何提交,因此您可能需要/希望注入'EntityManager'并在方法调用之间放置一个'entityManager.flush()'来模拟提交。你甚至使用SpringBootTest(刚刚注意到),这使得重启整个应用程序进行测试可能是一个更加糟糕的主意。 –

+0

另外我会说你的JMS设置有缺陷,你可以使它变得更容易。通过用'@ JmsLIstener'实现你的'onMessage',并在那里有一些队列名将会完成剩下的工作。 –

+0

我最初有'@ JmsListener',但有关自动配置设置的方式意味着它与Microsoft Service Bus结合时不能在生产环境中使用。 –

回答

0

使用@DirtiesContext这是一个可怕的想法(恕我直言)你应该做的就是让你的测试@Transactional。我也建议删除Thread.sleep并使用类似awaitility的东西来代替。

理论上,当您执行查询时,所有挂起的更改都应该提交,以便您可以使用等待性来检查最多6秒,以查看数据库中是否存在某些内容。如果这不起作用,您可以尝试在查询之前添加刷新。

@RunWith(SpringRunner.class) 
@SpringBootTest(classes = { TestGovernance.class }) 
@Transactional 
public class ActivitiIntegrationTest { 

    private static final String TEST_PROCESS_KEY = "oneTaskProcess"; 
    private static final String FIRST_TASK_KEY = "theTask"; 
    private static final String NEXT_TASK_KEY = "nextTask"; 

    @Autowired 
    private JmsTemplate jms; 

    @Autowired 
    private WorkflowEventRepository eventRepository; 

    @Autowired 
    private TaskService taskService; 

    @Autowired 
    private EntityManager em; 

    @Test 
    public void workFlowEventForRunningTaskMovesItToTheNextStage() throws InterruptedException 
    { 
     sendMessageToCreateNewInstanceOfProcess(TEST_PROCESS_KEY); 

     await().atMost(6, SECONDS).until(getActiveTask() != null); 

     Task activeTask = getActiveTask()); 
     assertThat(activeTask.getTaskDefinitionKey(), is(FIRST_TASK_KEY)); 

     sendMessageToUpdateExistingTask(activeTask.getProcessInstanceId(), FIRST_TASK_KEY); 

     Task nextTask = getActiveTask();   
     assertThat(nextTask.getTaskDefinitionKey(), is(NEXT_TASK_KEY)); 
    } 

    private Task getActiveTask() 
    { 
     em.flush(); // simulate a commit 
     // For some reason the tasks in the task service are hanging around even 
     // though the context is being reloaded. This means we have to get the 
     // ID of the only task in the database (since it has been cleaned 
     // properly) and use it to look up the task. 
     WorkflowEvent workflowEvent = eventRepository.findAll().get(0); 
     Task activeTask = taskService.createTaskQuery().processInstanceId(workflowEvent.getIdWorkflowInstance().toString()).singleResult(); 
     return activeTask; 
    } 

} 

您可能需要/想擦亮你的getActiveTask一点能够return null也许这种变化使得它甚至表现得像你希望做的事。

我只是做了一个单一的方法,其他人你可以弄清楚你自己。你用这种方法获得的收益可能是2倍,1它不会再等待5秒但是更少,你不必在测试之间重新加载整个应用程序。这两者都应该让你的测试更快。

+0

我已经更新了使用'@ Transactional'的测试,但它似乎没有清理出数据库。我可以在调试日志中看到它正在做些什么:'2017-03-16 19:24:11.216 DEBUG 12348 --- [main] osorm.jpa.JpaTransactionManager:在EntityManager上回滚JPA事务[org.hibernate.jpa .internal.EntityManagerImpl @ 18c820d2]'但是当我一起运行所有的测试时,它会到达最后一个测试方法,并且数据库中有4个项目。测试通过隔离,所以我知道这绝对不是测试将它们全部放入数据库中。 –

+0

我正在查看日志并想知道被测试的代码是否在不同的线程中运行,这与事实有关。 JMS消息处理在线程'[Session Task-1]'上运行,并且测试在线程'[main]'中运行。尽管如此,一切似乎都以正确的顺序发生。 –

+0

这确实是发生了什么事情,因为事务(和相关的'EntityManager')是基于线程的,不会起作用。当然这是JMS的全部想法:)。我将删除答案,因为在这方面它没有意义(忘记了在测试中实际使用JMS的原因)。尽管如此,我仍然会建议使用** awaitility **。 –

相关问题