Apply workaround for known issue to logsdb test data generation (#113214)

Author: lkts

muted-tests.yml

@@ -235,9 +235,6 @@ tests:
 - class: org.elasticsearch.xpack.inference.rest.ServerSentEventsRestActionListenerTests
   method: testErrorMidStream
   issue: https://github.com/elastic/elasticsearch/issues/113179
-- class: org.elasticsearch.logsdb.datageneration.DataGeneratorTests
-  method: testDataGeneratorProducesValidMappingAndDocument
-  issue: https://github.com/elastic/elasticsearch/issues/112966
 - class: org.elasticsearch.xpack.core.security.authz.RoleDescriptorTests
   method: testHasPrivilegesOtherThanIndex
   issue: https://github.com/elastic/elasticsearch/issues/113202

test/framework/src/main/java/org/elasticsearch/logsdb/datageneration/datasource/DataSourceRequest.java

@@ -104,9 +104,12 @@ public DataSourceResponse.ObjectArrayGenerator accept(DataSourceHandler handler)
         }
     }
 
-    record LeafMappingParametersGenerator(String fieldName, FieldType fieldType, Set<String> eligibleCopyToFields)
-        implements
-            DataSourceRequest<DataSourceResponse.LeafMappingParametersGenerator> {
+    record LeafMappingParametersGenerator(
+        String fieldName,
+        FieldType fieldType,
+        Set<String> eligibleCopyToFields,
+        DynamicMapping dynamicMapping
+    ) implements DataSourceRequest<DataSourceResponse.LeafMappingParametersGenerator> {
         public DataSourceResponse.LeafMappingParametersGenerator accept(DataSourceHandler handler) {
             return handler.handle(this);
         }

test/framework/src/main/java/org/elasticsearch/logsdb/datageneration/datasource/DefaultMappingParametersHandler.java

@@ -10,6 +10,7 @@
 package org.elasticsearch.logsdb.datageneration.datasource;
 
 import org.elasticsearch.index.mapper.Mapper;
+import org.elasticsearch.logsdb.datageneration.fields.DynamicMapping;
 import org.elasticsearch.test.ESTestCase;
 
 import java.util.HashMap;
@@ -48,7 +49,11 @@ private Supplier<Map<String, Object>> keywordMapping(
             // We only add copy_to to keywords because we get into trouble with numeric fields that are copied to dynamic fields.
             // If first copied value is numeric, dynamic field is created with numeric field type and then copy of text values fail.
             // Actual value being copied does not influence the core logic of copy_to anyway.
-            if (ESTestCase.randomDouble() <= 0.05) {
+            //
+            // TODO
+            // We don't use copy_to on fields that are inside an object with dynamic: strict
+            // because we'll hit https://github.com/elastic/elasticsearch/issues/113049.
+            if (request.dynamicMapping() != DynamicMapping.FORBIDDEN && ESTestCase.randomDouble() <= 0.05) {
                 var options = request.eligibleCopyToFields()
                     .stream()
                     .filter(f -> f.equals(request.fieldName()) == false)

test/framework/src/main/java/org/elasticsearch/logsdb/datageneration/fields/GenericSubObjectFieldDataGenerator.java

@@ -62,7 +62,8 @@ List<ChildField> generateChildFields(DynamicMapping dynamicMapping) {
                         new DataSourceRequest.LeafMappingParametersGenerator(
                             fieldName,
                             fieldTypeInfo.fieldType(),
-                            context.getEligibleCopyToDestinations()
+                            context.getEligibleCopyToDestinations(),
+                            dynamicMapping
                         )
                     );
                 var generator = fieldTypeInfo.fieldType()

test/framework/src/main/java/org/elasticsearch/logsdb/datageneration/fields/PredefinedField.java

@@ -21,7 +21,7 @@ public interface PredefinedField {
 
     FieldDataGenerator generator(DataSource dataSource);
 
-    record WithType(String fieldName, FieldType fieldType) implements PredefinedField {
+    record WithType(String fieldName, FieldType fieldType, DynamicMapping dynamicMapping) implements PredefinedField {
         @Override
         public String name() {
             return fieldName;
@@ -31,7 +31,7 @@ public String name() {
         public FieldDataGenerator generator(DataSource dataSource) {
             // copy_to currently not supported for predefined fields, use WithGenerator if needed
             var mappingParametersGenerator = dataSource.get(
-                new DataSourceRequest.LeafMappingParametersGenerator(fieldName, fieldType, Set.of())
+                new DataSourceRequest.LeafMappingParametersGenerator(fieldName, fieldType, Set.of(), dynamicMapping)
             );
             return fieldType().generator(fieldName, dataSource, mappingParametersGenerator);
         }