Fixed component tests assuming ast_process_enabled is true

Author: SergeyKleyman

agent/php/ElasticApm/Impl/Config/AllOptionsMetadata.php

@@ -82,7 +82,7 @@ public static function get(): array
         /** @var array<string, OptionMetadata<mixed>> $value */
         $value = [
             OptionNames::API_KEY                                    => new NullableStringOptionMetadata(),
-            OptionNames::AST_PROCESS_ENABLED                        => new BoolOptionMetadata(/* defaultValue: */ true),
+            OptionNames::AST_PROCESS_ENABLED                        => new BoolOptionMetadata(OptionDefaultValues::AST_PROCESS_ENABLED),
             OptionNames::AST_PROCESS_DEBUG_DUMP_CONVERTED_BACK_TO_SOURCE
                                                                     => new BoolOptionMetadata(/* defaultValue: */ true),
             OptionNames::AST_PROCESS_DEBUG_DUMP_FOR_PATH_PREFIX     => new NullableStringOptionMetadata(),

agent/php/ElasticApm/Impl/Config/OptionDefaultValues.php

@@ -34,6 +34,7 @@ final class OptionDefaultValues
 {
     use StaticClassTrait;
 
+    public const AST_PROCESS_ENABLED = false;
     public const SPAN_STACK_TRACE_MIN_DURATION = -1;
     public const STACK_TRACE_LIMIT = 50;
     public const TRANSACTION_MAX_SPANS = 500;

tests/ElasticApmTests/ComponentTests/Util/ComponentTestCaseBase.php

@@ -24,6 +24,7 @@
 namespace ElasticApmTests\ComponentTests\Util;
 
 use Elastic\Apm\Impl\AutoInstrument\AutoInstrumentationBase;
+use Elastic\Apm\Impl\Config\OptionDefaultValues;
 use Elastic\Apm\Impl\Config\OptionNames;
 use Elastic\Apm\Impl\GlobalTracerHolder;
 use Elastic\Apm\Impl\Log\Level as LogLevel;
@@ -184,6 +185,7 @@ protected function verifyOneTransactionNoSpans(DataFromAgent $dataFromAgent): Tr
     /**
      * @param class-string<AutoInstrumentationBase> $instrClassName
      * @param string[]                              $expectedNames
+     * @param bool                                  $isEnabledByDefault
      *
      * @return void
      */
@@ -197,7 +199,8 @@ protected static function implTestIsAutoInstrumentationEnabled(string $instrClas
         $actualNames = $instr->keywords();
         $actualNames[] = $instr->name();
         self::assertEqualAsSets($expectedNames, $actualNames);
-        self::assertTrue($instr->isEnabled());
+        $isEnabledByDefault = OptionDefaultValues::AST_PROCESS_ENABLED || (!$instr->requiresUserlandCodeInstrumentation());
+        self::assertSame($isEnabledByDefault, $instr->isEnabled());
 
         /**
          * @param string $name
@@ -238,7 +241,7 @@ protected static function implTestIsAutoInstrumentationEnabled(string $instrClas
             $dbgCtx->clearCurrentSubScope(['disableInstrumentationsOptVal' => $disableInstrumentationsOptVal]);
             $tracer = self::buildTracerForTests()->withConfig(OptionNames::DISABLE_INSTRUMENTATIONS, $disableInstrumentationsOptVal)->build();
             $instr = new $instrClassName($tracer);
-            self::assertTrue($instr->isEnabled());
+            self::assertSame($isEnabledByDefault, $instr->isEnabled());
         }
         $dbgCtx->popSubScope();