Fix league lookup exception

CHANGELOG.md

@@ -5,6 +5,13 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.0.2] - To be released
+
+### Fixed
+- League serializer should handle `null` response if ID was not found
+- Event serializer should handle `null` response if ID was not found
+- Skip schedule tests at the beginning and ending of the season
+
 ## [1.0.1] - 2021-05-04
 
 ### Fixed

src/Serializer/Event/EventSerializer.php

@@ -16,4 +16,9 @@ protected function getValidJsonRootNames(): array
     {
         return ['event', 'events', 'results'];
     }
+
+    protected function endpointReturnsNull(): bool
+    {
+        return true;
+    }
 }

src/Serializer/LeagueSerializer.php

@@ -15,4 +15,9 @@ protected function getValidJsonRootNames(): array
     {
         return ['countrys', 'leagues'];
     }
+
+    protected function endpointReturnsNull(): bool
+    {
+        return true;
+    }
 }

test/integration/ScheduleTest.php

@@ -33,6 +33,11 @@ public function testTeamNext(): void
         $events = $this->client->schedule()->teamNext(133602);
         $this->assertContainsOnlyInstancesOf(Event::class, $events);
 
+        // There are no events to check at the ending of a season
+        if (empty($events)) {
+            $this->markTestSkipped('No events to check');
+        }
+
         $event = $events[0];
         $this->assertTrue('Liverpool' === $event->strHomeTeam || 'Liverpool' === $event->strAwayTeam);
     }
@@ -47,6 +52,11 @@ public function testLeagueNext(): void
         TestUtils::setPatreonKey($this->client);
         $events = $this->client->schedule()->leagueNext(4328);
 
+        // There are no events to check at the ending of a season
+        if (empty($events)) {
+            $this->markTestSkipped('No events to check');
+        }
+
         $this->assertContainsOnlyInstancesOf(Event::class, $events);
         $this->assertSame('English Premier League', $events[0]->strLeague);
     }
@@ -61,6 +71,11 @@ public function testTeamLast(): void
         $events = $this->client->schedule()->teamLast(133602);
         $this->assertContainsOnlyInstancesOf(Event::class, $events);
 
+        // There are no events to check at the beginning of a season
+        if (empty($events)) {
+            $this->markTestSkipped('No events to check');
+        }
+
         $event = $events[0];
         $this->assertTrue('Liverpool' === $event->strHomeTeam || 'Liverpool' === $event->strAwayTeam);
     }
@@ -73,6 +88,12 @@ public function testTeamLast(): void
     public function testLeagueLast(): void
     {
         $events = $this->client->schedule()->leagueLast(4328);
+
+        // There are no events to check at the beginning of a season
+        if (empty($events)) {
+            $this->markTestSkipped('No events to check');
+        }
+
         $this->assertContainsOnlyInstancesOf(Event::class, $events);
         $this->assertSame('English Premier League', $events[0]->strLeague);
     }

test/unit/Serializer/ExtendedSerializerTest.php

@@ -48,11 +48,11 @@ public function serializerProvider(): array
             [ContractSerializer::class, false],
             [CountrySerializer::class, false],
             [EntrySerializer::class, false],
-            [EventSerializer::class, false],
+            [EventSerializer::class, true],
             [FormerTeamSerializer::class, false],
             [HighlightSerializer::class, false],
             [HonorSerializer::class, false],
-            [LeagueSerializer::class, false],
+            [LeagueSerializer::class, true],
             [LineupSerializer::class, false],
             [LivescoreSerializer::class, true],
             [LoveSerializer::class, false],