This is an automated email from the ASF dual-hosted git repository. rombert pushed a commit to annotated tag org.apache.sling.hc.junit.bridge-1.0.2 in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-hc-junit-bridge.git
commit 1a27335fda8d3ef93240c3879544d674bd3dfd3c Author: Bertrand Delacretaz <[email protected]> AuthorDate: Tue Jul 15 13:02:08 2014 +0000 Use HealthCheckMetadata to provide test name git-svn-id: https://svn.apache.org/repos/asf/sling/whiteboard/bdelacretaz/junit-bridge@1610678 13f79535-47bb-0310-9956-ffa450edef68 --- .../sling/hc/junitbridge/HealthCheckTest.java | 56 +++++++++++++--------- .../hc/junitbridge/HealthCheckTestsProvider.java | 45 +++++++++++++---- .../sling/hc/junitbridge/JUnitTestBridge.java | 6 +-- .../sling/hc/junitbridge/TestBridgeContext.java | 11 ++++- 4 files changed, 82 insertions(+), 36 deletions(-) diff --git a/src/main/java/org/apache/sling/hc/junitbridge/HealthCheckTest.java b/src/main/java/org/apache/sling/hc/junitbridge/HealthCheckTest.java index 8c5f36b..73f3d72 100644 --- a/src/main/java/org/apache/sling/hc/junitbridge/HealthCheckTest.java +++ b/src/main/java/org/apache/sling/hc/junitbridge/HealthCheckTest.java @@ -22,45 +22,55 @@ import junit.framework.TestCase; import org.apache.sling.hc.api.HealthCheck; import org.apache.sling.hc.api.Result; import org.apache.sling.hc.api.ResultLog; +import org.apache.sling.hc.util.HealthCheckMetadata; +import org.osgi.framework.BundleContext; +import org.osgi.framework.ServiceReference; public class HealthCheckTest extends TestCase { private final HealthCheck hc; - private final String name; + private final HealthCheckMetadata metadata; + private final BundleContext bundleContext; + private final ServiceReference serviceRef; - HealthCheckTest(HealthCheck hc) { + HealthCheckTest(TestBridgeContext context, ServiceReference ref) { super("testHealthCheck"); - this.hc = hc; - - // TODO HC name? see webconsole - name = hc.getClass().getName(); + bundleContext = context.getBundleContext(); + serviceRef = ref; + this.hc = (HealthCheck)bundleContext.getService(ref); + this.metadata = new HealthCheckMetadata(ref); } @Override public String getName() { - return name; + return metadata.getName(); } /** Execute our health check and dump its log * messages > INFO if it fails */ public void testHealthCheck() { - final Result r = hc.execute(); - final StringBuilder failMsg = new StringBuilder(); - if(!r.isOk()) { - failMsg.append(name); - failMsg.append("\n"); - for(ResultLog.Entry log : r) { - if(failMsg.length() > 0) { - failMsg.append("\n"); - } - if(log.getStatus().compareTo(Result.Status.INFO) > 0) { - failMsg.append(log.getStatus().toString()); - failMsg.append(" - "); - failMsg.append(log.getMessage()); + try { + final Result r = hc.execute(); + final StringBuilder failMsg = new StringBuilder(); + if(!r.isOk()) { + failMsg.append(metadata.getName()); + failMsg.append("\n"); + for(ResultLog.Entry log : r) { + if(log.getStatus().compareTo(Result.Status.INFO) > 0) { + if(failMsg.length() > 0) { + failMsg.append("\n"); + } + failMsg.append(log.getStatus().toString()); + failMsg.append(" - "); + failMsg.append(log.getMessage()); + } } } - } - if(failMsg.length() > 0) { - fail("Health Check failed: " + failMsg.toString()); + if(failMsg.length() > 0) { + fail("Health Check failed: " + failMsg.toString()); + } + } finally { + // TODO is that ok? service not used anymore after this? + bundleContext.ungetService(serviceRef); } } } \ No newline at end of file diff --git a/src/main/java/org/apache/sling/hc/junitbridge/HealthCheckTestsProvider.java b/src/main/java/org/apache/sling/hc/junitbridge/HealthCheckTestsProvider.java index 95849c4..bf0136d 100644 --- a/src/main/java/org/apache/sling/hc/junitbridge/HealthCheckTestsProvider.java +++ b/src/main/java/org/apache/sling/hc/junitbridge/HealthCheckTestsProvider.java @@ -22,9 +22,10 @@ import java.util.List; import org.apache.felix.scr.annotations.Activate; import org.apache.felix.scr.annotations.Component; +import org.apache.felix.scr.annotations.Deactivate; import org.apache.felix.scr.annotations.Service; -import org.apache.sling.hc.util.HealthCheckFilter; import org.apache.sling.junit.TestsProvider; +import org.osgi.framework.BundleContext; import org.osgi.framework.Constants; import org.osgi.service.component.ComponentContext; @@ -36,22 +37,49 @@ import org.osgi.service.component.ComponentContext; public class HealthCheckTestsProvider implements TestsProvider { private String servicePid; + private long lastModified; + private BundleContext bundleContext; + + public static final String TEST_NAME_PREFIX = "HealthChecks("; + public static final String TEST_NAME_SUFFIX = ")"; // TODO configurable - private String [] tags = { "script" }; - private HealthCheckFilter filter; + private String [] tags = { + "script", + "sling", + "bundles,script", + "bundles,-script" + }; @Activate protected void activate(ComponentContext ctx) { + bundleContext = ctx.getBundleContext(); servicePid = (String)ctx.getProperties().get(Constants.SERVICE_PID); - filter = new HealthCheckFilter(ctx.getBundleContext()); + lastModified = System.currentTimeMillis(); + } + + @Deactivate + protected void deactivate() { + bundleContext = null; + servicePid = null; + lastModified = -1; } @Override public Class<?> createTestClass(String testName) throws ClassNotFoundException { - JUnitTestBridge.setContext(new TestBridgeContext(filter, tags)); + // The test name is like "Health Checks(foo,bar)" and we need just 'foo,bar' + final String tagString = testName.substring(0, testName.length() - TEST_NAME_SUFFIX.length()).substring(TEST_NAME_PREFIX.length()); + JUnitTestBridge.setContext(new TestBridgeContext(bundleContext, splitTags(tagString))); return JUnitTestBridge.class; } + + private String [] splitTags(String tags) { + final List<String> result = new ArrayList<String>(); + for(String tag: tags.split(",")) { + result.add(tag.trim()); + } + return result.toArray(new String[]{}); + } @Override public String getServicePid() { @@ -61,13 +89,14 @@ public class HealthCheckTestsProvider implements TestsProvider { @Override public List<String> getTestNames() { final List<String> result = new ArrayList<String>(); - // TODO use a configurable name? - result.add("HealthChecks(sling,slow)"); + for(String t : tags) { + result.add(TEST_NAME_PREFIX + t + TEST_NAME_SUFFIX); + } return result; } @Override public long lastModified() { - return 0; + return lastModified; } } \ No newline at end of file diff --git a/src/main/java/org/apache/sling/hc/junitbridge/JUnitTestBridge.java b/src/main/java/org/apache/sling/hc/junitbridge/JUnitTestBridge.java index 51edad7..5996530 100644 --- a/src/main/java/org/apache/sling/hc/junitbridge/JUnitTestBridge.java +++ b/src/main/java/org/apache/sling/hc/junitbridge/JUnitTestBridge.java @@ -20,9 +20,9 @@ package org.apache.sling.hc.junitbridge; import static org.junit.Assert.assertNotNull; import junit.framework.TestSuite; -import org.apache.sling.hc.api.HealthCheck; import org.junit.runner.RunWith; import org.junit.runners.AllTests; +import org.osgi.framework.ServiceReference; @RunWith(AllTests.class) public class JUnitTestBridge { @@ -36,8 +36,8 @@ public class JUnitTestBridge { final TestBridgeContext context = testContext.get(); assertNotNull("Expecting non-null TestBridgeContext, via ThreadLocal", context); TestSuite suite = new TestSuite(); - for(HealthCheck hc : context.getFilter().getTaggedHealthChecks(context.getTags())) { - suite.addTest(new HealthCheckTest(hc)); + for(ServiceReference ref : context.getFilter().getTaggedHealthCheckServiceReferences(context.getTags())) { + suite.addTest(new HealthCheckTest(context, ref)); } return suite; } diff --git a/src/main/java/org/apache/sling/hc/junitbridge/TestBridgeContext.java b/src/main/java/org/apache/sling/hc/junitbridge/TestBridgeContext.java index 4072c50..030dfe5 100644 --- a/src/main/java/org/apache/sling/hc/junitbridge/TestBridgeContext.java +++ b/src/main/java/org/apache/sling/hc/junitbridge/TestBridgeContext.java @@ -18,14 +18,17 @@ package org.apache.sling.hc.junitbridge; import org.apache.sling.hc.util.HealthCheckFilter; +import org.osgi.framework.BundleContext; class TestBridgeContext { private final String [] tags; private final HealthCheckFilter filter; + private final BundleContext bundleContext; - TestBridgeContext(HealthCheckFilter filter, String [] tags) { + TestBridgeContext(BundleContext bundleContext, String [] tags) { + this.bundleContext = bundleContext; this.tags = tags; - this.filter = filter; + this.filter = new HealthCheckFilter(bundleContext); } String [] getTags() { @@ -35,4 +38,8 @@ class TestBridgeContext { HealthCheckFilter getFilter() { return filter; } + + BundleContext getBundleContext() { + return bundleContext; + } } \ No newline at end of file -- To stop receiving notification emails like this one, please contact "[email protected]" <[email protected]>.
