wip handle older SDK versions

Will either squash this into the first commit if acceptable, the
alternative is to just bump the required version of `sentry/sentry`.

Author: cs278

CHANGELOG.md

@@ -2,8 +2,8 @@
 
 ## Unreleased
 
-- Report exceptions to Sentry as unhandled by default.
-- Exceptions from messages which will be retried are sent to Sentry as handled.
+- Report exceptions to Sentry as unhandled by default (when `sentry/sentry` ≥ 3.11).
+- Exceptions from messages which will be retried are sent to Sentry as handled (when `sentry/sentry` ≥ 3.11).
 
 ## 4.4.0 (2022-10-20)
 

tests/EventListener/AbstractConsoleListenerTest.php

@@ -102,6 +102,9 @@ public function testHandleConsoleTerminateEvent(): void
      */
     public function testHandleConsoleErrorEvent(bool $captureErrors): void
     {
+        // For BC with sentry/sentry < 3.11
+        $checkMechanism = \property_exists(EventHint::class, 'mechanism');
+
         $scope = new Scope();
         $consoleEvent = new ConsoleErrorEvent(new ArrayInput([]), new NullOutput(), new \Exception());
         $listenerClass = static::getListenerClass();
@@ -119,10 +122,13 @@ public function testHandleConsoleErrorEvent(bool $captureErrors): void
                 $this->anything(),
                 $this->logicalAnd(
                     $this->isInstanceOf(EventHint::class),
-                    $this->callback(function (EventHint $subject) use ($consoleEvent) {
+                    $this->callback(function (EventHint $subject) use ($consoleEvent, $checkMechanism) {
                         self::assertSame($consoleEvent->getError(), $subject->exception);
-                        self::assertNotNull($subject->mechanism);
-                        self::assertFalse($subject->mechanism->isHandled());
+
+                        if ($checkMechanism) {
+                            self::assertNotNull($subject->mechanism);
+                            self::assertFalse($subject->mechanism->isHandled());
+                        }
 
                         return true;
                     })

tests/EventListener/ErrorListenerTest.php

@@ -41,6 +41,9 @@ protected function setUp(): void
      */
     public function testHandleExceptionEvent($event): void
     {
+        // For BC with sentry/sentry < 3.11
+        $checkMechanism = \property_exists(EventHint::class, 'mechanism');
+
         $expectedException = $event instanceof ExceptionEvent ? $event->getThrowable() : $event->getException();
 
         $this->hub->expects($this->once())
@@ -49,10 +52,13 @@ public function testHandleExceptionEvent($event): void
                 $this->anything(),
                 $this->logicalAnd(
                     $this->isInstanceOf(EventHint::class),
-                    $this->callback(function (EventHint $subject) use ($expectedException) {
+                    $this->callback(function (EventHint $subject) use ($expectedException, $checkMechanism) {
                         self::assertSame($expectedException, $subject->exception);
-                        self::assertNotNull($subject->mechanism);
-                        self::assertFalse($subject->mechanism->isHandled());
+
+                        if ($checkMechanism) {
+                            self::assertNotNull($subject->mechanism);
+                            self::assertFalse($subject->mechanism->isHandled());
+                        }
 
                         return true;
                     })

tests/EventListener/MessengerListenerTest.php

@@ -50,6 +50,9 @@ public function testHandleWorkerMessageFailedEvent(array $exceptions, WorkerMess
             $this->markTestSkipped('Messenger not supported in this environment.');
         }
 
+        // For BC with sentry/sentry < 3.11
+        $checkMechanism = \property_exists(EventHint::class, 'mechanism');
+
         $scope = new Scope();
 
         $this->hub->expects($this->once())
@@ -60,15 +63,18 @@ public function testHandleWorkerMessageFailedEvent(array $exceptions, WorkerMess
 
         $this->hub->expects($this->exactly(\count($exceptions)))
             ->method('captureEvent')
-            ->withConsecutive(...array_map(function (\Throwable $expectedException) use ($expectedIsHandled): array {
+            ->withConsecutive(...array_map(function (\Throwable $expectedException) use ($expectedIsHandled, $checkMechanism): array {
                 return [
                     $this->anything(),
                     $this->logicalAnd(
                         $this->isInstanceOf(EventHint::class),
-                        $this->callback(function (EventHint $subject) use ($expectedException, $expectedIsHandled) {
+                        $this->callback(function (EventHint $subject) use ($expectedException, $expectedIsHandled, $checkMechanism) {
                             self::assertSame($expectedException, $subject->exception);
-                            self::assertNotNull($subject->mechanism);
-                            self::assertSame($expectedIsHandled, $subject->mechanism->isHandled());
+
+                            if ($checkMechanism) {
+                                self::assertNotNull($subject->mechanism);
+                                self::assertSame($expectedIsHandled, $subject->mechanism->isHandled());
+                            }
 
                             return true;
                         })