Mockito mockStatic on InjectMocks Object

Question

I'm using Mockito 3.6 to take advantage of their mockStatic method. I am having issue when mocking a class that is also the main class to be used for testing.

public class MainClass{
    private HttpClient httpClient;

    public String sendRequest(){
        HttpRequest request = HttpRequest.newBuilder()
            .uri(buildUri(System.getEnv("host"), System.getEnv("port")))
            .GET().build()
        HttpResponse<String> response = httpClient.send(request, ofString());
        return response.body
    }

    public static URI buildURI(String host, String path) throws URISyntaxException {
        return new URI("http", host, path, null);
    }
}

This is my test class

@ExtendWith(MockitoExtension.class)
public class MainClassTest {
    @Spy
    private ObjectMapper mapper = new ObjectMapper();
    @Mock
    private HttpClient httpClientMock;
    @InjectMocks
    private MainClass mainClass;

    private MockedStatic<MainClass> systemMock;

    @BeforeEach
    void setUp() {
        systemMock = mockStatic(MainClass.class);
    }

    @AfterEach
    void tearDown() {
        systemMock.close();
    }

    @Test
    void sendRequest_successful(){
        HttpResponse response = mock(HttpResponse.class);
        when(response.body()).thenReturn("Hello, world!");
        when(httpClientMock.send(any(HttpRequest.class), any())).thenReturn(response);
        systemMock.when(() -> MainClass.buildURI(anyString(), anyString()))
                .thenReturn(URI.create("http://host.com/path"));

        assertEquals("Hello, world!", mainClass.sendRequest());
    }
}

Since the MainClass is being used to be mockStatic and injectMock, when calling buildURI, it always return null, which is not acceptable when creating HttpRequest.

Edit: To clarify my issue, I'm getting the host and port from environment variable, which will be null when running this test, and calling new URI() does not allow null values.

Why do we need to mock the static method at all? From the looks of it, the method is never called in the test presented. — Turing85, Nov 16 '21 at 21:32
Again the question: Why do you want to mock this call? Your test would pass without it. — Turing85, Nov 17 '21 at 17:24
Are you using maven? If so, we can [set system properties through the `maven-surefire-plugin`](http://maven.apache.org/surefire/maven-surefire-plugin/examples/system-properties.html), i.e. no mocking needed. — Turing85, Nov 18 '21 at 16:36

Mockito mockStatic on InjectMocks Object

0 Answers0