Support new default value of spring boot 2.1 of spring.main.allow-bean-definition-overriding

[huehnerlady]

Hi,

will you add support of the fact that spring.main.allow-bean-definition-overriding is now defaulted to false and therefore your annotation ends up in a org.springframework.beans.factory.support.BeanDefinitionOverrideException?

[huehnerlady]

I asked about a different annotation and they added information about it here:
spring-projects/spring-boot#15216 (comment)

[pchudzik]

Hi. I'll take a look into this but cannot promise when it will be fixed. To be honest, I've not used this library for some time now and I'm not working with spring-boot nor springboot2 at that moment, therefore, I'm not actively using this library now.

PR will be always welcomed :)

I've just created a branch on which I plan to work on this and added sample which fails the build:
https://github.com/pchudzik/springmock/tree/springboot2
https://travis-ci.org/pchudzik/springmock/builds/460429005

[huehnerlady]

Hi,

I am not sure we would use this library, but if/when I have a free moment I am happy to look at it :)

[pchudzik]

@huehnerlady You can give it a try. It's been deployed to maven snapshots repo. I'll release it in a day or two after I double check if everything is working as supposed.

You can configure use snapshots using this instruction: https://github.com/pchudzik/springmock#snapshots

[pchudzik]

I've just released it you can give it a try:

<dependency>
  <groupId>com.pchudzik.springmock</groupId>
  <artifactId>springmock-spock</artifactId>
  <version>1.2.0</version>
</dependency>

or

testCompile('com.pchudzik.springmock:springmock-spock:1.2.0')

[huehnerlady]

Hi,

Sorry that I didn't answer earlier.
Thank you for making the effort in fixing this.
Unfortunately I receive an Exception if I use @AutowiredMock now:

package my.package

import com.pchudzik.springmock.infrastructure.annotation.AutowiredMock
import org.springframework.boot.test.context.SpringBootTest
import org.springframework.stereotype.Service
import spock.lang.Specification

import static org.springframework.boot.test.context.SpringBootTest.WebEnvironment.RANDOM_PORT

@SpringBootTest(classes = [TestApplication], webEnvironment = RANDOM_PORT)
class IntegrationTest extends Specification {

  @Service
  class MyService {

    String helloworld() {
      "Hello world"
    }
  }
  @AutowiredMock
  MyService myService

  def "should change return"() {
    given:
    myService.helloworld() >> "HELLOOOOOO"

    when:
    String actual = myService.helloworld()

    then:
    actual == "HELLOOOOOO"
  }
}

TestApplication:

package my.package

import org.springframework.boot.SpringApplication
import org.springframework.boot.autoconfigure.SpringBootApplication
import org.springframework.context.annotation.EnableAspectJAutoProxy
import org.springframework.retry.annotation.EnableRetry
import org.springframework.scheduling.annotation.EnableAsync

@SpringBootApplication
@EnableAspectJAutoProxy(proxyTargetClass = true)
@EnableAsync
@EnableRetry(proxyTargetClass = true)
class TestApplication {

  static void main(String[] args) {
    SpringApplication.run TestApplication, args
  }
}

When I run the test, I receive

java.lang.IllegalStateException: Failed to load ApplicationContext

Caused by: org.springframework.beans.factory.BeanCreationException: 
Error creating bean with name 'myService': Bean instantiation via factory method failed; nested exception is org.springframework.beans.BeanInstantiationException: 
Failed to instantiate [java.lang.Object]: 
Factory method 'createMock' threw exception; nested exception is java.lang.IllegalArgumentException: 
Could not create type

do you have any idea what I am doing wrong?

spring version: 5.1.2.RELEASE
spring boot: 2.1.0.RELEASE
springmock-spock: 1.2.0

Many thanks

[pchudzik]

Hi. I'm not sure what you want to achieve.

You are defining class MyService (which is not even static so probably will not be picked up by spring) inside test class and want to mock it? It is not real life example is it?

Please take a look at:
Springboot2Application.java
AddOneTranslator.java
TwoRepository.java
MyService.java
SpockSamplesApplicationTest.groovy

It's a bit more complex and shows more examples of what can you do but in some way is very similar. It mocks behaviour of some service (AddOneTranslator), verifies interaction with other (spied TwoRepository) and shows mixed spring-mock with native spring mocks (LogService)

You can also find some more description in my blogpost

[huehnerlady]

Well basically I had the problem in our system and so I tried to make a small example which leads up to that problem.

I have a Service (marked with the annotation @Servioce) and I have a test. in that test I want to mock the Service. This happens usually our Controller Integration tests where you tests errors etc. of the endpoint and one scenario would be that the service is called with the correct data.

But when I use the @AutowiredMock Annotation, I get the error shown above, that the method createMock could not create that type.

We do not have Mockito at all in our dependencies as we use spock for everything.
So any idea of why this problem occurs?

[pchudzik]

Maybe you can implement this example inside springmock samples. Just clone the repo (git@github.com:pchudzik/springmock.git) there is project already configured with springbok2 in folder samples/spock-springboot2 you can import it in idea or eclipse and create classes that reflect how you are working with spring and springmock and the problem you have (from what you are saying it doesn't look like a lot of work and it will be much easier for me to debug it and see what's going on). Once you'll reproduce issue you have just create pull request for springmock (samples folder is also tested) and from there I'll take over and see what needs to be fixed.

If you could reproduce the issue you are having in there I'll be happy to help and fix any bug. From the stacktrace you have pasted I will not be able to help you out there is just not enough information for me to figure out what might be wrong.

[huehnerlady]

Hi, that sounds fair, so I will see when I can implement it :)