This is an automated email from the ASF dual-hosted git repository.
paulk 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 fa02ed1663 refactor: for a cleaner runtime classpath, keep logging
related tests in a separate subproject (remove old tests)
fa02ed1663 is described below
commit fa02ed1663b247210cab925543d32488fd697d30
Author: Paul King <[email protected]>
AuthorDate: Sat Mar 21 10:18:02 2026 +1000
refactor: for a cleaner runtime classpath, keep logging related tests in a
separate subproject (remove old tests)
---
src/test/groovy/bugs/Groovy6932.groovy | 52 -------------------------------
src/test/groovy/bugs/Groovy8060Bug.groovy | 43 -------------------------
2 files changed, 95 deletions(-)
diff --git a/src/test/groovy/bugs/Groovy6932.groovy
b/src/test/groovy/bugs/Groovy6932.groovy
deleted file mode 100644
index b9f4398a32..0000000000
--- a/src/test/groovy/bugs/Groovy6932.groovy
+++ /dev/null
@@ -1,52 +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 bugs
-
-import org.junit.jupiter.api.Test
-
-import static groovy.test.GroovyAssert.assertScript
-
-final class Groovy6932 {
- @Test
- void testLoggingWithinClosuresShouldHaveGuards() {
- assertScript '''
- @groovy.util.logging.Log
- class C {
- void m() {
- int info = 0
- int trace = 0
- log.info(createLogString(info++))
- log.finest(createLogString(trace++))
- Closure c1 = { log.info(createLogString(info++)) }
- c1()
- Closure c2 = { log.finest(createLogString(trace++)) }
- c2()
- assert info == 2
- assert trace == 0
- }
-
- String createLogString(p) {
- "called with $p"
- }
- }
-
- new C().m()
- '''
- }
-}
diff --git a/src/test/groovy/bugs/Groovy8060Bug.groovy
b/src/test/groovy/bugs/Groovy8060Bug.groovy
deleted file mode 100644
index af828f9bf2..0000000000
--- a/src/test/groovy/bugs/Groovy8060Bug.groovy
+++ /dev/null
@@ -1,43 +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 bugs
-
-import org.junit.jupiter.api.Test
-
-import static groovy.test.GroovyAssert.assertScript
-
-class Groovy8060Bug {
- @Test
- void testLoggingWithinClosuresThatAreMethodArgsShouldHaveGuards() {
- assertScript '''
- import groovy.util.logging.Slf4j
-
- @Slf4j
- class LogMain {
- public static int count = 0
-
- static void main(args) {
- assert !log.isTraceEnabled()
- 1.times { log.trace("${count++}") }
- assert !count
- }
- }
- '''
- }
-}
