SentryHandler: Replacing concrete type Hub with abstract HubInterface (#496)

* SentryHandler: Replacing concrete type Hub with abstract HubInterface

* CS

* Remove unneeded test

* Add changelog entry

Co-authored-by: Alex Bouma <[email protected]>

Author: mingalevme

CHANGELOG.md

@@ -2,6 +2,8 @@
 
 ## Unreleased
 
+- Replace type hint of concrete type (`Sentry\State\Hub`) with interface (`Sentry\State\HubInterface`) in `SentryHandler` constructor (#496)
+
 ## 2.6.0
 
 - Add all log context as `log_context` to events when using the log channel (#489)

src/Sentry/Laravel/SentryHandler.php

@@ -10,7 +10,7 @@
 use Sentry\Breadcrumb;
 use Sentry\Event;
 use Sentry\Severity;
-use Sentry\State\Hub;
+use Sentry\State\HubInterface;
 use Sentry\State\Scope;
 use Throwable;
 
@@ -28,7 +28,7 @@ class SentryHandler extends AbstractProcessingHandler
     protected $release;
 
     /**
-     * @var Hub the hub object that sends the message to the server
+     * @var HubInterface the hub object that sends the message to the server
      */
     protected $hub;
 
@@ -52,13 +52,13 @@ class SentryHandler extends AbstractProcessingHandler
     private $useFormattedMessage;
 
     /**
-     * @param Hub  $hub
-     * @param int  $level  The minimum logging level at which this handler will be triggered
-     * @param bool $bubble Whether the messages that are handled can bubble up the stack or not
-     * @param bool $reportExceptions
-     * @param bool $useFormattedMessage
+     * @param HubInterface $hub
+     * @param int          $level  The minimum logging level at which this handler will be triggered
+     * @param bool         $bubble Whether the messages that are handled can bubble up the stack or not
+     * @param bool         $reportExceptions
+     * @param bool         $useFormattedMessage
      */
-    public function __construct(Hub $hub, $level = Logger::DEBUG, bool $bubble = true, bool $reportExceptions = true, bool $useFormattedMessage = false)
+    public function __construct(HubInterface $hub, $level = Logger::DEBUG, bool $bubble = true, bool $reportExceptions = true, bool $useFormattedMessage = false)
     {
         parent::__construct($level, $bubble);