Browse Source

Merge pull request #18557 from dreis2211

* pr/18557:
  Polish
  Remove usage of @InjectMocks

Closes gh-18557
pull/18571/head
Stephane Nicoll 6 years ago
parent
commit
53b0569dae
  1. 25
      spring-boot-project/spring-boot-actuator/src/test/java/org/springframework/boot/actuate/integration/IntegrationGraphEndpointTests.java
  2. 2
      src/checkstyle/checkstyle.xml

25
spring-boot-project/spring-boot-actuator/src/test/java/org/springframework/boot/actuate/integration/IntegrationGraphEndpointTests.java

@ -16,11 +16,7 @@ @@ -16,11 +16,7 @@
package org.springframework.boot.actuate.integration;
import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;
import org.mockito.InjectMocks;
import org.mockito.Mock;
import org.mockito.MockitoAnnotations;
import org.springframework.integration.graph.Graph;
import org.springframework.integration.graph.IntegrationGraphServer;
@ -37,30 +33,23 @@ import static org.mockito.Mockito.verify; @@ -37,30 +33,23 @@ import static org.mockito.Mockito.verify;
*/
class IntegrationGraphEndpointTests {
@Mock
private IntegrationGraphServer integrationGraphServer;
private final IntegrationGraphServer server = mock(IntegrationGraphServer.class);
@InjectMocks
private IntegrationGraphEndpoint integrationGraphEndpoint;
@BeforeEach
void setUp() {
MockitoAnnotations.initMocks(this);
}
private final IntegrationGraphEndpoint endpoint = new IntegrationGraphEndpoint(this.server);
@Test
void readOperationShouldReturnGraph() {
Graph mockedGraph = mock(Graph.class);
given(this.integrationGraphServer.getGraph()).willReturn(mockedGraph);
Graph graph = this.integrationGraphEndpoint.graph();
verify(this.integrationGraphServer).getGraph();
given(this.server.getGraph()).willReturn(mockedGraph);
Graph graph = this.endpoint.graph();
verify(this.server).getGraph();
assertThat(graph).isEqualTo(mockedGraph);
}
@Test
void writeOperationShouldRebuildGraph() {
this.integrationGraphEndpoint.rebuild();
verify(this.integrationGraphServer).rebuild();
this.endpoint.rebuild();
verify(this.server).rebuild();
}
}

2
src/checkstyle/checkstyle.xml

@ -15,7 +15,7 @@ @@ -15,7 +15,7 @@
<property name="illegalPkgs"
value="^sun.*, ^org\.apache\.commons\.(?!compress|dbcp2|lang|lang3|logging|pool2).*, ^com\.google\.common.*, ^org\.flywaydb\.core\.internal.*, ^org\.testcontainers\.shaded.*" />
<property name="illegalClasses"
value="^com\.hazelcast\.util\.Base64, ^org\.junit\.rules\.ExpectedException, ^org\.slf4j\.LoggerFactory, ^reactor\.core\.support\.Assert" />
value="^com\.hazelcast\.util\.Base64, ^org\.junit\.rules\.ExpectedException, ^org\.mockito\.InjectMocks, ^org\.slf4j\.LoggerFactory, ^reactor\.core\.support\.Assert" />
</module>
<module
name="com.puppycrawl.tools.checkstyle.checks.imports.ImportControlCheck">

Loading…
Cancel
Save