You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@groovy.apache.org by pa...@apache.org on 2020/07/11 02:42:00 UTC
[groovy] 02/05: GROOVY-9293: add test cases
This is an automated email from the ASF dual-hosted git repository.
paulk pushed a commit to branch GROOVY_2_5_X
in repository https://gitbox.apache.org/repos/asf/groovy.git
commit e6f60942930531fdd08ed983772b999acccd0416
Author: Eric Milles <er...@thomsonreuters.com>
AuthorDate: Mon Oct 28 10:51:13 2019 -0500
GROOVY-9293: add test cases
---
src/test/groovy/bugs/Groovy9293.groovy | 329 +++++++++++++++++++++++++++++++++
1 file changed, 329 insertions(+)
diff --git a/src/test/groovy/bugs/Groovy9293.groovy b/src/test/groovy/bugs/Groovy9293.groovy
new file mode 100644
index 0000000..4d66cb1
--- /dev/null
+++ b/src/test/groovy/bugs/Groovy9293.groovy
@@ -0,0 +1,329 @@
+/*
+ * 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 groovy.test.NotYetImplemented
+import org.junit.Test
+
+import static groovy.test.GroovyAssert.shouldFail
+
+final class Groovy9293 {
+
+ private final GroovyShell shell = new GroovyShell()
+
+ @Test
+ void 'test accessing a package-private super class field inside a closure - same package'() {
+ shell.with {
+ evaluate '''
+ package a
+
+ class A {
+ @groovy.transform.PackageScope
+ String superField = 'works'
+ }
+
+ class B extends A {
+ @groovy.transform.CompileStatic
+ def test() {
+ 'something'.with {
+ return superField
+ }
+ }
+ }
+
+ def obj = new B()
+ assert obj.test() == "works"
+ '''
+ }
+ }
+
+ @Test
+ void 'test accessing a package-private super class field inside a closure - diff package'() {
+ shouldFail(MissingPropertyException) {
+ shell.with {
+ evaluate '''
+ package a
+
+ class A {
+ @groovy.transform.PackageScope
+ String superField = 'works'
+ }
+
+ assert true
+ '''
+
+ evaluate '''
+ package b
+
+ class B extends a.A {
+ @groovy.transform.CompileStatic
+ def test() {
+ 'something'.with {
+ return superField
+ }
+ }
+ }
+
+ def obj = new B()
+ assert obj.test() == "works"
+ '''
+ }
+ }
+ }
+
+ @Test
+ void 'test accessing a package-private super class field inside a closure - same package, this qualifier'() {
+ shell.with {
+ evaluate '''
+ package a
+
+ class A {
+ @groovy.transform.PackageScope
+ String superField = 'works'
+ }
+
+ class B extends A {
+ @groovy.transform.CompileStatic
+ def test() {
+ 'something'.with {
+ return this.superField
+ }
+ }
+ }
+
+ def obj = new B()
+ assert obj.test() == "works"
+ '''
+ }
+ }
+
+ @Test
+ void 'test accessing a package-private super class field inside a closure - diff package, this qualifier'() {
+ shouldFail(MissingPropertyException) {
+ shell.with {
+ evaluate '''
+ package a
+
+ class A {
+ @groovy.transform.PackageScope
+ String superField = 'works'
+ }
+
+ assert true
+ '''
+
+ evaluate '''
+ package b
+
+ class B extends a.A {
+ @groovy.transform.CompileStatic
+ def test() {
+ 'something'.with {
+ return this.superField
+ }
+ }
+ }
+
+ def obj = new B()
+ assert obj.test() == "works"
+ '''
+ }
+ }
+ }
+
+ @Test
+ void 'test accessing a package-private super class field inside a closure - same package, owner qualifier'() {
+ shell.with {
+ evaluate '''
+ package a
+
+ class A {
+ @groovy.transform.PackageScope
+ String superField = 'works'
+ }
+
+ class B extends A {
+ @groovy.transform.CompileStatic
+ def test() {
+ 'something'.with {
+ return owner.superField
+ }
+ }
+ }
+
+ def obj = new B()
+ assert obj.test() == "works"
+ '''
+ }
+ }
+
+ @Test @NotYetImplemented // GROOVY-9293
+ void 'test accessing a package-private super class field inside a closure - diff package, owner qualifier'() {
+ shouldFail(MissingPropertyException) {
+ shell.with {
+ evaluate '''
+ package a
+
+ class A {
+ @groovy.transform.PackageScope
+ String superField = 'works'
+ }
+
+ assert true
+ '''
+
+ evaluate '''
+ package b
+
+ class B extends a.A {
+ @groovy.transform.CompileStatic
+ def test() {
+ 'something'.with {
+ return owner.superField // Access to b.B#superField is forbidden
+ }
+ }
+ }
+
+ def obj = new B()
+ assert obj.test() == "works"
+ '''
+ }
+ }
+ }
+
+ @Test
+ void 'test accessing a package-private super class field inside a closure - same package, delegate qualifier'() {
+ shell.with {
+ evaluate '''
+ package a
+
+ class A {
+ @groovy.transform.PackageScope
+ String superField = 'works'
+ }
+
+ class B extends A {
+ @groovy.transform.CompileStatic
+ def test() {
+ with {
+ return delegate.superField
+ }
+ }
+ }
+
+ def obj = new B()
+ assert obj.test() == "works"
+ '''
+ }
+ }
+
+ @Test @NotYetImplemented // GROOVY-9293
+ void 'test accessing a package-private super class field inside a closure - diff package, delegate qualifier'() {
+ shouldFail(MissingPropertyException) {
+ shell.with {
+ evaluate '''
+ package a
+
+ class A {
+ @groovy.transform.PackageScope
+ String superField = 'works'
+ }
+
+ assert true
+ '''
+
+ evaluate '''
+ package b
+
+ class B extends a.A {
+ @groovy.transform.CompileStatic
+ def test() {
+ with {
+ return delegate.superField // Access to b.B#superField is forbidden
+ }
+ }
+ }
+
+ def obj = new B()
+ assert obj.test() == "works"
+ '''
+ }
+ }
+ }
+
+ @Test
+ void 'test accessing a package-private super class field inside a closure - same package, thisObject qualifier'() {
+ shell.with {
+ evaluate '''
+ package a
+
+ class A {
+ @groovy.transform.PackageScope
+ String superField = 'works'
+ }
+
+ class B extends A {
+ @groovy.transform.CompileStatic
+ def test() {
+ 'something'.with {
+ return thisObject.superField
+ }
+ }
+ }
+
+ def obj = new B()
+ assert obj.test() == "works"
+ '''
+ }
+ }
+
+ @Test @NotYetImplemented // GROOVY-9293
+ void 'test accessing a package-private super class field inside a closure - diff package, thisObject qualifier'() {
+ shouldFail(MissingPropertyException) {
+ shell.with {
+ evaluate '''
+ package a
+
+ class A {
+ @groovy.transform.PackageScope
+ String superField = 'works'
+ }
+
+ assert true
+ '''
+
+ evaluate '''
+ package b
+
+ class B extends a.A {
+ @groovy.transform.CompileStatic
+ def test() {
+ 'something'.with {
+ return thisObject.superField
+ }
+ }
+ }
+
+ def obj = new B()
+ assert obj.test() == "works"
+ '''
+ }
+ }
+ }
+}