spring-batch icon indicating copy to clipboard operation
spring-batch copied to clipboard

SpringBatchTest should support application contexts without javax.sql.DataSource

Open qwazer opened this issue 5 years ago • 5 comments
trafficstars

Spring-batch has 2 annotations to simply context creation

EnableBatchProcessing SpringBatchTest

EnableBatchProcessing support's app contexts without javax.sql.DataSource bean. It's mentioned in javadoc

  • If a user does not provide a {@link javax.sql.DataSource} within the context, a Map based
  • {@link org.springframework.batch.core.repository.JobRepository} will be used.

https://github.com/spring-projects/spring-batch/blob/4.2.x/spring-batch-core/src/main/java/org/springframework/batch/core/configuration/annotation/EnableBatchProcessing.java#L88

SpringBatchTest doesn't support contexts without javax.sql.DataSource and fails with

java.lang.IllegalStateException: Failed to load ApplicationContext

	at org.springframework.test.context.cache.DefaultCacheAwareContextLoaderDelegate.loadContext(DefaultCacheAwareContextLoaderDelegate.java:132)
	at org.springframework.test.context.support.DefaultTestContext.getApplicationContext(DefaultTestContext.java:123)
	at org.springframework.test.context.support.DependencyInjectionTestExecutionListener.injectDependencies(DependencyInjectionTestExecutionListener.java:118)
	at org.springframework.test.context.support.DependencyInjectionTestExecutionListener.prepareTestInstance(DependencyInjectionTestExecutionListener.java:83)
	at org.springframework.test.context.TestContextManager.prepareTestInstance(TestContextManager.java:244)
	at org.springframework.test.context.junit4.SpringJUnit4ClassRunner.createTest(SpringJUnit4ClassRunner.java:227)
	at org.springframework.test.context.junit4.SpringJUnit4ClassRunner$1.runReflectiveCall(SpringJUnit4ClassRunner.java:289)
	at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
	at org.springframework.test.context.junit4.SpringJUnit4ClassRunner.methodBlock(SpringJUnit4ClassRunner.java:291)
	at org.springframework.test.context.junit4.SpringJUnit4ClassRunner.runChild(SpringJUnit4ClassRunner.java:246)
	at org.springframework.test.context.junit4.SpringJUnit4ClassRunner.runChild(SpringJUnit4ClassRunner.java:97)
	at org.junit.runners.ParentRunner$4.run(ParentRunner.java:331)
	at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:79)
	at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:329)
	at org.junit.runners.ParentRunner.access$100(ParentRunner.java:66)
	at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:293)
	at org.springframework.test.context.junit4.statements.RunBeforeTestClassCallbacks.evaluate(RunBeforeTestClassCallbacks.java:61)
	at org.springframework.test.context.junit4.statements.RunAfterTestClassCallbacks.evaluate(RunAfterTestClassCallbacks.java:70)
	at org.junit.runners.ParentRunner$3.evaluate(ParentRunner.java:306)
	at org.junit.runners.ParentRunner.run(ParentRunner.java:413)
	at org.springframework.test.context.junit4.SpringJUnit4ClassRunner.run(SpringJUnit4ClassRunner.java:190)
	at org.junit.runner.JUnitCore.run(JUnitCore.java:137)
	at com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69)
	at com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33)
	at com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:220)
	at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:53)
Caused by: org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'jobRepositoryTestUtils': Unsatisfied dependency expressed through method 'setDataSource' parameter 0; nested exception is org.springframework.beans.factory.NoSuchBeanDefinitionException: No qualifying bean of type 'javax.sql.DataSource' available: expected at least 1 bean which qualifies as autowire candidate. Dependency annotations: {}
	at org.springframework.beans.factory.annotation.AutowiredAnnotationBeanPostProcessor$AutowiredMethodElement.inject(AutowiredAnnotationBeanPostProcessor.java:723)
	at org.springframework.beans.factory.annotation.InjectionMetadata.inject(InjectionMetadata.java:130)
	at org.springframework.beans.factory.annotation.AutowiredAnnotationBeanPostProcessor.postProcessProperties(AutowiredAnnotationBeanPostProcessor.java:399)
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.populateBean(AbstractAutowireCapableBeanFactory.java:1422)
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.doCreateBean(AbstractAutowireCapableBeanFactory.java:594)
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBean(AbstractAutowireCapableBeanFactory.java:517)
	at org.sp

qwazer avatar Aug 27 '20 13:08 qwazer

