MCR-3302 fix edge cases for MCRCategoryID.fromString()

[yagee-de]

Link to jira.

This pull request includes several changes to improve the handling of category IDs and update the testing framework in the mycore-base module. The most important changes include adding a new JUnit dependency, refactoring the fromString method in MCRCategoryID, and updating the tests to use JUnit 5.

Dependency Updates:

• Added junit-jupiter-params dependency to pom.xml for parameterized tests.

Code Refactoring:

• Refactored fromString method in MCRCategoryID to use split and switch instead of StringTokenizer for better readability and error handling.
• Removed unused import StringTokenizer from MCRCategoryID.java.

Testing Updates:

• Updated test class MCRCategoryIDTest to use JUnit 5 annotations and methods, including assertThrows for exception testing. [1] [2]
• Added parameterized tests for invalid category ID edge cases in MCRCategoryIDTest.

Copilot reviewed 2 out of 3 changed files in this pull request and generated no suggestions.

Files not reviewed (1)

• mycore-base/pom.xml: Language not supported

Comments skipped due to low confidence (2)

mycore-base/src/main/java/org/mycore/datamodel/classifications2/MCRCategoryID.java:99

• [nitpick] The error message 'CategoryId is ambiguous' is unclear. Consider changing it to 'CategoryId format is invalid or ambiguous: ' + categoryId.

default -> throw new IllegalArgumentException("CategoryId is ambiguous: " + categoryId);

mycore-base/src/main/java/org/mycore/datamodel/classifications2/MCRCategoryID.java:30

• There is a spelling mistake in the error message: 'chracters' should be 'characters'.

classification ID ''%s'' is more than %d chracters long: %d