govspeak 8.5.0 → 8.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 23ac40c67eb46b8265549568542bc48bb0cb96fe47c8b217a1e6135789ab2134
-  data.tar.gz: 26d7a270ab0163d056c0fefff53548727019128456deab21409ee3b3ae6e01eb
+  metadata.gz: 5a417c917e3816a6dc8f1dd69adb0264a6b89d679fa0fe40342e853913140885
+  data.tar.gz: 666d01ec5fc952dea1a2445aca9a29662b4208b6baf68833610a33487030249a
 SHA512:
-  metadata.gz: e637261f8a573479a054e8cdcba539ed296c4c9a1d70c263ff4caea059faf80992806332922259da0d52983013a4c302a11bed5421b12534f6d1de6b105d9620
-  data.tar.gz: fa04e11f539f372793d6420f9dcdc81964d244530f3d9e397f0587ca22fac9e66e41b85d79cd224c88f187946bddeb04f9afdfe223328240c353b060bb45fba2
+  metadata.gz: b140b11ff35bc4698a2f0a485556c5992abde565fd9fc1bdea76d2cb9f8494956a2f5038cda47e29e14fa18dad85b0e190f403f4362c76b263c56e248f50bbdf
+  data.tar.gz: 9f4f534254cf22208c8c71016f8b35ca0dc4908a07026ff034857ec4c33b6a192389e4c110d55c0231f54f5aa3b4177e6384f4a9ae22e25d51de80afc8c57e45

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+### 8.5.1
+
+* Rename embed-related code to `content block`
+* Do not attempt to parse embed codes if `content_blocks` option is missing
+
 ### 8.5.0
 
 * Support embeds in Govspeak

data/README.md

@@ -610,7 +610,7 @@ will output
 
 ### Content blocks
 
