Skip to content

Commit edcea63

Browse files
committed
Use match statement instead of switch
1 parent 8534b39 commit edcea63

File tree

1 file changed

+7
-14
lines changed

1 file changed

+7
-14
lines changed

tests/UnifiedSpecTests/EventObserver.php

Lines changed: 7 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -212,23 +212,16 @@ public function assert(array $expectedEvents, bool $ignoreExtraEvents): void
212212
}
213213
}
214214

215-
private function assertEvent($actual, stdClass $expected, string $message)
215+
private function assertEvent($actual, stdClass $expected, string $message): void
216216
{
217217
assertIsObject($actual);
218218

219-
switch ($actual::class) {
220-
case CommandStartedEvent::class:
221-
return $this->assertCommandStartedEvent($actual, $expected, $message);
222-
223-
case CommandSucceededEvent::class:
224-
return $this->assertCommandSucceededEvent($actual, $expected, $message);
225-
226-
case CommandFailedEvent::class:
227-
return $this->assertCommandFailedEvent($actual, $expected, $message);
228-
229-
default:
230-
Assert::fail($message . ': Unsupported event type: ' . $actual::class);
231-
}
219+
match ($actual::class) {
220+
CommandStartedEvent::class => $this->assertCommandStartedEvent($actual, $expected, $message),
221+
CommandSucceededEvent::class => $this->assertCommandSucceededEvent($actual, $expected, $message),
222+
CommandFailedEvent::class => $this->assertCommandFailedEvent($actual, $expected, $message),
223+
default => Assert::fail($message . ': Unsupported event type: ' . $actual::class)
224+
};
232225
}
233226

234227
private function assertCommandStartedEvent(CommandStartedEvent $actual, stdClass $expected, string $message): void

0 commit comments

Comments
 (0)