From 25cd97d9626dcc1cd3dd82b3ad5c934cdb137342 Mon Sep 17 00:00:00 2001 From: Andrew Gauld Date: Fri, 24 Jan 2020 19:14:27 +0000 Subject: Improve mod/runtimeapi code coverage Change-Id: Ic8b23955deed9411ac2c216571ec2548a9fc5e42 Issue-ID: DCAEGEN2-1860 Signed-off-by: Andrew Gauld --- .../web/controllers/TestFlowGraphController.java | 65 +++++++++++++--------- 1 file changed, 38 insertions(+), 27 deletions(-) (limited to 'mod/runtimeapi/runtime-web/src/test/java/org/onap/dcae/runtime/web/controllers/TestFlowGraphController.java') diff --git a/mod/runtimeapi/runtime-web/src/test/java/org/onap/dcae/runtime/web/controllers/TestFlowGraphController.java b/mod/runtimeapi/runtime-web/src/test/java/org/onap/dcae/runtime/web/controllers/TestFlowGraphController.java index e17e1c6..dbbfd8d 100644 --- a/mod/runtimeapi/runtime-web/src/test/java/org/onap/dcae/runtime/web/controllers/TestFlowGraphController.java +++ b/mod/runtimeapi/runtime-web/src/test/java/org/onap/dcae/runtime/web/controllers/TestFlowGraphController.java @@ -1,6 +1,6 @@ /*- * ============LICENSE_START======================================================= - * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved. + * Copyright (C) 2019-2020 AT&T Intellectual Property. All rights reserved. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -17,38 +17,53 @@ */ package org.onap.dcae.runtime.web.controllers; +import java.util.HashMap; +import java.util.LinkedList; + import org.onap.dcae.runtime.core.Edge; import org.onap.dcae.runtime.core.FlowGraph; import org.onap.dcae.runtime.core.Node; +import org.onap.dcae.runtime.web.ClientMocking; +import org.onap.dcae.runtime.web.Helper; import org.onap.dcae.runtime.web.TestUtils; +import org.onap.dcae.runtime.web.models.Action; +import org.onap.dcae.runtime.web.models.DistributeGraphRequest; import org.onap.dcae.runtime.web.models.GraphRequest; import org.onap.dcae.runtime.web.service.GraphServiceImpl; +import org.onap.dcae.runtime.web.service.BlueprintInventory; import org.junit.Before; import org.junit.Test; import org.junit.runner.RunWith; import org.springframework.beans.factory.annotation.Autowired; -import org.springframework.boot.test.autoconfigure.web.servlet.WebMvcTest; +import org.springframework.boot.test.autoconfigure.web.servlet.AutoConfigureMockMvc; +import org.springframework.boot.test.context.SpringBootTest; import org.springframework.boot.test.mock.mockito.MockBean; import org.springframework.http.MediaType; import org.springframework.test.context.junit4.SpringRunner; import org.springframework.test.web.servlet.MockMvc; import static org.mockito.Mockito.when; +import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.delete; import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get; import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.post; import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.jsonPath; import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status; @RunWith(SpringRunner.class) -@WebMvcTest +@SpringBootTest +@AutoConfigureMockMvc public class TestFlowGraphController { @Autowired private MockMvc mockMvc; - @MockBean + @Autowired + private BlueprintInventory blueprintInventory; + + @Autowired private GraphServiceImpl graphService; + private DistributeGraphRequest distributeGraphRequest; private GraphRequest graphRequest; @Before @@ -57,41 +72,37 @@ public class TestFlowGraphController { graphRequest.setId("1234"); graphRequest.setName("nifi-main"); graphRequest.setDescription("mock graph"); + distributeGraphRequest = new DistributeGraphRequest(); + ClientMocking inv = new ClientMocking() + .on("POST /ccsdk-app/api-if", "\"OK\"") + .applyTo(blueprintInventory); } @Test public void testInitializeGraph() throws Exception{ graphRequest.setMain(true); + mockMvc.perform(get("/api/graph/main")) + .andExpect(status().isNotFound()); mockMvc.perform(post("/api/graph/main") .contentType(MediaType.APPLICATION_JSON) .content(TestUtils.convertObjectToJsonBytes(graphRequest))) .andExpect(status().isCreated()); - } - - @Test - public void testGetMainGraph() throws Exception{ - FlowGraph flowGraph = new FlowGraph<>("1234","nifi-main",true,"demo"); - when(this.graphService.getMainGraph()).thenReturn(flowGraph); + mockMvc.perform(post("/api/graph/main") + .contentType(MediaType.APPLICATION_JSON) + .content(TestUtils.convertObjectToJsonBytes(graphRequest))) + .andExpect(status().isConflict()); mockMvc.perform(get("/api/graph/main")) .andExpect(status().isOk()) .andExpect(jsonPath("id").value("1234")) .andExpect(jsonPath("name").value("nifi-main")); + distributeGraphRequest.setActions(Helper.getAddNodesWithEdge()); + mockMvc.perform(post("/api/graph/" + graphRequest.getId() + "/distribute") + .contentType(MediaType.APPLICATION_JSON) + .content(TestUtils.convertObjectToJsonBytes(distributeGraphRequest))) + .andExpect(status().isOk()); + mockMvc.perform(delete("/api/graph/main")) + .andExpect(status().isOk()); + mockMvc.perform(delete("/api/graph/main")) + .andExpect(status().isNotFound()); } - -// @Test -// public void testMainGraphNotFound() throws Exception{ -// when(this.graphService.getMainGraph()).thenReturn(null); -// mockMvc.perform(get("/api/graph/main")) -// .andExpect(status().isNotFound()); -// } - -// @Test -// public void testConflictIfMainGraphExists() throws Exception{ -// when(this.graphService.initializeMainGraph(graphRequest)).thenThrow(new MainGraphAlreadyExistException("")); -// mockMvc.perform(post("/api/graph/main") -// .contentType(MediaType.APPLICATION_JSON) -// .content(TestUtils.convertObjectToJsonBytes(graphRequest))) -// .andExpect(status().isConflict()); -// } - } -- cgit 1.2.3-korg