Run validation mappings before validating ignored fields (#2389)

Reorder validation performed in system tests to execute validation
based on mappings just after the validation based on fields.

Author: mrodm

internal/testrunner/runners/system/tester.go

@@ -1394,6 +1394,8 @@ func (r *tester) prepareScenario(ctx context.Context, config *testConfig, stackC
 		return hits.size() > 0, nil
 	}, 1*time.Second, waitForDataTimeout)
 
+	// before checking "waitErr" error , it is necessary to check if the service has finished with error
+	// to report it as a test case failed
 	if service != nil && config.Service != "" && !config.IgnoreServiceError {
 		exited, code, err := service.ExitCode(ctx, config.Service)
 		if err != nil && !errors.Is(err, servicedeployer.ErrNotSupported) {
@@ -1637,16 +1639,6 @@ func (r *tester) validateTestScenario(ctx context.Context, result *testrunner.Re
 		})
 	}
 
-	stackVersion, err := semver.NewVersion(r.stackVersion.Number)
-	if err != nil {
-		return result.WithErrorf("failed to parse stack version: %w", err)
-	}
-
-	err = validateIgnoredFields(stackVersion, scenario, config)
-	if err != nil {
-		return result.WithError(err)
-	}
-
 	if r.fieldValidationMethod == mappingsMethod {
 		logger.Warn("Validation based on mappings enabled (technical preview)")
 		exceptionFields := listExceptionFields(scenario.docs, fieldsValidator)
@@ -1669,6 +1661,16 @@ func (r *tester) validateTestScenario(ctx context.Context, result *testrunner.Re
 		}
 	}
 
+	stackVersion, err := semver.NewVersion(r.stackVersion.Number)
+	if err != nil {
+		return result.WithErrorf("failed to parse stack version: %w", err)
+	}
+
+	err = validateIgnoredFields(stackVersion, scenario, config)
+	if err != nil {
+		return result.WithError(err)
+	}
+
 	docs := scenario.docs
 	if scenario.syntheticEnabled {
 		docs, err = fieldsValidator.SanitizeSyntheticSourceDocs(scenario.docs)