From ab9e2cddf169be536065e9b32d367fee978bbb0c Mon Sep 17 00:00:00 2001 From: Nik Everett Date: Fri, 7 Sep 2018 21:54:38 -0400 Subject: [PATCH] Logging: Clean up skipping test Clean up on top of the last fix: if we skip the entire test case then the test run would fail because we skipped all the tests. This adds a dummy test case to prevent that. It is a fairly nasty work around I plan to work on something that makes this not required any more anyway. --- .../unconfigured_node_name/NodeNameInLogsIT.java | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/qa/unconfigured-node-name/src/test/java/org/elasticsearch/unconfigured_node_name/NodeNameInLogsIT.java b/qa/unconfigured-node-name/src/test/java/org/elasticsearch/unconfigured_node_name/NodeNameInLogsIT.java index 76bc9fa4d55..9f36a600b68 100644 --- a/qa/unconfigured-node-name/src/test/java/org/elasticsearch/unconfigured_node_name/NodeNameInLogsIT.java +++ b/qa/unconfigured-node-name/src/test/java/org/elasticsearch/unconfigured_node_name/NodeNameInLogsIT.java @@ -43,4 +43,11 @@ public class NodeNameInLogsIT extends NodeNameInLogsIntegTestCase { } }); } + + public void testDummy() { + /* Dummy test case so that when we run this test on a platform that + * does not support our syscall filters and we skip the test above + * we don't fail the entire test run because we skipped all the tests. + */ + } }