Merge pull request #1245 from SomelifeDev/master

fix(tests): issue #1200, added testing compatibility for jdk9+

framework/src/play/mvc/Controller.java

@@ -1234,7 +1234,7 @@ private static void verifyContinuationsEnhancement() {
                 }
 
                 if (haveSeenFirstApplicationClass) {
-                    if (className.startsWith("sun.") || className.startsWith("play.")) {
+                    if (shouldBeCheckedForEnhancement(className)) {
                         // we're back into the play framework code...
                         return; // done checking
                     } else {
@@ -1252,6 +1252,17 @@ private static void verifyContinuationsEnhancement() {
         }
     }
 
+
+    /**
+     * Checks if the classname is from the jdk, sun or play package and therefore should not be checked for enhancement
+     * @param String className
+     * @return boolean
+     */
+    static boolean shouldBeCheckedForEnhancement(String className)
+    {
+        return className.startsWith("jdk.") || className.startsWith("sun.") || className.startsWith("play.");
+    }
+
     protected static <T> void await(Future<T> future, F.Action<T> callback) {
         Request.current().isNew = false;
         Request.current().args.put(ActionInvoker.F, future);

framework/test-src/play/mvc/ControllerTest.java

@@ -0,0 +1,14 @@
+package play.mvc;
+
+import org.junit.Test;
+import static org.junit.Assert.*;
+import static play.mvc.Controller.shouldBeCheckedForEnhancement;
+
+public class ControllerTest{
+    @Test
+    public void jdkAndPlayClassesShouldNeverBeenCheckedForEnhancement() {
+        assertTrue(shouldBeCheckedForEnhancement("sun.blah"));
+        assertTrue(shouldBeCheckedForEnhancement("play.foo"));
+        assertFalse(shouldBeCheckedForEnhancement("com.bar"));
+    }
+}