Ignore code that cannot be tested (and only exists to handle an edge case) from code coverage

sebastianbergmann · sebastianbergmann · commit e1ebcc435b5e · 2024-01-15T14:35:46.000+01:00
diff --git a/src/Event/Dispatcher/DirectDispatcher.php b/src/Event/Dispatcher/DirectDispatcher.php
@@ -86,9 +86,11 @@ public function dispatch(Event $event): void
         foreach ($this->tracers as $tracer) {
             try {
                 $tracer->trace($event);
+                // @codeCoverageIgnoreStart
             } catch (Throwable $t) {
                 $this->handleThrowable($t);
             }
+            // @codeCoverageIgnoreEnd
         }
 
         if (!array_key_exists($eventClassName, $this->subscribers)) {
@@ -122,7 +124,9 @@ public function handleThrowable(Throwable $t): void
             return;
         }
 
+        // @codeCoverageIgnoreStart
         throw $t;
+        // @codeCoverageIgnoreEnd
     }
 
     private function isThrowableFromThirdPartySubscriber(Throwable $t): bool

Original file line number	Diff line number	Diff line change
`@@ -86,9 +86,11 @@ public function dispatch(Event $event): void`
`86`	`86`	`foreach ($this->tracers as $tracer) {`
`87`	`87`	`try {`
`88`	`88`	`$tracer->trace($event);`
	`89`	`+ // @codeCoverageIgnoreStart`
`89`	`90`	`} catch (Throwable $t) {`
`90`	`91`	`$this->handleThrowable($t);`
`91`	`92`	`}`
	`93`	`+ // @codeCoverageIgnoreEnd`
`92`	`94`	`}`
`93`	`95`
`94`	`96`	`if (!array_key_exists($eventClassName, $this->subscribers)) {`
`@@ -122,7 +124,9 @@ public function handleThrowable(Throwable $t): void`
`122`	`124`	`return;`
`123`	`125`	`}`
`124`	`126`
	`127`	`+ // @codeCoverageIgnoreStart`
`125`	`128`	`throw $t;`
	`129`	`+ // @codeCoverageIgnoreEnd`
`126`	`130`	`}`
`127`	`131`
`128`	`132`	`private function isThrowableFromThirdPartySubscriber(Throwable $t): bool`