From ece736d5d4b3a567281d0abc30f1f28468941b65 Mon Sep 17 00:00:00 2001 From: Looly Date: Sat, 24 Jun 2023 19:36:10 +0800 Subject: [PATCH] fix code --- .../hutool/core/collection/CollUtil.java | 4 +-- .../hutool/core/collection/CollUtilTest.java | 18 +++++++++++++ .../hutool/setting/IssueI7EWGOTest.java | 27 +++++++++++++++++++ 3 files changed, 47 insertions(+), 2 deletions(-) create mode 100755 hutool-setting/src/test/java/org/dromara/hutool/setting/IssueI7EWGOTest.java diff --git a/hutool-core/src/main/java/org/dromara/hutool/core/collection/CollUtil.java b/hutool-core/src/main/java/org/dromara/hutool/core/collection/CollUtil.java index dc92bc867..39a1e444d 100644 --- a/hutool-core/src/main/java/org/dromara/hutool/core/collection/CollUtil.java +++ b/hutool-core/src/main/java/org/dromara/hutool/core/collection/CollUtil.java @@ -2110,7 +2110,7 @@ public class CollUtil { * @since 4.6.5 */ public static > T max(final Collection coll) { - return Collections.max(coll); + return isEmpty(coll) ? null : Collections.max(coll); } /** @@ -2123,7 +2123,7 @@ public class CollUtil { * @since 4.6.5 */ public static > T min(final Collection coll) { - return Collections.min(coll); + return isEmpty(coll) ? null : Collections.min(coll); } /** diff --git a/hutool-core/src/test/java/org/dromara/hutool/core/collection/CollUtilTest.java b/hutool-core/src/test/java/org/dromara/hutool/core/collection/CollUtilTest.java index e177e8e0d..8aebcef74 100644 --- a/hutool-core/src/test/java/org/dromara/hutool/core/collection/CollUtilTest.java +++ b/hutool-core/src/test/java/org/dromara/hutool/core/collection/CollUtilTest.java @@ -1163,4 +1163,22 @@ public class CollUtilTest { Assertions.assertFalse(CollUtil.allMatch(list, i -> i == 1)); Assertions.assertTrue(CollUtil.allMatch(list, i -> i <= 6)); } + + @Test + public void maxTest() { + final List list = Arrays.asList(1, 2, 3, 4, 5, 6); + Assertions.assertEquals((Integer) 6, CollUtil.max(list)); + } + + @SuppressWarnings({"rawtypes", "unchecked"}) + @Test + public void maxEmptyTest() { + final List emptyList = Collections.emptyList(); + Assertions.assertNull(CollUtil.max(emptyList)); + } + + @Test + public void minNullTest() { + Assertions.assertNull(CollUtil.max(null)); + } } diff --git a/hutool-setting/src/test/java/org/dromara/hutool/setting/IssueI7EWGOTest.java b/hutool-setting/src/test/java/org/dromara/hutool/setting/IssueI7EWGOTest.java new file mode 100755 index 000000000..9f326a956 --- /dev/null +++ b/hutool-setting/src/test/java/org/dromara/hutool/setting/IssueI7EWGOTest.java @@ -0,0 +1,27 @@ +/* + * Copyright (c) 2023 looly(loolly@aliyun.com) + * Hutool is licensed under Mulan PSL v2. + * You can use this software according to the terms and conditions of the Mulan PSL v2. + * You may obtain a copy of Mulan PSL v2 at: + * http://license.coscl.org.cn/MulanPSL2 + * THIS SOFTWARE IS PROVIDED ON AN "AS IS" BASIS, WITHOUT WARRANTIES OF ANY KIND, + * EITHER EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO NON-INFRINGEMENT, + * MERCHANTABILITY OR FIT FOR A PARTICULAR PURPOSE. + * See the Mulan PSL v2 for more details. + */ + +package org.dromara.hutool.setting; + +import org.junit.jupiter.api.Assertions; +import org.junit.jupiter.api.Test; + +public class IssueI7EWGOTest { + @Test + void loadTest() { + final Setting setting1 = SettingUtil.get("test"); + final Setting setting2 = new Setting("test.setting"); + + Assertions.assertNotNull(setting1); + Assertions.assertNotNull(setting2); + } +}