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 2015/07/10 18:05:23 UTC

[1/2] incubator-groovy git commit: delete DummyTestDerivation which has no purpose anymore

Repository: incubator-groovy
Updated Branches:
  refs/heads/master 09e9778e8 -> d6578c462


delete DummyTestDerivation which has no purpose anymore


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

Branch: refs/heads/master
Commit: 7817a0ad6b56543af40abdd1426230ab45bfb5a6
Parents: 09e9778
Author: pascalschumacher <pa...@gmx.net>
Authored: Thu Jul 9 20:34:17 2015 +0200
Committer: pascalschumacher <pa...@gmx.net>
Committed: Fri Jul 10 18:04:14 2015 +0200

----------------------------------------------------------------------
 gradle/test.gradle                              |  2 +-
 .../groovy/classgen/DummyTestDerivation.java    | 31 --------------------
 2 files changed, 1 insertion(+), 32 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-groovy/blob/7817a0ad/gradle/test.gradle
----------------------------------------------------------------------
diff --git a/gradle/test.gradle b/gradle/test.gradle
index 0c70671..3ce15a1 100644
--- a/gradle/test.gradle
+++ b/gradle/test.gradle
@@ -87,7 +87,7 @@ tasks.addRule('Pattern: testSingle<Name> will test **/<Name>.class') { String ta
 }
 
 def buildExcludeFilter() {
-    def excludes = ['GroovyTestCase', 'TestSupport', 'DummyTestDerivation', 'LineColumnChecker']
+    def excludes = ['GroovyTestCase', 'TestSupport', 'LineColumnChecker']
 
     // temporary disabling Groovy4393Bug because it requires a specific configuration
     excludes << 'Groovy4393Bug'

http://git-wip-us.apache.org/repos/asf/incubator-groovy/blob/7817a0ad/src/test/org/codehaus/groovy/classgen/DummyTestDerivation.java
----------------------------------------------------------------------
diff --git a/src/test/org/codehaus/groovy/classgen/DummyTestDerivation.java b/src/test/org/codehaus/groovy/classgen/DummyTestDerivation.java
deleted file mode 100644
index 944d556..0000000
--- a/src/test/org/codehaus/groovy/classgen/DummyTestDerivation.java
+++ /dev/null
@@ -1,31 +0,0 @@
-/*
- *  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 org.codehaus.groovy.classgen;
-
-import groovy.util.GroovyTestCase;
-
-/**
- * A dummy test case
- *
- * @author <a href="mailto:james@coredevelopers.net">James Strachan</a>
- */
-public class DummyTestDerivation extends GroovyTestCase {
-    public DummyTestDerivation() {
-    }
-}


[2/2] incubator-groovy git commit: use the JUnit ignore annotation to exclude helper classes and manual tests from test execution

Posted by pa...@apache.org.
use the JUnit ignore annotation to exclude helper classes and manual tests from test execution


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

Branch: refs/heads/master
Commit: d6578c462727d3c0d0a663b2d8f26f7f0aa7f7a1
Parents: 7817a0a
Author: pascalschumacher <pa...@gmx.net>
Authored: Fri Jul 10 17:56:05 2015 +0200
Committer: pascalschumacher <pa...@gmx.net>
Committed: Fri Jul 10 18:04:15 2015 +0200

----------------------------------------------------------------------
 gradle/test.gradle                                      | 2 +-
 src/test/groovy/bugs/TestSupport.java                   | 2 ++
 src/test/org/codehaus/groovy/ast/LineColumnChecker.java | 2 ++
 src/test/org/codehaus/groovy/classgen/TestSupport.java  | 2 ++
 4 files changed, 7 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-groovy/blob/d6578c46/gradle/test.gradle
----------------------------------------------------------------------
diff --git a/gradle/test.gradle b/gradle/test.gradle
index 3ce15a1..c09a847 100644
--- a/gradle/test.gradle
+++ b/gradle/test.gradle
@@ -87,7 +87,7 @@ tasks.addRule('Pattern: testSingle<Name> will test **/<Name>.class') { String ta
 }
 
 def buildExcludeFilter() {
-    def excludes = ['GroovyTestCase', 'TestSupport', 'LineColumnChecker']
+    def excludes = []
 
     // temporary disabling Groovy4393Bug because it requires a specific configuration
     excludes << 'Groovy4393Bug'

http://git-wip-us.apache.org/repos/asf/incubator-groovy/blob/d6578c46/src/test/groovy/bugs/TestSupport.java
----------------------------------------------------------------------
diff --git a/src/test/groovy/bugs/TestSupport.java b/src/test/groovy/bugs/TestSupport.java
index e5ea111..401ac28 100644
--- a/src/test/groovy/bugs/TestSupport.java
+++ b/src/test/groovy/bugs/TestSupport.java
@@ -19,6 +19,7 @@
 package groovy.bugs;
 
 import groovy.util.GroovyTestCase;
+import org.junit.Ignore;
 
 import java.util.Arrays;
 import java.util.Iterator;
@@ -28,6 +29,7 @@ import java.util.Iterator;
  *
  * @author <a href="mailto:james@coredevelopers.net">James Strachan</a>
  */
+@Ignore("base class for tests")
 public abstract class TestSupport extends GroovyTestCase {
 
     public String[] getMockArguments() {

http://git-wip-us.apache.org/repos/asf/incubator-groovy/blob/d6578c46/src/test/org/codehaus/groovy/ast/LineColumnChecker.java
----------------------------------------------------------------------
diff --git a/src/test/org/codehaus/groovy/ast/LineColumnChecker.java b/src/test/org/codehaus/groovy/ast/LineColumnChecker.java
index 1abcdc6..1200a38 100644
--- a/src/test/org/codehaus/groovy/ast/LineColumnChecker.java
+++ b/src/test/org/codehaus/groovy/ast/LineColumnChecker.java
@@ -55,6 +55,7 @@ import org.codehaus.groovy.ast.expr.UnaryPlusExpression;
 import org.codehaus.groovy.ast.expr.VariableExpression;
 import org.codehaus.groovy.ast.stmt.Statement;
 import org.codehaus.groovy.control.SourceUnit;
+import org.junit.Ignore;
 
 /**
  * Tests the LineColumn information of the groovy source obtained in the
@@ -63,6 +64,7 @@ import org.codehaus.groovy.control.SourceUnit;
  * @author <a href="mailto:martin.kempf@gmail.com">Martin Kempf</a>
  *
  */
+@Ignore("helper class for manual testing")
 public class LineColumnChecker extends ASTTest {
     
     private LineCheckVisitor visitor;

http://git-wip-us.apache.org/repos/asf/incubator-groovy/blob/d6578c46/src/test/org/codehaus/groovy/classgen/TestSupport.java
----------------------------------------------------------------------
diff --git a/src/test/org/codehaus/groovy/classgen/TestSupport.java b/src/test/org/codehaus/groovy/classgen/TestSupport.java
index b92189b..579562d 100644
--- a/src/test/org/codehaus/groovy/classgen/TestSupport.java
+++ b/src/test/org/codehaus/groovy/classgen/TestSupport.java
@@ -30,6 +30,7 @@ import org.codehaus.groovy.ast.expr.MethodCallExpression;
 import org.codehaus.groovy.ast.stmt.ExpressionStatement;
 import org.codehaus.groovy.control.CompilerConfiguration;
 import org.codehaus.groovy.runtime.InvokerHelper;
+import org.junit.Ignore;
 import org.objectweb.asm.Opcodes;
 
 import java.beans.BeanInfo;
@@ -47,6 +48,7 @@ import java.security.PrivilegedAction;
  *
  * @author <a href="mailto:james@coredevelopers.net">James Strachan</a>
  */
+@Ignore("base class for tests")
 public class TestSupport extends GroovyTestCase implements Opcodes {
 
     protected static boolean DUMP_CLASS = false;