Merge pull request #1387 from gzhao9/refactor-OkHttpRequestAdapterTest

Refactor to Eliminate Repetitive Mock Object Creation in OkHttpRequestAdapterTest.java

Author: andrueastman

components/http/okHttp/src/test/java/com/microsoft/kiota/http/OkHttpRequestAdapterTest.java

@@ -167,8 +167,7 @@ void sendReturnsNullOnNoContent(int statusCode) throws Exception {
                         httpMethod = HttpMethod.GET;
                     }
                 };
-        final var mockEntity = mock(Parsable.class);
-        when(mockEntity.getFieldDeserializers()).thenReturn(new HashMap<>());
+        final var mockEntity = creatMockEntity();
         final var response = requestAdapter.send(requestInformation, null, (node) -> mockEntity);
         assertNull(response);
     }
@@ -198,14 +197,9 @@ void sendReturnsObjectOnContent(int statusCode) throws Exception {
                         httpMethod = HttpMethod.GET;
                     }
                 };
-        final var mockEntity = mock(Parsable.class);
-        when(mockEntity.getFieldDeserializers()).thenReturn(new HashMap<>());
-        final var mockParseNode = mock(ParseNode.class);
-        when(mockParseNode.getObjectValue(any(ParsableFactory.class))).thenReturn(mockEntity);
-        final var mockFactory = mock(ParseNodeFactory.class);
-        when(mockFactory.getParseNode(any(String.class), any(InputStream.class)))
-                .thenReturn(mockParseNode);
-        when(mockFactory.getValidContentType()).thenReturn("application/json");
+        final var mockEntity = creatMockEntity();
+        final var mockParseNode = creatMockParseNode(mockEntity);
+        final var mockFactory = creatMockParseNodeFactory(mockParseNode, "application/json");
         final var requestAdapter =
                 new OkHttpRequestAdapter(authenticationProviderMock, mockFactory, null, client);
         final var response = requestAdapter.send(requestInformation, null, (node) -> mockEntity);
@@ -260,16 +254,11 @@ void throwsAPIException(
                         httpMethod = HttpMethod.GET;
                     }
                 };
-        final var mockEntity = mock(Parsable.class);
-        when(mockEntity.getFieldDeserializers()).thenReturn(new HashMap<>());
+        final var mockEntity = creatMockEntity();
         final var mockParsableFactory = mock(ParsableFactory.class);
         when(mockParsableFactory.create(any(ParseNode.class))).thenReturn(mockEntity);
-        final var mockParseNode = mock(ParseNode.class);
-        when(mockParseNode.getObjectValue(any(ParsableFactory.class))).thenReturn(mockEntity);
-        final var mockFactory = mock(ParseNodeFactory.class);
-        when(mockFactory.getParseNode(any(String.class), any(InputStream.class)))
-                .thenReturn(mockParseNode);
-        when(mockFactory.getValidContentType()).thenReturn("application/json");
+        final var mockParseNode = creatMockParseNode(mockEntity);
+        final var mockFactory = creatMockParseNodeFactory(mockParseNode, "application/json");
 
         final var requestAdapter =
                 new OkHttpRequestAdapter(authenticationProviderMock, mockFactory, null, client);
@@ -430,4 +419,25 @@ public static OkHttpClient getMockClient(final Response response) throws IOExcep
         when(mockClient.newCall(any())).thenReturn(remoteCall);
         return mockClient;
     }
+
+    public Parsable creatMockEntity() {
+        final var mockEntity = mock(Parsable.class);
+        when(mockEntity.getFieldDeserializers()).thenReturn(new HashMap<>());
+        return mockEntity;
+    }
+
+    public ParseNode creatMockParseNode(Parsable entity) {
+        final var mockParseNode = mock(ParseNode.class);
+        when(mockParseNode.getObjectValue(any(ParsableFactory.class))).thenReturn(entity);
+        return mockParseNode;
+    }
+
+    public ParseNodeFactory creatMockParseNodeFactory(
+            ParseNode mockParseNode, String validContentType) {
+        final var mockFactory = mock(ParseNodeFactory.class);
+        when(mockFactory.getParseNode(any(String.class), any(InputStream.class)))
+                .thenReturn(mockParseNode);
+        when(mockFactory.getValidContentType()).thenReturn(validContentType);
+        return mockFactory;
+    }
 }