Merge pull request #9552 from alphagov/landing-page-attachments

Add attachable to LandingPage model
alphagov · Oct 29, 2024 · c7478f8 · c7478f8
2 parents 951a2d1 + a212e4a
commit c7478f8
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 0 deletions.
diff --git a/app/models/landing_page.rb b/app/models/landing_page.rb
@@ -1,4 +1,5 @@
 class LandingPage < Edition
+  include ::Attachable
   include Edition::Organisations
 
   skip_callback :validation, :before, :update_document_slug

diff --git a/app/presenters/publishing_api/landing_page_presenter.rb b/app/presenters/publishing_api/landing_page_presenter.rb
@@ -48,6 +48,7 @@ def document_type
 
     def details
       YAML.load(item.body, permitted_classes: [Date])
+        .merge(PayloadBuilder::Attachments.for(item))
     end
   end
 end
diff --git a/test/unit/app/presenters/publishing_api/landing_page_presenter_test.rb b/test/unit/app/presenters/publishing_api/landing_page_presenter_test.rb
@@ -7,6 +7,7 @@ class PublishingApi::LandingPagePresenterTest < ActiveSupport::TestCase
       document: create(:document, id: 12_345, slug: "/landing-page/test"),
       title: "Landing Page title",
       summary: "Landing Page summary",
+      attachments: [create(:file_attachment)],
       first_published_at: @first_published_at = Time.zone.now,
       updated_at: 1.year.ago,
     )
@@ -75,4 +76,10 @@ class PublishingApi::LandingPagePresenterTest < ActiveSupport::TestCase
     expected_links = {}
     assert expected_links, @presented_content[:links]
   end
+
+  test "it presents attachments" do
+    attachment = @landing_page.attachments.first
+    assert_equal @presented_content.dig(:details, :attachments, 0, :id),
+                 attachment.id.to_s
+  end
 end