RFR 8072853: SimpleScriptContext used by NashornScriptEngine doesn't completely complies to the spec regarding exception throwing
Marcus Lagergren
marcus.lagergren at oracle.com
Mon May 18 13:07:50 UTC 2015
Also +1
> On 18 May 2015, at 14:59, A. Sundararajan <sundararajan.athijegannathan at oracle.com> wrote:
>
> Thanks for the review. Updated test as per your suggestion. Uploaded fresh review @ http://cr.openjdk.java.net/~sundar/8072853/webrev.01/
>
> Thanks
> -Sundar
>
> Paul Sandoz wrote:
>> On May 18, 2015, at 12:44 PM, A. Sundararajan <sundararajan.athijegannathan at oracle.com> wrote:
>>
>>
>>> Please review http://cr.openjdk.java.net/~sundar/8072853/webrev.00/ for https://bugs.openjdk.java.net/browse/JDK-8072853
>>>
>>>
>>
>> Changes to SimpleScriptContext look good.
>>
>> Test-wise you could reduce the duplication with a method accepting Consumer<ScriptContext>.
>>
>> e.g.
>>
>> @Test
>> public void getAttributeEmptyName() {
>> test(sc -> sc.getAttribute("", ScriptContext.GLOBAL_SCOPE));
>> }
>>
>> void test(Consumer<ScriptContext> c) {
>> for (ScriptEngineFactory fac : getFactories()) {
>> ScriptContext sc = fac.getScriptEngine().getContext();
>> String name = fac.getEngineName();
>> try {
>> c.accept(sc);
>> throw new RuntimeException("no exception for " + name);
>> } catch (IllegalArgumentException iae) {
>> System.out.println("got " + iae + " as expected for " + name);
>> }
>> }
>> }
>>
>> Paul.
>>
>
More information about the core-libs-dev
mailing list