From: Alexandre Montplaisir Date: Thu, 18 Feb 2016 22:11:15 +0000 (-0500) Subject: Replace test skipping with straight failures X-Git-Url: http://git.liburcu.org/?p=lttng-ust-java-tests.git;a=commitdiff_plain;h=65a36bff2c5a362152e6448d7891e94f008e4a8b Replace test skipping with straight failures Chasing after skipped tests is annoying. If the environment is not set correctly, that should still count as a failure. Signed-off-by: Alexandre Montplaisir --- diff --git a/lttng-ust-java-tests-common/src/test/java/org/lttng/ust/agent/integration/client/TcpClientIT.java b/lttng-ust-java-tests-common/src/test/java/org/lttng/ust/agent/integration/client/TcpClientIT.java index ab27da6..69ef96f 100644 --- a/lttng-ust-java-tests-common/src/test/java/org/lttng/ust/agent/integration/client/TcpClientIT.java +++ b/lttng-ust-java-tests-common/src/test/java/org/lttng/ust/agent/integration/client/TcpClientIT.java @@ -20,7 +20,6 @@ package org.lttng.ust.agent.integration.client; import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertTrue; -import static org.junit.Assume.assumeTrue; import java.util.ArrayList; import java.util.Arrays; @@ -97,7 +96,7 @@ public class TcpClientIT { clientThread = new Thread(client); clientThread.start(); - assumeTrue("Timed out waiting for root sessiond", client.waitForConnection(5)); + assertTrue("Timed out waiting for root sessiond", client.waitForConnection(5)); } /** diff --git a/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/context/JulAppContextIT.java b/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/context/JulAppContextIT.java index aaecf41..a6b3f68 100644 --- a/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/context/JulAppContextIT.java +++ b/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/context/JulAppContextIT.java @@ -18,7 +18,7 @@ package org.lttng.ust.agent.integration.context; -import static org.junit.Assume.assumeTrue; +import static org.junit.Assert.assertTrue; import java.io.IOException; import java.util.logging.Handler; @@ -49,9 +49,9 @@ public class JulAppContextIT extends AppContextITBase { */ @BeforeClass public static void julClassSetup() { - /* Skip tests if we can't find the JNI library or lttng-tools */ - assumeTrue(JulTestUtils.checkForJulLibrary()); - assumeTrue(LttngUtils.checkForLttngTools(Domain.JUL)); + /* Make sure we can find the JNI library and lttng-tools */ + assertTrue(JulTestUtils.checkForJulLibrary()); + assertTrue(LttngUtils.checkForLttngTools(Domain.JUL)); LttngToolsHelper.destroyAllSessions(); } diff --git a/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/context/JulAppContextOrderingIT.java b/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/context/JulAppContextOrderingIT.java index d3a2f64..5fbea4c 100644 --- a/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/context/JulAppContextOrderingIT.java +++ b/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/context/JulAppContextOrderingIT.java @@ -18,8 +18,8 @@ package org.lttng.ust.agent.integration.context; +import static org.junit.Assert.assertTrue; import static org.junit.Assert.fail; -import static org.junit.Assume.assumeTrue; import java.io.IOException; import java.util.logging.Handler; @@ -46,9 +46,9 @@ public class JulAppContextOrderingIT extends AppContextOrderingITBase { */ @BeforeClass public static void julClassSetup() { - /* Skip tests if we can't find the JNI library or lttng-tools */ - assumeTrue(JulTestUtils.checkForJulLibrary()); - assumeTrue(LttngUtils.checkForLttngTools(Domain.JUL)); + /* Make sure we can find the JNI library and lttng-tools */ + assertTrue(JulTestUtils.checkForJulLibrary()); + assertTrue(LttngUtils.checkForLttngTools(Domain.JUL)); LttngToolsHelper.destroyAllSessions(); } diff --git a/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/events/JulEnabledEventsIT.java b/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/events/JulEnabledEventsIT.java index 72cb283..f5aaaac 100644 --- a/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/events/JulEnabledEventsIT.java +++ b/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/events/JulEnabledEventsIT.java @@ -18,7 +18,7 @@ package org.lttng.ust.agent.integration.events; -import static org.junit.Assume.assumeTrue; +import static org.junit.Assert.assertTrue; import java.io.IOException; import java.util.logging.Handler; @@ -52,9 +52,9 @@ public class JulEnabledEventsIT extends EnabledEventsITBase { */ @BeforeClass public static void julClassSetup() { - /* Skip tests if we can't find the JNI library or lttng-tools */ - assumeTrue(JulTestUtils.checkForJulLibrary()); - assumeTrue(LttngUtils.checkForLttngTools(Domain.JUL)); + /* Make sure we can find the JNI library and lttng-tools */ + assertTrue(JulTestUtils.checkForJulLibrary()); + assertTrue(LttngUtils.checkForLttngTools(Domain.JUL)); LttngToolsHelper.destroyAllSessions(); } diff --git a/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/events/JulLegacyApiIT.java b/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/events/JulLegacyApiIT.java index 35d70ba..88ed2ab 100644 --- a/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/events/JulLegacyApiIT.java +++ b/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/events/JulLegacyApiIT.java @@ -22,7 +22,6 @@ import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertNotNull; import static org.junit.Assert.assertTrue; import static org.junit.Assert.fail; -import static org.junit.Assume.assumeTrue; import java.lang.reflect.Field; import java.util.List; @@ -66,9 +65,9 @@ public class JulLegacyApiIT { */ @BeforeClass public static void julClassSetup() { - /* Skip tests if we can't find the JNI library or lttng-tools */ - assumeTrue(JulTestUtils.checkForJulLibrary()); - assumeTrue(LttngUtils.checkForLttngTools(Domain.JUL)); + /* Make sure we can find the JNI library and lttng-tools */ + assertTrue(JulTestUtils.checkForJulLibrary()); + assertTrue(LttngUtils.checkForLttngTools(Domain.JUL)); LttngToolsHelper.destroyAllSessions(); } diff --git a/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/events/JulListEventsIT.java b/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/events/JulListEventsIT.java index 6b78cb2..3a4cef6 100644 --- a/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/events/JulListEventsIT.java +++ b/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/events/JulListEventsIT.java @@ -18,7 +18,7 @@ package org.lttng.ust.agent.integration.events; -import static org.junit.Assume.assumeTrue; +import static org.junit.Assert.assertTrue; import java.io.IOException; import java.util.logging.Handler; @@ -48,9 +48,9 @@ public class JulListEventsIT extends ListEventsITBase { */ @BeforeClass public static void testClassSetup() { - /* Skip tests if we can't find the JNI library or lttng-tools */ - assumeTrue(JulTestUtils.checkForJulLibrary()); - assumeTrue(LttngUtils.checkForLttngTools(Domain.JUL)); + /* Make sure we can find the JNI library and lttng-tools */ + assertTrue(JulTestUtils.checkForJulLibrary()); + assertTrue(LttngUtils.checkForLttngTools(Domain.JUL)); LttngToolsHelper.destroyAllSessions(); } diff --git a/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/events/JulMultiSessionIT.java b/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/events/JulMultiSessionIT.java index 33116b0..4f34b1b 100644 --- a/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/events/JulMultiSessionIT.java +++ b/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/events/JulMultiSessionIT.java @@ -18,7 +18,7 @@ package org.lttng.ust.agent.integration.events; -import static org.junit.Assume.assumeTrue; +import static org.junit.Assert.assertTrue; import java.io.IOException; import java.util.logging.Handler; @@ -53,9 +53,9 @@ public class JulMultiSessionIT extends MultiSessionITBase { */ @BeforeClass public static void julClassSetup() { - /* Skip tests if we can't find the JNI library or lttng-tools */ - assumeTrue(JulTestUtils.checkForJulLibrary()); - assumeTrue(LttngUtils.checkForLttngTools(Domain.JUL)); + /* Make sure we can find the JNI library and lttng-tools */ + assertTrue(JulTestUtils.checkForJulLibrary()); + assertTrue(LttngUtils.checkForLttngTools(Domain.JUL)); LttngToolsHelper.destroyAllSessions(); } diff --git a/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/filter/JulFilterListenerIT.java b/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/filter/JulFilterListenerIT.java index 3db70b5..70390a4 100644 --- a/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/filter/JulFilterListenerIT.java +++ b/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/filter/JulFilterListenerIT.java @@ -18,7 +18,7 @@ package org.lttng.ust.agent.integration.filter; -import static org.junit.Assume.assumeTrue; +import static org.junit.Assert.assertTrue; import java.io.IOException; @@ -43,9 +43,9 @@ public class JulFilterListenerIT extends FilterListenerITBase { */ @BeforeClass public static void julClassSetup() { - /* Skip tests if we can't find the JNI library or lttng-tools */ - assumeTrue(JulTestUtils.checkForJulLibrary()); - assumeTrue(LttngUtils.checkForLttngTools(ILttngSession.Domain.JUL)); + /* Make sure we can find the JNI library and lttng-tools */ + assertTrue(JulTestUtils.checkForJulLibrary()); + assertTrue(LttngUtils.checkForLttngTools(ILttngSession.Domain.JUL)); LttngToolsHelper.destroyAllSessions(); } diff --git a/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/filter/JulFilterListenerOrderingIT.java b/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/filter/JulFilterListenerOrderingIT.java index f52b90d..4d599cd 100644 --- a/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/filter/JulFilterListenerOrderingIT.java +++ b/lttng-ust-java-tests-jul/src/test/java/org/lttng/ust/agent/integration/filter/JulFilterListenerOrderingIT.java @@ -18,8 +18,8 @@ package org.lttng.ust.agent.integration.filter; +import static org.junit.Assert.assertTrue; import static org.junit.Assert.fail; -import static org.junit.Assume.assumeTrue; import java.io.IOException; import java.util.logging.Handler; @@ -27,8 +27,8 @@ import java.util.logging.Level; import java.util.logging.Logger; import org.junit.BeforeClass; -import org.lttng.tools.LttngToolsHelper; import org.lttng.tools.ILttngSession.Domain; +import org.lttng.tools.LttngToolsHelper; import org.lttng.ust.agent.jul.LttngLogHandler; import org.lttng.ust.agent.utils.JulTestUtils; import org.lttng.ust.agent.utils.LttngUtils; @@ -46,9 +46,9 @@ public class JulFilterListenerOrderingIT extends FilterListenerOrderingITBase { */ @BeforeClass public static void julClassSetup() { - /* Skip tests if we can't find the JNI library or lttng-tools */ - assumeTrue(JulTestUtils.checkForJulLibrary()); - assumeTrue(LttngUtils.checkForLttngTools(Domain.JUL)); + /* Make sure we can find the JNI library and lttng-tools */ + assertTrue(JulTestUtils.checkForJulLibrary()); + assertTrue(LttngUtils.checkForLttngTools(Domain.JUL)); LttngToolsHelper.destroyAllSessions(); } diff --git a/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/context/Log4jAppContextIT.java b/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/context/Log4jAppContextIT.java index bc8e631..00e8b37 100644 --- a/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/context/Log4jAppContextIT.java +++ b/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/context/Log4jAppContextIT.java @@ -18,7 +18,7 @@ package org.lttng.ust.agent.integration.context; -import static org.junit.Assume.assumeTrue; +import static org.junit.Assert.assertTrue; import java.io.IOException; @@ -49,9 +49,9 @@ public class Log4jAppContextIT extends AppContextITBase { */ @BeforeClass public static void julClassSetup() { - /* Skip tests if we can't find the log4j library or lttng-tools */ - assumeTrue(Log4jTestUtils.checkForLog4jLibrary()); - assumeTrue(LttngUtils.checkForLttngTools(Domain.LOG4J)); + /* Make sure we can find the JNI library and lttng-tools */ + assertTrue(Log4jTestUtils.checkForLog4jLibrary()); + assertTrue(LttngUtils.checkForLttngTools(Domain.LOG4J)); LttngToolsHelper.destroyAllSessions(); } diff --git a/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/context/Log4jAppContextOrderingIT.java b/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/context/Log4jAppContextOrderingIT.java index 7b69351..59b7d1b 100644 --- a/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/context/Log4jAppContextOrderingIT.java +++ b/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/context/Log4jAppContextOrderingIT.java @@ -18,8 +18,8 @@ package org.lttng.ust.agent.integration.context; +import static org.junit.Assert.assertTrue; import static org.junit.Assert.fail; -import static org.junit.Assume.assumeTrue; import java.io.IOException; @@ -46,9 +46,9 @@ public class Log4jAppContextOrderingIT extends AppContextOrderingITBase { */ @BeforeClass public static void log4jClassSetup() { - /* Skip tests if we can't find the JNI library or lttng-tools */ - assumeTrue(Log4jTestUtils.checkForLog4jLibrary()); - assumeTrue(LttngUtils.checkForLttngTools(Domain.LOG4J)); + /* Make sure we can find the JNI library and lttng-tools */ + assertTrue(Log4jTestUtils.checkForLog4jLibrary()); + assertTrue(LttngUtils.checkForLttngTools(Domain.LOG4J)); LttngToolsHelper.destroyAllSessions(); } diff --git a/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/events/Log4jEnabledEventsIT.java b/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/events/Log4jEnabledEventsIT.java index 1642b6a..dcca40d 100644 --- a/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/events/Log4jEnabledEventsIT.java +++ b/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/events/Log4jEnabledEventsIT.java @@ -18,7 +18,7 @@ package org.lttng.ust.agent.integration.events; -import static org.junit.Assume.assumeTrue; +import static org.junit.Assert.assertTrue; import java.io.IOException; @@ -53,9 +53,9 @@ public class Log4jEnabledEventsIT extends EnabledEventsITBase { */ @BeforeClass public static void log4jClassSetup() { - /* Skip tests if we can't find the JNI library or lttng-tools */ - assumeTrue(Log4jTestUtils.checkForLog4jLibrary()); - assumeTrue(LttngUtils.checkForLttngTools(Domain.LOG4J)); + /* Make sure we can find the JNI library and lttng-tools */ + assertTrue(Log4jTestUtils.checkForLog4jLibrary()); + assertTrue(LttngUtils.checkForLttngTools(Domain.LOG4J)); LttngToolsHelper.destroyAllSessions(); } diff --git a/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/events/Log4jLegacyApiIT.java b/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/events/Log4jLegacyApiIT.java index fe613d3..af46cf4 100644 --- a/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/events/Log4jLegacyApiIT.java +++ b/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/events/Log4jLegacyApiIT.java @@ -22,7 +22,6 @@ import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertNotNull; import static org.junit.Assert.assertTrue; import static org.junit.Assert.fail; -import static org.junit.Assume.assumeTrue; import java.lang.reflect.Field; import java.util.List; @@ -67,9 +66,9 @@ public class Log4jLegacyApiIT { */ @BeforeClass public static void classSetup() { - /* Skip tests if we can't find the JNI library or lttng-tools */ - assumeTrue(Log4jTestUtils.checkForLog4jLibrary()); - assumeTrue(LttngUtils.checkForLttngTools(Domain.LOG4J)); + /* Make sure we can find the JNI library and lttng-tools */ + assertTrue(Log4jTestUtils.checkForLog4jLibrary()); + assertTrue(LttngUtils.checkForLttngTools(Domain.LOG4J)); LttngToolsHelper.destroyAllSessions(); } diff --git a/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/events/Log4jListEventsIT.java b/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/events/Log4jListEventsIT.java index e06abf8..885ba4b 100644 --- a/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/events/Log4jListEventsIT.java +++ b/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/events/Log4jListEventsIT.java @@ -18,7 +18,7 @@ package org.lttng.ust.agent.integration.events; -import static org.junit.Assume.assumeTrue; +import static org.junit.Assert.assertTrue; import java.io.IOException; @@ -48,9 +48,9 @@ public class Log4jListEventsIT extends ListEventsITBase { */ @BeforeClass public static void testClassSetup() { - /* Skip tests if we can't find the JNI library or lttng-tools */ - assumeTrue(Log4jTestUtils.checkForLog4jLibrary()); - assumeTrue(LttngUtils.checkForLttngTools(Domain.LOG4J)); + /* Make sure we can find the JNI library and lttng-tools */ + assertTrue(Log4jTestUtils.checkForLog4jLibrary()); + assertTrue(LttngUtils.checkForLttngTools(Domain.LOG4J)); LttngToolsHelper.destroyAllSessions(); } diff --git a/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/events/Log4jMultiSessionIT.java b/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/events/Log4jMultiSessionIT.java index c32a825..00e7183 100644 --- a/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/events/Log4jMultiSessionIT.java +++ b/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/events/Log4jMultiSessionIT.java @@ -18,7 +18,7 @@ package org.lttng.ust.agent.integration.events; -import static org.junit.Assume.assumeTrue; +import static org.junit.Assert.assertTrue; import java.io.IOException; @@ -53,9 +53,9 @@ public class Log4jMultiSessionIT extends MultiSessionITBase { */ @BeforeClass public static void log4jClassSetup() { - /* Skip tests if we can't find the JNI library or lttng-tools */ - assumeTrue(Log4jTestUtils.checkForLog4jLibrary()); - assumeTrue(LttngUtils.checkForLttngTools(Domain.LOG4J)); + /* Make sure we can find the JNI library and lttng-tools */ + assertTrue(Log4jTestUtils.checkForLog4jLibrary()); + assertTrue(LttngUtils.checkForLttngTools(Domain.LOG4J)); LttngToolsHelper.destroyAllSessions(); } diff --git a/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/filter/Log4jFilterListenerIT.java b/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/filter/Log4jFilterListenerIT.java index d40dd95..3647963 100644 --- a/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/filter/Log4jFilterListenerIT.java +++ b/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/filter/Log4jFilterListenerIT.java @@ -18,7 +18,7 @@ package org.lttng.ust.agent.integration.filter; -import static org.junit.Assume.assumeTrue; +import static org.junit.Assert.assertTrue; import java.io.IOException; @@ -43,9 +43,9 @@ public class Log4jFilterListenerIT extends FilterListenerITBase { */ @BeforeClass public static void log4jClassSetup() { - /* Skip tests if we can't find the JNI library or lttng-tools */ - assumeTrue(Log4jTestUtils.checkForLog4jLibrary()); - assumeTrue(LttngUtils.checkForLttngTools(ILttngSession.Domain.LOG4J)); + /* Make sure we can find the JNI library and lttng-tools */ + assertTrue(Log4jTestUtils.checkForLog4jLibrary()); + assertTrue(LttngUtils.checkForLttngTools(ILttngSession.Domain.LOG4J)); LttngToolsHelper.destroyAllSessions(); } diff --git a/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/filter/Log4jFilterListenerOrderingIT.java b/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/filter/Log4jFilterListenerOrderingIT.java index 5db9d2b..0399b71 100644 --- a/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/filter/Log4jFilterListenerOrderingIT.java +++ b/lttng-ust-java-tests-log4j/src/test/java/org/lttng/ust/agent/integration/filter/Log4jFilterListenerOrderingIT.java @@ -18,8 +18,8 @@ package org.lttng.ust.agent.integration.filter; +import static org.junit.Assert.assertTrue; import static org.junit.Assert.fail; -import static org.junit.Assume.assumeTrue; import java.io.IOException; @@ -27,8 +27,8 @@ import org.apache.log4j.Appender; import org.apache.log4j.Level; import org.apache.log4j.Logger; import org.junit.BeforeClass; -import org.lttng.tools.LttngToolsHelper; import org.lttng.tools.ILttngSession.Domain; +import org.lttng.tools.LttngToolsHelper; import org.lttng.ust.agent.log4j.LttngLogAppender; import org.lttng.ust.agent.utils.Log4jTestUtils; import org.lttng.ust.agent.utils.LttngUtils; @@ -46,9 +46,9 @@ public class Log4jFilterListenerOrderingIT extends FilterListenerOrderingITBase */ @BeforeClass public static void julClassSetup() { - /* Skip tests if we can't find the JNI library or lttng-tools */ - assumeTrue(Log4jTestUtils.checkForLog4jLibrary()); - assumeTrue(LttngUtils.checkForLttngTools(Domain.LOG4J)); + /* Make sure we can find the JNI library and lttng-tools */ + assertTrue(Log4jTestUtils.checkForLog4jLibrary()); + assertTrue(LttngUtils.checkForLttngTools(Domain.LOG4J)); LttngToolsHelper.destroyAllSessions(); }