[GitHub] ant pull request #:

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view

[GitHub] ant pull request #:

Github user bodewig commented on the pull request:

    In src/tests/junit/org/apache/tools/ant/taskdefs/optional/image/ImageTest.java:
    In src/tests/junit/org/apache/tools/ant/taskdefs/optional/image/ImageTest.java on line 117:
    This test is ignored anyway, but your change doesn't seem correct.
    The original code does not expect a `RuntimeException` always - there is no `fail` inside the try block. It has an assertion that is only expected to hold true if there is no `RuntimeException` (so it may be wrong in the `finally` block) and an assertion about the exception if one occurs at all.


To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]