[ML] Fix bug with fields.yml not defined, add jest tests for new tran…

…sform installation mechanism in Fleet (#142159) Co-authored-by: Kibana Machine <[email protected]>
elastic · Oct 5, 2022 · a3e2af4 · a3e2af4
1 parent d3370cf
commit a3e2af4
Show file tree

Hide file tree

Showing 3 changed files with 677 additions and 4 deletions.
diff --git a/x-pack/plugins/fleet/server/services/epm/elasticsearch/transform/install.ts b/x-pack/plugins/fleet/server/services/epm/elasticsearch/transform/install.ts
@@ -261,9 +261,10 @@ const installTransformsAssets = async (
     await Promise.all(
       destinationIndexTemplates
         .map((destinationIndexTemplate) => {
-          const customMappings = transformsSpecifications
-            .get(destinationIndexTemplate.transformModuleId)
-            ?.get('mappings');
+          const customMappings =
+            transformsSpecifications
+              .get(destinationIndexTemplate.transformModuleId)
+              ?.get('mappings') ?? {};
           const registryElasticsearch: RegistryElasticsearch = {
             'index_template.settings': destinationIndexTemplate.template.settings,
             'index_template.mappings': destinationIndexTemplate.template.mappings,

diff --git a/...elasticsearch/transform/transform.test.ts → ...earch/transform/legacy_transforms.test.ts b/...elasticsearch/transform/transform.test.ts → ...earch/transform/legacy_transforms.test.ts
@@ -37,7 +37,7 @@ import { PACKAGES_SAVED_OBJECT_TYPE } from '../../../../constants';
 import { getAsset } from './common';
 import { installTransforms } from './install';
 
-describe('test transform install', () => {
+describe('test transform install with legacy schema', () => {
   let esClient: ReturnType<typeof elasticsearchClientMock.createElasticsearchClient>;
   let savedObjectsClient: jest.Mocked<SavedObjectsClientContract>;
   beforeEach(() => {