govspeak 6.5.0 → 6.5.5

data/lib/govspeak/header_extractor.rb

@@ -20,7 +20,7 @@ module Govspeak
   private
 
     def id(element)
-      element.attr.fetch('id', generate_id(element.options[:raw_text]))
+      element.attr.fetch("id", generate_id(element.options[:raw_text]))
     end
 
     def build_header(element)

data/lib/govspeak/html_sanitizer.rb

@@ -1,4 +1,4 @@
-require 'addressable/uri'
+require "addressable/uri"
 
 class Govspeak::HtmlSanitizer
   class ImageSourceWhitelister
@@ -10,7 +10,7 @@ class Govspeak::HtmlSanitizer
       return unless sanitize_context[:node_name] == "img"
 
       node = sanitize_context[:node]
-      image_uri = Addressable::URI.parse(node['src'])
+      image_uri = Addressable::URI.parse(node["src"])
       unless image_uri.relative? || @allowed_image_hosts.include?(image_uri.host)
         node.unlink # the node isn't sanitary. Remove it from the document.
       end
@@ -25,8 +25,8 @@ class Govspeak::HtmlSanitizer
 
       # Kramdown uses text-align to allow table cells to be aligned
       # http://kramdown.gettalong.org/quickref.html#tables
-      if invalid_style_attribute?(node['style'])
-        node.remove_attribute('style')
+      if invalid_style_attribute?(node["style"])
+        node.remove_attribute("style")
       end
     end
 
@@ -51,14 +51,17 @@ class Govspeak::HtmlSanitizer
   def sanitize_config
     Sanitize::Config.merge(
       Sanitize::Config::RELAXED,
-      elements: Sanitize::Config::RELAXED[:elements] + %w[govspeak-embed-attachment govspeak-embed-attachment-link],
+      elements: Sanitize::Config::RELAXED[:elements] + %w[govspeak-embed-attachment govspeak-embed-attachment-link svg path],
       attributes: {
         :all => Sanitize::Config::RELAXED[:attributes][:all] + %w[role aria-label],
-        "a"  => Sanitize::Config::RELAXED[:attributes]["a"] + [:data],
-        "th"  => Sanitize::Config::RELAXED[:attributes]["th"] + %w[style],
-        "td"  => Sanitize::Config::RELAXED[:attributes]["td"] + %w[style],
+        "a" => Sanitize::Config::RELAXED[:attributes]["a"] + [:data],
+        "svg" => Sanitize::Config::RELAXED[:attributes][:all] + %w[xmlns width height viewbox focusable],
+        "path" => Sanitize::Config::RELAXED[:attributes][:all] + %w[fill d],
+        "div" => [:data],
+        "th" => Sanitize::Config::RELAXED[:attributes]["th"] + %w[style],
+        "td" => Sanitize::Config::RELAXED[:attributes]["td"] + %w[style],
         "govspeak-embed-attachment" => %w[content-id],
-      }
+      },
     )
   end
 end

data/lib/govspeak/kramdown_overrides.rb

@@ -1,9 +1,9 @@
 module Govspeak
   module KramdownOverrides
-    # This depends on two internal parts of Kramdown.
+    # This depends on two internal parts of Kramdown.
     # 1. Parser registry (kramdown/parser/kramdown.rb#define_parser)
     # 2. Kramdown list regexes (kramdown/parser/kramdown/list.rb)
-    # Updating the Kramdown gem therefore also means updating this file to to
+    # Updating the Kramdown gem therefore also means updating this file to to
     # match Kramdown's internals.
 
     def self.with_kramdown_ordered_lists_disabled

data/lib/govspeak/link_extractor.rb

@@ -14,14 +14,14 @@ module Govspeak
     attr_reader :document, :website_root
 
     def extract_links
-      document_anchors.
-        map { |link| extract_href_from_link(link) }.
-        reject(&:blank?)
+      document_anchors
+        .map { |link| extract_href_from_link(link) }
+        .reject(&:blank?)
     end
 
     def extract_href_from_link(link)
-      href = link['href'] || ''
-      if website_root && href.start_with?('/')
+      href = link["href"] || ""
+      if website_root && href.start_with?("/")
         "#{website_root}#{href}"
       else
         href
@@ -29,7 +29,7 @@ module Govspeak
     end
 
     def document_anchors
