fix sitemap just needed to treat it as dto

LlmLaraHub · Jul 7, 2024 · 8b33b02 · 8b33b02
1 parent b24232b
commit 8b33b02
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/app/Domains/Sources/SiteMapSource.php b/app/Domains/Sources/SiteMapSource.php
@@ -39,10 +39,10 @@ public function handle(Source $source): void
 
         foreach ($feedItems as $feedItem) {
             $webResponseDto = WebResponseDto::from([
-                'url' => $feedItem['link'],
-                'title' => $feedItem['title'],
-                'description' => $feedItem['description'],
-                'meta_data' => $feedItem,
+                'url' => $feedItem->link,
+                'title' => $feedItem->title,
+                'description' => $feedItem->description,
+                'meta_data' => $feedItem->toArray(),
                 'profile' => [],
             ]);
             $jobs[] = new GetWebContentJob($source, $webResponseDto);

diff --git a/tests/Feature/SiteMapSourceTest.php b/tests/Feature/SiteMapSourceTest.php
@@ -2,9 +2,11 @@
 
 namespace Tests\Feature;
 
+use App\Domains\Sources\FeedSource\FeedItemDto;
 use App\Domains\Sources\SourceTypeEnum;
 use App\Models\Source;
 use Facades\App\Domains\Sources\SiteMapSource\SiteMapParserWrapper;
+use Illuminate\Support\Arr;
 use Illuminate\Support\Facades\Bus;
 use Tests\TestCase;
 
@@ -17,9 +19,13 @@ public function test_run()
 
         $data = get_fixture('sitemap_parsed_results.json');
 
+        $item = Arr::first($data);
+
+        $item = FeedItemDto::from($item);
+
         SiteMapParserWrapper::shouldReceive('handle')
             ->once()->andReturn(
-                collect($data)
+                collect([$item])
             );
 
         $source = Source::factory()->create([