Release 0.1.6 #198

mercyblitz · 2025-10-04T07:25:12Z

This pull request introduces several improvements and updates across the codebase, focusing mainly on utility enhancements, test coverage, and project configuration. The most significant changes include a refactor of the StringUtils.split method for better handling of edge cases, the addition of a utility method for converting collections to string arrays, expanded test coverage for string utilities, and updates to project dependencies and build configurations.

String Utilities Enhancements:

Refactored StringUtils.split to handle null and empty delimiters more robustly, and improved its logic for splitting strings, including support for splitting by empty strings and handling edge cases. [1] [2]
Added StringUtils.toStringArray(Collection<String>) utility method to convert collections of strings to string arrays safely.

Test Coverage Improvements:

Expanded StringUtilsTest to cover more cases for split, replace, substringBeforeLast, substringAfterLast, and added tests for the new toStringArray method. [1] [2] [3] [4] [5] [6]

Project Configuration and Documentation:

Added a CODE_OF_CONDUCT.md file to establish contributor guidelines and foster a welcoming community.
Updated the Maven build workflow to include Java 25 in the test matrix, ensuring compatibility with the latest Java version.
Upgraded the Spring Framework dependency to version 6.2.11 in the parent POM for improved security and features.

Minor Code and Test Updates:

Improved imports and usage of static constants in annotation processor test files and utility tests for clarity and consistency. [1] [2] [3] [4]
Fixed substring extraction logic in URLUtils.findSubProtocolsString to correctly handle subprotocols in URLs.

Sync dev code

Removed the 'spring-core' test dependency from microsphere-java-core/pom.xml and added multiple Spring Framework source files under src/test/java for testing purposes. This change allows tests to run without relying on the external spring-core dependency.

This reverts commit 66405ee.

Bumped the default Spring version to 6.2.11. Added a Maven profile 'spring5' to use Spring 5.3.31 for JDK versions 1.8 to 16, ensuring compatibility with older Java versions.

Reimplemented StringUtils.split to avoid StringTokenizer, handle null and empty delimiters, and added toStringArray utility. Updated StringUtilsTest to cover new edge cases and ensure compatibility with Spring's delimitedListToStringArray.

Corrects the substring indices when extracting the protocol from a URL string, ensuring the returned value does not include a leading character.

Eliminated redundant sleep call in FileUtilsTest to improve test efficiency and clarity.

…into dev

Release 0.1.6

Copilot

Pull Request Overview

Release update introducing enhanced String utilities, expanded tests, new collection-to-array helper, Spring version/profile adjustments, build matrix update, and added community guidelines.

Refactored StringUtils.split to support null and empty delimiters plus character-wise splitting when delimiter is empty.
Added toStringArray(Collection) and broadened related unit tests.
Introduced Spring 5 fallback profile for older JDKs and added Java 25 to CI matrix.

Reviewed Changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
pom.xml	Bumped project revision and parent build version.
microsphere-java-parent/pom.xml	Upgraded Spring to 6.2.11 with JDK-conditional Spring 5 profile.
microsphere-java-core/src/main/java/io/microsphere/util/StringUtils.java	Refactored split logic; added toStringArray and annotations; updated examples.
microsphere-java-core/src/main/java/io/microsphere/net/URLUtils.java	Corrected substring extraction for subprotocols.
microsphere-java-core/src/main/java/io/microsphere/util/ClassPathUtils.java	Minor documentation formatting tweak.
microsphere-java-core/src/test/java/io/microsphere/util/StringUtilsTest.java	Expanded split/replace/substring tests and added toStringArray coverage.
microsphere-java-core/src/test/java/io/microsphere/io/FileUtilsTest.java	Refactored an asynchronous test pattern for delete directory IOException scenario.
microsphere-java-core/src/test/java/io/microsphere/lang/ClassDataRepositoryTest.java	Removed immutability assertion on returned Set.
microsphere-annotation-processor/src/test/java/io/microsphere/annotation/processor/TestAnnotation.java	Switched to static imports for retention/target constants.
CODE_OF_CONDUCT.md	Added contributor code of conduct.
.github/workflows/maven-build.yml	Added Java 25 to CI build matrix.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

Copilot · 2025-10-04T07:27:19Z

microsphere-java-core/src/test/java/io/microsphere/util/StringUtilsTest.java

+        assertEquals(TEST_EMPTY_STRING, replace(TEST_EMPTY_STRING, "null", null));
+        assertEquals(TEST_CSV_STRING, replace(TEST_CSV_STRING, "null", "null"));
+
        assertEquals(TEST_EMPTY_STRING, replace(TEST_EMPTY_STRING, null, null));


Duplicate assertion on lines 180 and 184 adds noise without increasing coverage; remove one of them.

Suggested change

assertEquals(TEST_EMPTY_STRING, replace(TEST_EMPTY_STRING, null, null));

microsphere-java-core/src/test/java/io/microsphere/util/StringUtilsTest.java

