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.

----


---

Reply via email to