skip tests by optionally using describe.skip

elastic · Jun 18, 2020 · 36b19aa · 36b19aa
1 parent 0230e32
commit 36b19aa
Showing 1 changed file with 12 additions and 8 deletions.
diff --git a/x-pack/test/epm_api_integration/apis/index.ts b/x-pack/test/epm_api_integration/apis/index.ts
@@ -7,15 +7,19 @@
 import { FtrProviderContext } from '../../api_integration/ftr_provider_context';
 
 export default function ({ loadTestFile, getService }: FtrProviderContext) {
-  describe('EPM Endpoints', function () {
-    this.tags('ciGroup7');
+  const dockerServers = getService('dockerServers');
+  const log = getService('log');
+
+  let maybeSkipped: typeof global.describe | typeof global.describe.skip = describe;
 
-    const dockerServers = getService('dockerServers');
-    const log = getService('log');
-    if (!dockerServers.isEnabled('registry')) {
-      log.warning('skipping EPM Endpoints tests because registry docker server is disabled');
-      return;
-    }
+  if (!dockerServers.isEnabled('registry')) {
+    log.warning('skipping EPM Endpoints tests because registry docker server is disabled');
+    maybeSkipped = describe.skip.bind(describe);
+    return;
+  }
+
+  maybeSkipped('EPM Endpoints', function () {
+    this.tags('ciGroup7');
 
     loadTestFile(require.resolve('./list'));
     // loadTestFile(require.resolve('./file'));