Merge pull request #489 from jikan-me/moar-tests2

Added more tests and improved existing ones
jikan-me · Jan 31, 2024 · 02613f5 · 02613f5
2 parents 289bdfb + 99fa9f9
commit 02613f5
Show file tree

Hide file tree

Showing 4 changed files with 122 additions and 20 deletions.
diff --git a/database/factories/AnimeFactory.php b/database/factories/AnimeFactory.php
@@ -23,9 +23,14 @@ protected function definitionInternal(): array
     {
         $mal_id = $this->createMalId();
         $title = $this->createTitle();
-        $status = $this->faker->randomElement(["Currently Airing", "Completed", "Upcoming"]);
+        $status = $this->faker->randomElement(["Currently Airing", "Finished Airing", "Not yet aired"]);
         [$aired_from, $aired_to] = $this->createActiveDateRange($status, "Currently Airing");
 
+        $premiered = $this->faker->randomElement(["Winter", "Spring", "Fall", "Summer"]) . " " . $this->faker->year();
+        if ($status === "Not yet aired") {
+            $premiered = null;
+        }
+
         return [
             "mal_id" => $mal_id,
             "url" => $this->createMalUrl($mal_id, "anime"),
@@ -68,7 +73,7 @@ protected function definitionInternal(): array
             "synopsis" => "test",
             "approved" => true,
             "background" => "test",
-            "premiered" => $this->faker->randomElement(["Winter", "Spring", "Fall", "Summer"]) . " " . $this->faker->year(),
+            "premiered" => $premiered,
             "broadcast" => [
                 "day" => "",
                 "time" => "",

diff --git a/tests/Integration/AnimeSearchEndpointTest.php b/tests/Integration/AnimeSearchEndpointTest.php
@@ -120,24 +120,24 @@ public function emptyDateRangeProvider(): array
     public function commonParameterProvider(): array
     {
         return [
-            [["status" => "airing"]],
-            [["status" => "complete"]],
-            [["status" => "upcoming"]],
-            [["status" => "Airing"]],
-            [["status" => "Complete"]],
-            [["status" => "Upcoming"]],
-            [["max_score" => "8"]],
-            [["min_score" => "6"]],
-            [["max_score" => "7", "min_score" => "3"]],
-            [["rating" => "pg"]],
-            [["rating" => "rx"]],
-            [["rating" => "r"]],
-            [["rating" => "pg13"]],
-            [["rating" => "g"]],
-            [["rating" => "r17"]],
-            [["type" => "movie"]],
-            [["type" => "ova"]],
-            [["type" => "special"]],
+            "status = airing" => [["status" => "airing"]],
+            "status = complete" => [["status" => "complete"]],
+            "status = upcoming" => [["status" => "upcoming"]],
+            "status = Airing" => [["status" => "Airing"]],
+            "status = Complete" => [["status" => "Complete"]],
+            "status = Upcoming" => [["status" => "Upcoming"]],
+            "max_score = 8" => [["max_score" => "8"]],
+            "min_score = 6" => [["min_score" => "6"]],
+            "max_score = 7, min_score = 3" => [["max_score" => "7", "min_score" => "3"]],
+            "rating = pg" => [["rating" => "pg"]],
+            "rating = rx" => [["rating" => "rx"]],
+            "rating = r" => [["rating" => "r"]],
+            "rating = pg13" => [["rating" => "pg13"]],
+            "rating = g" => [["rating" => "g"]],
+            "rating = r17" => [["rating" => "r17"]],
+            "type = movie" => [["type" => "movie"]],
+            "type = ova" => [["type" => "ova"]],
+            "type = special" => [["type" => "special"]],
         ];
     }
 

diff --git a/tests/Integration/SeasonControllerTest.php b/tests/Integration/SeasonControllerTest.php
@@ -0,0 +1,52 @@
+<?php
+/** @noinspection PhpIllegalPsrClassPathInspection */
+namespace Tests\Integration;
+
+use App\Anime;
+use App\CarbonDateRange;
+use App\Testing\ScoutFlush;
+use App\Testing\SyntheticMongoDbTransaction;
+use Illuminate\Support\Carbon;
+use Tests\TestCase;
+
+
+class SeasonControllerTest extends TestCase
+{
+    use SyntheticMongoDbTransaction;
+    use ScoutFlush;
+
+    public function testShouldFilterOutAnimeWithGarbledAiredString()
+    {
+        Carbon::setTestNow(Carbon::parse("2024-01-11"));
+        // the wrong item
+        $f = Anime::factory(1);
+        $startDate = "2024-01-01";
+        $carbonStartDate = Carbon::parse($startDate);
+        $state = $f->serializeStateDefinition([
+            "aired" => new CarbonDateRange($carbonStartDate, null)
+        ]);
+        $state["aired"]["string"] = "Jan 1, 2024 to ?";
+        $state["premiered"] = null;
+        $state["status"] = "Not yet aired";
+        $state["airing"] = false;
+        $f->create($state);
+
+        // the correct item
+        $f = Anime::factory(1);
+        $state = $f->serializeStateDefinition([
+            "aired" => new CarbonDateRange(Carbon::parse("2024-01-10"), Carbon::parse("2024-02-15"))
+        ]);
+        $state["premiered"] = "Winter 2024";
+        $state["status"] = "Currently Airing";
+        $state["airing"] = true;
+        $f->create($state);
+
+        $content = $this->getJsonResponse([], "/v4/seasons/2024/winter");
+
+        Carbon::setTestNow();
+
+        $this->seeStatusCode(200);
+        $this->assertIsArray($content["data"]);
+        $this->assertCount(1, $content["data"]);
+    }
+}
diff --git a/tests/Unit/PreparesDataTraitTest.php b/tests/Unit/PreparesDataTraitTest.php
@@ -0,0 +1,45 @@
+<?php
+/** @noinspection PhpIllegalPsrClassPathInspection */
+namespace Tests\Unit;
+use App\Dto\Concerns\HasLimitParameter;
+use App\Dto\Concerns\HasPageParameter;
+use App\Dto\Concerns\PreparesData;
+use Spatie\LaravelData\Data;
+use Spatie\LaravelData\Optional;
+use Tests\TestCase;
+
+final class PreparesDataFixture extends Data
+{
+    use PreparesData, HasLimitParameter, HasPageParameter;
+
+    public string|Optional $filter;
+}
+
+final class PreparesDataTraitTest extends TestCase
+{
+    public function paramsDataProvider(): array
+    {
+        return [
+            'limit = empty, page = empty, filter = empty' => [['limit' => '', 'page' => '', 'filter' => ''], []],
+            'limit = 1, page = empty, filter = empty' => [['limit' => '1', 'page' => '', 'filter' => ''], ['limit' => 1]],
+            'limit = 1, page = 2, filter = empty' => [
+                ['limit' => '1', 'page' => '2', 'filter' => ''],
+                ['limit' => 1, 'page' => 2]
+            ],
+            'limit = 1, page = 2, filter = somefilter' => [
+                ['limit' => '1', 'page' => '2', 'filter' => 'somefilter'],
+                ['limit' => 1, 'page' => 2, 'filter' => 'somefilter']
+            ],
+        ];
+    }
+
+    /**
+     * @dataProvider paramsDataProvider
+     */
+    public function testShouldIgnoreEmptyParams($actual, $expected)
+    {
+        $sut = PreparesDataFixture::prepareForPipeline(collect($actual));
+
+        $this->assertCollectionsStrictlyEqual(collect($expected), $sut);
+    }
+}