-
Bug
-
Resolution: Fixed
-
P4
-
9
-
b128
-
Verified
The code in VersionProps.versionNumbers() could be slightly refactored in order to provide an entry point for unit testing:
...
class VersionProps {
. . .
static List<Integer> parseVersionNumbers(String versionNumber) {
// parsing code goes there
}
static List<Integer> versionNumbers() {
return parseVersionNumbers(VERSION_NUMBER);
}
. . .
}
...
This would allow writing/adding a unit test for the algorithm implemented in parseVersionNumbers (either using white-box with -Xpatch or using reflection and the proper incantations to invoke parseVersionNumbers from outside the package).
...
class VersionProps {
. . .
static List<Integer> parseVersionNumbers(String versionNumber) {
// parsing code goes there
}
static List<Integer> versionNumbers() {
return parseVersionNumbers(VERSION_NUMBER);
}
. . .
}
...
This would allow writing/adding a unit test for the algorithm implemented in parseVersionNumbers (either using white-box with -Xpatch or using reflection and the proper incantations to invoke parseVersionNumbers from outside the package).
- relates to
-
JDK-8160457 VersionProps.versionNumbers() is broken
-
- Resolved
-
-
JDK-8160000 Runtime.version() cause startup regressions in 9+119
-
- Closed
-
-
JDK-8156907 Runtime.Version.{major(),security()} return value of 0 may be ambiguous
-
- Closed
-