govspeak 3.6.2 → 4.0.0

data/lib/presenters/h_card_presenter.rb

@@ -0,0 +1,69 @@
+class HCardPresenter
+  def self.from_contact(contact)
+    new(contact_properties(contact), contact.country_code)
+  end
+
+  def self.contact_properties(contact)
+    { 'fn' => contact.recipient,
+      'street-address' => contact.street_address,
+      'postal-code' => contact.postal_code,
+      'locality' => contact.locality,
+      'region' => contact.region,
+      'country-name' => country_name(contact) }
+  end
+
+  def self.country_name(contact)
+    contact.country_name unless contact.country_code == 'GB'
+  end
+
+  def self.property_keys
+    ['fn', 'street-address', 'postal-code', 'locality', 'region', 'country-name']
+  end
+
+  def self.address_formats
+    @address_formats ||= YAML.load_file('config/address_formats.yml')
+  end
+
+  attr_reader :properties, :country_code
+
+  def initialize(properties, country_code)
+    @properties = properties
+    @country_code = country_code
+  end
+
+  def render
+    "<p class=\"adr\">\n#{interpolate_address_template}\n</p>\n".html_safe
+  end
+
+  def interpolate_address_property(property_name)
+    value = properties[property_name]
+
+    if value.present?
+      "<span class=\"#{property_name}\">#{ERB::Util.html_escape(value)}</span>"
+    else
+      ""
+    end
+  end
+
+  private
+
+  def interpolate_address_template
+    address = address_template
+
+    self.class.property_keys.each do |key|
+      address.gsub!(/\{\{#{key}\}\}/, interpolate_address_property(key))
+    end
+
+    address.gsub(/^\n/, '')         # get  rid of blank lines
+           .strip                   # get rid of any trailing whitespace
+           .gsub(/\n/, "<br />\n")  # add break tags where appropriate
+  end
+
+  def address_template
+    (self.class.address_formats[country_code.to_s.downcase] || default_format_string).dup
+  end
+
+  def default_format_string
+    self.class.address_formats['gb']
+  end
+end

data/lib/templates/contact.html.erb

@@ -0,0 +1,42 @@
+<%
+  extra_class = []
+  extra_class << 'postal-address' if contact.has_postal_address?
+%>
+<div id="contact_<%= contact.id %>" class="<%= ['contact', extra_class].flatten.join(' ') %>">
+  <div class="content">
+    <h3><%= contact.title %></h3>
+    <div class="vcard contact-inner">
+    <% if contact.has_postal_address? %>
+      <%= render_hcard_address(contact) %>
+    <% end %>
+    <% if contact.email.present? || contact.contact_form_url.present? || contact.contact_numbers.any? %>
+      <div class="email-url-number">
+      <% if contact.email.present? %>
+        <p class="email">
+          <span class="type"><%= t('contact.email') %></span>
+          <a href="mailto:<%= contact.email %>" class="email"><%= contact.email %></a>
+        </p>
+      <% end %>
+      <% if contact.contact_form_url.present? %>
+        <p class="contact_form_url">
+          <span class="type"><%= t('contact.contact_form') %></span>
+          <a href="<%= contact.contact_form_url %>"><%= contact.contact_form_url.truncate(25) %></a>
+        </p>
+      <% end %>
+      <% contact.contact_numbers.each do |number| %>
+        <p class="tel">
+          <span class="type"><%= number.label %></span>
+          <%= number.number %>
+        </p>
+      <% end %>
+      </div>
+    <% end %>
+    <% if contact.comments.present? %>
+      <p class="comments"><%= auto_link(format_with_html_line_breaks(h(contact.comments))) %></p>
+    <% end %>
+    <% if contact.worldwide_organisation_path %>
+      <a href="<%= contact.worldwide_organisation_path %>">Access and opening times</a>
+    <% end %>
+    </div>
+  </div>
+</div>

data/test/blockquote_extra_quote_remover_test.rb

@@ -0,0 +1,91 @@
+# encoding: UTF-8
+
+require "test_helper"
+
+class BlockquoteExtraQuoteRemoverTest < Minitest::Test
+  def assert_remover_transforms(options)
+    from = options.keys.first
+    to = options.values.first
+    assert_equal to, Govspeak::BlockquoteExtraQuoteRemover.remove(from)
+  end
+
+  def assert_leaves_untouched(candidate)
+    assert_remover_transforms candidate => candidate
+  end
+
+  test "ignores nil" do
+    assert_leaves_untouched nil
+  end
+
+  test "ignores text without double quotes" do
+    assert_leaves_untouched %{no quotes\na few lines,\n\n\n\nbut no quotes\n\n}
+  end
+
+  test "ignores text without a quote symbol" do
+    assert_leaves_untouched %{quotes\n"but not with a > symbol"}
+  end
+
+  test "transforms text with surrounding quotes" do
+    assert_remover_transforms(
+      %{He said:\n> "yes, it's true!"\n\napparently.} =>
+      %{He said:\n> yes, it's true!\n\napparently.}
+    )
+  end
+
+  test "leaves quotes in the middle of the string" do
+    assert_remover_transforms(
+      %{He said:\n> "yes, it's true!" whilst sipping a cocktail. "And yes, I did rather enjoy it." \n\napparently.} =>
+      %{He said:\n> yes, it's true!" whilst sipping a cocktail. "And yes, I did rather enjoy it.\n\napparently.}
+    )
+  end
+
+  test "leaves trailing text and quote intact" do
+    assert_remover_transforms(
+      %{He said:\n> "yes, it's true!" whilst sipping a cocktail.} =>
+      %{He said:\n> yes, it's true!" whilst sipping a cocktail.}
+    )
+  end
+
+  test "windows line breaks" do
+    assert_remover_transforms(
+      %{Sir George Young MP, said\r\n>  "I welcome the positive public response to the e-petitions site, which is important way of building a bridge between people and Parliament.”\r\n## The special relationship} =>
+      %{Sir George Young MP, said\r\n> I welcome the positive public response to the e-petitions site, which is important way of building a bridge between people and Parliament.\r\n## The special relationship}
+    )
+  end
+
+  test "no space in front" do
+    assert_remover_transforms(
+      %{>"As we continue with the redundancy process we will ensure we retain the capabilities that our armed forces will require to meet the challenges of the future. The redundancy programme will not impact adversely on the current operations in Afghanistan, where our armed forces continue to fight so bravely on this country's behalf."} =>
+      %{> As we continue with the redundancy process we will ensure we retain the capabilities that our armed forces will require to meet the challenges of the future. The redundancy programme will not impact adversely on the current operations in Afghanistan, where our armed forces continue to fight so bravely on this country's behalf.}
+    )
+  end
+
+  test "remove double double quotes" do
+    assert_remover_transforms(
+      %{We heard it said:\n\n> ""Today the coalition is remedying those deficiencies by putting in place a new fast track process where the people's elected representatives have responsibility for the final decisions about Britain's future instead of unelected commissioners.""} =>
+      %{We heard it said:\n\n> Today the coalition is remedying those deficiencies by putting in place a new fast track process where the people's elected representatives have responsibility for the final decisions about Britain's future instead of unelected commissioners.}
+    )
+    assert_remover_transforms(
+      %{> ""Today the coalition is remedying those deficiencies by putting in place a new fast track process where the people's elected representatives have responsibility for the final decisions about Britain's future instead of unelected commissioners.} =>
+      %{> Today the coalition is remedying those deficiencies by putting in place a new fast track process where the people's elected representatives have responsibility for the final decisions about Britain's future instead of unelected commissioners.}
+    )
+  end
+
+  test "removes quotes correctly from multi-line blockquotes" do
+    assert_remover_transforms(
+      %{> "Here is a block quote using 2 lines and two of the arrows.\n> I am not sure how this will render.  I think it will mash them together."} =>
+      %{> Here is a block quote using 2 lines and two of the arrows.\n> I am not sure how this will render.  I think it will mash them together.}
+    )
+  end
+
+  test "preserves multiline blockquotes with plain newlines quotes" do
+    assert_remover_transforms(
+      %{> "line 1\n> \n> "line 2\n> \n> "line 3"} =>
+      %{> line 1\n> \n> line 2\n> \n> line 3}
+      )
+  end
+
+  test "preserve newlines when there's a blockquote with additional text after" do
+    assert_leaves_untouched(%{> \n> blah})
+  end
+end

data/test/govspeak_contacts_test.rb

@@ -0,0 +1,112 @@
+# encoding: UTF-8
+
+require 'test_helper'
+require 'ostruct'
+
+class GovspeakContactsTest < Minitest::Test
+
+  def build_contact(attrs={})
+    OpenStruct.new(
+      has_postal_address?: attrs.fetch(:has_postal_address?, true),
+      id: attrs.fetch(:id, 123456),
+      content_id: attrs.fetch(:content_id, "4f3383e4-48a2-4461-a41d-f85ea8b89ba0"),
+      title: attrs.fetch(:title, "Government Digital Service"),
+      recipient: attrs.fetch(:recipient, ""),
+      street_address: attrs.fetch(:street_address, "125 Kingsway"),
+      postal_code: attrs.fetch(:postal_code, "WC2B 6NH"),
+      locality: attrs.fetch(:locality, "Holborn"),
+      region: attrs.fetch(:region, "London"),
+      country_code: attrs.fetch(:country_code, "gb"),
+      email: attrs.fetch(:email, "people@digital.cabinet-office.gov.uk"),
+      contact_form_url: attrs.fetch(:contact_form_url, ""),
+      contact_numbers: attrs.fetch(:contact_numbers,
+                                   [OpenStruct.new(label: "helpdesk", number: "+4412345 67890")]),
+      comments: attrs.fetch(:comments, ""),
+      worldwide_organisation_path: attrs.fetch(:worldwide_organisation_path, nil),
+    )
+  end
+
+  def compress_html(html)
+    html.gsub(/[\n\r]+[\s]*/,'')
+  end
+
+  test "contact is rendered when present in options[:contacts]" do
+    contact = build_contact
+    govspeak = "[Contact:4f3383e4-48a2-4461-a41d-f85ea8b89ba0]"
+
+    rendered = Govspeak::Document.new(govspeak, { contacts: [contact] }).to_html
+    expected_html_output = %{
+      <div id="contact_123456" class="contact postal-address">
+        <div class="content">
+          <h3>Government Digital Service</h3>
+          <div class="vcard contact-inner">
+            <p class="adr">
+              <span class="street-address">125 Kingsway</span><br>
+              <span class="locality">Holborn</span><br>
+              <span class="region">London</span><br>
+              <span class="postal-code">WC2B 6NH</span>
+            </p>
+            <div class="email-url-number">
+              <p class="email">
+                <span class="type">Email</span>
+                <a href="mailto:people@digital.cabinet-office.gov.uk" class="email">people@digital.cabinet-office.gov.uk</a>
+              </p>
+              <p class="tel">
+                <span class="type">helpdesk</span>
+                +4412345 67890
+              </p>
+            </div>
+          </div>
+        </div>
+      </div>
+    }
+
+    assert_equal(compress_html(expected_html_output), compress_html(rendered))
+  end
+
+  test "no contact is rendered when contact not present in options[:contacts]" do
+    contact = build_contact(content_id: "19f06142-1b4a-47ce-b257-964badd0a5e2")
+    govspeak = "[Contact:4f3383e4-48a2-4461-a41d-f85ea8b89ba0]"
+    rendered = Govspeak::Document.new(govspeak, { contacts: [contact]}).to_html
+    assert_match("", rendered)
+  end
+
+  test "no contact is rendered when no contacts are supplied" do
+    rendered = Govspeak::Document.new("[Contact:4f3383e4-48a2-4461-a41d-f85ea8b89ba0]").to_html
+    assert_match("", rendered)
+  end
+
+  test "contact with no postal address omits the address info" do
+    contact = build_contact(has_postal_address?: false)
+    govspeak = "[Contact:4f3383e4-48a2-4461-a41d-f85ea8b89ba0]"
+    rendered = Govspeak::Document.new(govspeak, { contacts: [contact] }).to_html
+    expected_html_output = %{
+      <div id="contact_123456" class="contact">
+        <div class="content">
+          <h3>Government Digital Service</h3>
+          <div class="vcard contact-inner">
+            <div class="email-url-number">
+              <p class="email">
+                <span class="type">Email</span>
+                <a href="mailto:people@digital.cabinet-office.gov.uk" class="email">people@digital.cabinet-office.gov.uk</a>
+              </p>
+              <p class="tel">
+                <span class="type">helpdesk</span>
+                +4412345 67890
+              </p>
+            </div>
+          </div>
+        </div>
+      </div>
+    }
+    assert_equal(compress_html(expected_html_output), compress_html(rendered))
+  end
+
+  test "worldwide office contact renders worldwide organisation link" do
+    contact = build_contact(worldwide_organisation_path: "/government/world/organisations/british-antarctic-territory")
+    govspeak = "[Contact:4f3383e4-48a2-4461-a41d-f85ea8b89ba0]"
+    rendered = Govspeak::Document.new(govspeak, { contacts: [contact] }).to_html
+    organisation_link = %Q(<a href="/government/world/organisations/british-antarctic-territory">Access and opening times</a>)
+    assert_match(organisation_link, rendered)
+  end
+end

data/test/govspeak_test.rb

@@ -18,11 +18,6 @@ class GovspeakTest < Minitest::Test
     assert_equal "<p><em>this is markdown</em></p>\n", rendered
   end
 
-  test "simple block extension" do
-    rendered =  Govspeak::Document.new("this \n{::reverse}\n*is*\n{:/reverse}\n markdown").to_html
-    assert_equal "<p>this</p>\n\n<p><em>si</em></p>\n\n<p>markdown</p>\n", rendered
-  end
-
   test "highlight-answer block extension" do
     rendered =  Govspeak::Document.new("this \n{::highlight-answer}Lead in to *BIG TEXT*\n{:/highlight-answer}").to_html
     assert_equal %Q{<p>this</p>\n\n<div class="highlight-answer">\n<p>Lead in to <em>BIG TEXT</em></p>\n</div>\n}, rendered
@@ -106,7 +101,29 @@ Testcase Cliffs
 Teston
 0123 456 7890 $A    }
     doc = Govspeak::Document.new(input)
-    assert_equal %{\n<div class="address"><div class="adr org fn"><p>\n123 Test Street<br />Testcase Cliffs<br />Teston<br />0123 456 7890 \n</p></div></div>\n}, doc.to_html
+    assert_equal %{\n<div class="address"><div class="adr org fn"><p>\n123 Test Street<br>Testcase Cliffs<br>Teston<br>0123 456 7890 \n</p></div></div>\n}, doc.to_html
+  end
+
+  test "should convert barchart" do
+    input = <<-END
+|col|
+|---|
+|val|
+{barchart}
+    END
+    html = Govspeak::Document.new(input).to_html
+    assert_equal %{<table class=\"js-barchart-table mc-auto-outdent\">\n  <thead>\n    <tr>\n      <th>col</th>\n    </tr>\n  </thead>\n  <tbody>\n    <tr>\n      <td>val</td>\n    </tr>\n  </tbody>\n</table>\n}, html
+  end
+
+  test "should convert barchart with stacked compact and negative" do
+    input = <<-END
+|col|
+|---|
+|val|
+{barchart stacked compact negative}
+    END
+    html = Govspeak::Document.new(input).to_html
+    assert_equal %{<table class=\"js-barchart-table mc-stacked compact mc-negative mc-auto-outdent\">\n  <thead>\n    <tr>\n      <th>col</th>\n    </tr>\n  </thead>\n  <tbody>\n    <tr>\n      <td>val</td>\n    </tr>\n  </tbody>\n</table>\n}, html
   end
 
   test "address div is separated from paragraph text by a couple of line-breaks" do
@@ -119,7 +136,7 @@ Testcase Cliffs
 Teston
 0123 456 7890 $A}
     doc = Govspeak::Document.new(input)
-    assert_equal %{<p>Paragraph1</p>\n\n<div class="address"><div class="adr org fn"><p>\n123 Test Street<br />Testcase Cliffs<br />Teston<br />0123 456 7890 \n</p></div></div>\n}, doc.to_html
+    assert_equal %{<p>Paragraph1</p>\n\n<div class="address"><div class="adr org fn"><p>\n123 Test Street<br>Testcase Cliffs<br>Teston<br>0123 456 7890 \n</p></div></div>\n}, doc.to_html
   end
 
   test_given_govspeak("^ I am very informational ^") do
@@ -156,7 +173,8 @@ Teston
 
   test_given_govspeak "@ I am very important @" do
     assert_html_output %{
-      <div role="note" aria-label="Important" class="advisory"><p><strong>I am very important</strong></p>
+      <div role="note" aria-label="Important" class="advisory">
+      <p><strong>I am very important</strong></p>
       </div>}
     assert_text_output "I am very important"
   end
@@ -168,7 +186,8 @@ Teston
     assert_html_output %{
       <p>The following is very important</p>
 
-      <div role="note" aria-label="Important" class="advisory"><p><strong>I am very important</strong></p>
+      <div role="note" aria-label="Important" class="advisory">
+      <p><strong>I am very important</strong></p>
       </div>}
     assert_text_output "The following is very important I am very important"
   end
@@ -212,12 +231,12 @@ Teston
   end
 
   test_given_govspeak "This is a [link](http://www.gov.uk) isn't it?" do
-    assert_html_output '<p>This is a <a href="http://www.gov.uk">link</a> isn&rsquo;t it?</p>'
+    assert_html_output '<p>This is a <a href="http://www.gov.uk">link</a> isn’t it?</p>'
     assert_text_output "This is a link isn’t it?"
   end
 
   test_given_govspeak "This is a [link with an at sign in it](http://www.gov.uk/@dg/@this) isn't it?" do
-    assert_html_output '<p>This is a <a href="http://www.gov.uk/@dg/@this">link with an at sign in it</a> isn&rsquo;t it?</p>'
+    assert_html_output '<p>This is a <a href="http://www.gov.uk/@dg/@this">link with an at sign in it</a> isn’t it?</p>'
     assert_text_output "This is a link with an at sign in it isn’t it?"
   end
 
@@ -281,7 +300,7 @@ Teston
   end
 
   test_given_govspeak "![image with external url](http://www.example.com/image.jpg)" do
-    assert_html_output '<p><img src="http://www.example.com/image.jpg" alt="image with external url" /></p>'
+    assert_html_output '<p><img src="http://www.example.com/image.jpg" alt="image with external url"></p>'
   end
 
   test "should be able to override default 'document_domains' option" do
@@ -299,9 +318,9 @@ Teston
     refute html.include?('rel="external"'), "should not automatically add rel external attribute"
   end
 
-  test "should be able to override default 'entity output' option" do
-    html = Govspeak::Document.new("&yen;", entity_output: :numeric).to_html
-    assert html.include?("&#165;")
+  test "should not be able to override default 'entity output' option" do
+    html = Govspeak::Document.new("&gt;", entity_output: :numeric).to_html
+    assert html.include?("&gt;")
   end
 
   test "should assume a link with an invalid uri is internal" do
@@ -364,7 +383,7 @@ Teston
     $A" do
     assert_html_output %{
       <div class="address"><div class="adr org fn"><p>
-      street<br />road<br />
+      street<br>road<br>
       </p></div></div>}
     assert_text_output "street road"
   end
@@ -444,11 +463,14 @@ $CTA
     " do
     assert_html_output %{
       <ol class="steps">
-      <li><p>zippy</p>
+      <li>
+      <p>zippy</p>
       </li>
-      <li><p>bungle</p>
+      <li>
+      <p>bungle</p>
       </li>
-      <li><p>george</p>
+      <li>
+      <p>george</p>
       </li>
       </ol>}
     assert_text_output "zippy bungle george"
@@ -468,9 +490,11 @@ $CTA
       </ul>
 
       <ol class="steps">
-      <li><p>step</p>
+      <li>
+      <p>step</p>
       </li>
-      <li><p>list</p>
+      <li>
+      <p>list</p>
       </li>
       </ol>}
     assert_text_output "unordered list step list"
@@ -591,7 +615,8 @@ $CTA
     assert_html_output '
       <div class="devolved-content scotland">
       <p class="devolved-header">This section applies to Scotland</p>
-      <div class="devolved-body"><p>I am very devolved
+      <div class="devolved-body">
+      <p>I am very devolved
        and very scottish</p>
       </div>
       </div>
@@ -600,7 +625,8 @@ $CTA
 
   test_given_govspeak "@ Message with [a link](http://foo.bar/)@" do
     assert_html_output %{
-      <div role="note" aria-label="Important" class="advisory"><p><strong>Message with <a rel="external" href="http://foo.bar/">a link</a></strong></p>
+      <div role="note" aria-label="Important" class="advisory">
+      <p><strong>Message with <a rel="external" href="http://foo.bar/">a link</a></strong></p>
       </div>
       }
   end
@@ -610,19 +636,19 @@ $CTA
     given_govspeak "!!1", images do
       assert_html_output %Q{
           <figure class="image embedded">
-            <div class="img"><img alt="my alt" src="http://example.com/image.jpg" /></div>
+            <div class="img"><img alt="my alt" src="http://example.com/image.jpg"></div>
           </figure>
-        }
-    end
+      }
   end
+end
 
-  test "alt text of referenced images is escaped" do
-    images = [OpenStruct.new(alt_text: %Q{my alt '&"<>}, url: "http://example.com/image.jpg")]
-    given_govspeak "!!1", images do
-      assert_html_output %Q{
-          <figure class="image embedded">
-            <div class="img"><img alt="my alt &apos;&amp;&quot;&lt;&gt;" src="http://example.com/image.jpg" /></div>
-          </figure>
+test "alt text of referenced images is escaped" do
+  images = [OpenStruct.new(alt_text: %Q{my alt '&"<>}, url: "http://example.com/image.jpg")]
+  given_govspeak "!!1", images do
+    assert_html_output %Q{
+        <figure class="image embedded">
+          <div class="img"><img alt="my alt '&amp;&quot;&lt;&gt;" src="http://example.com/image.jpg"></div>
+        </figure>
         }
     end
   end
@@ -639,7 +665,7 @@ $CTA
     given_govspeak "!!1", images do
       assert_html_output %Q{
           <figure class="image embedded">
-            <div class="img"><img alt="my alt" src="http://example.com/image.jpg" /></div>
+            <div class="img"><img alt="my alt" src="http://example.com/image.jpg"></div>
             <figcaption>My Caption &amp; so on</figcaption>
           </figure>
         }
@@ -651,7 +677,7 @@ $CTA
     given_govspeak "!!1", images do
       assert_html_output %Q{
           <figure class="image embedded">
-            <div class="img"><img alt="my alt" src="http://example.com/image.jpg" /></div>
+            <div class="img"><img alt="my alt" src="http://example.com/image.jpg"></div>
           </figure>
         }
     end
@@ -810,4 +836,111 @@ $PriorityList:1
       |
     end
   end
+
+  test "should remove quotes surrounding a blockquote" do
+    govspeak = %Q{
+He said:
+
+> "I'm not sure what you mean!"
+
+Or so we thought.}
+
+    given_govspeak(govspeak) do
+      assert_html_output %|
+        <p>He said:</p>
+
+        <blockquote>
+          <p class="last-child">I’m not sure what you mean!</p>
+        </blockquote>
+
+        <p>Or so we thought.</p>
+      |
+    end
+  end
+
+  test "should add class to last paragraph of blockquote" do
+    govspeak = "
+    > first line
+    >
+    > last line"
+
+    given_govspeak(govspeak) do
+      assert_html_output %|
+        <blockquote>
+          <p>first line</p>
+
+          <p class="last-child">last line</p>
+        </blockquote>
+      |
+    end
+  end
+
+  test "inline attachment" do
+    attachment = OpenStruct.new(
+      content_id: "2b4d92f3-f8cd-4284-aaaa-25b3a640d26c",
+      id: 456,
+    )
+    govspeak = "[embed:attachments:inline:2b4d92f3-f8cd-4284-aaaa-25b3a640d26c]"
+    rendered = Govspeak::Document.new(govspeak, {attachments: attachment}).to_html
+    assert_match(/<span id=\"attachment_456\" class=\"attachment-inline\">/, rendered)
+  end
+
+  test "attachment" do
+    attachment = OpenStruct.new(
+      url: "www.test.com",
+      content_id: "2b4d92f3-f8cd-4284-aaaa-25b3a640d26c",
+      id: 123,
+      opendocument?: true,
+      order_url: "www.test.com",
+      price: 12.3,
+      csv?: true,
+      unnumbered_command_paper?: true,
+      isbn: 'isbn-123',
+    )
+    govspeak = "[embed:attachments:2b4d92f3-f8cd-4284-aaaa-25b3a640d26c]"
+    rendered = Govspeak::Document.new(govspeak, {attachments: attachment}).to_html
+    assert_match(/<section class=\"attachment embedded\">/, rendered)
+    assert_match(/<div class=\"attachment-thumb\">/, rendered)
+    assert_match(/attachment-123-accessibility-help/, rendered)
+    assert_match(/If you use assistive technology/, rendered)
+    assert_match(/<p class=\"opendocument-help\">/, rendered)
+    assert_match(/<span class=\"price\">£12.30/, rendered)
+    assert_match(/<span class=\"preview\">/, rendered)
+    assert_match(/<span class=\"unnumbered-paper\">/, rendered)
+    assert_match(/<span class=\"references\">/, rendered)
+  end
+
+  test "attachment that isn't provided" do
+    govspeak = "[embed:attachments:906ac8b7-850d-45c6-98e0-9525c680f891]"
+    rendered = Govspeak::Document.new(govspeak).to_html
+    assert_match("", rendered)
+  end
+
+  test "embedded link with link provided" do
+    link = OpenStruct.new(
+      content_id: "5572fee5-f38f-4641-8ffa-64fed9230ad4",
+      url: "https://www.example.com/",
+      title: "Example website"
+    )
+    govspeak = "[embed:link:5572fee5-f38f-4641-8ffa-64fed9230ad4]"
+    rendered = Govspeak::Document.new(govspeak, {links: link}).to_html
+    expected = %r{<a href="https://www.example.com/">Example website</a>}
+    assert_match(expected, rendered)
+  end
+
+  test "embedded link with link not provided" do
+    link = OpenStruct.new(
+      content_id: "e510f1c1-4862-4333-889c-8d3acd443fbf",
+      title: "Example website",
+    )
+    govspeak = "[embed:link:e510f1c1-4862-4333-889c-8d3acd443fbf]"
+    rendered = Govspeak::Document.new(govspeak, {links: link}).to_html
+    assert_match("Example website", rendered)
+  end
+
+  test "embedded link without link object" do
+    govspeak = "[embed:link:0726637c-8c66-47ad-834a-d815cbf51e0e]"
+    rendered = Govspeak::Document.new(govspeak).to_html
+    assert_match("", rendered)
+  end
 end