Fixed test cases and member validation for Java 8 interfaces.

Author: Rafael Winterhalter

byte-buddy-dep/src/main/java/net/bytebuddy/dynamic/scaffold/TypeWriter.java

@@ -1031,7 +1031,7 @@ protected enum Constraint {
                 /**
                  * Constrains for an annotation type since Java 8.
                  */
-                JAVA8_ANNOTATION("annotation (Java 8+)", false, false, false, true, true, false, true);
+                JAVA8_ANNOTATION("annotation (Java 8+)", false, false, false, true, true, true, true);
 
                 /**
                  * A name to represent the type being validated within an error message.
@@ -1141,7 +1141,7 @@ protected void assertMethod(String name, boolean isAbstract, boolean isPublic, b
                     } else if (!isStatic && isAbstract && !allowsAbstract) {
                         throw new IllegalStateException("Cannot define abstract method " + name + " for " + sortName);
                     } else if (!isAbstract && !allowsNonAbstract) {
-                        throw new IllegalStateException("Cannot define non-abstract method " + name + " + for " + sortName);
+                        throw new IllegalStateException("Cannot define non-abstract method " + name + " for " + sortName);
                     } else if (!isStatic && isDefaultIncompatible && allowsDefaultValue) {
                         throw new IllegalStateException("The signature of " + name + " is not compatible for a property of " + sortName);
                     }

byte-buddy-dep/src/test/java/net/bytebuddy/dynamic/scaffold/TypeWriterDefaultTest.java

@@ -5,6 +5,7 @@
 import net.bytebuddy.description.modifier.TypeManifestation;
 import net.bytebuddy.description.modifier.Visibility;
 import net.bytebuddy.dynamic.scaffold.subclass.ConstructorStrategy;
+import net.bytebuddy.implementation.StubMethod;
 import net.bytebuddy.implementation.SuperMethodCall;
 import net.bytebuddy.test.utility.JavaVersionRule;
 import net.bytebuddy.test.utility.ObjectPropertyAssertion;
@@ -128,12 +129,12 @@ public void testStaticMethodOnInterfaceAssertion() throws Exception {
     }
 
     @Test
-    @JavaVersionRule.Enforce(value = 8)
+    @JavaVersionRule.Enforce(8)
     public void testStaticMethodOnAnnotationAssertionJava8() throws Exception {
         new ByteBuddy()
                 .makeInterface()
                 .defineMethod(FOO, String.class, Collections.<Class<?>>emptyList(), Visibility.PUBLIC, Ownership.STATIC)
-                .withoutCode()
+                .intercept(StubMethod.INSTANCE)
                 .make();
     }
 
@@ -143,16 +144,17 @@ public void testStaticMethodOnAnnotationAssertion() throws Exception {
         new ByteBuddy()
                 .makeAnnotation()
                 .defineMethod(FOO, String.class, Collections.<Class<?>>emptyList(), Visibility.PUBLIC, Ownership.STATIC)
-                .withoutCode()
+                .intercept(StubMethod.INSTANCE)
                 .make();
     }
 
     @Test
-    @JavaVersionRule.Enforce(value = 8)
+    @JavaVersionRule.Enforce(8)
     public void testStaticMethodOnInterfaceAssertionJava8() throws Exception {
         new ByteBuddy()
                 .makeAnnotation()
-                .defineField(FOO, String.class, Visibility.PUBLIC, Ownership.STATIC)
+                .defineMethod(FOO, String.class, Collections.<Class<?>>emptyList(), Visibility.PUBLIC, Ownership.STATIC)
+                .intercept(StubMethod.INSTANCE)
                 .make();
     }