[groovy] branch master updated: GROOVY-8361: add test case

[em...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

emilles pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/groovy.git


The following commit(s) were added to refs/heads/master by this push:
     new 73eb720  GROOVY-8361: add test case
73eb720 is described below

commit 73eb7200cd3cb64f4ac4c829bc56daab30aa9b79
Author: Eric Milles <er...@thomsonreuters.com>
AuthorDate: Wed Nov 27 09:23:41 2019 -0600

    GROOVY-8361: add test case
---
 src/test/groovy/bugs/Groovy8361.groovy | 115 +++++++++++++++++++++++++++++++++
 1 file changed, 115 insertions(+)

diff --git a/src/test/groovy/bugs/Groovy8361.groovy b/src/test/groovy/bugs/Groovy8361.groovy
new file mode 100644
index 0000000..c3bee1d
--- /dev/null
+++ b/src/test/groovy/bugs/Groovy8361.groovy
@@ -0,0 +1,115 @@
+/*
+ *  Licensed to the Apache Software Foundation (ASF) under one
+ *  or more contributor license agreements.  See the NOTICE file
+ *  distributed with this work for additional information
+ *  regarding copyright ownership.  The ASF licenses this file
+ *  to you under the Apache License, Version 2.0 (the
+ *  "License"); you may not use this file except in compliance
+ *  with the License.  You may obtain a copy of the License at
+ *
+ *    http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *  Unless required by applicable law or agreed to in writing,
+ *  software distributed under the License is distributed on an
+ *  "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ *  KIND, either express or implied.  See the License for the
+ *  specific language governing permissions and limitations
+ *  under the License.
+ */
+package groovy.bugs
+
+import org.codehaus.groovy.control.CompilationUnit
+import org.codehaus.groovy.control.CompilerConfiguration
+import org.codehaus.groovy.tools.javac.JavaAwareCompilationUnit
+import org.junit.Test
+
+import static groovy.test.GroovyAssert.shouldFail
+
+final class Groovy8361 {
+
+    @Test
+    void testAliasedImportNotUsed1() {
+        def config = new CompilerConfiguration(
+            targetDirectory: File.createTempDir(),
+            jointCompilationOptions: [memStub: true]
+        )
+
+        def parentDir = File.createTempDir()
+        new File(parentDir, 'p').mkdir()
+        new File(parentDir, 'q').mkdir()
+        try {
+            def a = new File(parentDir, 'p/A.java')
+            a.write '''
+                package p;
+
+                public class A {
+                    public static class AA {
+                    }
+                }
+            '''
+            def b = new File(parentDir, 'q/B.groovy')
+            b.write '''
+                package q
+
+                import p.A.AA as AAA
+
+                println new AA() // should be unresolved
+            '''
+
+            def loader = new GroovyClassLoader(this.class.classLoader)
+            def cu = new JavaAwareCompilationUnit(config, loader)
+            cu.addSources(a, b)
+
+            def err = shouldFail {
+                cu.compile()
+            }
+            assert err =~ /unable to resolve class AA/
+        } finally {
+            parentDir.deleteDir()
+            config.targetDirectory.deleteDir()
+        }
+    }
+
+    @Test
+    void testAliasedImportNotUsed2() {
+        def config = new CompilerConfiguration(
+            targetDirectory: File.createTempDir(),
+            jointCompilationOptions: [memStub: true]
+        )
+
+        def parentDir = File.createTempDir()
+        new File(parentDir, 'p').mkdir()
+        new File(parentDir, 'q').mkdir()
+        try {
+            def a = new File(parentDir, 'p/A.groovy')
+            a.write '''
+                package p
+
+                class A {
+                    static class AA {
+                    }
+                }
+            '''
+            def b = new File(parentDir, 'q/B.groovy')
+            b.write '''
+                package q
+
+                import p.A.AA as AAA
+
+                println new AA() // should be unresolved
+            '''
+
+            def loader = new GroovyClassLoader(this.class.classLoader)
+            def cu = new CompilationUnit(config, null, loader)
+            cu.addSources(a, b)
+
+            def err = shouldFail {
+                cu.compile()
+            }
+            assert err =~ /unable to resolve class AA/
+        } finally {
+            parentDir.deleteDir()
+            config.targetDirectory.deleteDir()
+        }
+    }
+}