[CI] Auto-commit changed files from 'node scripts/eslint --no-cache -…

…-fix'
elastic · Dec 12, 2024 · 47dbd14 · 47dbd14
1 parent 4982f7f
commit 47dbd14
Show file tree

Hide file tree

Showing 81 changed files with 152 additions and 190 deletions.
diff --git a/packages/kbn-eslint-plugin-i18n/helpers/get_i18n_identifier_from_file_path.test.ts b/packages/kbn-eslint-plugin-i18n/helpers/get_i18n_identifier_from_file_path.test.ts
@@ -26,7 +26,10 @@ const testMap = [
     'xpack.synthetics',
   ],
   ['src/plugins/vis_types/gauge/public/editor/collections.ts', 'visTypeGauge'],
-  ['src/platform/packages/shared/kbn-alerts-ui-shared/src/alert_lifecycle_status_badge/index.tsx', 'alertsUIShared'],
+  [
+    'src/platform/packages/shared/kbn-alerts-ui-shared/src/alert_lifecycle_status_badge/index.tsx',
+    'alertsUIShared',
+  ],
 ];
 
 describe('Get i18n Identifier for file', () => {

diff --git a/packages/kbn-eslint-plugin-telemetry/helpers/get_app_name.test.ts b/packages/kbn-eslint-plugin-telemetry/helpers/get_app_name.test.ts
@@ -18,7 +18,10 @@ const testMap = [
   ['x-pack/plugins/observability/foo/bar/baz/header_actions.tsx', 'o11y'],
   ['x-pack/plugins/observability_solution/apm/baz/header_actions.tsx', 'apm'],
   ['x-pack/platform/plugins/shared/cases/public/components/foo.tsx', 'cases'],
-  ['src/platform/packages/shared/kbn-alerts-ui-shared/src/alert_lifecycle_status_badge/index.tsx', 'kbnAlertsUiShared'],
+  [
+    'src/platform/packages/shared/kbn-alerts-ui-shared/src/alert_lifecycle_status_badge/index.tsx',
+    'kbnAlertsUiShared',
+  ],
 ];
 
 describe('Get App Name', () => {

diff --git a/src/platform/packages/shared/kbn-alerts-ui-shared/jest.config.js b/src/platform/packages/shared/kbn-alerts-ui-shared/jest.config.js
@@ -11,5 +11,7 @@ module.exports = {
   preset: '@kbn/test',
   rootDir: '../../../../..',
   roots: ['<rootDir>/src/platform/packages/shared/kbn-alerts-ui-shared'],
-  setupFilesAfterEnv: ['<rootDir>/src/platform/packages/shared/kbn-alerts-ui-shared/setup_tests.ts'],
+  setupFilesAfterEnv: [
+    '<rootDir>/src/platform/packages/shared/kbn-alerts-ui-shared/setup_tests.ts',
+  ],
 };
diff --git a/src/platform/packages/shared/kbn-cases-components/jest.config.js b/src/platform/packages/shared/kbn-cases-components/jest.config.js
@@ -11,5 +11,7 @@ module.exports = {
   preset: '@kbn/test',
   rootDir: '../../../../..',
   roots: ['<rootDir>/src/platform/packages/shared/kbn-cases-components'],
-  setupFilesAfterEnv: ['<rootDir>/src/platform/packages/shared/kbn-cases-components/setup_tests.ts'],
+  setupFilesAfterEnv: [
+    '<rootDir>/src/platform/packages/shared/kbn-cases-components/setup_tests.ts',
+  ],
 };
diff --git a/x-pack/platform/plugins/shared/actions/common/index.ts b/x-pack/platform/plugins/shared/actions/common/index.ts
@@ -6,7 +6,6 @@
  */
 
 // TODO: https://github.com/elastic/kibana/issues/110895
-/* eslint-disable @kbn/eslint/no_export_all */
 
 export * from './types';
 export * from './alert_history_schema';

diff --git a/x-pack/platform/plugins/shared/actions/jest.config.js b/x-pack/platform/plugins/shared/actions/jest.config.js
@@ -11,5 +11,7 @@ module.exports = {
   roots: ['<rootDir>/x-pack/platform/plugins/shared/actions'],
   coverageDirectory: '<rootDir>/target/kibana-coverage/jest/x-pack/platform/plugins/shared/actions',
   coverageReporters: ['text', 'html'],
-  collectCoverageFrom: ['<rootDir>/x-pack/platform/plugins/shared/actions/{common,server}/**/*.{js,ts,tsx}'],
+  collectCoverageFrom: [
+    '<rootDir>/x-pack/platform/plugins/shared/actions/{common,server}/**/*.{js,ts,tsx}',
+  ],
 };
diff --git a/x-pack/platform/plugins/shared/actions/server/constants/plugin.ts b/x-pack/platform/plugins/shared/actions/server/constants/plugin.ts
@@ -10,7 +10,7 @@ import { LicenseType } from '@kbn/licensing-plugin/server';
 export const PLUGIN = {
   ID: 'actions',
   MINIMUM_LICENSE_REQUIRED: 'basic' as LicenseType, // TODO: supposed to be changed up on requirements
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
   getI18nName: (i18n: any): string =>
     i18n.translate('xpack.actions.appName', {
       defaultMessage: 'Actions',

diff --git a/...k/platform/plugins/shared/actions/server/integration_tests/axios_utils_connection.test.ts b/...k/platform/plugins/shared/actions/server/integration_tests/axios_utils_connection.test.ts
@@ -61,7 +61,7 @@ const AuthB64 = Buffer.from(Auth).toString('base64');
 
 describe('axios connections', () => {
   let testServer: http.Server | https.Server | null;
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
   let savedAxiosDefaultsAdapter: any;
 
   beforeEach(() => {

diff --git a/x-pack/platform/plugins/shared/actions/server/integration_tests/axios_utils_proxy.test.ts b/x-pack/platform/plugins/shared/actions/server/integration_tests/axios_utils_proxy.test.ts
@@ -48,7 +48,7 @@ const ServerResponse = 'A unique response returned by the server!';
 
 describe('axios connections', () => {
   let testServer: http.Server | https.Server | null;
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
   let savedAxiosDefaultsAdapter: any;
 
   beforeEach(() => {

diff --git a/x-pack/platform/plugins/shared/actions/server/lib/ensure_sufficient_license.test.ts b/x-pack/platform/plugins/shared/actions/server/lib/ensure_sufficient_license.test.ts
@@ -65,7 +65,7 @@ describe('ensureSufficientLicense()', () => {
       ensureSufficientLicense({
         ...sampleActionType,
         // we're faking an invalid value, this requires stripping the typing
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
         minimumLicenseRequired: 'foo' as any,
       })
     ).toThrowErrorMatchingInlineSnapshot(`"\\"foo\\" is not a valid license type"`);

diff --git a/x-pack/platform/plugins/shared/actions/server/lib/mustache_renderer.ts b/x-pack/platform/plugins/shared/actions/server/lib/mustache_renderer.ts
@@ -113,7 +113,7 @@ function addToStringDeep(object: unknown): void {
   // walk arrays, but don't add a toString() as mustache already does something
   if (Array.isArray(object)) {
     // instead, add an asJSON()
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
     (object as any).asJSON = () => JSON.stringify(object);
     object.forEach((element) => addToStringDeep(element));
     return;

diff --git a/x-pack/platform/plugins/shared/actions/server/plugin.test.ts b/x-pack/platform/plugins/shared/actions/server/plugin.test.ts
@@ -298,7 +298,7 @@ describe('Actions Plugin', () => {
 
       beforeEach(async () => {
         // coreMock.createSetup doesn't support Plugin generics
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
         setup = await plugin.setup(coreSetup as any, pluginsSetup);
       });
 
@@ -307,7 +307,7 @@ describe('Actions Plugin', () => {
           setup.registerType({
             ...sampleActionType,
             // we're faking an invalid value, this requires stripping the typing
-            // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
             minimumLicenseRequired: 'foo' as any,
           })
         ).toThrowErrorMatchingInlineSnapshot(`"\\"foo\\" is not a valid license type"`);
@@ -358,7 +358,7 @@ describe('Actions Plugin', () => {
       it('should correctly return whether connector is preconfigured', async () => {
         setup(getConfig());
         // coreMock.createSetup doesn't support Plugin generics
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
         const pluginSetup = await plugin.setup(coreSetup as any, pluginsSetup);
 
         expect(pluginSetup.isPreconfiguredConnector('preconfiguredServerLog')).toEqual(true);
@@ -386,7 +386,7 @@ describe('Actions Plugin', () => {
       it('should set connector type enabled', async () => {
         setup(getConfig());
         // coreMock.createSetup doesn't support Plugin generics
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
         const pluginSetup = await plugin.setup(coreSetup as any, pluginsSetup);
         const coreStart = coreMock.createStart();
         const pluginsStart = {
@@ -429,7 +429,7 @@ describe('Actions Plugin', () => {
       it('should set connector type enabled and check isActionTypeEnabled with plugin setup method', async () => {
         setup(getConfig());
         // coreMock.createSetup doesn't support Plugin generics
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
         const pluginSetup = await plugin.setup(coreSetup as any, pluginsSetup);
 
         pluginSetup.registerType({
@@ -466,7 +466,7 @@ describe('Actions Plugin', () => {
       it('should set all the connector types enabled when null or ["*"] passed', async () => {
         setup(getConfig());
         // coreMock.createSetup doesn't support Plugin generics
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
         const pluginSetup = await plugin.setup(coreSetup as any, pluginsSetup);
         const coreStart = coreMock.createStart();
         const pluginsStart = {
@@ -509,7 +509,7 @@ describe('Actions Plugin', () => {
       it('should set all the connector types disabled when [] passed', async () => {
         setup(getConfig());
         // coreMock.createSetup doesn't support Plugin generics
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
         const pluginSetup = await plugin.setup(coreSetup as any, pluginsSetup);
         const coreStart = coreMock.createStart();
         const pluginsStart = {
@@ -552,7 +552,7 @@ describe('Actions Plugin', () => {
       it('should throw if the enabledActionTypes is already set by the config', async () => {
         setup({ ...getConfig(), enabledActionTypes: ['.email'] });
         // coreMock.createSetup doesn't support Plugin generics
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
         const pluginSetup = await plugin.setup(coreSetup as any, pluginsSetup);
 
         expect(() => pluginSetup.setEnabledConnectorTypes(['.index'])).toThrow(
@@ -702,7 +702,7 @@ describe('Actions Plugin', () => {
         it('should handle preconfigured actions', async () => {
           setup(getConfig());
           // coreMock.createSetup doesn't support Plugin generics
-          // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
           const pluginSetup = await plugin.setup(coreSetup as any, pluginsSetup);
           pluginSetup.registerType({
             id: '.server-log',
@@ -728,7 +728,6 @@ describe('Actions Plugin', () => {
         it('should handle preconfiguredAlertHistoryEsIndex = true', async () => {
           setup(getConfig({ preconfiguredAlertHistoryEsIndex: true }));
 
-          // eslint-disable-next-line @typescript-eslint/no-explicit-any
           const pluginSetup = await plugin.setup(coreSetup as any, pluginsSetup);
           pluginSetup.registerType({
             id: '.index',
@@ -765,7 +764,7 @@ describe('Actions Plugin', () => {
             })
           );
           // coreMock.createSetup doesn't support Plugin generics
-          // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
           await plugin.setup(coreSetup as any, pluginsSetup);
           const pluginStart = await plugin.start(coreStart, pluginsStart);
 
@@ -780,7 +779,7 @@ describe('Actions Plugin', () => {
         it('should set system actions correctly', async () => {
           setup(getConfig());
           // coreMock.createSetup doesn't support Plugin generics
-          // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
           const pluginSetup = await plugin.setup(coreSetup as any, pluginsSetup);
 
           const platinumLicense = licensingMock.createLicense({
@@ -848,7 +847,7 @@ describe('Actions Plugin', () => {
           );
 
           // coreMock.createSetup doesn't support Plugin generics
-          // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
           const pluginSetup = await plugin.setup(coreSetup as any, pluginsSetup);
 
           pluginSetup.registerType({
@@ -890,7 +889,7 @@ describe('Actions Plugin', () => {
 
       it('passes through the notifyUsage option when set to true', async () => {
         // coreMock.createSetup doesn't support Plugin generics
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
         const pluginSetup = await plugin.setup(coreSetup as any, pluginsSetup);
         pluginSetup.registerType(actionType);
         const pluginStart = plugin.start(coreStart, pluginsStart);
@@ -918,7 +917,7 @@ describe('Actions Plugin', () => {
 
       it('passes through the notifyUsage option when set to true', async () => {
         // coreMock.createSetup doesn't support Plugin generics
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
         const pluginSetup = await plugin.setup(coreSetup as any, pluginsSetup);
         pluginSetup.registerType(actionType);
         const pluginStart = plugin.start(coreStart, pluginsStart);
@@ -933,7 +932,7 @@ describe('Actions Plugin', () => {
     describe('getActionsHealth()', () => {
       it('should return hasPermanentEncryptionKey false if canEncrypt of encryptedSavedObjects is false', async () => {
         // coreMock.createSetup doesn't support Plugin generics
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
         const pluginSetup = await plugin.setup(coreSetup as any, pluginsSetup);
         expect(pluginSetup.getActionsHealth()).toEqual({ hasPermanentEncryptionKey: false });
       });
@@ -952,7 +951,7 @@ describe('Actions Plugin', () => {
     describe('isSystemActionConnector()', () => {
       it('should return true if the connector is a system connector', async () => {
         // coreMock.createSetup doesn't support Plugin generics
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
         const pluginSetup = await plugin.setup(coreSetup as any, pluginsSetup);
 
         pluginSetup.registerType({
@@ -975,7 +974,7 @@ describe('Actions Plugin', () => {
 
       it('should return false if the connector is not a system connector', async () => {
         // coreMock.createSetup doesn't support Plugin generics
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
         const pluginSetup = await plugin.setup(coreSetup as any, pluginsSetup);
 
         pluginSetup.registerType({

diff --git a/x-pack/platform/plugins/shared/actions/server/types.ts b/x-pack/platform/plugins/shared/actions/server/types.ts
@@ -228,7 +228,7 @@ export interface RawAction extends Record<string, unknown> {
 export interface ActionTaskParams extends SavedObjectAttributes {
   actionId: string;
   // Saved Objects won't allow us to enforce unknown rather than any
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
   params: Record<string, any>;
   apiKey?: string;
   executionId?: string;

diff --git a/x-pack/platform/plugins/shared/alerting/common/constants/plugin.ts b/x-pack/platform/plugins/shared/alerting/common/constants/plugin.ts
@@ -11,7 +11,7 @@ export const PLUGIN = {
   ID: 'alerting',
   MINIMUM_LICENSE_REQUIRED: 'basic' as LicenseType, // TODO: supposed to be changed up on requirements
   // all plugins seem to use getI18nName with any
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
   getI18nName: (i18n: any): string =>
     i18n.translate('xpack.alerting.appName', {
       defaultMessage: 'Alerting',

diff --git a/x-pack/platform/plugins/shared/alerting/common/index.ts b/x-pack/platform/plugins/shared/alerting/common/index.ts
@@ -6,7 +6,7 @@
  */
 
 // TODO: https://github.com/elastic/kibana/issues/110895
-/* eslint-disable @kbn/eslint/no_export_all */
+
 export * from './rule';
 export * from './rules_settings';
 export * from './rule_type';

diff --git a/x-pack/platform/plugins/shared/alerting/jest.config.js b/x-pack/platform/plugins/shared/alerting/jest.config.js
@@ -9,7 +9,10 @@ module.exports = {
   preset: '@kbn/test',
   rootDir: '../../../../..',
   roots: ['<rootDir>/x-pack/platform/plugins/shared/alerting'],
-  coverageDirectory: '<rootDir>/target/kibana-coverage/jest/x-pack/platform/plugins/shared/alerting',
+  coverageDirectory:
+    '<rootDir>/target/kibana-coverage/jest/x-pack/platform/plugins/shared/alerting',
   coverageReporters: ['text', 'html'],
-  collectCoverageFrom: ['<rootDir>/x-pack/platform/plugins/shared/alerting/{common,public,server}/**/*.{ts,tsx}'],
+  collectCoverageFrom: [
+    '<rootDir>/x-pack/platform/plugins/shared/alerting/{common,public,server}/**/*.{ts,tsx}',
+  ],
 };
diff --git a/x-pack/platform/plugins/shared/alerting/server/alerts_client/lib/alert_conflict_resolver.ts b/x-pack/platform/plugins/shared/alerting/server/alerts_client/lib/alert_conflict_resolver.ts
@@ -160,7 +160,7 @@ async function refreshFieldsInDocs(
 
     // @ts-expect-error @elastic/elasticsearch _source is not in the type!
     const freshDoc = freshResponse._source;
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
     const conflictDoc = conflictRequest.doc as Record<string, any>;
     if (!freshDoc || !conflictDoc) continue;
 

diff --git a/x-pack/platform/plugins/shared/alerting/server/alerts_service/alerts_service.test.ts b/x-pack/platform/plugins/shared/alerting/server/alerts_service/alerts_service.test.ts
@@ -2054,13 +2054,11 @@ describe('Alerts Service', () => {
           // leverage the outcome of the first retry
           expect(
             logger.info.mock.calls.filter(
-              // eslint-disable-next-line @typescript-eslint/no-explicit-any
               (calls: any[]) => calls[0] === `Retrying resource initialization for context "test"`
             ).length
           ).toEqual(1);
           expect(
             logger.info.mock.calls.filter(
-              // eslint-disable-next-line @typescript-eslint/no-explicit-any
               (calls: any[]) =>
                 calls[0] === `Resource installation for "test" succeeded after retry`
             ).length
@@ -2133,7 +2131,6 @@ describe('Alerts Service', () => {
           // Should only log the retry once because the second and third retries should be throttled
           expect(
             logger.info.mock.calls.filter(
-              // eslint-disable-next-line @typescript-eslint/no-explicit-any
               (calls: any[]) => calls[0] === `Retrying resource initialization for context "test"`
             ).length
           ).toEqual(1);

diff --git a/...tform/plugins/shared/alerting/server/alerts_service/lib/retry_transient_es_errors.test.ts b/...tform/plugins/shared/alerting/server/alerts_service/lib/retry_transient_es_errors.test.ts
@@ -66,16 +66,16 @@ describe('retryTransientErrors', () => {
     const errors = [
       new EsErrors.NoLivingConnectionsError('no living connection', {
         warnings: [],
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
         meta: {} as any,
       }),
       new EsErrors.ConnectionError('no connection'),
       new EsErrors.TimeoutError('timeout'),
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
       new EsErrors.ResponseError({ statusCode: 503, meta: {} as any, warnings: [] }),
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
       new EsErrors.ResponseError({ statusCode: 408, meta: {} as any, warnings: [] }),
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
       new EsErrors.ResponseError({ statusCode: 410, meta: {} as any, warnings: [] }),
     ];