autofixes, round 2

elastic · Dec 16, 2019 · 0414339 · 0414339
1 parent 5da6e82
commit 0414339
Show file tree

Hide file tree

Showing 13 changed files with 34 additions and 19 deletions.
diff --git a/packages/kbn-ui-framework/doc_site/src/components/guide_page/guide_page.js b/packages/kbn-ui-framework/doc_site/src/components/guide_page/guide_page.js
@@ -31,9 +31,9 @@ export class GuidePage extends Component {
 
   onClickLink(id) {
     // Scroll to element.
+    // eslint-disable-next-line no-undef
     $('html, body').animate(
       {
-        // eslint-disable-line no-undef
         scrollTop: $(`#${id}`).offset().top - 100, // eslint-disable-line no-undef
       },
       250

diff --git a/...c/directives/timelion_expression_suggestions/__tests__/timelion_expression_suggestions.js b/...c/directives/timelion_expression_suggestions/__tests__/timelion_expression_suggestions.js
@@ -27,11 +27,12 @@ describe('Timelion expression suggestions directive', function() {
 
   beforeEach(ngMock.module('kibana'));
 
-  beforeEach(inject(function($injector) {
-    // eslint-disable-line no-undef
-    $compile = $injector.get('$compile');
-    scope = $injector.get('$rootScope').$new();
-  }));
+  beforeEach(
+    ngMock.inject(function($injector) {
+      $compile = $injector.get('$compile');
+      scope = $injector.get('$rootScope').$new();
+    })
+  );
 
   describe('attributes', function() {
     describe('suggestions', function() {

diff --git a/src/legacy/server/http/integration_tests/xsrf.test.js b/src/legacy/server/http/integration_tests/xsrf.test.js
@@ -110,8 +110,8 @@ describe('xsrf request filter', () => {
   });
 
   for (const method of destructiveMethods) {
+    // eslint-disable-next-line no-loop-func
     describe(`destructiveMethod: ${method}`, function() {
-      // eslint-disable-line no-loop-func
       it('accepts requests with the xsrf header', async function() {
         await kbnTestServer.request[method.toLowerCase()](root, testPath)
           .set(xsrfHeader, 'anything')

diff --git a/src/legacy/server/logging/log_with_metadata.js b/src/legacy/server/logging/log_with_metadata.js
@@ -18,10 +18,10 @@
  */
 import { isPlainObject } from 'lodash';
 
-// eslint-disable-next-line @kbn/eslint/no-restricted-paths
 import {
   metadataSymbol,
   attachMetaData,
+  // eslint-disable-next-line @kbn/eslint/no-restricted-paths
 } from '../../../../src/core/server/legacy/logging/legacy_logging_server';
 
 export const logWithMetadata = {

diff --git a/...cy/plugins/cross_cluster_replication/common/services/auto_follow_pattern_serialization.js b/...cy/plugins/cross_cluster_replication/common/services/auto_follow_pattern_serialization.js
@@ -5,9 +5,19 @@
  */
 
 export const deserializeAutoFollowPattern = (
-  { name, pattern: { remote_cluster, leader_index_patterns, follow_index_pattern } } = {
+  {
+    name,
+    pattern: {
+      // eslint-disable-next-line camelcase
+      remote_cluster,
+      // eslint-disable-next-line camelcase
+      leader_index_patterns,
+      // eslint-disable-next-line camelcase
+      follow_index_pattern,
+    },
+  } = {
     pattern: {},
-  } // eslint-disable-line camelcase
+  }
 ) => ({
   name,
   remoteCluster: remote_cluster,

diff --git a/...ross_cluster_replication/public/app/components/follower_index_form/follower_index_form.js b/...ross_cluster_replication/public/app/components/follower_index_form/follower_index_form.js
@@ -117,8 +117,10 @@ export class FollowerIndexForm extends PureComponent {
           ...getEmptyFollowerIndex(),
           ...this.props.followerIndex,
         };
+
+    // eslint-disable-next-line no-nested-ternary
     const areAdvancedSettingsVisible = isNew
-      ? false // eslint-disable-line no-nested-ternary
+      ? false
       : areAdvancedSettingsEdited(followerIndex)
       ? true
       : false;

diff --git a/x-pack/legacy/plugins/cross_cluster_replication/server/lib/error_wrappers/wrap_es_error.js b/x-pack/legacy/plugins/cross_cluster_replication/server/lib/error_wrappers/wrap_es_error.js
@@ -13,8 +13,8 @@ function extractCausedByChain(causedBy = {}, accumulator = []) {
     accumulator.push(reason);
   }
 
+  // eslint-disable-next-line camelcase
   if (caused_by) {
-    // eslint-disable-line camelcase
     return extractCausedByChain(caused_by, accumulator);
   }
 

diff --git a/...gins/monitoring/public/components/elasticsearch/shard_allocation/components/table_head.js b/...gins/monitoring/public/components/elasticsearch/shard_allocation/components/table_head.js
@@ -48,9 +48,8 @@ class IndexLabel extends React.Component {
   }
 }
 
+// eslint-disable-next-line react/no-multi-comp
 export class TableHead extends React.Component {
-  // eslint-disable-line react/no-multi-comp
-
   constructor(props) {
     super(props);
   }

diff --git a/x-pack/legacy/plugins/monitoring/server/lib/elasticsearch/convert_metric_names.js b/x-pack/legacy/plugins/monitoring/server/lib/elasticsearch/convert_metric_names.js
@@ -55,11 +55,14 @@ export function uncovertMetricNames(byDateBucketResponse) {
     unconverted[metricName] = {
       buckets: byDateBucketResponse.buckets.map(bucket => {
         const {
+          // eslint-disable-next-line camelcase
           key_as_string,
+          // eslint-disable-next-line camelcase
           key,
+          // eslint-disable-next-line camelcase
           doc_count,
           ...rest
-        } = bucket; /* eslint-disable-line camelcase */
+        } = bucket;
         const metrics = Object.entries(rest).reduce((accum, [key, value]) => {
           if (key.startsWith(`${CONVERTED_TOKEN}${metricName}`)) {
             const name = key.split('__')[1];

diff --git a/x-pack/legacy/plugins/rollup/public/crud_app/app.js b/x-pack/legacy/plugins/rollup/public/crud_app/app.js
@@ -39,8 +39,8 @@ class ShareRouterComponent extends Component {
 
 const ShareRouter = withRouter(ShareRouterComponent);
 
+// eslint-disable-next-line react/no-multi-comp
 export class App extends Component {
-  // eslint-disable-line react/no-multi-comp
   componentDidMount() {
     trackUiMetric(METRIC_TYPE.LOADED, UIM_APP_LOAD);
   }

diff --git a/x-pack/legacy/plugins/rollup/server/lib/error_wrappers/wrap_es_error.js b/x-pack/legacy/plugins/rollup/server/lib/error_wrappers/wrap_es_error.js
@@ -13,8 +13,8 @@ function extractCausedByChain(causedBy = {}, accumulator = []) {
     accumulator.push(reason);
   }
 
+  // eslint-disable-next-line camelcase
   if (caused_by) {
-    // eslint-disable-line camelcase
     return extractCausedByChain(caused_by, accumulator);
   }
 

diff --git a/x-pack/legacy/server/lib/create_router/error_wrappers/wrap_es_error.js b/x-pack/legacy/server/lib/create_router/error_wrappers/wrap_es_error.js
@@ -13,8 +13,8 @@ function extractCausedByChain(causedBy = {}, accumulator = []) {
     accumulator.push(reason);
   }
 
+  // eslint-disable-next-line camelcase
   if (caused_by) {
-    // eslint-disable-line camelcase
     return extractCausedByChain(caused_by, accumulator);
   }
 

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
@@ -56,8 +56,8 @@ export default function({ getService }) {
         // eslint-disable-line no-loop-func
         for (const metric of Object.values(metrics)) {
           for (const field of metric.getFields()) {
+            // eslint-disable-next-line no-loop-func
             it(`${field} should exist in the mappings`, () => {
-              // eslint-disable-line no-loop-func
               const propertyGetter = field
                 .split('.')
                 .reduce((list, field) => {