Strip spaces between elements on inline attachments

Outputting <span class="attachment-inline"> <a href="">test</a> </span>
will output as ` test ` which is often not what an author wants.

This is a rather ugly fix for it.

lib/govspeak.rb

@@ -202,7 +202,9 @@ def insert_strong_inside_p(body, parser=Govspeak::Document)
       next "" unless attachment
       attachment = AttachmentPresenter.new(attachment)
       content = File.read(__dir__ + '/templates/inline_attachment.html.erb')
-      ERB.new(content).result(binding).gsub(/\n/, "")
+      rendered = ERB.new(content).result(binding)
+      # remove new lines with spaces, add a space between link and open bracket
+      rendered.gsub(/\n\s*/, "").gsub(%r{(</a>)(\()}, "\\1 \\2")
     end
 
     def render_image(url, alt_text, caption = nil)

test/govspeak_attachments_inline_test.rb

@@ -60,6 +60,14 @@ def render_govspeak(govspeak, attachments = [])
     assert_match(%r{<span id="attachment[^\n]*</span>}, rendered)
   end
 
+  test "doesn't have spaces between the span and the link" do
+    rendered = render_govspeak(
+      "[embed:attachments:inline:2bc1]",
+      [build_attachment(content_id: "2bc1", id: nil)]
+    )
+    assert_match(%r{<span class="attachment-inline"><a href=".*">.*</a></span>}, rendered)
+  end
+
   test "will show HTML type (in brackets) if file_extension is specified as html" do
     rendered = render_govspeak(
       "[embed:attachments:inline:1fe8]",
@@ -129,7 +137,7 @@ def render_govspeak(govspeak, attachments = [])
     type = %{<span class="type">Plain text</span>}
     file_size = %{<span class="file-size">2 KB</span>}
     pages = %{<span class="page-length">2 pages</span>}
-    assert_match(/#{link}\s*\(#{type}, #{file_size}, #{pages}\)/, rendered)
+    assert_match(/#{link}\s+\(#{type}, #{file_size}, #{pages}\)/, rendered)
   end
 
   test "can render two inline attachments on the same line" do