From 73cc5e28b7dbb03a20b5a7a200f41bbbf4a0dc08 Mon Sep 17 00:00:00 2001 From: Daniel Wilkowski Date: Tue, 30 Jan 2024 16:00:40 +0100 Subject: [PATCH] Extract BaseFixture\Forum\Models.newThread() --- tests/Unit/BaseFixture/Forum/Models.php | 17 +++++++++++++++++ tests/Unit/Canonical/Topic/Fixture/Models.php | 5 ++--- tests/Unit/Post/Fixture/Models.php | 5 ++--- .../Schema/BreadcrumbList/Fixture/Models.php | 8 -------- .../Schema/BreadcrumbList/Fixture/Schema.php | 7 +++++-- 5 files changed, 26 insertions(+), 16 deletions(-) create mode 100644 tests/Unit/BaseFixture/Forum/Models.php diff --git a/tests/Unit/BaseFixture/Forum/Models.php b/tests/Unit/BaseFixture/Forum/Models.php new file mode 100644 index 0000000000..3701de10b3 --- /dev/null +++ b/tests/Unit/BaseFixture/Forum/Models.php @@ -0,0 +1,17 @@ +storeThread( + new Forum(['slug' => $forumSlug]), + new Topic(['title' => $topicTitle])); + } +} diff --git a/tests/Unit/Canonical/Topic/Fixture/Models.php b/tests/Unit/Canonical/Topic/Fixture/Models.php index b1ee9d6215..28787fa3a2 100644 --- a/tests/Unit/Canonical/Topic/Fixture/Models.php +++ b/tests/Unit/Canonical/Topic/Fixture/Models.php @@ -8,6 +8,7 @@ trait Models { use BaseFixture\Forum\Store; + use BaseFixture\Forum\Models; function newTopic(): string { @@ -17,9 +18,7 @@ function newTopic(): string function newForumTopic(string $forumSlug, string $topicTitle): int { - $topic = $this->storeThread( - new Forum(['slug' => $forumSlug]), - new Topic(['title' => $topicTitle])); + $topic = $this->newThread($forumSlug, $topicTitle); return $topic->id; } diff --git a/tests/Unit/Post/Fixture/Models.php b/tests/Unit/Post/Fixture/Models.php index 8dad7f932b..bd362722cb 100644 --- a/tests/Unit/Post/Fixture/Models.php +++ b/tests/Unit/Post/Fixture/Models.php @@ -8,13 +8,12 @@ trait Models { + use BaseFixture\Forum\Models; use BaseFixture\Forum\Store; function newPostInCategoryTopic(string $forumSlug, string $topicTitle): array { - $origin = $this->storeThread( - new Forum(['slug' => $forumSlug]), - new Topic(['title' => $topicTitle])); + $origin = $this->newThread($forumSlug, $topicTitle); return [$origin->id, $origin->firstPost->id]; } diff --git a/tests/Unit/Seo/Schema/BreadcrumbList/Fixture/Models.php b/tests/Unit/Seo/Schema/BreadcrumbList/Fixture/Models.php index c1c4d76210..fdb29afbf4 100644 --- a/tests/Unit/Seo/Schema/BreadcrumbList/Fixture/Models.php +++ b/tests/Unit/Seo/Schema/BreadcrumbList/Fixture/Models.php @@ -2,20 +2,12 @@ namespace Tests\Unit\Seo\Schema\BreadcrumbList\Fixture; use Coyote\Forum; -use Coyote\Topic; use Tests\Unit\BaseFixture; trait Models { use BaseFixture\Forum\Store; - function newThread(string $topicTitle, string $forumSlug): Topic - { - return $this->storeThread( - new Forum(['slug' => $forumSlug]), - new Topic(['title' => $topicTitle])); - } - function newForum(string $name, string $slug): void { $forum = new Forum([ diff --git a/tests/Unit/Seo/Schema/BreadcrumbList/Fixture/Schema.php b/tests/Unit/Seo/Schema/BreadcrumbList/Fixture/Schema.php index 379e7d2cfa..1a1eb13991 100644 --- a/tests/Unit/Seo/Schema/BreadcrumbList/Fixture/Schema.php +++ b/tests/Unit/Seo/Schema/BreadcrumbList/Fixture/Schema.php @@ -1,11 +1,14 @@ newThread($topicTitle, $forumSlug); + $topic = $this->newThread($forumSlug, $topicTitle); return [ $this->schema("/Forum/$forumSlug/{$topic->id}-{$topic->slug}", 'BreadcrumbList'), $topic->id,