[data view mgmt] fix field refresh when index pattern is changed. (#1…

…50403) Data view management now updates the field list of an edited data view upon save. Closes: #150248
elastic · Feb 8, 2023 · a2c5bf8 · a2c5bf8
1 parent 447cbfd
commit a2c5bf8
Show file tree

Hide file tree

Showing 4 changed files with 48 additions and 6 deletions.
diff --git a/src/plugins/data_view_management/public/components/edit_index_pattern/edit_index_pattern.tsx b/src/plugins/data_view_management/public/components/edit_index_pattern/edit_index_pattern.tsx
@@ -128,7 +128,10 @@ export const EditIndexPattern = withRouter(
     const isRollup = new URLSearchParams(useLocation().search).get('type') === 'rollup';
     const displayIndexPatternEditor = showEditDialog ? (
       <IndexPatternEditor
-        onSave={() => setShowEditDialog(false)}
+        onSave={() => {
+          setFields(indexPattern.getNonScriptedFields());
+          setShowEditDialog(false);
+        }}
         onCancel={() => setShowEditDialog(false)}
         defaultTypeIsRollup={isRollup}
         editData={indexPattern}

diff --git a/...dit_index_pattern/indexed_fields_table/components/table/__snapshots__/table.test.tsx.snap b/...dit_index_pattern/indexed_fields_table/components/table/__snapshots__/table.test.tsx.snap
diff --git a/...ment/public/components/edit_index_pattern/indexed_fields_table/components/table/table.tsx b/...ment/public/components/edit_index_pattern/indexed_fields_table/components/table/table.tsx
@@ -220,7 +220,7 @@ const getItems = (conflictDescriptions: IndexedFieldItem['conflictDescriptions']
 };
 
 export const renderFieldName = (field: IndexedFieldItem, timeFieldName?: string) => (
-  <span>
+  <span data-test-subj={`field-name-${field.name}`}>
     {field.name}
     {field.info && field.info.length ? (
       <span>

diff --git a/test/functional/apps/management/_index_pattern_create_delete.ts b/test/functional/apps/management/_index_pattern_create_delete.ts
@@ -22,11 +22,20 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
   describe('creating and deleting default index', function describeIndexTests() {
     before(async function () {
       await esArchiver.emptyKibanaIndex();
+      await esArchiver.loadIfNeeded(
+        'test/functional/fixtures/es_archiver/kibana_sample_data_flights_index_pattern'
+      );
       await kibanaServer.uiSettings.replace({});
       await PageObjects.settings.navigateTo();
       await PageObjects.settings.clickKibanaIndexPatterns();
     });
 
+    after(async function () {
+      await esArchiver.unload(
+        'test/functional/fixtures/es_archiver/kibana_sample_data_flights_index_pattern'
+      );
+    });
+
     describe('can open and close editor', function () {
       it('without creating index pattern', async function () {
         await PageObjects.settings.clickKibanaIndexPatterns();
@@ -157,6 +166,28 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
       });
     });
 
+    describe('index pattern edit', function () {
+      it('should update field list', async function () {
+        await PageObjects.settings.editIndexPattern(
+          'kibana_sample_data_flights',
+          'timestamp',
+          undefined,
+          true
+        );
+
+        await retry.try(async () => {
+          // verify initial field list
+          expect(await testSubjects.exists('field-name-AvgTicketPrice')).to.be(true);
+        });
+
+        await PageObjects.settings.editIndexPattern('logstash-*', '@timestamp', undefined, true);
+        await retry.try(async () => {
+          // verify updated field list
+          expect(await testSubjects.exists('field-name-agent')).to.be(true);
+        });
+      });
+    });
+
     describe('index pattern deletion', function indexDelete() {
       before(function () {
         const expectedAlertText = 'Delete data view';