RFR of JDK-8170839: failed test case is not checked in java/rmi/activation/ActivateFailedException/activateFails/ActivateFails.java

Roger Riggs Roger.Riggs at Oracle.com
Wed Dec 7 18:01:05 UTC 2016


Hi Hamlin,

Looks fine.

Roger


On 12/7/2016 1:47 AM, Hamlin Li wrote:
> correct the subject
>
>
> On 2016/12/7 14:46, Hamlin Li wrote:
>> Would you please review the below patch?
>>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8170839
>>
>> webrev: http://cr.openjdk.java.net/~mli/8170839/webrev.00/
>>
>>
>> Thank you
>> -Hamlin
>>
>> ------------------------------------------------------------------------
>>
>> diff -r 10b191e1793b 
>> test/java/rmi/activation/ActivateFailedException/activateFails/ActivateFails.java
>> --- 
>> a/test/java/rmi/activation/ActivateFailedException/activateFails/ActivateFails.java 
>> Tue Dec 06 17:53:22 2016 -0800
>> +++ 
>> b/test/java/rmi/activation/ActivateFailedException/activateFails/ActivateFails.java 
>> Tue Dec 06 22:44:29 2016 -0800
>> @@ -1,5 +1,5 @@
>>  /*
>> - * Copyright (c) 1998, 2014, Oracle and/or its affiliates. All 
>> rights reserved.
>> + * Copyright (c) 1998, 2016, Oracle and/or its affiliates. All 
>> rights reserved.
>>   * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
>>   *
>>   * This code is free software; you can redistribute it and/or modify it
>> @@ -140,7 +140,7 @@
>>              System.err.println("invoking method on activatable 
>> object...");
>>              try {
>>                  obj1.ping();
>> -
>> +                throw new RuntimeException("ActivateFailedException 
>> is expected");
>>              } catch (ActivateFailedException e) {
>>
>>                  /*
>>
>



More information about the core-libs-dev mailing list