I do confirm this issue, here is a failing test with v4.3.3:

import org.junit.Assert;
import org.junit.Test;
import org.junit.runner.RunWith;

import org.springframework.batch.core.ExitStatus;
import org.springframework.batch.core.Job;
import org.springframework.batch.core.JobExecution;
import org.springframework.batch.core.configuration.annotation.EnableBatchProcessing;
import org.springframework.batch.core.configuration.annotation.JobBuilderFactory;
import org.springframework.batch.core.configuration.annotation.StepBuilderFactory;
import org.springframework.batch.repeat.RepeatStatus;
import org.springframework.batch.test.JobLauncherTestUtils;
import org.springframework.batch.test.context.SpringBatchTest;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.context.annotation.Bean;
import org.springframework.context.annotation.Configuration;
import org.springframework.test.context.junit4.SpringRunner;

@SpringBatchTest
@RunWith(SpringRunner.class)
public class springBatchTestWithoutDataSource {

    @Autowired
    private JobLauncherTestUtils jobLauncherTestUtils;

    @Test
    public void testJob() throws Exception {
        JobExecution jobExecution = jobLauncherTestUtils.launchJob();
        Assert.assertEquals(ExitStatus.COMPLETED, jobExecution.getExitStatus());
    }

    @Configuration
    @EnableBatchProcessing
    public static class JobConfiguration {

        @Bean
        public Job job(JobBuilderFactory jobBuilderFactory, StepBuilderFactory stepBuilderFactory) {
            return jobBuilderFactory.get("job")
                    .start(
                            stepBuilderFactory.get("step")
                                    .tasklet((contribution, chunkContext) -> RepeatStatus.FINISHED)
                                    .build())
                    .build();
        }
    }

}

This is not directly related to the @SpringBatchTest annotation, but rather to the way the underlying utilities it imports are designed to work (ie the JobLauncherTestUtils which autowires the job under test and JobRepositoryTestUtils which autowires a datasource). I wrote a similar comment about this in https://github.com/spring-projects/spring-batch/issues/3699#issuecomment-702966663. There is a similar/related open issue here: https://github.com/spring-projects/spring-batch/issues/1237.

Since the Map-based job repository has been deprecated for removal (see #3780), we need to revisit how this should work in v5 (it might be fixed by design since a datasource will be mandatory).

fmbenhassine avatar Apr 21 '21 09:04 fmbenhassine

Hi, do we have update on this enhancement

munnema avatar Nov 22 '21 12:11 munnema

@munnema you could have a look at spring-batch-inmemory that gives you a NullDataSource, you will have to use one of the custom repositories and job explorers though.

marschall avatar Nov 22 '21 16:11 marschall

Hi @marschall Thanks for getting back on query Can I mock JobRepositoryTestUtils bean that is implicitly registered by @SpringBatchTest and couldn't see any option to exclude this bean to be registered by @SpringBatchTest, I really don't want to save and remove jobexecution.

munnema avatar Nov 22 '21 16:11 munnema

@munnema maybe it's better to raise an issue in that project and keep the discussion on this issue on topic

marschall avatar Nov 22 '21 19:11 marschall

The dependency to a datasource in JobRepositoryTestUtils was removed in #4070. As a result, SpringBatchTest should now work with a test context that does not contain a datasource.

Closing this as superseded by #4070 .

fmbenhassine avatar Aug 24 '22 13:08 fmbenhassine

Hi @fmbenhassine, Is there any workaround on spring-batch 2.7.2 to unit test jobs and steps without needing DataSource ? Thank you.

oha85 avatar May 18 '23 13:05 oha85

@oha85 This issue is closed. Please check https://github.com/spring-projects/spring-batch#getting-help for how to ask for support and I will try to help. Thank you.

fmbenhassine avatar May 19 '23 04:05 fmbenhassine

No indication how to implement the test case where DataSource is used or DataSource No qualifying bean of type is coming

javaHelper avatar Sep 24 '23 08:09 javaHelper

With Batch 5 it's now easily possible to write integration tests that do not need a javax.sql.DataSource.

An example can be found here:

https://github.com/marschall/spring-batch-inmemory/blob/master/src/test/java/com/github/marschall/spring/batch/inmemory/InMemoryBatchConfigurationTests.java

Or here:

https://github.com/marschall/spring-batch-inmemory/blob/master/src/test/java/com/github/marschall/spring/batch/inmemory/NullBatchConfigurationTests.java

marschall avatar Sep 24 '23 11:09 marschall