-      processed_govspeak.css('a[href]').css('a:not([href^="mailto"])').css('a:not([href^="#"])')
+      processed_govspeak.css("a[href]").css('a:not([href^="mailto"])').css('a:not([href^="#"])')
     end
 
     def processed_govspeak

data/lib/govspeak/post_processor.rb

@@ -2,8 +2,8 @@ module Govspeak
   class PostProcessor
     @extensions = []
 
-    def self.extensions
-      @extensions
+    class << self
+      attr_reader :extensions
     end
 
     def self.process(html, govspeak_document)
@@ -38,11 +38,11 @@ module Govspeak
         el.children = xml
           .gsub(
             %r{&lt;(div class="img")&gt;(.*?)&lt;(/div)&gt;},
-            "<\\1>\\2<\\3>"
+            "<\\1>\\2<\\3>",
           )
           .gsub(
             %r{&lt;(figcaption)&gt;(.*?)&lt;(/figcaption&)gt;},
-            "<\\1>\\2<\\3>"
+            "<\\1>\\2<\\3>",
           )
       end
     end
@@ -59,7 +59,7 @@ module Govspeak
         attachment_html = GovukPublishingComponents.render(
           "govuk_publishing_components/components/attachment",
           attachment: attachment,
-          locale: govspeak_document.locale
+          locale: govspeak_document.locale,
         )
         el.swap(attachment_html)
       end
@@ -77,7 +77,7 @@ module Govspeak
         attachment_html = GovukPublishingComponents.render(
           "govuk_publishing_components/components/attachment_link",
           attachment: attachment,
-          locale: govspeak_document.locale
+          locale: govspeak_document.locale,
         )
         el.swap(attachment_html)
       end
@@ -85,18 +85,38 @@ module Govspeak
 
     extension("Add table headers and row / column scopes") do |document|
       document.css("thead th").map do |el|