Copilot · 2025-10-04T07:27:20Z

microsphere-java-core/src/test/java/io/microsphere/io/FileUtilsTest.java

+        ExecutorService executor = newSingleThreadExecutor();
+        executor.submit(this::testDeleteDirectoryOnIOException0);
+        shutdown(executor);
+        while (!executor.isTerminated()) {
+        }


Exceptions thrown in the submitted task are never observed (no Future captured), so the test can pass despite failures; capture the Future and call get() (e.g., Future<?> f = executor.submit(...); f.get();), eliminating the silent failure.

Copilot · 2025-10-04T07:27:20Z

microsphere-java-core/src/test/java/io/microsphere/io/FileUtilsTest.java

+        while (!executor.isTerminated()) {
+        }


Busy-wait loop wastes CPU cycles; replace with executor.awaitTermination(timeout, unit) or Future.get() to block efficiently.

Suggested change

while (!executor.isTerminated()) {

}

executor.awaitTermination(1000, MILLISECONDS);

microsphere-java-core/src/test/java/io/microsphere/io/FileUtilsTest.java

Copilot · 2025-10-04T07:27:20Z

microsphere-java-core/src/main/java/io/microsphere/util/StringUtils.java

     * <h3>Example Usage</h3>
     * <pre>{@code
     * StringUtils.split(null, ",")       = []
+     * StringUtils.split("", null)        = []
     * StringUtils.split("", ";")         = []
+     * StringUtils.split("abc", "")       = ["a", "b", "c"]
     * StringUtils.split("a,b,c", ",")    = ["a", "b", "c"]


Document the behavior for a non-empty String with a null delimiter (returns the whole String as a single element) to clarify the difference from empty and blank inputs.

…into dev

Copilot

Pull Request Overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated 4 comments.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

Copilot · 2025-10-04T07:32:10Z

microsphere-java-core/src/test/java/io/microsphere/io/FileUtilsTest.java

+        executor.submit(this::testDeleteDirectoryOnIOException0);
+        shutdown(executor);
+        while (!executor.isTerminated()) {


Assertions and exceptions inside testDeleteDirectoryOnIOException0 will not fail the test because the Future returned by submit(...) is ignored; any AssertionError thrown in the worker thread is swallowed. Capture the Future and call future.get() (or use executor.submit(...).get()) so failures propagate, or invoke the helper method directly on the test thread.

Suggested change

executor.submit(this::testDeleteDirectoryOnIOException0);

shutdown(executor);

while (!executor.isTerminated()) {

try {

// Capture the Future and call get() to propagate exceptions

executor.submit(this::testDeleteDirectoryOnIOException0).get();

} finally {

shutdown(executor);

while (!executor.isTerminated()) {

}

Copilot · 2025-10-04T07:32:11Z

microsphere-java-core/src/main/java/io/microsphere/util/StringUtils.java

+            if (startIndex <= length) {
+                // Add rest of String, but not in case of empty input.
+                result.add(value.substring(startIndex));
+            }


Condition 'if (startIndex <= length)' is always true here (startIndex cannot exceed length) given earlier flow and the empty-input guard at line 162, making the check redundant. Remove the conditional and add the tail element unconditionally to simplify.

Suggested change

if (startIndex <= length) {

// Add rest of String, but not in case of empty input.

result.add(value.substring(startIndex));

}

// Add rest of String, but not in case of empty input.

result.add(value.substring(startIndex));

Copilot · 2025-10-04T07:32:11Z

microsphere-java-core/src/test/java/io/microsphere/util/StringUtilsTest.java

    @Test
    void testReplace() {
        assertNull(replace(null, null, null));
+        assertEquals(TEST_EMPTY_STRING, replace(TEST_EMPTY_STRING, null, null));


Duplicate assertion (line 184 repeats line 180) adds no additional coverage; remove the duplicate to reduce noise.

Copilot · 2025-10-04T07:32:11Z

microsphere-java-core/src/main/java/io/microsphere/util/StringUtils.java

     * <h3>Example Usage</h3>
     * <pre>{@code
     * StringUtils.split(null, ",")       = []
+     * StringUtils.split("", null)        = []
     * StringUtils.split("", ";")         = []
+     * StringUtils.split("abc", "")       = ["a", "b", "c"]
     * StringUtils.split("a,b,c", ",")    = ["a", "b", "c"]
     * StringUtils.split("a;b;c", ",")    = ["a;b;c"]
     * StringUtils.split("a,,b,c", ",")   = ["a", "", "b", "c"]


Javadoc omits the behavior for a non-empty input with a null delimiter (returns the entire String as a single-element array). Add an example (e.g., StringUtils.split("abc", null) = ["abc"]) to fully document the null-delimiter case.

Updated the assertSplit method to correctly handle delimiter length checks and assertions. Now, assertions for single-character and multi-character delimiters are separated, ensuring accurate test validation.

Replaces busy-wait loop with assertTrue using awaitTermination for better test reliability and readability in testDeleteDirectoryOnIOException.

Changed the return type of testDeleteDirectoryOnIOException0 from Void to File and updated the return statement to return the test directory. This improves clarity and allows the caller to access the created directory.

Fix the issues from the code-review

sonarqubecloud · 2025-10-04T07:47:57Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

codecov · 2025-10-04T07:48:43Z

Codecov Report

❌ Patch coverage is 95.00000% with 1 line in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
...src/main/java/io/microsphere/util/StringUtils.java	94.73%	0 Missing and 1 partial ⚠️

Files with missing lines	Coverage Δ	Complexity Δ
...ore/src/main/java/io/microsphere/net/URLUtils.java	`90.05% <100.00%> (+1.70%)`	`119.00 <0.00> (+2.00)`
.../main/java/io/microsphere/util/ClassPathUtils.java	`84.61% <ø> (ø)`	`12.00 <0.00> (ø)`
...src/main/java/io/microsphere/util/StringUtils.java	`99.26% <94.73%> (-0.74%)`	`89.00 <8.00> (+5.00)`	⬇️

... and 4 files with indirect coverage changes

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

mercyblitz added 25 commits August 26, 2025 09:08

Update pom.xml

3ce2def

Update ClassPathUtils.java

0e5e451

Update TestAnnotation.java

188b794

Update maven-build.yml

8c28af9

Merge pull request #192 from mercyblitz/dev

1d3d639

Sync dev code

Revert "Remove spring-core test dependency and add test sources #193"

413a9df

This reverts commit 66405ee.

Update Spring version and add spring5 profile #194

4e75e71

Bumped the default Spring version to 6.2.11. Added a Maven profile 'spring5' to use Spring 5.3.31 for JDK versions 1.8 to 16, ensuring compatibility with older Java versions.

Update FileUtilsTest.java

da1dfa8

Update FileUtilsTest.java

c4ec584

Fix protocol extraction in URLUtils

df7d7d7

Corrects the substring indices when extracting the protocol from a URL string, ensuring the returned value does not include a leading character.

Remove unnecessary sleep in testDeleteDirectoryOnIOException

fed4170

Eliminated redundant sleep call in FileUtilsTest to improve test efficiency and clarity.

Update pom.xml

d73d621

Update pom.xml

72d1219

Create CODE_OF_CONDUCT.md

0c17b14

Update pom.xml

1607fc2

Update pom.xml

d6ed398

Merge branch 'microsphere-projects:dev' into dev

bd57ada

Update ClassDataRepositoryTest.java

86ce818

Merge branch 'dev' of https://github.com/mercyblitz/microsphere-java …

5c20964

…into dev

Update pom.xml

49094a2

Update StringUtils.java

2818838

Update StringUtilsTest.java

2b29afd

Merge pull request #195 from mercyblitz/dev

70770cf

Release 0.1.6

mercyblitz added this to the 0.1.6 milestone Oct 4, 2025

mercyblitz requested a review from Copilot October 4, 2025 07:25

Copilot AI reviewed Oct 4, 2025

View reviewed changes

mercyblitz added 2 commits October 4, 2025 15:30

Update StringUtilsTest.java

9882dc3

Merge branch 'dev' of https://github.com/mercyblitz/microsphere-java …

ce40551

…into dev

mercyblitz requested a review from Copilot October 4, 2025 07:30

Copilot AI reviewed Oct 4, 2025

View reviewed changes

mercyblitz added 5 commits October 4, 2025 15:38

Fix split assertion logic in StringUtilsTest

d42bbbb

Updated the assertSplit method to correctly handle delimiter length checks and assertions. Now, assertions for single-character and multi-character delimiters are separated, ensuring accurate test validation.

Improve executor termination in FileUtilsTest

985ddc8

Replaces busy-wait loop with assertTrue using awaitTermination for better test reliability and readability in testDeleteDirectoryOnIOException.

Update return type of testDeleteDirectoryOnIOException0

ed3ea11

Changed the return type of testDeleteDirectoryOnIOException0 from Void to File and updated the return statement to return the test directory. This improves clarity and allows the caller to access the created directory.

Update FileUtilsTest.java

8b07ae3

Merge pull request #199 from mercyblitz/dev

6c42c68

Fix the issues from the code-review

mercyblitz merged commit ec6cba8 into release Oct 4, 2025
7 checks passed

	while (!executor.isTerminated()) {
	}
	executor.awaitTermination(1000, MILLISECONDS);

Release 0.1.6 #198

Release 0.1.6 #198

Uh oh!

Conversation

mercyblitz commented Oct 4, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI Oct 4, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Copilot AI Oct 4, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 4, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Copilot AI Oct 4, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Copilot AI Oct 4, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 4, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 4, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 4, 2025

Choose a reason for hiding this comment

Uh oh!

sonarqubecloud bot commented Oct 4, 2025

Quality Gate passed

Uh oh!

codecov bot commented Oct 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

codecov bot commented Oct 4, 2025 •

edited

Loading