WIP ✨ Support Sentry V4

.github/workflows/ci.yml

@@ -24,6 +24,7 @@ jobs:
           - php: 8.1 # should use monolog v3
           - php: 8.2 # should use monolog v3
           - php: 8.3 # should use monolog v3
+          # TODO matrix should support Both versions of Sentry SDK
     steps:
       - uses: actions/checkout@v2
 

composer.json

@@ -37,7 +37,7 @@
     "require": {
         "php": "^7.4 || ^8.0",
         "monolog/monolog": "^1.6 || ^2.0 || ^3.0",
-        "sentry/sentry": "^3.1"
+        "sentry/sentry": "^3.1 || ^4.0"
     },
     "require-dev": {
         "coduo/php-matcher": "^6.0.8",

tests/SentryHandlerTest.php

@@ -14,6 +14,7 @@
 use Monolog\LogRecord;
 use PHPUnit\Framework\TestCase;
 use Sentry\Breadcrumb;
+use Sentry\Client;
 use Sentry\ClientBuilder;
 use Sentry\Event;
 use Sentry\Integration\EnvironmentIntegration;
@@ -570,62 +571,66 @@ public function resetSpy(): void
     }
 }
 
-class SpyTransport implements TransportInterface
-{
-    /**
-     * @var Event|null
-     */
-    public $spiedEvent;
-
-    /**
-     * @var bool
-     */
-    public $isFlushed = false;
-
-    public function send(Event $event): PromiseInterface
+if (defined(Client::class.'::SDK_VERSION') && version_compare(Client::SDK_VERSION, '4.0.0') >= 0) {
+    dump('TODO');
+} else {
+    class SpyTransport implements TransportInterface
     {
-        $this->spiedEvent = $event;
+        /**
+         * @var Event|null
+         */
+        public $spiedEvent;
 
-        return new FulfilledPromise(new Response(ResponseStatus::skipped(), $event));
-    }
+        /**
+         * @var bool
+         */
+        public $isFlushed = false;
 
-    public function resetSpy(): void
-    {
-        $this->spiedEvent = null;
-        $this->isFlushed = false;
-    }
+        public function send(Event $event): PromiseInterface
+        {
+            $this->spiedEvent = $event;
 
-    public function getSpiedEvent(): Event
-    {
-        if (null === $this->spiedEvent) {
-            throw new \RuntimeException('No spied scope');
+            return new FulfilledPromise(new Response(ResponseStatus::skipped(), $event));
         }
 
-        return $this->spiedEvent;
-    }
+        public function resetSpy(): void
+        {
+            $this->spiedEvent = null;
+            $this->isFlushed  = false;
+        }
 
-    public function close(?int $timeout = null): PromiseInterface
-    {
-        $this->isFlushed = true;
+        public function getSpiedEvent(): Event
+        {
+            if (null === $this->spiedEvent) {
+                throw new \RuntimeException('No spied scope');
+            }
 
-        return new FulfilledPromise(true);
-    }
-}
+            return $this->spiedEvent;
+        }
 
-class FakeTransportFactory implements TransportFactoryInterface
-{
-    /**
-     * @var SpyTransport
-     */
-    private $transport;
+        public function close(?int $timeout = null): PromiseInterface
+        {
+            $this->isFlushed = true;
 
-    public function __construct(SpyTransport $transport)
-    {
-        $this->transport = $transport;
+            return new FulfilledPromise(true);
+        }
     }
 
-    public function create(Options $options): TransportInterface
+    class FakeTransportFactory implements TransportFactoryInterface
     {
-        return $this->transport;
+        /**
+         * @var SpyTransport
+         */
+        private $transport;
+
+        public function __construct(SpyTransport $transport)
+        {
+            $this->transport = $transport;
+        }
+
+        public function create(Options $options): TransportInterface
+        {
+            return $this->transport;
+        }
     }
 }