style: Add void return types to test methods (#2068)

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-authored-by: Martin Linzmayer <martin.linzmayer@sentry.io>

Author: 3 people

src/ErrorHandler.php

@@ -76,7 +76,7 @@ final class ErrorHandler
     /**
      * @var callable|null The previous exception handler, if any
      *
-     * @phpstan-var null|callable(\Throwable): void
+     * @phpstan-var (callable(\Throwable): void)|null
      */
     private $previousExceptionHandler;
 

src/State/HubAdapter.php

@@ -206,7 +206,7 @@ public function __clone()
     /**
      * @see https://www.php.net/manual/en/language.oop5.magic.php#object.wakeup
      */
-    public function __wakeup()
+    public function __wakeup(): void
     {
         throw new \BadMethodCallException('Unserializing instances of this class is forbidden.');
     }

tests/ClientBuilderTest.php

@@ -23,7 +23,7 @@
 
 final class ClientBuilderTest extends TestCase
 {
-    public function testGetOptions()
+    public function testGetOptions(): void
     {
         $options = new Options();
         $clientBuilder = new ClientBuilder($options);

tests/ExceptionDataBagTest.php

@@ -15,7 +15,7 @@ final class ExceptionDataBagTest extends TestCase
     /**
      * @dataProvider constructorDataProvider
      */
-    public function testConstructor(array $constructorArgs, string $expectedType, string $expectedValue, ?Stacktrace $expectedStackTrace, ?ExceptionMechanism $expectedExceptionMechansim)
+    public function testConstructor(array $constructorArgs, string $expectedType, string $expectedValue, ?Stacktrace $expectedStackTrace, ?ExceptionMechanism $expectedExceptionMechansim): void
     {
         $exceptionDataBag = new ExceptionDataBag(...$constructorArgs);
 

tests/HttpClient/ResponseTest.php

@@ -9,7 +9,7 @@
 
 final class ResponseTest extends TestCase
 {
-    public function testResponseSuccess()
+    public function testResponseSuccess(): void
     {
         $response = new Response(
             200,
@@ -32,7 +32,7 @@ public function testResponseSuccess()
         $this->assertFalse($response->hasError());
     }
 
-    public function testResponseFailure()
+    public function testResponseFailure(): void
     {
         $response = new Response(
             500,
@@ -51,7 +51,7 @@ public function testResponseFailure()
         $this->assertTrue($response->hasError());
     }
 
-    public function testResponseMultiValueHeader()
+    public function testResponseMultiValueHeader(): void
     {
         $response = new Response(
             200,

tests/Metrics/TraceMetricsTest.php

@@ -141,7 +141,7 @@ public function testEnableMetrics(): void
         $this->assertEmpty(StubTransport::$events);
     }
 
-    public function testBeforeSendMetricAltersContent()
+    public function testBeforeSendMetricAltersContent(): void
     {
         HubAdapter::getInstance()->bindClient(new Client(new Options([
             'before_send_metric' => static function (Metric $metric) {
@@ -162,7 +162,7 @@ public function testBeforeSendMetricAltersContent()
         $this->assertEquals(99999, $metric->getValue());
     }
 
-    public function testIntType()
+    public function testIntType(): void
     {
         traceMetrics()->count('test-count', 2, ['foo' => 'bar']);
         traceMetrics()->flush();

tests/Monolog/LogsHandlerTest.php

@@ -100,7 +100,7 @@ public function testFiltersAndMapsUsingMonologEnumThreshold($threshold, $recordL
         }
     }
 
-    public function testLogsHandlerDestructor()
+    public function testLogsHandlerDestructor(): void
     {
         $transport = new StubTransport();
         $client = ClientBuilder::create([
@@ -135,7 +135,7 @@ public function testOriginTagAppliedWithHandler(): void
         $this->assertSame('auto.log.monolog', $log->attributes()->toSimpleArray()['sentry.origin']);
     }
 
-    public function testOriginTagNotAppliedWhenUsingDirectly()
+    public function testOriginTagNotAppliedWhenUsingDirectly(): void
     {
         \Sentry\logger()->info('No origin attribute');
 

tests/OptionsTest.php

@@ -587,7 +587,7 @@ public function excludedPathProviders(): array
     /**
      * @dataProvider includedPathProviders
      */
-    public function testIncludedAppPathsOverrideExcludedAppPaths(string $value, string $expected)
+    public function testIncludedAppPathsOverrideExcludedAppPaths(string $value, string $expected): void
     {
         $configuration = new Options(['in_app_include' => [$value]]);
 

tests/State/HubAdapterTest.php

@@ -270,7 +270,7 @@ public static function captureLastErrorDataProvider(): \Generator
         ];
     }
 
-    public function testCaptureCheckIn()
+    public function testCaptureCheckIn(): void
     {
         $hub = new Hub();
 

tests/Tracing/PropagationContextTest.php

@@ -14,7 +14,7 @@
 
 final class PropagationContextTest extends TestCase
 {
-    public function testFromDefaults()
+    public function testFromDefaults(): void
     {
         $propagationContext = PropagationContext::fromDefaults();
 
@@ -27,7 +27,7 @@ public function testFromDefaults()
     /**
      * @dataProvider tracingDataProvider
      */
-    public function testFromHeaders(string $sentryTraceHeader, string $baggageHeader, ?TraceId $expectedTraceId, ?SpanId $expectedParentSpanId, ?bool $expectedDynamicSamplingContextFrozen)
+    public function testFromHeaders(string $sentryTraceHeader, string $baggageHeader, ?TraceId $expectedTraceId, ?SpanId $expectedParentSpanId, ?bool $expectedDynamicSamplingContextFrozen): void
     {
         $propagationContext = PropagationContext::fromHeaders($sentryTraceHeader, $baggageHeader);
 
@@ -49,7 +49,7 @@ public function testFromHeaders(string $sentryTraceHeader, string $baggageHeader
     /**
      * @dataProvider tracingDataProvider
      */
-    public function testFromEnvironment(string $sentryTrace, string $baggage, ?TraceId $expectedTraceId, ?SpanId $expectedParentSpanId, ?bool $expectedDynamicSamplingContextFrozen)
+    public function testFromEnvironment(string $sentryTrace, string $baggage, ?TraceId $expectedTraceId, ?SpanId $expectedParentSpanId, ?bool $expectedDynamicSamplingContextFrozen): void
     {
         $propagationContext = PropagationContext::fromEnvironment($sentryTrace, $baggage);
 
@@ -95,7 +95,7 @@ public static function tracingDataProvider(): iterable
         ];
     }
 
-    public function testToTraceparent()
+    public function testToTraceparent(): void
     {
         $propagationContext = PropagationContext::fromDefaults();
         $propagationContext->setTraceId(new TraceId('566e3688a61d4bc888951642d6f14a19'));
@@ -104,7 +104,7 @@ public function testToTraceparent()
         $this->assertSame('566e3688a61d4bc888951642d6f14a19-566e3688a61d4bc8', $propagationContext->toTraceparent());
     }
 
-    public function testToBaggage()
+    public function testToBaggage(): void
     {
         $dynamicSamplingContext = DynamicSamplingContext::fromHeader('sentry-trace_id=566e3688a61d4bc888951642d6f14a19');
         $propagationContext = PropagationContext::fromDefaults();
@@ -113,7 +113,7 @@ public function testToBaggage()
         $this->assertSame('sentry-trace_id=566e3688a61d4bc888951642d6f14a19', $propagationContext->toBaggage());
     }
 
-    public function testGetTraceContext()
+    public function testGetTraceContext(): void
     {
         $propagationContext = PropagationContext::fromDefaults();
         $propagationContext->setTraceId(new TraceId('566e3688a61d4bc888951642d6f14a19'));