[1/2] groovy git commit: GROOVY-8496: Can't call default getProperty() implementation from Java subclass

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[1/2] groovy git commit: GROOVY-8496: Can't call default getProperty() implementation from Java subclass

paulk
Repository: groovy
Updated Branches:
  refs/heads/master 406ffc12a -> 4523d0446


GROOVY-8496: Can't call default getProperty() implementation from Java subclass


Project: http://git-wip-us.apache.org/repos/asf/groovy/repo
Commit: http://git-wip-us.apache.org/repos/asf/groovy/commit/78c02181
Tree: http://git-wip-us.apache.org/repos/asf/groovy/tree/78c02181
Diff: http://git-wip-us.apache.org/repos/asf/groovy/diff/78c02181

Branch: refs/heads/master
Commit: 78c0218140887472a01db40a57eb97d7f6ac0228
Parents: 406ffc1
Author: Paul King <[hidden email]>
Authored: Sat Sep 15 00:11:46 2018 +1000
Committer: Paul King <[hidden email]>
Committed: Sat Sep 15 00:11:46 2018 +1000

----------------------------------------------------------------------
 .../groovy/groovy/ant/Groovy8496Test.groovy     | 57 ++++++++++++++++++++
 1 file changed, 57 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/groovy/blob/78c02181/subprojects/groovy-ant/src/test/groovy/groovy/ant/Groovy8496Test.groovy
----------------------------------------------------------------------
diff --git a/subprojects/groovy-ant/src/test/groovy/groovy/ant/Groovy8496Test.groovy b/subprojects/groovy-ant/src/test/groovy/groovy/ant/Groovy8496Test.groovy
new file mode 100644
index 0000000..981c0a0
--- /dev/null
+++ b/subprojects/groovy-ant/src/test/groovy/groovy/ant/Groovy8496Test.groovy
@@ -0,0 +1,57 @@
+/*
+ *  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.ant
+
+class Groovy8496Test extends AntTestCase {
+    void testGetProperty() {
+//        def debugLogger = new org.apache.tools.ant.DefaultLogger()
+//        debugLogger.setMessageOutputLevel(4)
+//        debugLogger.setOutputPrintStream(System.out)
+//        debugLogger.setErrorPrintStream(System.err)
+
+        doInTmpDir { ant, baseDir ->
+            baseDir.src {
+                'GroovyClass8496.groovy'('''
+                    class GroovyClass8496 {
+                        String getBar() { 'BAR!' }
+                    }
+                ''')
+                'OverrideGetProperty.java'('''
+                    public class OverrideGetProperty extends GroovyClass8496 {
+                        @Override
+                        public Object getProperty(String propertyName) {
+                            return super.getProperty(propertyName).toString().toLowerCase();
+                        }
+                        public static void main(String[] args) {
+                            System.out.println(new OverrideGetProperty().getProperty("bar"));
+                        }
+                    }
+                ''')
+            }
+//            ant.project.addBuildListener(debugLogger)
+            ant.mkdir(dir: 'build')
+            def cp = System.getProperty('java.class.path') + ':build'
+            ant.taskdef(name: 'groovyc', classname: 'org.codehaus.groovy.ant.Groovyc')
+            ant.groovyc(srcdir: 'src', destdir: 'build')
+            ant.javac(classpath: cp, destdir: 'build', srcdir: 'src', includeantruntime: 'false', fork: 'true')
+            ant.java(classpath: cp, outputproperty: 'result', classname: 'OverrideGetProperty')
+            assert ant.project.properties.result == 'bar!'
+        }
+    }
+}

Reply | Threaded
Open this post in threaded view
|

[2/2] groovy git commit: GROOVY-8497: Can't call getProperty() from Java

paulk
GROOVY-8497: Can't call getProperty() from Java


Project: http://git-wip-us.apache.org/repos/asf/groovy/repo
Commit: http://git-wip-us.apache.org/repos/asf/groovy/commit/4523d044
Tree: http://git-wip-us.apache.org/repos/asf/groovy/tree/4523d044
Diff: http://git-wip-us.apache.org/repos/asf/groovy/diff/4523d044

Branch: refs/heads/master
Commit: 4523d044615561e10bd7de79666b50b70b8f7f3f
Parents: 78c0218
Author: Paul King <[hidden email]>
Authored: Sat Sep 15 00:12:31 2018 +1000
Committer: Paul King <[hidden email]>
Committed: Sat Sep 15 00:12:31 2018 +1000

----------------------------------------------------------------------
 .../groovy/groovy/ant/Groovy8497Test.groovy     | 53 ++++++++++++++++++++
 1 file changed, 53 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/groovy/blob/4523d044/subprojects/groovy-ant/src/test/groovy/groovy/ant/Groovy8497Test.groovy
----------------------------------------------------------------------
diff --git a/subprojects/groovy-ant/src/test/groovy/groovy/ant/Groovy8497Test.groovy b/subprojects/groovy-ant/src/test/groovy/groovy/ant/Groovy8497Test.groovy
new file mode 100644
index 0000000..1991c2d
--- /dev/null
+++ b/subprojects/groovy-ant/src/test/groovy/groovy/ant/Groovy8497Test.groovy
@@ -0,0 +1,53 @@
+/*
+ *  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.ant
+
+class Groovy8497Test extends AntTestCase {
+    void testGetProperty() {
+//        def debugLogger = new org.apache.tools.ant.DefaultLogger()
+//        debugLogger.setMessageOutputLevel(4)
+//        debugLogger.setOutputPrintStream(System.out)
+//        debugLogger.setErrorPrintStream(System.err)
+
+        doInTmpDir { ant, baseDir ->
+            baseDir.src {
+                'GroovyClass8497.groovy'('''
+                    class GroovyClass8497 {
+                        String getFoo() { 'FOO!' }
+                    }
+                ''')
+                'AccessGetProperty.java'('''
+                    public class AccessGetProperty {
+                        public static void main(String[] args) {
+                            System.out.println(new GroovyClass8497().getProperty("foo"));
+                        }
+                    }
+                ''')
+            }
+//            ant.project.addBuildListener(debugLogger)
+            ant.mkdir(dir: 'build')
+            def cp = System.getProperty('java.class.path') + ':build'
+            ant.taskdef(name: 'groovyc', classname: 'org.codehaus.groovy.ant.Groovyc')
+            ant.groovyc(srcdir: 'src', destdir: 'build')
+            ant.javac(classpath: cp, destdir: 'build', srcdir: 'src', includeantruntime: 'false', fork: 'true')
+            ant.java(classpath: cp, outputproperty: 'result', classname: 'AccessGetProperty')
+            assert ant.project.properties.result == 'FOO!'
+        }
+    }
+}