set suite.pending to disable the suite, rather than calling describe.…

…skip
elastic · Jun 18, 2020 · 4426ead · 4426ead
1 parent a1494fc
commit 4426ead
Showing 1 changed file with 6 additions and 9 deletions.
diff --git a/x-pack/test/epm_api_integration/apis/index.ts b/x-pack/test/epm_api_integration/apis/index.ts
@@ -10,17 +10,14 @@ export default function ({ loadTestFile, getService }: FtrProviderContext) {
   const dockerServers = getService('dockerServers');
   const log = getService('log');
 
-  let maybeSkipped: typeof global.describe | typeof global.describe.skip = describe;
-
-  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 () {
+  describe('EPM Endpoints', function () {
     this.tags('ciGroup1');
 
+    if (!dockerServers.isEnabled('registry')) {
+      log.warning('skipping EPM Endpoints tests because registry docker server is disabled');
+      this.pending = true;
+    }
+
     loadTestFile(require.resolve('./list'));
     // loadTestFile(require.resolve('./file'));
     // temporarily disabled, see https://github.com/elastic/kibana/issues/67943