Refator JacksonUtils and JacksonUtilsTest in 2021 (#367)

CHANGELOG.md

@@ -22,3 +22,4 @@
 - [rm code: Condition 'null != interceptors' is always 'true' ](https://github.com/Tencent/spring-cloud-tencent/pull/357)
 - [docs:update logo in README.](https://github.com/Tencent/spring-cloud-tencent/pull/360)
 - [Use jdk constants instead of magic variables](https://github.com/Tencent/spring-cloud-tencent/pull/361)
+- [Refator JacksonUtils and JacksonUtilsTest](https://github.com/Tencent/spring-cloud-tencent/pull/367)

spring-cloud-tencent-commons/src/main/java/com/tencent/cloud/common/util/JacksonUtils.java

@@ -30,7 +30,7 @@ import org.springframework.util.StringUtils;
 /**
  * Utils for Jackson.
  *
- * @author Haotian Zhang
+ * @author Haotian Zhang, cheese8
  */
 public final class JacksonUtils {
 
@@ -80,8 +80,7 @@ public final class JacksonUtils {
 		}
 		catch (JsonProcessingException e) {
 			LOG.error(
-					"Json to map failed. check if the format of the json string[{}] is correct.",
+					"Json to map failed. check if the format of the json string[{}] is correct.", jsonStr, e);
-					jsonStr, e);
 			throw new RuntimeException("Json to map failed.", e);
 		}
 	}

spring-cloud-tencent-commons/src/test/java/com/tencent/cloud/common/util/JacksonUtilsTest.java

@@ -26,12 +26,12 @@ import org.junit.runner.RunWith;
 import org.mockito.junit.MockitoJUnitRunner;
 
 import static org.assertj.core.api.Assertions.assertThat;
-import static org.assertj.core.api.Assertions.fail;
+import static org.assertj.core.api.Assertions.assertThatThrownBy;
 
 /**
  * Test for {@link JacksonUtils}.
  *
- * @author lepdou, Haotian Zhang
+ * @author lepdou, Haotian Zhang, cheese8
  */
 @RunWith(MockitoJUnitRunner.class)
 public class JacksonUtilsTest {
@@ -42,10 +42,7 @@ public class JacksonUtilsTest {
 		sourceMap.put("k1", "v1");
 		sourceMap.put("k2", "v2");
 		sourceMap.put("k3", "v3");
-
+		assertThat(JacksonUtils.serialize2Json(sourceMap)).isEqualTo("{\"k1\":\"v1\",\"k2\":\"v2\",\"k3\":\"v3\"}");
-		String jsonStr = JacksonUtils.serialize2Json(sourceMap);
-
-		assertThat(jsonStr).isEqualTo("{\"k1\":\"v1\",\"k2\":\"v2\",\"k3\":\"v3\"}");
 	}
 
 	@Test
@@ -56,20 +53,19 @@ public class JacksonUtilsTest {
 		assertThat(map.get("k1")).isEqualTo("v1");
 		assertThat(map.get("k2")).isEqualTo("v2");
 		assertThat(map.get("k3")).isEqualTo("v3");
+	}
 
-		assertThat(JacksonUtils.deserialize2Map("")).isNotNull();
+	@Test
-		assertThat(JacksonUtils.deserialize2Map("")).isEmpty();
+	public void testDeserializeBlankIntoEmptyMap() {
+		Map<String, String> map = JacksonUtils.deserialize2Map("");
+		assertThat(map).isNotNull();
+		assertThat(map).isEmpty();
+	}
 
-		jsonStr = "{\"k1\":\"v1\",\"k2\":\"v2\",\"k3\":\"v3\"";
+	@Test
-		try {
+	public void testDeserializeThrowsRuntimeException() {
-			JacksonUtils.deserialize2Map(jsonStr);
+		String jsonStr = "{\"k1\":\"v1\",\"k2\":\"v2\",\"k3\":\"v3\"";
-			fail("RuntimeException should be thrown.");
+		assertThatThrownBy(() -> JacksonUtils.deserialize2Map(jsonStr))
-		}
+				.isExactlyInstanceOf(RuntimeException.class).hasMessage("Json to map failed.");
-		catch (RuntimeException exception) {
-			assertThat(exception.getMessage()).isEqualTo("Json to map failed.");
-		}
-		catch (Throwable throwable) {
-			fail("RuntimeException should be thrown.");
-		}
 	}
 }