groovy-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sun...@apache.org
Subject [groovy] 08/08: GROOVY-9293: add test cases
Date Mon, 28 Oct 2019 16:50:52 GMT
This is an automated email from the ASF dual-hosted git repository.

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

commit db5b985a5e4d22f4b717af7d514a50ae99d1ae78
Author: Eric Milles <eric.milles@thomsonreuters.com>
AuthorDate: Mon Oct 28 10:51:13 2019 -0500

    GROOVY-9293: add test cases
    
    (cherry picked from commit 9be48ebff4b9cf31b02dcc7d627ddbcfdaf3d6a8)
---
 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"
+                '''
+            }
+        }
+    }
+}


Mime
View raw message