From 36114c6ace37f2b5fe4042878a3685c7919cf887 Mon Sep 17 00:00:00 2001
From: Jason Rhodes <jason.matthew.rhodes@gmail.com>
Date: Fri, 26 Oct 2018 10:03:48 -0400
Subject: [PATCH] Skipping a Kibana test to mirror a fix going into master in
 PR #24650

---
 .../apis/monitoring/common/mappings_exist.js  | 33 +++++++++++--------
 1 file changed, 19 insertions(+), 14 deletions(-)

diff --git a/x-pack/test/api_integration/apis/monitoring/common/mappings_exist.js b/x-pack/test/api_integration/apis/monitoring/common/mappings_exist.js
index 8d78a150f28c216..653a49d3c586188 100644
--- a/x-pack/test/api_integration/apis/monitoring/common/mappings_exist.js
+++ b/x-pack/test/api_integration/apis/monitoring/common/mappings_exist.js
@@ -12,38 +12,38 @@ import * as logstashMetrics from '../../../../../plugins/monitoring/server/lib/m
 import * as beatsMetrics from '../../../../../plugins/monitoring/server/lib/metrics/beats/metrics';
 import * as apmMetrics from '../../../../../plugins/monitoring/server/lib/metrics/apm/metrics';
 
-export default function ({ getService }) {
+export default function({ getService }) {
   const es = getService('es');
 
   const metricSets = [
     {
       metrics: esMetrics.metrics,
       name: 'es metrics',
-      indexTemplate: '.monitoring-es'
+      indexTemplate: '.monitoring-es',
     },
     {
       metrics: kibanaMetrics.metrics,
       name: 'kibana metrics',
-      indexTemplate: '.monitoring-kibana'
+      indexTemplate: '.monitoring-kibana',
     },
     {
       metrics: logstashMetrics.metrics,
       name: 'logstash metrics',
-      indexTemplate: '.monitoring-logstash'
+      indexTemplate: '.monitoring-logstash',
     },
     {
       metrics: beatsMetrics.metrics,
       name: 'beats metrics',
-      indexTemplate: '.monitoring-beats'
+      indexTemplate: '.monitoring-beats',
     },
     {
       metrics: apmMetrics.metrics,
       name: 'apm metrics',
-      indexTemplate: '.monitoring-beats' // apm uses the same as beats
+      indexTemplate: '.monitoring-beats', // apm uses the same as beats
     },
   ];
 
-  describe('mappings', () => {
+  describe.skip('mappings', () => {
     for (const { indexTemplate, metrics, name } of metricSets) {
       let mappings;
 
@@ -52,15 +52,20 @@ export default function ({ getService }) {
         mappings = get(template, [indexTemplate, 'mappings', 'doc', 'properties']);
       });
 
-      describe(`for ${name}`, () => { // eslint-disable-line no-loop-func
+      describe(`for ${name}`, () => {
+        // eslint-disable-line no-loop-func
         for (const metric of Object.values(metrics)) {
           for (const field of metric.getFields()) {
-            it(`${field} should exist in the mappings`, () => { // eslint-disable-line no-loop-func
-              const propertyGetter = field.split('.').reduce((list, field) => {
-                list.push(field);
-                list.push('properties');
-                return list;
-              }, []).slice(0, -1); // Remove the trailing 'properties'
+            it(`${field} should exist in the mappings`, () => {
+              // eslint-disable-line no-loop-func
+              const propertyGetter = field
+                .split('.')
+                .reduce((list, field) => {
+                  list.push(field);
+                  list.push('properties');
+                  return list;
+                }, [])
+                .slice(0, -1); // Remove the trailing 'properties'
 
               const foundMapping = get(mappings, propertyGetter, null);
               expect(foundMapping).to.not.equal(null);