-Authors can embed different types of [supported content](https://github.com/alphagov/govspeak/blob/main/lib/govspeak/embedded_content.rb#L3) created by the Content Block Manager
+Authors can embed different types of [supported content](https://github.com/alphagov/govspeak/blob/main/lib/govspeak/content_block.rb#L3) created by the Content Block Manager
 
 ```
 {{embed:content_block_email_address:d308f561-e5ee-45b5-90b2-3ac36a23fad9}}
@@ -620,7 +620,7 @@ with options provided
 
 ```
 {
-  embeds: [
+  content_blocks: [
     {
       content_id: "d308f561-e5ee-45b5-90b2-3ac36a23fad9",
       title: "Government Digital Service",

data/lib/govspeak/{embedded_content.rb → content_block.rb}

@@ -1,5 +1,5 @@
 module Govspeak
-  class EmbeddedContent
+  class ContentBlock
     SUPPORTED_DOCUMENT_TYPES = %w[contact content_block_email_address].freeze
     UUID_REGEX = /([0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12})/
     EMBED_REGEX = /({{embed:(#{SUPPORTED_DOCUMENT_TYPES.join('|')}):#{UUID_REGEX}}})/

data/lib/govspeak/{embed_extractor.rb → content_block_extractor.rb}

@@ -1,12 +1,12 @@
 module Govspeak
-  class EmbedExtractor
+  class ContentBlockExtractor
     def initialize(document)
       @document = document
     end
 
     def content_references
-      @content_references ||= @document.scan(EmbeddedContent::EMBED_REGEX).map { |match|
-        EmbeddedContent.new(document_type: match[1], content_id: match[2], embed_code: match[0])
+      @content_references ||= @document.scan(ContentBlock::EMBED_REGEX).map { |match|
+        ContentBlock.new(document_type: match[1], content_id: match[2], embed_code: match[0])
       }.uniq
     end
 

data/lib/govspeak/presenters/{embed_presenter.rb → content_block_presenter.rb}

@@ -2,7 +2,7 @@ require "action_view"
 require "htmlentities"
 
 module Govspeak
-  class EmbedPresenter
+  class ContentBlockPresenter
     include ActionView::Helpers::TagHelper
 
     attr_reader :embed

data/lib/govspeak/version.rb

@@ -1,3 +1,3 @@
 module Govspeak
-  VERSION = "8.5.0".freeze
+  VERSION = "8.5.1".freeze
 end

data/lib/govspeak.rb

@@ -14,14 +14,14 @@ require "govspeak/structured_header_extractor"
 require "govspeak/html_validator"
 require "govspeak/html_sanitizer"
 require "govspeak/blockquote_extra_quote_remover"
-require "govspeak/embed_extractor"
-require "govspeak/embedded_content"
+require "govspeak/content_block_extractor"
+require "govspeak/content_block"
 require "govspeak/post_processor"
 require "govspeak/link_extractor"
 require "govspeak/template_renderer"
 require "govspeak/presenters/attachment_presenter"
 require "govspeak/presenters/contact_presenter"
-require "govspeak/presenters/embed_presenter"
+require "govspeak/presenters/content_block_presenter"
 require "govspeak/presenters/h_card_presenter"
 require "govspeak/presenters/image_presenter"
 require "govspeak/presenters/attachment_image_presenter"
@@ -40,7 +40,7 @@ module Govspeak
     @extensions = []
 
     attr_accessor :images
-    attr_reader :attachments, :contacts, :links, :locale, :embeds
+    attr_reader :attachments, :contacts, :links, :locale, :content_blocks
 
     def self.to_html(source, options = {})
       new(source, options).to_html
@@ -60,7 +60,7 @@ module Govspeak
       @attachments = Array.wrap(options.delete(:attachments))
       @links = Array.wrap(options.delete(:links))
       @contacts = Array.wrap(options.delete(:contacts))
-      @embeds = Array.wrap(options.delete(:embeds))
+      @content_blocks = Array.wrap(options.delete(:content_blocks))
       @locale = options.fetch(:locale, "en")
       @options = { input: PARSER_CLASS_NAME,
                    sanitize: true,
@@ -259,11 +259,13 @@ module Govspeak
       render_image(AttachmentImagePresenter.new(attachment))
     end
 
-    extension("embeds", Govspeak::EmbeddedContent::EMBED_REGEX) do |_embed_code, _document_type, content_id|
-      embed = embeds.detect { |e| e[:content_id] == content_id }
+    extension("content blocks", Govspeak::ContentBlock::EMBED_REGEX) do |embed_code, _document_type, content_id|
+      next embed_code if content_blocks.empty?
+
+      embed = content_blocks.detect { |e| e[:content_id] == content_id }
       next "" unless embed
 
-      EmbedPresenter.new(embed).render
+      ContentBlockPresenter.new(embed).render
     end
 
     # As of version 1.12.0 of Kramdown the block elements (div & figcaption)

data/test/{embed_extractor_test.rb → content_block_extractor_test.rb}

@@ -1,10 +1,10 @@
 require "test_helper"
 
-class EmbedExtractorTest < Minitest::Test
+class ContentBlockExtractorTest < Minitest::Test
   extend Minitest::Spec::DSL
 
-  describe "EmbedExtractor" do
-    subject { Govspeak::EmbedExtractor.new(document) }
+  describe "ContentBlockExtractor" do
+    subject { Govspeak::ContentBlockExtractor.new(document) }
 
     describe "when there is no embedded content" do
       let(:document) { "foo" }

data/test/{govspeak_embeds_test.rb → govspeak_content_blocks_test.rb}

@@ -1,6 +1,6 @@
 require "test_helper"
 
-class GovspeakEmbedsTest < Minitest::Test
+class GovspeakContentBlocksTest < Minitest::Test
   extend Minitest::Spec::DSL
 
   def compress_html(html)
@@ -10,7 +10,7 @@ class GovspeakEmbedsTest < Minitest::Test
   let(:content_id) { SecureRandom.uuid }
 
   it "renders an email address when present in options[:embeds]" do
-    embed = {
+    content_block = {
       content_id:,
       document_type: "content_block_email_address",
       title: "foo",
@@ -20,38 +20,52 @@ class GovspeakEmbedsTest < Minitest::Test
     }
     govspeak = "{{embed:content_block_email_address:#{content_id}}}"
 
-    rendered = Govspeak::Document.new(govspeak, embeds: [embed]).to_html
+    rendered = Govspeak::Document.new(govspeak, content_blocks: [content_block]).to_html
 
-    expected = "<p><span class=\"embed embed-content_block_email_address\" id=\"embed_#{content_id}\">#{embed[:details][:email_address]}</span></p>"
+    expected = "<p><span class=\"embed embed-content_block_email_address\" id=\"embed_#{content_id}\">#{content_block[:details][:email_address]}</span></p>"
 
     assert_equal compress_html(expected), compress_html(rendered)
   end
 
   it "renders the title when the document type is a contact" do
-    embed = {
+    content_block = {
       content_id:,
       document_type: "contact",
       title: "foo",
     }
     govspeak = "{{embed:contact:#{content_id}}}"
 
-    rendered = Govspeak::Document.new(govspeak, embeds: [embed]).to_html
+    rendered = Govspeak::Document.new(govspeak, content_blocks: [content_block]).to_html
 
-    expected = "<p><span class=\"embed embed-contact\" id=\"embed_#{content_id}\">#{embed[:title]}</span></p>"
+    expected = "<p><span class=\"embed embed-contact\" id=\"embed_#{content_id}\">#{content_block[:title]}</span></p>"
 
     assert_equal compress_html(expected), compress_html(rendered)
   end
 
-  it "ignores missing embeds" do
+  it "removes embed code if a content block cannot be found" do
+    content_block = {
+      content_id: SecureRandom.uuid,
+      document_type: "contact",
+      title: "foo",
+    }
+
     govspeak = "{{embed:contact:#{content_id}}}"
 
-    rendered = Govspeak::Document.new(govspeak, embeds: []).to_html
+    rendered = Govspeak::Document.new(govspeak, content_blocks: [content_block]).to_html
 
     assert_equal compress_html(""), compress_html(rendered)
   end
 
+  it "retains an embed code if content_blocks are not specified" do
+    govspeak = "{{embed:contact:#{content_id}}}"
+
+    rendered = Govspeak::Document.new(govspeak).to_html
+
+    assert_equal compress_html("<p>#{govspeak}</p>"), compress_html(rendered)
+  end
+
   it "supports multiple embeds" do
-    embeds = [
+    content_blocks = [
       {
         content_id: SecureRandom.uuid,
         document_type: "contact",
@@ -67,16 +81,16 @@ class GovspeakEmbedsTest < Minitest::Test
       },
     ]
 
-    govspeak = %(Here is a contact: {{embed:contact:#{embeds[0][:content_id]}}}
+    govspeak = %(Here is a contact: {{embed:contact:#{content_blocks[0][:content_id]}}}
 
-Here is an email address: {{embed:content_block_email_address:#{embeds[1][:content_id]}}}
+Here is an email address: {{embed:content_block_email_address:#{content_blocks[1][:content_id]}}}
     )
 
-    rendered = Govspeak::Document.new(govspeak, embeds:).to_html
+    rendered = Govspeak::Document.new(govspeak, content_blocks:).to_html
 
     expected = """
-      <p>Here is a contact: <span class=\"embed embed-contact\" id=\"embed_#{embeds[0][:content_id]}\">#{embeds[0][:title]}</span></p>
-      <p>Here is an email address: <span class=\"embed embed-content_block_email_address\" id=\"embed_#{embeds[1][:content_id]}\">#{embeds[1][:details][:email_address]}</span></p>
+      <p>Here is a contact: <span class=\"embed embed-contact\" id=\"embed_#{content_blocks[0][:content_id]}\">#{content_blocks[0][:title]}</span></p>
+      <p>Here is an email address: <span class=\"embed embed-content_block_email_address\" id=\"embed_#{content_blocks[1][:content_id]}\">#{content_blocks[1][:details][:email_address]}</span></p>
     """
 
     assert_equal compress_html(expected), compress_html(rendered)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: govspeak
 version: !ruby/object:Gem::Version
-  version: 8.5.0
+  version: 8.5.1
 platform: ruby
 authors:
 - GOV.UK Dev
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-10-21 00:00:00.000000000 Z
+date: 2024-10-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionview
@@ -250,8 +250,8 @@ files:
 - config/address_formats.yml
 - lib/govspeak.rb
 - lib/govspeak/blockquote_extra_quote_remover.rb
-- lib/govspeak/embed_extractor.rb
-- lib/govspeak/embedded_content.rb
+- lib/govspeak/content_block.rb
+- lib/govspeak/content_block_extractor.rb
 - lib/govspeak/header_extractor.rb
 - lib/govspeak/html_sanitizer.rb
 - lib/govspeak/html_validator.rb
@@ -260,7 +260,7 @@ files:
 - lib/govspeak/presenters/attachment_image_presenter.rb
 - lib/govspeak/presenters/attachment_presenter.rb
 - lib/govspeak/presenters/contact_presenter.rb
-- lib/govspeak/presenters/embed_presenter.rb
+- lib/govspeak/presenters/content_block_presenter.rb
 - lib/govspeak/presenters/h_card_presenter.rb
 - lib/govspeak/presenters/image_presenter.rb
 - lib/govspeak/structured_header_extractor.rb
@@ -309,14 +309,14 @@ files:
 - locales/zh-tw.yml
 - locales/zh.yml
 - test/blockquote_extra_quote_remover_test.rb
-- test/embed_extractor_test.rb
+- test/content_block_extractor_test.rb
 - test/govspeak_attachment_link_test.rb
 - test/govspeak_attachment_test.rb
 - test/govspeak_attachments_image_test.rb
 - test/govspeak_attachments_inline_test.rb
 - test/govspeak_button_test.rb
 - test/govspeak_contacts_test.rb
-- test/govspeak_embeds_test.rb
+- test/govspeak_content_blocks_test.rb
 - test/govspeak_extract_contact_content_ids_test.rb
 - test/govspeak_footnote_test.rb
 - test/govspeak_images_bang_test.rb
@@ -355,14 +355,14 @@ specification_version: 4
 summary: Markup language for single domain
 test_files:
 - test/blockquote_extra_quote_remover_test.rb
-- test/embed_extractor_test.rb
+- test/content_block_extractor_test.rb
 - test/govspeak_attachment_link_test.rb
 - test/govspeak_attachment_test.rb
 - test/govspeak_attachments_image_test.rb
 - test/govspeak_attachments_inline_test.rb
 - test/govspeak_button_test.rb
 - test/govspeak_contacts_test.rb
-- test/govspeak_embeds_test.rb
+- test/govspeak_content_blocks_test.rb
 - test/govspeak_extract_contact_content_ids_test.rb
 - test/govspeak_footnote_test.rb
 - test/govspeak_images_bang_test.rb