RFR: nestmates and methodhandles

Stas Smirnov stanislav.smirnov at oracle.com
Tue Mar 1 12:42:07 UTC 2016


Hi all,

so I decided to use testNG since there are a number of negative tests 
that will be easier to track, please take a look.
webrev: http://cr.openjdk.java.net/~stsmirno/webrev.02

On 25/02/16 20:49, Ryan Schmitt wrote:
> If you're using a recent TestNG, just call Assert#assertThrows. Or, if 
> you want the thrown exception back, call Assert#expectThrows. These 
> methods work even with checked exceptions, and they allow you to be 
> much more specific (compared to expectedExceptions) about where 
> exactly the exception is supposed to come from, which helps prevent 
> false positives in tests.
>
> On Thu, Feb 25, 2016 at 6:39 AM, Brian Goetz <brian.goetz at oracle.com 
> <mailto:brian.goetz at oracle.com>> wrote:
>
>     Can I suggest doing this in TestNG?  You'll be far more productive
>     -- and failures are easier to debug.  You can use sophisticated
>     asserts rather than just comparing to a golden file.  jtreg
>     already supports TestNG tests.
>
>     For example, each of the negative tests (try-catch-assert
>     expected) can be its own test method, and TestNG has a way to mark
>     a test method as expected to throw an exception.  So these could
>     be written:
>
>         @Test(expectedExceptions = ClassNotFoundException.class)
>         public void testProtectedMethodNeg() {
>             inner.getSuperMethodMH(METHOD_NAME_PROTECTED);
>         }
>
>     It's far more clear what's going on, and each one is its own test
>     case, meaning that you can pinpoint a failure more easily.
>
>     Happy to help you get up to speed on TestNG if that helps.
>
>
>     On 2/25/2016 8:12 AM, Stas Smirnov wrote:
>
>         Hello,
>
>         please review this set of tests to verify "parent-child"
>         access using MethodHandle in terms of nestmates vm exploration.
>         1. MethodHandleSubTop class contains methods to verify
>         private/protected methods/fields of the "super" class using
>         MethodHandle from a single one and double level of nesting
>         using access static bridges and direct access in negative and
>         positive test cases
>         2. MethodHandleInheritTest executes a number of positive and
>         negative test cases using MethodHandleSubTop methods
>         it will allow us to verify current implementation and future,
>         when access bridges will be removed and MethodHandle access
>         checks will be modified.
>
>         webrev:
>         http://cr.openjdk.java.net/~stsmirno/nestmates/webrev.01
>         <http://cr.openjdk.java.net/%7Estsmirno/nestmates/webrev.01>
>         Testing: RBT
>
>
>

-- 
Best regards,
Stanislav




More information about the valhalla-dev mailing list