Content type error when mocking static method

Question

I am trying to test a method inside a controller. The test passes if I comment out logic inside a static method that is called in the method I am testing.

I can't be commenting out that logic and instead I just want to mock it. Now the mock works, but I get a new error as follows:

java.lang.AssertionError: Content type not set

But I do have the content type indicated. Please advie what I am doing wrong.

@Test
public void testMethod() throws Exception{

    // If I don't mock this, test will fail. 
    // If I don't mock this comment out logic in this method, test passes. 
    // If I mock this, test passes if I don't check for content type.
    // I am using Power Mockito. 

    mockStatic(MyStaticClass.class);
    doReturn("").when(MyStaticClass.class, "someMethod", any(Config.class), anyString());

    //also tried this, works. 
    //when(MyStaticClass.someMethod(any(Config.class), anyString())).thenReturn("");


    //as mentioned above this would work if I comment out logic in MyStaticClass. 
    mockMvc.perform(
                get("/api/some/a/b/c/d").accept(
                        MediaType.APPLICATION_JSON))
                .andExpect(status().isForbidden())
                .andExpect(content().contentType("text/html")); // when I mock, I need to comment this out to get test to work.  
}



// Controller 

@RequestMapping(value = "/{a}/{b}/{c}", method = RequestMethod.GET, produces = MediaType.APPLICATION_JSON_VALUE) // I do have content type
@ResponseBody
public MyResponse getSomething(
            HttpServletRequest request, HttpServletResponse response,
            @PathVariable String a, @PathVariable String b,
            @PathVariable String c,
            @RequestParam(value = "some", required = false) String some)
            throws Exception {

            // some logic 

            //static method being called
            MyStaticClass.someMethod("sample", "sample2");

            try {
                MyResponse myPageResponse = new MyResponse(anotherStr, someStr); // it breaks here and throws that error msg. Doesn't reach return.
                return MyResponse;
            } catch (NullPointerException npe) {}
}

score 1 · Accepted Answer · answered Oct 21 '18 at 18:02

1

This is a get request it doesn't have a body hence Ideally specifying content-type header using contentType("text/html") may not be a right approach. Secondly your content-type header in request must match with the @consumes value, it clearly state that you are expecting to send text/html but there is no @consumes to support that.

answered Oct 21 '18 at 18:02

Amit Kumar Lal

5,537
3
19
37

Hi, do you mean ideally I shouldn't be doing this? -> .andExpect(content().contentType("text/html")); ? – kang Oct 22 '18 at 08:25
Ideally Get request do not have a body hence contentType doesn't make sense, but even if you want to implement it then you have to add the code for the same using @consumes (its similar to how you have used produces) in the code. – Amit Kumar Lal Oct 22 '18 at 09:08
request you to go through https://stackoverflow.com/questions/5661596/do-i-need-a-content-type-for-http-get-requests – Amit Kumar Lal Oct 22 '18 at 09:13
please upvote and accept the answer if you find this useful :) – Amit Kumar Lal Oct 22 '18 at 09:14
1

Thanks for answering. Makes sense. – kang Oct 22 '18 at 12:20

Content type error when mocking static method

1 Answers1