-        el.content = el.content.gsub(/^# /, '')
-        el.content = el.content.gsub(/[[:space:]]/, '') if el.content.blank? # Removes a strange whitespace in the cell if the cell is already blank.
-        el.name = 'td' if el.content.blank? # This prevents a `th` with nothing inside it; a `td` is preferable.
+        el.content = el.content.gsub(/^# /, "")
+        el.content = el.content.gsub(/[[:space:]]/, "") if el.content.blank? # Removes a strange whitespace in the cell if the cell is already blank.
+        el.name = "td" if el.content.blank? # This prevents a `th` with nothing inside it; a `td` is preferable.
         el[:scope] = "col" if el.content.present? # `scope` shouldn't be used if there's nothing in the table heading.
       end
 
       document.css(":not(thead) tr td:first-child").map do |el|
-        if el.content.match?(/^#($|\s.*$)/)
-          el.content = el.content.gsub(/^#($|\s)/, '') # Replace '# ' and '#', but not '#Word'.
-          el.name = 'th' if el.content.present? # This also prevents a `th` with nothing inside it; a `td` is preferable.
-          el[:scope] = 'row' if el.content.present? # `scope` shouldn't be used if there's nothing in the table heading.
-        end
+        next unless el.content.match?(/^#($|\s.*$)/)
+
+        # Replace '# ' and '#', but not '#Word'.
+        # This runs on the first child of the element to preserve any links
+        el.children.first.content = el.children.first.content.gsub(/^#($|\s)/, "")
+        el.name = "th" if el.content.present? # This also prevents a `th` with nothing inside it; a `td` is preferable.
+        el[:scope] = "row" if el.content.present? # `scope` shouldn't be used if there's nothing in the table heading.
+      end
+    end
+
+    extension("use gem component for buttons") do |document|
+      document.css(".govuk-button").map do |el|
+        button_html = GovukPublishingComponents.render(
+          "govuk_publishing_components/components/button",
+          text: el.content,
+          href: el["href"],
+          start: el["data-start"],
+          data_attributes: {
+            module: el["data-module"],
+            "tracking-code": el["data-tracking-code"],
+            "tracking-name": el["data-tracking-name"],
+          },
+        ).squish.gsub("> <", "><").gsub!(/\s+/, " ")
+
+        el.swap(button_html)
       end
     end
 

data/lib/govspeak/presenters/attachment_presenter.rb

@@ -34,15 +34,15 @@ module Govspeak
     def attachment_attributes
       attributes = []
       if file_extension == "html"
-        attributes << content_tag(:span, 'HTML', class: 'type')
+        attributes << content_tag(:span, "HTML", class: "type")
       elsif attachment[:external?]
-        attributes << content_tag(:span, url, class: 'url')
+        attributes << content_tag(:span, url, class: "url")
       else
-        attributes << content_tag(:span, humanized_content_type(file_extension), class: 'type') if file_extension
-        attributes << content_tag(:span, number_to_human_size(attachment[:file_size]), class: 'file-size') if attachment[:file_size]
-        attributes << content_tag(:span, pluralize(attachment[:number_of_pages], "page"), class: 'page-length') if attachment[:number_of_pages]
+        attributes << content_tag(:span, humanized_content_type(file_extension), class: "type") if file_extension
+        attributes << content_tag(:span, number_to_human_size(attachment[:file_size]), class: "file-size") if attachment[:file_size]
+        attributes << content_tag(:span, pluralize(attachment[:number_of_pages], "page"), class: "page-length") if attachment[:number_of_pages]
       end
-      attributes.join(', ').html_safe
+      attributes.join(", ").html_safe
     end
 
     MS_WORD_DOCUMENT_HUMANIZED_CONTENT_TYPE = "MS Word Document".freeze
@@ -55,46 +55,46 @@ module Govspeak
 
     def humanized_content_type(file_extension)
       file_extension_vs_humanized_content_type = {
-        "chm"  => file_abbr_tag('CHM', 'Microsoft Compiled HTML Help'),
-        "csv"  => file_abbr_tag('CSV', 'Comma-separated Values'),
-        "diff" => file_abbr_tag('DIFF', 'Plain text differences'),
-        "doc"  => MS_WORD_DOCUMENT_HUMANIZED_CONTENT_TYPE,
+        "chm" => file_abbr_tag("CHM", "Microsoft Compiled HTML Help"),
+        "csv" => file_abbr_tag("CSV", "Comma-separated Values"),
+        "diff" => file_abbr_tag("DIFF", "Plain text differences"),
+        "doc" => MS_WORD_DOCUMENT_HUMANIZED_CONTENT_TYPE,
         "docx" => MS_WORD_DOCUMENT_HUMANIZED_CONTENT_TYPE,
-        "dot"  => file_abbr_tag('DOT', 'MS Word Document Template'),
-        "dxf"  => file_abbr_tag('DXF', 'AutoCAD Drawing Exchange Format'),
-        "eps"  => file_abbr_tag('EPS', 'Encapsulated PostScript'),
-        "gif"  => file_abbr_tag('GIF', 'Graphics Interchange Format'),
-        "gml"  => file_abbr_tag('GML', 'Geography Markup Language'),
-        "html" => file_abbr_tag('HTML', 'Hypertext Markup Language'),
-        "ics" => file_abbr_tag('ICS', 'iCalendar file'),
-        "jpg"  => "JPEG",
-        "odp"  => file_abbr_tag('ODP', 'OpenDocument Presentation'),
-        "ods"  => file_abbr_tag('ODS', 'OpenDocument Spreadsheet'),
-        "odt"  => file_abbr_tag('ODT', 'OpenDocument Text document'),
-        "pdf"  => file_abbr_tag('PDF', 'Portable Document Format'),
-        "png"  => file_abbr_tag('PNG', 'Portable Network Graphic'),
-        "ppt"  => MS_POWERPOINT_PRESENTATION_HUMANIZED_CONTENT_TYPE,
+        "dot" => file_abbr_tag("DOT", "MS Word Document Template"),
+        "dxf" => file_abbr_tag("DXF", "AutoCAD Drawing Exchange Format"),
+        "eps" => file_abbr_tag("EPS", "Encapsulated PostScript"),
+        "gif" => file_abbr_tag("GIF", "Graphics Interchange Format"),
+        "gml" => file_abbr_tag("GML", "Geography Markup Language"),
+        "html" => file_abbr_tag("HTML", "Hypertext Markup Language"),
+        "ics" => file_abbr_tag("ICS", "iCalendar file"),
+        "jpg" => "JPEG",
+        "odp" => file_abbr_tag("ODP", "OpenDocument Presentation"),
+        "ods" => file_abbr_tag("ODS", "OpenDocument Spreadsheet"),
+        "odt" => file_abbr_tag("ODT", "OpenDocument Text document"),
+        "pdf" => file_abbr_tag("PDF", "Portable Document Format"),
+        "png" => file_abbr_tag("PNG", "Portable Network Graphic"),
+        "ppt" => MS_POWERPOINT_PRESENTATION_HUMANIZED_CONTENT_TYPE,
         "pptx" => MS_POWERPOINT_PRESENTATION_HUMANIZED_CONTENT_TYPE,
-        "ps"   => file_abbr_tag('PS', 'PostScript'),
-        "rdf"  => file_abbr_tag('RDF', 'Resource Description Framework'),
-        "rtf"  => file_abbr_tag('RTF', 'Rich Text Format'),
-        "sch"  => file_abbr_tag('SCH', 'XML based Schematic'),
-        "txt"  => "Plain text",
-        "wsdl" => file_abbr_tag('WSDL', 'Web Services Description Language'),
-        "xls"  => MS_EXCEL_SPREADSHEET_HUMANIZED_CONTENT_TYPE,
-        "xlsm" => file_abbr_tag('XLSM', 'MS Excel Macro-Enabled Workbook'),
+        "ps" => file_abbr_tag("PS", "PostScript"),
+        "rdf" => file_abbr_tag("RDF", "Resource Description Framework"),
+        "rtf" => file_abbr_tag("RTF", "Rich Text Format"),
+        "sch" => file_abbr_tag("SCH", "XML based Schematic"),
+        "txt" => "Plain text",
+        "wsdl" => file_abbr_tag("WSDL", "Web Services Description Language"),
+        "xls" => MS_EXCEL_SPREADSHEET_HUMANIZED_CONTENT_TYPE,
+        "xlsm" => file_abbr_tag("XLSM", "MS Excel Macro-Enabled Workbook"),
         "xlsx" => MS_EXCEL_SPREADSHEET_HUMANIZED_CONTENT_TYPE,
-        "xlt"  => file_abbr_tag('XLT', 'MS Excel Spreadsheet Template'),
-        "xsd"  => file_abbr_tag('XSD', 'XML Schema'),
-        "xslt" => file_abbr_tag('XSLT', 'Extensible Stylesheet Language Transformation'),
-        "zip"  => file_abbr_tag('ZIP', 'Zip archive'),
+        "xlt" => file_abbr_tag("XLT", "MS Excel Spreadsheet Template"),
+        "xsd" => file_abbr_tag("XSD", "XML Schema"),
+        "xslt" => file_abbr_tag("XSLT", "Extensible Stylesheet Language Transformation"),
+        "zip" => file_abbr_tag("ZIP", "Zip archive"),
       }
-      file_extension_vs_humanized_content_type.fetch(file_extension.to_s.downcase, '')
+      file_extension_vs_humanized_content_type.fetch(file_extension.to_s.downcase, "")
     end
 
     def link(body, url, options = {})
-      options_str = options.map { |k, v| %{#{encode(k)}="#{encode(v)}"} }.join(" ")
-      %{<a href="#{encode(url)}" #{options_str}>#{body}</a>}
+      options_str = options.map { |k, v| %(#{encode(k)}="#{encode(v)}") }.join(" ")
+      %(<a href="#{encode(url)}" #{options_str}>#{body}</a>)
     end
 
   private

data/lib/govspeak/presenters/contact_presenter.rb

@@ -1,5 +1,5 @@
-require 'active_support/core_ext/array'
-require 'active_support/core_ext/hash'
+require "active_support/core_ext/array"
+require "active_support/core_ext/hash"
 
 module Govspeak
   class ContactPresenter

data/lib/govspeak/presenters/h_card_presenter.rb

@@ -2,7 +2,7 @@ module Govspeak
   class HCardPresenter
     def self.address_formats
       @address_formats ||= YAML.load_file(
-        File.expand_path('config/address_formats.yml', Govspeak.root)
+        File.expand_path("config/address_formats.yml", Govspeak.root),
       )
     end
 
@@ -44,7 +44,7 @@ module Govspeak
         address.gsub!(/\{\{#{hcard_name}\}\}/, interpolate_address_property(our_name, hcard_name))
       end
 
-      address.gsub(/^\n/, '')         # get  rid of blank lines
+      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
@@ -55,7 +55,7 @@ module Govspeak
     end
 
     def default_format_string
-      self.class.address_formats['gb']
+      self.class.address_formats["gb"]
     end
   end
 end

data/lib/govspeak/presenters/image_presenter.rb

@@ -32,10 +32,10 @@ module Govspeak
 
     def figcaption_html
       lines = []
-      lines << '<figcaption>'
-      lines << %{<p>#{caption}</p>} if caption.present?
-      lines << %{<p>#{I18n.t('govspeak.image.figure.credit', credit: credit)}</p>} if credit.present?
-      lines << '</figcaption>'
+      lines << "<figcaption>"
+      lines << %(<p>#{caption}</p>) if caption.present?
+      lines << %(<p>#{I18n.t('govspeak.image.figure.credit', credit: credit)}</p>) if credit.present?
+      lines << "</figcaption>"
       lines.join
     end
   end

data/lib/govspeak/structured_header_extractor.rb

@@ -48,9 +48,9 @@ module Govspeak
     private :doc, :stack, :structured_headers
 
     def headers_list
-      @headers_list ||= doc.headers.map { |h|
+      @headers_list ||= doc.headers.map do |h|
         StructuredHeader.new(h.text, h.level, h.id, [])
-      }
+      end
     end
 
     def add_top_level(header)

data/lib/govspeak/version.rb

@@ -1,3 +1,3 @@
 module Govspeak
-  VERSION = "6.5.0".freeze
+  VERSION = "6.5.5".freeze
 end

data/lib/kramdown/parser/govuk.rb

@@ -9,7 +9,7 @@ module Kramdown
       end
     end
 
-    define(:document_domains, Object, %w{www.gov.uk}, <<~DESCRIPTION) do |val|
+    define(:document_domains, Object, %w[www.gov.uk], <<~DESCRIPTION) do |val|
       Defines the domains which are considered local to the document
 
       Default: www.gov.uk
@@ -21,10 +21,10 @@ module Kramdown
 
   module Parser
     class Govuk < Kramdown::Parser::Kramdown
-      CUSTOM_INLINE_ELEMENTS = %w(govspeak-embed-attachment-link).freeze
+      CUSTOM_INLINE_ELEMENTS = %w[govspeak-embed-attachment-link].freeze
 
       def initialize(source, options)
-        @document_domains = options[:document_domains] || %w(www.gov.uk)
+        @document_domains = options[:document_domains] || %w[www.gov.uk]
         super
       end
 
@@ -33,13 +33,12 @@ module Kramdown
           begin
             host = Addressable::URI.parse(href).host
             unless host.nil? || @document_domains.compact.include?(host)
-              element.attr['rel'] = 'external'
+              element.attr["rel"] = "external"
             end
-          # rubocop:disable Lint/HandleExceptions
-          rescue Addressable::URI::InvalidURIError
+          rescue Addressable::URI::InvalidURIError # rubocop:disable Lint/SuppressedException
             # it's safe to ignore these very *specific* exceptions
           end
-          # rubocop:enable Lint/HandleExceptions
+
         end
         super
       end

data/test/blockquote_extra_quote_remover_test.rb

@@ -1,5 +1,3 @@
-# encoding: UTF-8
-
 require "test_helper"
 
 class BlockquoteExtraQuoteRemoverTest < Minitest::Test
@@ -18,81 +16,81 @@ class BlockquoteExtraQuoteRemoverTest < Minitest::Test
   end
 
   test "ignores text without double quotes" do
-    assert_leaves_untouched %{no quotes\na few lines,\n\n\n\nbut no quotes\n\n}
+    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"}
+    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.}
+      %(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.}
+      %(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.}
+      %(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}
+      %(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.}
+      %(>"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 "handles space after a quote" do
     assert_remover_transforms(
-      %{>" Test"} =>
-      %{> Test}
+      %(>" Test") =>
+      %(> Test),
     )
   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.}
+      %(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.}
+      %(> ""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.}
+      %(> "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}
-      )
+      %(> "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})
+    assert_leaves_untouched(%(> \n> blah))
   end
 end