#112 Now using jsonNode.get instead of jsonNode.findPath to select returned

fields. By wrongly using findPath a different createdOn field was taken from the response.

Author: Chris Wiechmann

CHANGELOG.md

@@ -4,6 +4,10 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/)
 and this project adheres to [Semantic Versioning](http://semver.org/).
 
+## [Unreleased]
+### Fixed
+- Created BE-API response was parsing the response wrong. This could lead to an issue, if the API contains a createdOn field. (See issue [#112](https://github.com/Axway-API-Management-Plus/apim-cli/issues/112))
+
 ## [1.3.0] 2020-11-10
 ### Added
 - Search for APIs based on configured Inbound- and Outbound-Security (See issue [#86](https://github.com/Axway-API-Management-Plus/apim-cli/issues/86))

modules/apim-adapter/src/main/java/com/axway/apim/adapter/apis/APIManagerAPIAdapter.java

@@ -718,9 +718,9 @@ public API importBackendAPI(API api) throws AppException {
 				jsonNode =  importFromSwagger(api);
 			}
 			API createdAPI = new APIBaseDefinition();
-			createdAPI.setApiId(jsonNode.findPath("id").asText());
-			createdAPI.setName(jsonNode.findPath("name").asText());
-			createdAPI.setCreatedOn(Long.parseLong(jsonNode.findPath("createdOn").asText()));
+			createdAPI.setApiId(jsonNode.get("id").asText());
+			createdAPI.setName(jsonNode.get("name").asText());
+			createdAPI.setCreatedOn(Long.parseLong(jsonNode.get("createdOn").asText()));
 			return createdAPI;
 		} catch (Exception e) {
 			throw new AppException("Can't import definition / Create BE-API.", ErrorCode.CANT_CREATE_BE_API, e);

modules/apis/src/test/java/com/axway/apim/model/BEAPICreatedResponseTest.java

@@ -0,0 +1,35 @@
+package com.axway.apim.model;
+
+import java.io.IOException;
+
+import org.testng.Assert;
+import org.testng.annotations.Test;
+
+import com.axway.apim.api.API;
+import com.axway.apim.api.APIBaseDefinition;
+import com.fasterxml.jackson.core.JsonParseException;
+import com.fasterxml.jackson.databind.JsonMappingException;
+import com.fasterxml.jackson.databind.JsonNode;
+import com.fasterxml.jackson.databind.ObjectMapper;
+
+public class BEAPICreatedResponseTest {
+	
+	private static final String testPackage = "com/axway/apim/model/";
+	
+	ObjectMapper mapper = new ObjectMapper();
+	
+	@Test
+	public void parseResponseHavingCreatedOn() throws JsonParseException, JsonMappingException, IOException {
+		JsonNode jsonNode = mapper.readTree(this.getClass().getClassLoader().getResourceAsStream(testPackage + "BEAPICreatedWithCreatedOn.json"));
+		API createdAPI = new APIBaseDefinition();
+		createdAPI.setApiId(jsonNode.get("id").asText());
+		createdAPI.setName(jsonNode.get("name").asText());
+		createdAPI.setName(jsonNode.get("description").asText());
+		
+		createdAPI.setCreatedOn(Long.parseLong(jsonNode.get("createdOn").asText()));
+		
+		Assert.assertEquals(new Long("1605099396581"), createdAPI.getCreatedOn());
+		// Tests if the Unicode-Description is parsed without giving a encoding
+		Assert.assertEquals("提供銀行身份認證整合 Axway APIM OAuth Authorization Code Grant Flow\n", jsonNode.get("description").asText());
+	}
+}