commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Henri Biestro (JIRA)" <>
Subject [jira] [Resolved] (JEXL-145) Incorrect test case in SandboxTest
Date Thu, 19 Mar 2015 15:23:51 GMT


Henri Biestro resolved JEXL-145.
    Resolution: Fixed

> Incorrect test case in SandboxTest
> ----------------------------------
>                 Key: JEXL-145
>                 URL:
>             Project: Commons JEXL
>          Issue Type: Bug
>    Affects Versions: 2.1.1
>         Environment: JDK 1.7.0_45
>            Reporter: Ian Connor
>             Fix For: 3.0
> Whilst debugging blacklisting of static functions using the Sandbox class and unable
to get it to work I checked the testcase SandboxTest. I subsequently found that the test of
System.exit() in testRestrict() passes not because System.currentTimeMillis() is whitelisted
but because of a signature mismatch i.e System.exit() requires an int parameter. Changing
the expression to System.exit(1) causes the test to end prematruely due to exit() being called.
> The white listing fails in SandboxUberspectImpl.getMethod() because obj is a Class and
getClass().getName() returns "java.lang.Class" which doesn't match "java.lang.System" as specified
in SandboxTest.testRestrict().

This message was sent by Atlassian JIRA

View raw message