Rename values also

elastic · Oct 13, 2021 · 6905991 · 6905991
1 parent 00d307d
commit 6905991
Show file tree

Hide file tree

Showing 9 changed files with 22 additions and 22 deletions.
diff --git a/...jest__/client_integration/overview/migrate_system_indices/migrate_system_indices.test.tsx b/...jest__/client_integration/overview/migrate_system_indices/migrate_system_indices.test.tsx
@@ -45,7 +45,7 @@ describe('Overview - Migrate system indices', () => {
       component.update();
 
       httpRequestsMockHelpers.setLoadSystemIndicesMigrationStatus({
-        migration_status: 'NO_UPGRADE_NEEDED',
+        migration_status: 'NO_MIGRATION_NEEDED',
       });
 
       await actions.clickRetrySystemIndicesButton();
@@ -56,7 +56,7 @@ describe('Overview - Migrate system indices', () => {
 
   test('No migration needed', async () => {
     httpRequestsMockHelpers.setLoadSystemIndicesMigrationStatus({
-      migration_status: 'NO_UPGRADE_NEEDED',
+      migration_status: 'NO_MIGRATION_NEEDED',
     });
 
     testBed = await setupOverviewPage();
@@ -91,7 +91,7 @@ describe('Overview - Migrate system indices', () => {
   describe('Migration needed', () => {
     test('Initial state', async () => {
       httpRequestsMockHelpers.setLoadSystemIndicesMigrationStatus({
-        migration_status: 'UPGRADE_NEEDED',
+        migration_status: 'MIGRATION_NEEDED',
       });
 
       testBed = await setupOverviewPage();
@@ -109,7 +109,7 @@ describe('Overview - Migrate system indices', () => {
 
     test('Handles errors when migrating', async () => {
       httpRequestsMockHelpers.setLoadSystemIndicesMigrationStatus({
-        migration_status: 'UPGRADE_NEEDED',
+        migration_status: 'MIGRATION_NEEDED',
       });
       httpRequestsMockHelpers.setSystemIndicesMigrationResponse(undefined, {
         statusCode: 400,

diff --git a/...ns/upgrade_assistant/__jest__/client_integration/overview/migrate_system_indices/mocks.ts b/...ns/upgrade_assistant/__jest__/client_integration/overview/migrate_system_indices/mocks.ts
@@ -8,7 +8,7 @@
 import { SystemIndicesMigrationStatus } from '../../../../common/types';
 
 export const systemIndicesMigrationStatus: SystemIndicesMigrationStatus = {
-  migration_status: 'UPGRADE_NEEDED',
+  migration_status: 'MIGRATION_NEEDED',
   features: [
     {
       feature_name: 'security',
@@ -35,7 +35,7 @@ export const systemIndicesMigrationStatus: SystemIndicesMigrationStatus = {
     {
       feature_name: 'kibana',
       minimum_index_version: '7.1.3',
-      migration_status: 'UPGRADE_NEEDED',
+      migration_status: 'MIGRATION_NEEDED',
       indices: [
         {
           index: '.kibana',
@@ -46,7 +46,7 @@ export const systemIndicesMigrationStatus: SystemIndicesMigrationStatus = {
     {
       feature_name: 'logstash',
       minimum_index_version: '7.1.4',
-      migration_status: 'NO_UPGRADE_NEEDED',
+      migration_status: 'NO_MIGRATION_NEEDED',
       indices: [
         {
           index: '.logstash-config',

diff --git a/...stant/__jest__/client_integration/overview/migrate_system_indices/step_completion.test.ts b/...stant/__jest__/client_integration/overview/migrate_system_indices/step_completion.test.ts
@@ -21,7 +21,7 @@ describe('Overview - Migrate system indices - Step completion', () => {
 
   test(`It's complete when no upgrade is needed`, async () => {
     httpRequestsMockHelpers.setLoadSystemIndicesMigrationStatus({
-      migration_status: 'NO_UPGRADE_NEEDED',
+      migration_status: 'NO_MIGRATION_NEEDED',
     });
 
     await act(async () => {
@@ -37,7 +37,7 @@ describe('Overview - Migrate system indices - Step completion', () => {
 
   test(`It's incomplete when migration is needed`, async () => {
     httpRequestsMockHelpers.setLoadSystemIndicesMigrationStatus({
-      migration_status: 'UPGRADE_NEEDED',
+      migration_status: 'MIGRATION_NEEDED',
     });
 
     await act(async () => {
@@ -73,7 +73,7 @@ describe('Overview - Migrate system indices - Step completion', () => {
       expect(exists('migrateSystemIndicesStep-incomplete')).toBe(true);
 
       httpRequestsMockHelpers.setLoadSystemIndicesMigrationStatus({
-        migration_status: 'NO_UPGRADE_NEEDED',
+        migration_status: 'NO_MIGRATION_NEEDED',
       });
 
       // Resolve the polling timeout.

diff --git a/x-pack/plugins/upgrade_assistant/common/types.ts b/x-pack/plugins/upgrade_assistant/common/types.ts
@@ -254,7 +254,7 @@ export interface DeprecationLoggingStatus {
   isDeprecationLoggingEnabled: boolean;
 }
 
-export type MIGRATION_STATUS = 'UPGRADE_NEEDED' | 'NO_UPGRADE_NEEDED' | 'IN_PROGRESS' | 'ERROR';
+export type MIGRATION_STATUS = 'MIGRATION_NEEDED' | 'NO_MIGRATION_NEEDED' | 'IN_PROGRESS' | 'ERROR';
 export interface SystemIndicesMigrationFeature {
   id?: string;
   feature_name: string;

diff --git a/...pgrade_assistant/public/application/components/overview/migrate_system_indices/flyout.tsx b/...pgrade_assistant/public/application/components/overview/migrate_system_indices/flyout.tsx
@@ -85,7 +85,7 @@ const i18nTexts = {
 };
 
 const renderMigrationStatus = (status: MIGRATION_STATUS) => {
-  if (status === 'NO_UPGRADE_NEEDED') {
+  if (status === 'NO_MIGRATION_NEEDED') {
     return (
       <EuiFlexGroup alignItems="center" gutterSize="s">
         <EuiFlexItem grow={false}>
@@ -100,7 +100,7 @@ const renderMigrationStatus = (status: MIGRATION_STATUS) => {
     );
   }
 
-  if (status === 'UPGRADE_NEEDED') {
+  if (status === 'MIGRATION_NEEDED') {
     return (
       <EuiText size="s" data-test-subj="featureUpgradeNeeded">
         <p>{i18nTexts.needsMigrationLabel}</p>

diff --git a/.../public/application/components/overview/migrate_system_indices/migrate_system_indices.tsx b/.../public/application/components/overview/migrate_system_indices/migrate_system_indices.tsx
@@ -83,7 +83,7 @@ const MigrateSystemIndicesStep: FunctionComponent<Props> = ({ setIsComplete }) =
     useMigrateSystemIndices();
 
   useEffect(() => {
-    setIsComplete(migrationStatus.data?.migration_status === 'NO_UPGRADE_NEEDED');
+    setIsComplete(migrationStatus.data?.migration_status === 'NO_MIGRATION_NEEDED');
     // Depending upon setIsComplete would create an infinite loop.
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [migrationStatus.data?.migration_status]);
@@ -111,7 +111,7 @@ const MigrateSystemIndicesStep: FunctionComponent<Props> = ({ setIsComplete }) =
     );
   }
 
-  if (migrationStatus.data?.migration_status === 'NO_UPGRADE_NEEDED') {
+  if (migrationStatus.data?.migration_status === 'NO_MIGRATION_NEEDED') {
     return (
       <EuiFlexGroup alignItems="center" gutterSize="s" data-test-subj="noMigrationNeededSection">
         <EuiFlexItem grow={false}>

diff --git a/x-pack/plugins/upgrade_assistant/server/lib/es_deprecations_status.test.ts b/x-pack/plugins/upgrade_assistant/server/lib/es_deprecations_status.test.ts
@@ -46,7 +46,7 @@ describe('getESUpgradeStatus', () => {
         {
           feature_name: 'machine_learning',
           minimum_index_version: '7.1.1',
-          migration_status: 'UPGRADE_NEEDED',
+          migration_status: 'MIGRATION_NEEDED',
           indices: [
             {
               index: '.ml-config',
@@ -55,7 +55,7 @@ describe('getESUpgradeStatus', () => {
           ],
         },
       ],
-      migration_status: 'UPGRADE_NEEDED',
+      migration_status: 'MIGRATION_NEEDED',
     })
   );
 

diff --git a/x-pack/plugins/upgrade_assistant/server/lib/es_system_indices_migration.test.ts b/x-pack/plugins/upgrade_assistant/server/lib/es_system_indices_migration.test.ts
@@ -13,7 +13,7 @@ const esUpgradeSystemIndicesStatusMock: SystemIndicesMigrationStatus = {
     {
       feature_name: 'machine_learning',
       minimum_index_version: '7.1.1',
-      migration_status: 'UPGRADE_NEEDED',
+      migration_status: 'MIGRATION_NEEDED',
       indices: [
         {
           index: '.ml-config',
@@ -28,7 +28,7 @@ const esUpgradeSystemIndicesStatusMock: SystemIndicesMigrationStatus = {
     {
       feature_name: 'security',
       minimum_index_version: '7.1.1',
-      migration_status: 'UPGRADE_NEEDED',
+      migration_status: 'MIGRATION_NEEDED',
       indices: [
         {
           index: '.ml-config',
@@ -37,7 +37,7 @@ const esUpgradeSystemIndicesStatusMock: SystemIndicesMigrationStatus = {
       ],
     },
   ],
-  migration_status: 'UPGRADE_NEEDED',
+  migration_status: 'MIGRATION_NEEDED',
 };
 
 describe('convertFeaturesToIndicesArray', () => {

diff --git a/x-pack/plugins/upgrade_assistant/server/routes/system_indices_migration.test.ts b/x-pack/plugins/upgrade_assistant/server/routes/system_indices_migration.test.ts
@@ -21,7 +21,7 @@ const mockedResponse = {
     {
       feature_name: 'security',
       minimum_index_version: '7.1.1',
-      migration_status: 'NO_UPGRADE_NEEDED',
+      migration_status: 'NO_MIGRATION_NEEDED',
       indices: [
         {
           index: '.security-7',
@@ -30,7 +30,7 @@ const mockedResponse = {
       ],
     },
   ],
-  migration_status: 'UPGRADE_NEEDED',
+  migration_status: 'MIGRATION_NEEDED',
 };
 
 /**