Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor(web/test): replace deprecated mockito api verifyZeroInteractions() with verifyNoMoreInteractions() during upgrade to spring boot 2.6.x #1207

Merged
merged 3 commits into from Dec 20, 2023

Conversation

j-sandy
Copy link
Contributor

@j-sandy j-sandy commented Dec 20, 2023

While upgrading spring boot 2.6.15, encounter below errors in igor-web module during test compilation:

> Task :igor-web:compileTestJava FAILED
/igor/igor-web/src/test/java/com/netflix/spinnaker/igor/nexus/NexusEventPosterTest.java:21: error: cannot find symbol
import static org.mockito.Mockito.verifyZeroInteractions;
^
  symbol:   static verifyZeroInteractions
  location: class Mockito
/igor/igor-web/src/test/java/com/netflix/spinnaker/igor/nexus/NexusEventPosterTest.java:119: error: cannot find symbol
    verifyZeroInteractions(echoService);
    ^
  symbol:   method verifyZeroInteractions(EchoService)
  location: class NexusEventPosterTest
/igor/igor-web/src/test/java/com/netflix/spinnaker/igor/nexus/NexusEventPosterTest.java:129: error: cannot find symbol
    verifyZeroInteractions(echoService);
    ^
  symbol:   method verifyZeroInteractions(EchoService)
  location: class NexusEventPosterTest
/igor/igor-web/src/test/java/com/netflix/spinnaker/igor/nexus/NexusEventPosterTest.java:139: error: cannot find symbol
    verifyZeroInteractions(echoService);
    ^
  symbol:   method verifyZeroInteractions(EchoService)
  location: class NexusEventPosterTest
4 errors

Spring boot 2.6.15 brings mockito 4.0.0 as transitive dependency. The verifyZeroInteractions() was deprecated in 3.x as mentioned here: mockito/mockito-kotlin#383 And now it is removed from mockito 4.0.0.
To fix these issues, replacing verifyZeroInteractions() with verifyNoMoreInteractions()

…ions() with verifyNoMoreInteractions() during upgrade to spring boot 2.6.x

While upgrading spring boot 2.6.15, encounter below errors in igor-web module during test compilation:
```
> Task :igor-web:compileTestJava FAILED
/igor/igor-web/src/test/java/com/netflix/spinnaker/igor/nexus/NexusEventPosterTest.java:21: error: cannot find symbol
import static org.mockito.Mockito.verifyZeroInteractions;
^
  symbol:   static verifyZeroInteractions
  location: class Mockito
/igor/igor-web/src/test/java/com/netflix/spinnaker/igor/nexus/NexusEventPosterTest.java:119: error: cannot find symbol
    verifyZeroInteractions(echoService);
    ^
  symbol:   method verifyZeroInteractions(EchoService)
  location: class NexusEventPosterTest
/igor/igor-web/src/test/java/com/netflix/spinnaker/igor/nexus/NexusEventPosterTest.java:129: error: cannot find symbol
    verifyZeroInteractions(echoService);
    ^
  symbol:   method verifyZeroInteractions(EchoService)
  location: class NexusEventPosterTest
/igor/igor-web/src/test/java/com/netflix/spinnaker/igor/nexus/NexusEventPosterTest.java:139: error: cannot find symbol
    verifyZeroInteractions(echoService);
    ^
  symbol:   method verifyZeroInteractions(EchoService)
  location: class NexusEventPosterTest
4 errors
```
Spring boot [2.6.15](https://repo1.maven.org/maven2/org/springframework/boot/spring-boot-dependencies/2.6.15/spring-boot-dependencies-2.6.15.pom) brings mockito 4.0.0 as transitive dependency. The `verifyZeroInteractions()` was deprecated in 3.x as mentioned here:
mockito/mockito-kotlin#383
And now it is removed from mockito 4.0.0.
To fix these issues, replacing `verifyZeroInteractions()` with `verifyNoMoreInteractions()`
@dbyron-sf dbyron-sf added the ready to merge Approved and ready for merge label Dec 20, 2023
@mergify mergify bot added the auto merged label Dec 20, 2023
@mergify mergify bot merged commit 793665f into spinnaker:master Dec 20, 2023
4 checks passed
@j-sandy j-sandy deleted the mockito-dep branch December 21, 2023 04:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants