govspeak 6.8.3 → 6.8.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ca0f29b1dceee03154a3f7f535e27c5bf7ebf5ae5e42845919c7bfea3eb2e132
-  data.tar.gz: 428ff21aa80eaccd670ee643aed80d3081148417161f1a1bf10b78742e01c6b4
+  metadata.gz: 35ed6a74578a19fb89e4c855494d27708d7809e925a6029b5b391fbe474f5d49
+  data.tar.gz: 74f0b44824ff948767ae8e8f1a5f95ae784f07ea3ab424f66a46ee0d07efb6af
 SHA512:
-  metadata.gz: cef697b4026db708378ede49588032d2b95e603a4f4ce87053b8c47ce119ff94b6ad5363e4ec7079d914941c48db0fb6d93ed3b75fff1225459957ed3d24736a
-  data.tar.gz: e78af326c264e4d4d92dedebb23a8db4a08927e55101f435a4ebe4714142e5432262bf1b1ce45553e83f6bb969af859ee4bcba7388a62fc63c987cc83f86ad99
+  metadata.gz: c022e6a898d40e41ea571d9f39863f6235822810d2823f99d17e58eb559ab7e0b2b35c2a60cdec627270b9fe3d31c39224f820e5e8f3378f8df811478ca6203d
+  data.tar.gz: facb052f32abfcdc7e54fb6799d7e052d1a951aedce6db6b582fa716654bb6cc6730ff99af65a2af511dc0039b4f07f8f9669accbadf8821b01f2280c22933c8

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 6.8.4
+
+* Fix bug where inconsistent linebreaks trigger validations [#250](https://github.com/alphagov/govspeak/pull/250)
+
 ## 6.8.3
 
 * Require Kramdown minimum version of 2.3.1 to avoid CVE-2021-28834 [#246](https://github.com/alphagov/govspeak/pull/246)

data/lib/govspeak/html_validator.rb

@@ -20,7 +20,7 @@ private
 
   # Make whitespace in html tags consistent
   def normalise_html(html)
-    Nokogiri::HTML5.fragment(html).to_s
+    Nokogiri::HTML5.fragment(html).to_s.gsub("\n", "")
   end
 
   def govspeak_to_html(sanitize:)

data/lib/govspeak/version.rb

@@ -1,3 +1,3 @@
 module Govspeak
-  VERSION = "6.8.3".freeze
+  VERSION = "6.8.4".freeze
 end

data/test/html_validator_test.rb

@@ -108,4 +108,9 @@ class HtmlValidatorTest < Minitest::Test
     assert Govspeak::HtmlValidator.new("<table><tr><td>Hello</td></tr></table>").valid?, "No <tbody> is valid"
     assert Govspeak::HtmlValidator.new("<table><tbody><tr><td>Hello</td></tr></tbody></table>").valid?, "<tbody> is valid"
   end
+
+  test "allow HTML tables with \n" do
+    html = "<table>\n<thead>\n<tr>\n<th></th>\n<th></th>\n<th colspan=\"3\"></th>\n\n</tr>\n</thead>\n<tr>\n<th></th>\n<td></td>\n<td></td>\n<td></td>\n<td></td>\n</tr>\n</table>"
+    assert Govspeak::HtmlValidator.new(html).valid?, 'Tables with no <tbody> and \n are valid'
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: govspeak
 version: !ruby/object:Gem::Version
-  version: 6.8.3
+  version: 6.8.4
 platform: ruby
 authors:
 - GOV.UK Dev
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-06-21 00:00:00.000000000 Z
+date: 2022-08-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionview
@@ -190,14 +190,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 4.5.0
+        version: 4.6.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 4.5.0
+        version: 4.6.0
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
@@ -327,29 +327,29 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.16
+rubygems_version: 3.3.20
 signing_key: 
 specification_version: 4
 summary: Markup language for single domain
 test_files:
-- test/govspeak_attachment_link_test.rb
-- test/test_helper.rb
-- test/govspeak_button_test.rb
-- test/govspeak_footnote_test.rb
-- test/govspeak_images_test.rb
-- test/govspeak_images_bang_test.rb
-- test/blockquote_extra_quote_remover_test.rb
-- test/html_sanitizer_test.rb
+- test/govspeak_attachments_image_test.rb
+- test/govspeak_link_extractor_test.rb
 - test/govspeak_table_with_headers_test.rb
+- test/govspeak_attachment_test.rb
 - test/govspeak_extract_contact_content_ids_test.rb
+- test/test_helper.rb
 - test/html_validator_test.rb
-- test/govspeak_contacts_test.rb
+- test/govspeak_test_helper.rb
+- test/html_sanitizer_test.rb
+- test/govspeak_footnote_test.rb
+- test/govspeak_images_test.rb
+- test/govspeak_button_test.rb
 - test/govspeak_test.rb
-- test/govspeak_attachments_image_test.rb
-- test/govspeak_attachment_test.rb
-- test/govspeak_link_extractor_test.rb
-- test/govspeak_link_test.rb
+- test/blockquote_extra_quote_remover_test.rb
+- test/govspeak_images_bang_test.rb
 - test/presenters/h_card_presenter_test.rb
+- test/govspeak_contacts_test.rb
+- test/govspeak_attachment_link_test.rb
 - test/govspeak_attachments_inline_test.rb
-- test/govspeak_test_helper.rb
+- test/govspeak_link_test.rb
 - test/govspeak_structured_headers_test.rb