GitHub user mureinik opened a pull request:
https://github.com/apache/commons-lang/pull/301
Clean up EventUtilsTest
Clean up the `EventUtilsTest` class in order to make it easier to read and
maintain,
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mureinik/commons-lang EventUtilsTest
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/commons-lang/pull/301.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #301
----
commit 73d0c3617f392223b92e90f16365d7d2af649971
Author: Allon Mureinik <[email protected]>
Date: 2017-10-21T10:29:06Z
EventUtilsTest ExpectedException usage
Use the ExpectedException @Rule to verify thrown exception instead of
boiler-plate implementing its logic, in order to clean up the code
and make it easier to read and maintain.
commit 182e3aac0c12f1043497c156042ef2b9deb14a0c
Author: Allon Mureinik <[email protected]>
Date: 2017-10-21T10:31:42Z
EventUtilsTest boxing
Removed unnecessary explicit boxing and unboxing in order to make the
code cleaner and easier to read.
----
---