Fix checkstyle issues in new test code

Merge branch 'pr-379'

This closes pr #379

Thanks to @apirom9

add test use ImmutablePair class with java.util.HashMap and java.util.TreeMap

Update spotbugs-maven-plugin to version 3.1.10

Travis: "openjdk-ea" now means Java 13, so add Java 12 to JDKs to build on. (closes #396)

Update to JaCoCo 0.8.2 with works Java 11+.

LANG-1427: Add SystemUtils.isJavaVersionAtMost(JavaVersion)

Fix checkstyle violations.

Merge branch 'master' of https://git-wip-us.apache.org/repos/asf/commons-lang.git

[LANG-1427] Add API
org.apache.commons.lang3.SystemUtils.isJavaVersionAtMost(JavaVersion)

Add additional tests for StandardToStringStyle (closes #390)

Update to JUnit version 5.3.2

Travis: Stop building with Java 9 and 10, as these are superseded by Java 11 and not supported anymore.

Update Spotbugs Maven Plugin to version 3.1.8

Make whitespace use around generic tokens consistent and add a checkstyle rule to enforce it.

Make whitespace use before tokens consistent and add a checkstyle rule to enforce it.

Make whitespace use after tokens consistent and add a checkstyle rule to enforce it.

Merge branch 'pr-380'

This closes #380 thanks to @oorijsonar

fix javadoc typo

Update JUnit Pioneer to version 0.3.0

Update maven-spotbugs-plugin to version 3.1.7

Update EasyMock from 3.6 to 4.0 to fix testing on Java 11.

Use surefire plugin version from parent POM.

Remove unused test code

LANG-1422: Add null-safe StringUtils.valueOf(char[]) to delegate to String.valueOf(char[])

Fix checkstyle violations

[LANG-1422] Add null-safe StringUtils.valueOf(char[]) to delegate to
String.valueOf(char[])

Travis: Remove Oracle JDK 10 because it is no longer available

Clean up assertions (closes #376)

Use built-in assertion methods provides by
org.junit.jupiter.api.Assertions instead of reimplementing the same
logic with assertTrue and assertFalse.

Note that JUnit Jupiter 5.3.1 does not support deltas of 0 for
assertEquals(double, double, double) and
assertEquals(float, float, float), so these usages of assertTrue were
left untouched, and will be addressed when JUnit Jupiter 5.4, that
addresses this isssue, will be available (see
https://github.com/junit-team/junit5/pull/1613 for details).

Clean up redundant throws clauses

Use assertTrue/assertFalse instead of reimplementing them

Use assertTrue and assertFalse instead of reimplemeting their logic by
having an if statement conditionalling call fail.

Clean up testing of exceptions

Now that the entire project is ported to JUnit Jupiter, there are more
elegant ways to test for exceptions, which this patch applies
throughtout the code base.

If throwing an exception is supposed to fail a test, just throwing it
outside of the method cleans up the code and makes it more elegant,
instead of catching it and calling Assertions#fail.

If an exception is supposed to be thrown, calling
Assertions#assertThrows is a more elegant option than calling
Assertions#fail in the try block and then catching and ignoring the
expected exception.
Note that assertThrows uses a lambda block, so the variables inside it
should be final or effectively final. Reusing variables is a common
practice in the tests, so where needed new final variables were
introduced, or the variables used were inlined.

Remove double stop() test in StopWatchTest

StopWatchTest#testBadStates has the same block of code testing
StopWatch#stop copy-pasted.
This patch cleans it up by removing one of those blocks.

Make LocaleUtilsTest#parseAllLocales parameterized

This patch converts testParseAllLocales to a @ParameterizedTest instead
of iterating over the locales inside the method's body.
This changes allows using standard asserts for each case individually
instead of having to count failures and print the problematic locales
to System.out.

Remove ConversionTest#assertBinaryEquals

JUnit Jupiter's Assertions has an
assertArraysEuqals(boolean[], boolean[]) method, so there's no longer a
need for the assertBinaryEquals method.

Update checkstyle to version 8.13

Update tests to JUnit Jupiter (closes #375)

This patch finalizes the upgrade of commons-lang's tests to use JUnit
Jupiter and remove the Vintage Engine dependency entirely.

While most of these changes are drop-in replacements with no functional
benefit, there are some non-obvious changes worth mentioning.

Unlike org.junit.Assert.assertEquals(double, double, double),
org.junit.jupiter.api.Assertions.assertEquals(double, double, double)
does not support deltas of zero, only strictly positive deltas.
This issue will be addressed in JUnit Jupiter 5.4 (see
https://github.com/junit-team/junit5/pull/1613 for details). In the
meanwhile, assertTrue(expected==actual) was used, and TODO comments were
placed in the code to refactor it to assertEquals once JUnit 5.4 is
available.

Unlike org.junit.Test, org.junit.jupiter.api.Test does not have an
"expected" argument. Instead, an explicit call to
org.junit.jupiter.api.Assertions.assertThrows is used.

Unlike org.junit.Test, org.junit.jupiter.api.Test does not have a
"timeout" argument either. Instead, an explicit call to
org.junit.jupiter.api.Assertions.assertTimeoutPreemptively is used.

JUnit Jupiter also no longer has the concept of Rules. Usages of the
SystemDefaultsSwitch rule and its accompanying annotates were replaced
with the @DefaultLocale annotation that Benedikt Ritter contributed to
JUnit Pioneer, the semi-official JUnit extension project.
Following the removal of their usages, the SystemDefaults annotation,
the SystemDefaultsSwitch rule and the SystemDefaultsSwitchTest class
that tests them had no more use, and they were removed entirely.

It's also worth noting this is a minimal patch for migrating the
package's tests to Jupiter. There are several tests that can be made
more elegant with Jupiter's new features, but that work is left for
subsequent patches.

Update time tests to JUnit Jupiter

Upgrade the tests in the time package to use JUnit Jupiter as
part of the effort to remove the dependency on the Vintage Engine.

While most of these changes are drop-in replacements with no functional
benefit, there are some non-obvious changes worth mentioning.

Unlike org.junit.Test, org.junit.jupiter.api.Test does not have an
"expected" argument. Instead, an explicit call to
org.junit.jupiter.api.Assertions.assertThrows is used.

JUnit Jupiter no longer has a concept of runners. Tests previously run
with the org.junit.runners.Parameterized runner were rewritten to use
@ParameterizedTest and a @MethodSource.

JUnit Jupiter also no longer has the concept of Rules. Usages of the
SystemDefaultsSwitch rule and its accompanying annotates were replaced
with the @DefaultLocale and @DefaultTimezone annotations that
Benedikt Ritter contributed to JUnit Pioneer, the semi-official JUnit
extension project.

It's also worth noting this is a minimal patch for migrating the
package's tests to Jupiter. There are several tests that can be made
more elegant with Jupiter's new features, but that work is left for
subsequent patches.

Update mutable tests to JUnit Jupiter

Upgrade the tests in the mutable package to use JUnit Jupiter as
part of the effort to remove the dependency on the Vintage Engine.

While most of these changes are drop-in replacements with no functional
benefit, there are some non-obvious changes worth mentioning.

Unlike org.junit.Test, org.junit.jupiter.api.Test does not have an
"expected" argument. Instead, an explicit call to
org.junit.jupiter.api.Assertions.assertThrows is used.

Unlike org.junit.Assert.assertEquals(double, double, double),
org.junit.jupiter.api.Assertions.assertEquals(double, double, double)
does not support deltas of zero, only strictly positive deltas.
This issue will be addressed in JUnit Jupiter 5.4 (see
https://github.com/junit-team/junit5/pull/1613 for details). In the
meanwhile, assertTrue(expected==actual) was used, and TODO comments were
placed in the code to refactor it to assertEquals once JUnit 5.4 is
available.

It's also worth noting this is a minimal patch for migrating the
package's tests to Jupiter. There are several tests that can be made
more elegant with Jupiter's new features, but that work is left for
subsequent patches.

Update math tests to JUnit Jupiter

Upgrade the tests in the math package to use JUnit Jupiter as
part of the effort to remove the dependency on the Vintage Engine.

While most of these changes are drop-in replacements with no functional
benefit, there are some non-obvious changes worth mentioning.

Unlike org.junit.Test, org.junit.jupiter.api.Test does not have an
"expected" argument. Instead, an explicit call to
org.junit.jupiter.api.Assertions.assertThrows is used.

Unlike org.junit.Assert.assertEquals(double, double, double),
org.junit.jupiter.api.Assertions.assertEquals(double, double, double)
does not support deltas of zero, only strictly positive deltas.
This issue will be addressed in JUnit Jupiter 5.4 (see
https://github.com/junit-team/junit5/pull/1613 for details). In the
meanwhile, assertTrue(expected==actual) was used, and TODO comments were
placed in the code to refactor it to assertEquals once JUnit 5.4 is
available.

It's also worth noting this is a minimal patch for migrating the
package's tests to Jupiter. There are several tests that can be made
more elegant with Jupiter's new features, but that work is left for
subsequent patches.

Update tuple tests to JUnit Jupiter

Upgrade the tests in the text package to use JUnit Jupiter as
part of the effort to remove the dependency on the Vintage Engine.

Update text tests to JUnit Jupiter

Upgrade the tests in the text package to use JUnit Jupiter as
part of the effort to remove the dependency on the Vintage Engine.

While most of these changes are drop-in replacements with no functional
benefit, there are some non-obvious changes worth mentioning.

Unlike org.junit.Test, org.junit.jupiter.api.Test does not have an
"expected" argument. Instead, an explicit call to
org.junit.jupiter.api.Assertions.assertThrows is used.

It's also worth noting this is a minimal patch for migrating the
package's tests to Jupiter. There are several tests that can be made
more elegant with Jupiter's new features, but that work is left for
subsequent patches.

Update reflect tests to JUnit Jupiter

Upgrade the tests in the reflect package to use JUnit Jupiter as
part of the effort to remove the dependency on the Vintage Engine.

While most of these changes are drop-in replacements with no functional
benefit, there are some non-obvious changes worth mentioning.

Unlike org.junit.Test, org.junit.jupiter.api.Test does not have an
"expected" argument. Instead, an explicit call to
org.junit.jupiter.api.Assertions.assertThrows is used.

Unlike org.junit.Assume, org.junit.jupiter.api.Assumptions does not have
an assumtNotNull method. Instead, assumeTrue was used with an explicit
condition of a variable being different than null.

It's also worth noting this is a minimal patch for migrating the
package's tests to Jupiter. There are several tests that can be made
more elegant with Jupiter's new features, but that work is left for
subsequent patches.

Update exception tests to JUnit Jupiter

Upgrade the tests in the exception package to use JUnit Jupiter as
part of the effort to remove the dependency on the Vintage Engine.

While most of these changes are drop-in replacements with no functional
benefit, there are some non-obvious changes worth mentioning.

Unlike org.junit.Test, org.junit.jupiter.api.Test does not have an
"expected" argument. Instead, an explicit call to
org.junit.jupiter.api.Assertions.assertThrows is used.

Another non-obvious change was performed in
ContextedRuntimeExceptionTest. Unlike JUnit Vintages's @Before, JUnit
Jupiter's @BeforeEach does not apply if a parent's method is annotated
with it and the overriding method is not, so an explicit @BeforeEach
annotation had to be added to ContexedTuntimeExceptionTest#setUp().

It's also worth noting this is a minimal patch for migrating the
package's tests to Jupiter. There are several tests that can be made
made more elegant with Jupiter's new features, but that work is left
for subsequent patches.

Update event tests to JUnit Jupiter

Upgrade the tests in the event package to use JUnit Jupiter as
part of the effort to remove the dependency on the Vintage Engine.

While most of these changes are drop-in replacements with no functional
benefit, it is worth noting the tests that test thrown excetpions.
Prior to this patch, this package tested for exceptions in two ways,
neither of which are supported in JUnit Jupiter:
1. With the "expected" argument of the org.junit.Test annotation.
2. With the org.junit.rules.ExpectedException @Rule

Both of these usages were replaced with calls to
org.junit.jupiter.api.Assertions#assertThrows.

Update concurrent tests to JUnit Jupiter

Upgrade the tests in the concurrent package to use JUnit Jupiter as
part of the effort to remove the dependency on the Vintage Engine.

While most of these changes are drop-in replacements with no functional
benefit, there are some non-obvious changes worth mentioning.

Unlike org.junit.Test, org.junit.jupiter.api.Test does not have an
"expected" argument. Instead, an explicit call to
org.junit.jupiter.api.Assertions.assertThrows is used. This call allows
the test to pinpoint the exact statement that is expected to throw the
exception and allows making the tests a bit stricter by preventing
false-positives that could occur if the setup code would throw the
expected exception instead of the statement that was supposed to throw
it.

Another notable change was performed in MemoizerTest.
JUnit Jupiter does not support JUnit 4's runners, and EasyMock has no
equivalent @Extension, so a setup method was added and the mock was
explicitly initialized.

It's also worth noting this is a minimal patch for migrating the
package's tests to Jupiter. There are several tests that can be made
made more elegant with Jupiter's new features, but that work is left
for subsequent patches.

Travis: Also build with openjdk-ea, but allow openjdk-ea builds to fail.

Use JDKs provided by travis

Replace FindBugs with SpotBugs: Fix source assembly

Update builder tests to JUnit Jupiter

Upgrade the tests in the builder package to use JUnit Jupiter as part of
the effort to remove the dependency on the Vintage Engine.

While most of these changes are drop-in replacements with no functional
benefit, it is worth mentioning the change to how expected exceptions
are tested.
Unlike org.junit.Test, org.junit.jupiter.api.Test does not have an
"expected" argument. Instead, an explicit call to
org.junit.jupiter.api.Assertions.assertThrows is used. This call allows
the test to pinpoint the exact statement that is expected to throw the
expcetion and allows making the tests a bit stricter by preventing
false-positives that could occur if the setup code would throw the
expected exception instead of the statement that was supposed to throw
it.

Fix dead links in README.md (closes #359)

StringUtils: trivial formatting change.

This replaces <code> with {@code} primarily as a way to test
INFRA-17094.

(fix) download page pointed to lang3 in uri

Adding TODO to index.xml for 3.9 release

RELEASE-NOTES: fix template 3.9+ targets java 8

(fix) RELEASE-NOTES.txt on master to look like 3.8.1 dist

Updates for 3.8.1 release

Prepare 3.8.1 release

LANG-1419: Restore BundleSymbolicName for OSGi

Update to latest JUnit version

Merge branch 'spotbugs'

Merge branch 'nested-validate-test'

Convert util class convention tests to @Nested test

Convert tests for Validate.isAssignable overloads to @Nested test

Convert tests for Validate.isInstanceOf overloads to @Nested test

Convert tests for Validate.exclusiveBetween overloads to @Nested test

Convert tests for Validate.inclusiveBetween overloads to @Nested test

Convert tests for Validate.finite overloads to @Nested test

Convert tests for Validate.notNaN overloads to @Nested test

Convert tests for Validate.matchesPattern overloads to @Nested test

Convert tests for Validate.validIndex overloads to @Nested test

Move constructor test to top

Convert tests for Validate.noNullElements overloads to @Nested test

Convert tests for Validate.notBlank overloads to @Nested test

Convert tests for Validate.notEmpty overloads to @Nested test

Convert tests for Validate.notNull overloads to @Nested test

Merge branch 'parameterized-FastDateParser_TimeZoneStrategyTest'

Use @ParameterizedTest to iterate over available locales

Convert tests for Validate.isTrue overloads to @Nested test

JUnit 5 tests do not need to be public

Merge branch 'LANG-1416'

LANG-1416: Add changes.xml entry

LANG-1416: Update ValidateTest to JUnit 5 API

LANG-1416: Update to JUnit 5.3.0

LANG-1416: Use latest Apache Maven Surefire Plugin release

Merge branch 'LANG-1417'

LANG-1417: Add @FunctionalInterface to ThreadPredicate and ThreadGroupPredicate

Skip SpotBugs on Java 11

Replace FindBugs with SpotBugs

Add missing spaces

Use build in methods for comparing numerical values

Identical catch blocks can be combined

Merge branch 'remove-type-parameters'

Explicit type parameters can be removed

Merge branch 'java-11-build-passes'

Looks like Java 11 no longer fails the build

Convert JavaDoc license headers to block comments

Merge branch 'LANG-1415'

LANG-1415: Update Java Language requirement to 1.8

(fix) Add missing @Test annotation

LANG-1411: Add changes.xml entry and @since tags.

This fixes #342 from GitHub. Thanks to Alexander Tsvetkov.

Merge branch 'LANG-1411'