This is an automated email from the ASF dual-hosted git repository.
emilles 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 a6d4d3e854 refactor setup
a6d4d3e854 is described below
commit a6d4d3e85481c4ef0208b7d26aab897398a9f395
Author: Eric Milles <[email protected]>
AuthorDate: Sat Feb 14 21:32:30 2026 -0600
refactor setup
---
.../jmx/builder/JmxBeanExportFactoryTest.groovy | 20 +++++++++++++-------
.../groovy/jmx/builder/JmxTimerFactoryTest.groovy | 12 ++++++------
2 files changed, 19 insertions(+), 13 deletions(-)
diff --git
a/subprojects/groovy-jmx/src/test/groovy/groovy/jmx/builder/JmxBeanExportFactoryTest.groovy
b/subprojects/groovy-jmx/src/test/groovy/groovy/jmx/builder/JmxBeanExportFactoryTest.groovy
index 963928cbbc..797c94ee71 100644
---
a/subprojects/groovy-jmx/src/test/groovy/groovy/jmx/builder/JmxBeanExportFactoryTest.groovy
+++
b/subprojects/groovy-jmx/src/test/groovy/groovy/jmx/builder/JmxBeanExportFactoryTest.groovy
@@ -20,6 +20,7 @@ package groovy.jmx.builder
import groovy.jmx.GroovyMBean
import org.junit.jupiter.api.Assumptions
+import org.junit.jupiter.api.BeforeEach
import org.junit.jupiter.api.Test
import javax.management.MBeanServerConnection
@@ -29,7 +30,18 @@ import static groovy.test.GroovyAssert.shouldFail
final class JmxBeanExportFactoryTest {
- private final JmxBuilder builder = new JmxBuilder()
+ private JmxBuilder builder
+ private MBeanServerConnection server
+
+ @BeforeEach
+ void setUp() {
+ builder = new JmxBuilder()
+ try {
+ server = builder.getMBeanServer()
+ } catch (e) {
+ Assumptions.abort(e.getMessage())
+ }
+ }
@Test
void testBeanExportReplacePolicy() {
@@ -76,12 +88,6 @@ final class JmxBeanExportFactoryTest {
bean(object)
}
- MBeanServerConnection server
- try {
- server = builder.getMBeanServer()
- } catch (e) {
- Assumptions.abort(e.getMessage())
- }
GroovyMBean bean = new GroovyMBean(server, new ObjectName(objName))
assert server.isRegistered(new ObjectName(objName))
diff --git
a/subprojects/groovy-jmx/src/test/groovy/groovy/jmx/builder/JmxTimerFactoryTest.groovy
b/subprojects/groovy-jmx/src/test/groovy/groovy/jmx/builder/JmxTimerFactoryTest.groovy
index b764e0e6ee..eb1c238cdf 100644
---
a/subprojects/groovy-jmx/src/test/groovy/groovy/jmx/builder/JmxTimerFactoryTest.groovy
+++
b/subprojects/groovy-jmx/src/test/groovy/groovy/jmx/builder/JmxTimerFactoryTest.groovy
@@ -31,23 +31,23 @@ import static groovy.test.GroovyAssert.shouldFail
final class JmxTimerFactoryTest {
private JmxBuilder builder
+ private MBeanServerConnection server
@BeforeEach
void setUp() {
builder = new JmxBuilder()
builder.registerFactory('timer', new JmxTimerFactory())
+ try {
+ server = builder.getMBeanServer()
+ } catch (e) {
+ Assumptions.abort(e.getMessage())
+ }
}
@Test
void testSimpleTimerSetup() {
GroovyMBean timer = builder.timer()
assert timer
- MBeanServerConnection server
- try {
- server = builder.getMBeanServer()
- } catch (e) {
- Assumptions.abort(e.getMessage())
- }
Set<ObjectName> objectNames = server.queryNames(new
ObjectName('jmx.builder:type=TimerService,*'), null)
assert !objectNames.isEmpty()
shouldFail {
