Skip to content

[LANG-1808] remove redundant test case in CharSetUtilsTest.testDelete_StringStringarray#1540

Merged
garydgregory merged 1 commit intoapache:masterfrom
IcoreE:RemoveRedundansTestCaseCharSetUtilsTest
Dec 23, 2025
Merged

[LANG-1808] remove redundant test case in CharSetUtilsTest.testDelete_StringStringarray#1540
garydgregory merged 1 commit intoapache:masterfrom
IcoreE:RemoveRedundansTestCaseCharSetUtilsTest

Conversation

@IcoreE
Copy link
Copy Markdown
Contributor

@IcoreE IcoreE commented Dec 23, 2025

The unit test method testDelete_StringStringarray() in CharSetUtilsTest contains a redundant assertion

        assertEquals("hello", CharSetUtils.delete("hello", ""));
        assertEquals("hello", CharSetUtils.delete("hello", ""));

@garydgregory
Copy link
Copy Markdown
Member

@IcoreE
You don't need to create Jira tickets for trivial changes.
TY!

@garydgregory garydgregory merged commit 67cbf81 into apache:master Dec 23, 2025
20 checks passed
@IcoreE
Copy link
Copy Markdown
Contributor Author

IcoreE commented Dec 23, 2025

Okay, I'll be more careful with this from now on. Thanks for reminding me. @garydgregory

@garydgregory
Copy link
Copy Markdown
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants