Explorar o código

Add NoDiscard attr, fixes #2012 (#2013)

Jordi Boggiano hai 1 semana
pai
achega
edf9cb618b
Modificáronse 1 ficheiros con 8 adicións e 0 borrados
  1. 8 0
      src/Monolog/Handler/TestHandler.php

+ 8 - 0
src/Monolog/Handler/TestHandler.php

@@ -15,6 +15,7 @@ use Monolog\Level;
 use Monolog\Logger;
 use Psr\Log\LogLevel;
 use Monolog\LogRecord;
+use NoDiscard;
 
 /**
  * Used for testing purposes.
@@ -79,6 +80,7 @@ class TestHandler extends AbstractProcessingHandler
     /**
      * @return array<LogRecord>
      */
+    #[NoDiscard]
     public function getRecords(): array
     {
         return $this->records;
@@ -107,6 +109,7 @@ class TestHandler extends AbstractProcessingHandler
      *
      * @phpstan-param value-of<Level::VALUES>|value-of<Level::NAMES>|Level|LogLevel::* $level
      */
+    #[NoDiscard]
     public function hasRecords(int|string|Level $level): bool
     {
         return isset($this->recordsByLevel[Logger::toMonologLevel($level)->value]);
@@ -117,6 +120,7 @@ class TestHandler extends AbstractProcessingHandler
      *
      * @phpstan-param array{message: string, context?: mixed[]}|string $recordAssertions
      */
+    #[NoDiscard]
     public function hasRecord(string|array $recordAssertions, Level $level): bool
     {
         if (\is_string($recordAssertions)) {
@@ -135,11 +139,13 @@ class TestHandler extends AbstractProcessingHandler
         }, $level);
     }
 
+    #[NoDiscard]
     public function hasRecordThatContains(string $message, Level $level): bool
     {
         return $this->hasRecordThatPasses(fn (LogRecord $rec) => str_contains($rec->message, $message), $level);
     }
 
+    #[NoDiscard]
     public function hasRecordThatMatches(string $regex, Level $level): bool
     {
         return $this->hasRecordThatPasses(fn (LogRecord $rec) => preg_match($regex, $rec->message) > 0, $level);
@@ -148,6 +154,7 @@ class TestHandler extends AbstractProcessingHandler
     /**
      * @phpstan-param callable(LogRecord, int): mixed $predicate
      */
+    #[NoDiscard]
     public function hasRecordThatPasses(callable $predicate, Level $level): bool
     {
         $level = Logger::toMonologLevel($level);
@@ -177,6 +184,7 @@ class TestHandler extends AbstractProcessingHandler
     /**
      * @param mixed[] $args
      */
+    #[NoDiscard]
     public function __call(string $method, array $args): bool
     {
         if ((bool) preg_match('/(.*)(Debug|Info|Notice|Warning|Error|Critical|Alert|Emergency)(.*)/', $method, $matches)) {