X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=binding%2Fmdsal-binding-java-api-generator%2Fsrc%2Ftest%2Fjava%2Forg%2Fopendaylight%2Fmdsal%2Fbinding%2Fjava%2Fapi%2Fgenerator%2Ftest%2FCompilationTest.java;h=16aa921b6da77484bcb042c0050f9fc3d85bbfb0;hb=62d1bb7f2cf0c96ce34ca181462132dc95555daf;hp=3ede285aba6dca0a1f3864b1143cd035e0144389;hpb=d08575a3a4d04e190b31c182e280d9040e560a2d;p=mdsal.git diff --git a/binding/mdsal-binding-java-api-generator/src/test/java/org/opendaylight/mdsal/binding/java/api/generator/test/CompilationTest.java b/binding/mdsal-binding-java-api-generator/src/test/java/org/opendaylight/mdsal/binding/java/api/generator/test/CompilationTest.java index 3ede285aba..16aa921b6d 100644 --- a/binding/mdsal-binding-java-api-generator/src/test/java/org/opendaylight/mdsal/binding/java/api/generator/test/CompilationTest.java +++ b/binding/mdsal-binding-java-api-generator/src/test/java/org/opendaylight/mdsal/binding/java/api/generator/test/CompilationTest.java @@ -33,10 +33,10 @@ import java.util.Arrays; import java.util.Collection; import java.util.List; import java.util.stream.Collectors; -import javax.annotation.Nullable; import org.junit.Test; import org.opendaylight.yangtools.yang.binding.ChildOf; import org.opendaylight.yangtools.yang.binding.annotations.RoutingContext; +import org.opendaylight.yangtools.yang.common.Empty; /** * Test correct code generation. @@ -73,7 +73,7 @@ public class CompilationTest extends BaseCompilationTest { assertTrue(linksBuilder.exists()); assertTrue(linksKey.exists()); assertTrue(testData.exists()); - CompilationTestUtils.assertFilesCount(parent, 6); + CompilationTestUtils.assertFilesCount(parent, 7); parent = new File(sourcesOutputDir, CompilationTestUtils.NS_TEST + CompilationTestUtils.FS + "links"); final File level = new File(parent, "Level.java"); @@ -90,7 +90,7 @@ public class CompilationTest extends BaseCompilationTest { assertTrue(nodeList.exists()); assertTrue(nodeListBuilder.exists()); assertTrue(nodesType.exists()); - CompilationTestUtils.assertFilesCount(parent, 7); + CompilationTestUtils.assertFilesCount(parent, 8); // Test if sources are compilable CompilationTestUtils.testCompilation(sourcesOutputDir, compiledOutputDir); @@ -107,13 +107,12 @@ public class CompilationTest extends BaseCompilationTest { assertTrue(keyArgsClass.isInterface()); CompilationTestUtils.assertContainsMethod(keyArgsClass, String.class, "getName"); CompilationTestUtils.assertContainsMethod(keyArgsClass, Integer.class, "getSize"); - assertEquals(2, abstractMethods(keyArgsClass).size()); + assertEquals(3, abstractMethods(keyArgsClass).size()); // Test generated 'list links' assertTrue(linksClass.isInterface()); CompilationTestUtils.assertImplementsIfc(linksClass, keyArgsClass); - // TODO: anyxml - assertEquals(6, abstractMethods(linksClass).size()); + assertEquals(7, abstractMethods(linksClass).size()); // Test list key constructor arguments ordering CompilationTestUtils.assertContainsConstructor(linksKeyClass, Byte.class, String.class, Integer.class); @@ -138,7 +137,7 @@ public class CompilationTest extends BaseCompilationTest { assertTrue(new File(parent, "OpenObject.java").exists()); assertTrue(new File(parent, "ExplicitRouteObject.java").exists()); assertTrue(new File(parent, "PathKeySubobject.java").exists()); - CompilationTestUtils.assertFilesCount(parent, 9); + CompilationTestUtils.assertFilesCount(parent, 10); parent = new File(parent, "object"); assertTrue(new File(parent, "Nodes.java").exists()); @@ -196,7 +195,7 @@ public class CompilationTest extends BaseCompilationTest { assertTrue(new File(parent, "BasicExplicitRouteSubobjects.java").exists()); assertTrue(new File(parent, "ExplicitRouteSubobjects.java").exists()); assertTrue(new File(parent, "RouteSubobjects.java").exists()); - CompilationTestUtils.assertFilesCount(parent, 5); + CompilationTestUtils.assertFilesCount(parent, 6); parent = new File(parent, "route"); CompilationTestUtils.assertFilesCount(parent, 1); @@ -243,7 +242,7 @@ public class CompilationTest extends BaseCompilationTest { assertTrue(pathAttributes.exists()); assertTrue(update.exists()); assertTrue(updateBuilder.exists()); - CompilationTestUtils.assertFilesCount(parent, 6); + CompilationTestUtils.assertFilesCount(parent, 7); parent = new File(sourcesOutputDir, CompilationTestUtils.NS_FOO + CompilationTestUtils.FS + "path"); CompilationTestUtils.assertFilesCount(parent, 1); @@ -269,7 +268,7 @@ public class CompilationTest extends BaseCompilationTest { assertTrue(destination.exists()); assertTrue(pathAttributes1.exists()); assertTrue(pathAttributes1Builder.exists()); - CompilationTestUtils.assertFilesCount(parent, 5); + CompilationTestUtils.assertFilesCount(parent, 6); parent = new File(sourcesOutputDir, CompilationTestUtils.NS_BAR + CompilationTestUtils.FS + "destination"); CompilationTestUtils.assertFilesCount(parent, 2); @@ -309,7 +308,7 @@ public class CompilationTest extends BaseCompilationTest { // Test if all sources were generated from 'module baz' parent = new File(sourcesOutputDir, CompilationTestUtils.NS_BAZ); - CompilationTestUtils.assertFilesCount(parent, 2); + CompilationTestUtils.assertFilesCount(parent, 3); final File linkstateDestination = new File(parent, "LinkstateDestination.java"); assertTrue(linkstateDestination.exists()); @@ -376,7 +375,7 @@ public class CompilationTest extends BaseCompilationTest { assertTrue(new File(parent, "NodesBuilder.java").exists()); assertTrue(new File(parent, "Alg.java").exists()); assertTrue(new File(parent, "NodesIdUnionBuilder.java").exists()); - CompilationTestUtils.assertFilesCount(parent, 5); + CompilationTestUtils.assertFilesCount(parent, 6); // Test if sources are compilable CompilationTestUtils.testCompilation(sourcesOutputDir, compiledOutputDir); @@ -392,7 +391,7 @@ public class CompilationTest extends BaseCompilationTest { CompilationTestUtils.assertContainsMethod(nodesClass, pkg + ".Nodes$IdBits", "getIdBits", loader); CompilationTestUtils.assertContainsMethod(nodesClass, Boolean.class, "isIdBoolean"); CompilationTestUtils.assertContainsMethod(nodesClass, BigDecimal.class, "getIdDecimal64"); - CompilationTestUtils.assertContainsMethod(nodesClass, Boolean.class, "isIdEmpty"); + CompilationTestUtils.assertContainsMethod(nodesClass, Empty.class, "getIdEmpty"); CompilationTestUtils.assertContainsMethod(nodesClass, pkg + ".Nodes$IdEnumeration", "getIdEnumeration", loader); testReturnTypeIdentityref(nodesClass, "getIdIdentityref", pkg + ".Alg"); testReturnTypeInstanceIdentitifer(loader, nodesClass, "getIdInstanceIdentifier"); @@ -408,22 +407,24 @@ public class CompilationTest extends BaseCompilationTest { CompilationTestUtils.assertContainsMethod(nodesClass, BigInteger.class, "getIdU64"); CompilationTestUtils.assertContainsMethod(nodesClass, pkg + ".Nodes$IdUnion", "getIdUnion", loader); - final Object builderObj = builderClass.newInstance(); + final Object builderObj = builderClass.getDeclaredConstructor().newInstance(); - Method m = CompilationTestUtils.assertContainsMethod(builderClass, builderClass, "setIdBinary", b.getClass()); + Method method = CompilationTestUtils.assertContainsMethod(builderClass, builderClass, "setIdBinary", + b.getClass()); final List> lengthConstraints = new ArrayList<>(); lengthConstraints.add(Range.closed(1, 10)); byte[] arg = new byte[] {}; String expectedMsg = String.format("Invalid length: %s, expected: %s.", Arrays.toString(arg), lengthConstraints); - CompilationTestUtils.assertContainsRestrictionCheck(builderObj, m, expectedMsg, arg); + CompilationTestUtils.assertContainsRestrictionCheck(builderObj, method, expectedMsg, arg); - m = CompilationTestUtils.assertContainsMethod(builderClass, builderClass, "setIdDecimal64", BigDecimal.class); + method = CompilationTestUtils.assertContainsMethod(builderClass, builderClass, "setIdDecimal64", + BigDecimal.class); final List> rangeConstraints = new ArrayList<>(); rangeConstraints.add(Range.closed(new BigDecimal("1.5"), new BigDecimal("5.5"))); Object arg1 = new BigDecimal("1.4"); expectedMsg = String.format("Invalid range: %s, expected: %s.", arg1, rangeConstraints); - CompilationTestUtils.assertContainsRestrictionCheck(builderObj, m, expectedMsg, arg1); + CompilationTestUtils.assertContainsRestrictionCheck(builderObj, method, expectedMsg, arg1); CompilationTestUtils.cleanUp(sourcesOutputDir, compiledOutputDir); } @@ -437,13 +438,13 @@ public class CompilationTest extends BaseCompilationTest { // Test if all sources are generated final File fooParent = new File(sourcesOutputDir, CompilationTestUtils.NS_FOO); - CompilationTestUtils.assertFilesCount(fooParent, 3); + CompilationTestUtils.assertFilesCount(fooParent, 4); assertTrue(new File(fooParent, "FooData.java").exists()); assertTrue(new File(fooParent, "Nodes.java").exists()); assertTrue(new File(fooParent, "NodesBuilder.java").exists()); final File barParent = new File(sourcesOutputDir, CompilationTestUtils.NS_BAR); - CompilationTestUtils.assertFilesCount(barParent, 1); + CompilationTestUtils.assertFilesCount(barParent, 2); assertTrue(new File(barParent, "IdentityClass.java").exists()); // Test if sources are compilable @@ -468,7 +469,7 @@ public class CompilationTest extends BaseCompilationTest { throw new AssertionError("Method getId() not found", e); } - assertEquals(ImmutableSet.of(RoutingContext.class, Nullable.class), Arrays.stream(getId.getAnnotations()) + assertEquals(ImmutableSet.of(RoutingContext.class), Arrays.stream(getId.getAnnotations()) .map(Annotation::annotationType).collect(Collectors.toSet())); CompilationTestUtils.cleanUp(sourcesOutputDir, compiledOutputDir); } @@ -511,8 +512,6 @@ public class CompilationTest extends BaseCompilationTest { /** * Test handling nested uses-augmentations. - * - * @throws Exception */ @Test public void testBug1172() throws Exception { @@ -552,49 +551,48 @@ public class CompilationTest extends BaseCompilationTest { assertTrue(new File(parent, "TypedefCurrent.java").exists()); assertTrue(new File(parent, "TypedefDeprecated.java").exists()); - final String pkg = CompilationTestUtils.BASE_PKG + ".urn.yang.foo.rev160102"; - final ClassLoader loader = new URLClassLoader(new URL[] { compiledOutputDir.toURI().toURL() }); - final Class cls = loader.loadClass(pkg + ".FooData"); - final Class clsContainer = loader.loadClass(pkg + ".ContainerMain"); - final Class clsTypedefDepr = loader.loadClass(pkg + ".TypedefDeprecated"); - final Class clsTypedefCur = loader.loadClass(pkg + ".TypedefCurrent"); - final Class clsGroupingDepr = loader.loadClass(pkg + ".GroupingDeprecated"); - final Class clsGroupingCur = loader.loadClass(pkg + ".GroupingCurrent"); - final Class clsTypeDef1 = loader.loadClass(pkg + ".Typedef1"); - final Class clsTypeDef2 = loader.loadClass(pkg + ".Typedef2"); - final Class clsTypeDef3 = loader.loadClass(pkg + ".Typedef3"); - assertTrue(clsTypedefDepr.getAnnotations()[0].toString().contains("Deprecated")); - assertTrue(clsTypedefCur.getAnnotations().length == 0); - assertTrue(clsGroupingDepr.getAnnotations()[0].toString().contains("Deprecated")); - assertTrue(clsGroupingCur.getAnnotations().length == 0); - assertTrue(clsTypeDef1.getAnnotations().length == 0); - assertTrue(clsTypeDef3.getAnnotations().length == 0); - assertTrue(clsTypeDef2.getAnnotations()[0].toString().contains("Deprecated")); - - /*methods inside container*/ - assertTrue(clsContainer.getMethod("getContainerMainLeafDepr").isAnnotationPresent(Deprecated.class)); - assertTrue(clsContainer.getMethod("getContainerMainListDepr").isAnnotationPresent(Deprecated.class)); - assertTrue(clsContainer.getMethod("getContainerMainChoiceDepr").isAnnotationPresent(Deprecated.class)); - assertFalse(clsContainer.getMethod("getContainerMainLeafCurrent").isAnnotationPresent(Deprecated.class)); - assertFalse(clsContainer.getMethod("getContainerMainListCurrent").isAnnotationPresent(Deprecated.class)); - assertFalse(clsContainer.getMethod("getContainerMainChoiceCur").isAnnotationPresent(Deprecated.class)); - - /*methods inside module*/ - assertTrue(cls.getMethod("getContainerMainLeafDepr").isAnnotationPresent(Deprecated.class)); - assertTrue(cls.getMethod("getContainerMainListDepr").isAnnotationPresent(Deprecated.class)); - assertTrue(cls.getMethod("getContainerMainChoiceDepr").isAnnotationPresent(Deprecated.class)); - assertFalse(cls.getMethod("getContainerMainLeafCurrent").isAnnotationPresent(Deprecated.class)); - assertFalse(cls.getMethod("getContainerMainListCurrent").isAnnotationPresent(Deprecated.class)); - assertFalse(cls.getMethod("getContainerMainChoiceCur").isAnnotationPresent(Deprecated.class)); - assertTrue(cls.getMethod("getLeafDeprecated").isAnnotationPresent(Deprecated.class)); + try (URLClassLoader loader = new URLClassLoader(new URL[] { compiledOutputDir.toURI().toURL() })) { + final String pkg = CompilationTestUtils.BASE_PKG + ".urn.yang.foo.rev160102"; + final Class cls = loader.loadClass(pkg + ".FooData"); + final Class clsContainer = loader.loadClass(pkg + ".ContainerMain"); + final Class clsTypedefDepr = loader.loadClass(pkg + ".TypedefDeprecated"); + final Class clsTypedefCur = loader.loadClass(pkg + ".TypedefCurrent"); + final Class clsGroupingDepr = loader.loadClass(pkg + ".GroupingDeprecated"); + final Class clsGroupingCur = loader.loadClass(pkg + ".GroupingCurrent"); + final Class clsTypeDef1 = loader.loadClass(pkg + ".Typedef1"); + final Class clsTypeDef2 = loader.loadClass(pkg + ".Typedef2"); + final Class clsTypeDef3 = loader.loadClass(pkg + ".Typedef3"); + assertTrue(clsTypedefDepr.getAnnotations()[0].toString().contains("Deprecated")); + assertTrue(clsTypedefCur.getAnnotations().length == 0); + assertTrue(clsGroupingDepr.getAnnotations()[0].toString().contains("Deprecated")); + assertTrue(clsGroupingCur.getAnnotations().length == 0); + assertTrue(clsTypeDef1.getAnnotations().length == 0); + assertTrue(clsTypeDef3.getAnnotations().length == 0); + assertTrue(clsTypeDef2.getAnnotations()[0].toString().contains("Deprecated")); + + /*methods inside container*/ + assertTrue(clsContainer.getMethod("getContainerMainLeafDepr").isAnnotationPresent(Deprecated.class)); + assertTrue(clsContainer.getMethod("getContainerMainListDepr").isAnnotationPresent(Deprecated.class)); + assertTrue(clsContainer.getMethod("getContainerMainChoiceDepr").isAnnotationPresent(Deprecated.class)); + assertFalse(clsContainer.getMethod("getContainerMainLeafCurrent").isAnnotationPresent(Deprecated.class)); + assertFalse(clsContainer.getMethod("getContainerMainListCurrent").isAnnotationPresent(Deprecated.class)); + assertFalse(clsContainer.getMethod("getContainerMainChoiceCur").isAnnotationPresent(Deprecated.class)); + + /*methods inside module*/ + assertTrue(cls.getMethod("getContainerMainLeafDepr").isAnnotationPresent(Deprecated.class)); + assertTrue(cls.getMethod("getContainerMainListDepr").isAnnotationPresent(Deprecated.class)); + assertTrue(cls.getMethod("getContainerMainChoiceDepr").isAnnotationPresent(Deprecated.class)); + assertFalse(cls.getMethod("getContainerMainLeafCurrent").isAnnotationPresent(Deprecated.class)); + assertFalse(cls.getMethod("getContainerMainListCurrent").isAnnotationPresent(Deprecated.class)); + assertFalse(cls.getMethod("getContainerMainChoiceCur").isAnnotationPresent(Deprecated.class)); + assertTrue(cls.getMethod("getLeafDeprecated").isAnnotationPresent(Deprecated.class)); + } CompilationTestUtils.cleanUp(sourcesOutputDir, compiledOutputDir); } /** * Test if class generated for node from grouping implements ChildOf. - * - * @throws Exception */ @Test public void testBug1377() throws Exception { @@ -629,6 +627,24 @@ public class CompilationTest extends BaseCompilationTest { CompilationTestUtils.cleanUp(sourcesOutputDir, compiledOutputDir); } + @Test + public void testMdsal365() throws Exception { + final File sourcesOutputDir = CompilationTestUtils.generatorOutput("mdsal365"); + final File compiledOutputDir = CompilationTestUtils.compilerOutput("mdsal365"); + generateTestSources("/compilation/mdsal365", sourcesOutputDir); + CompilationTestUtils.testCompilation(sourcesOutputDir, compiledOutputDir); + CompilationTestUtils.cleanUp(sourcesOutputDir, compiledOutputDir); + } + + @Test + public void testMdsal395() throws Exception { + final File sourcesOutputDir = CompilationTestUtils.generatorOutput("mdsal395"); + final File compiledOutputDir = CompilationTestUtils.compilerOutput("mdsal395"); + generateTestSources("/compilation/mdsal395", sourcesOutputDir); + CompilationTestUtils.testCompilation(sourcesOutputDir, compiledOutputDir); + CompilationTestUtils.cleanUp(sourcesOutputDir, compiledOutputDir); + } + @Test public void classNamesColisionTest() throws Exception { final File sourcesOutputDir = CompilationTestUtils.generatorOutput("class-name-collision"); @@ -647,6 +663,33 @@ public class CompilationTest extends BaseCompilationTest { CompilationTestUtils.cleanUp(sourcesOutputDir, compiledOutputDir); } + @Test + public void twoNestedUnionsTest() throws Exception { + final File sourcesOutputDir = CompilationTestUtils.generatorOutput("mdsal320"); + final File compiledOutputDir = CompilationTestUtils.compilerOutput("mdsal320"); + generateTestSources("/compilation/mdsal320", sourcesOutputDir); + CompilationTestUtils.testCompilation(sourcesOutputDir, compiledOutputDir); + CompilationTestUtils.cleanUp(sourcesOutputDir, compiledOutputDir); + } + + @Test + public void testMdsal425() throws Exception { + final File sourcesOutputDir = CompilationTestUtils.generatorOutput("mdsal425"); + final File compiledOutputDir = CompilationTestUtils.compilerOutput("mdsal425"); + generateTestSources("/compilation/mdsal425", sourcesOutputDir); + CompilationTestUtils.testCompilation(sourcesOutputDir, compiledOutputDir); + CompilationTestUtils.cleanUp(sourcesOutputDir, compiledOutputDir); + } + + @Test + public void testMdsal529() throws Exception { + final File sourcesOutputDir = CompilationTestUtils.generatorOutput("mdsal529"); + final File compiledOutputDir = CompilationTestUtils.compilerOutput("mdsal529"); + generateTestSources("/compilation/mdsal529", sourcesOutputDir); + CompilationTestUtils.testCompilation(sourcesOutputDir, compiledOutputDir); + CompilationTestUtils.cleanUp(sourcesOutputDir, compiledOutputDir); + } + private static void testReturnTypeIdentityref(final Class clazz, final String methodName, final String returnTypeStr) throws NoSuchMethodException { Method method = clazz.getMethod(methodName); @@ -664,13 +707,11 @@ public class CompilationTest extends BaseCompilationTest { private static void testReturnTypeInstanceIdentitifer(final ClassLoader loader, final Class clazz, final String methodName) throws ClassNotFoundException, NoSuchMethodException, SecurityException { - Method method; - Class rawReturnType; - java.lang.reflect.Type returnType; - method = clazz.getMethod(methodName); - rawReturnType = Class.forName("org.opendaylight.yangtools.yang.binding.InstanceIdentifier", true, loader); + final Method method = clazz.getMethod(methodName); + final Class rawReturnType = Class.forName("org.opendaylight.yangtools.yang.binding.InstanceIdentifier", true, + loader); assertEquals(rawReturnType, method.getReturnType()); - returnType = method.getGenericReturnType(); + final java.lang.reflect.Type returnType = method.getGenericReturnType(); assertTrue(returnType instanceof ParameterizedType); final ParameterizedType pt = (ParameterizedType) returnType; final java.lang.reflect.Type[] parameters = pt.getActualTypeArguments();