sanitize 6.1.3 → 7.0.0

data/test/test_clean_comment.rb

@@ -1,47 +1,48 @@
-# encoding: utf-8
-require_relative 'common'
+# frozen_string_literal: true
 
-describe 'Sanitize::Transformers::CleanComment' do
+require_relative "common"
+
+describe "Sanitize::Transformers::CleanComment" do
   make_my_diffs_pretty!
   parallelize_me!
 
-  describe 'when :allow_comments is false' do
+  describe "when :allow_comments is false" do
     before do
-      @s = Sanitize.new(:allow_comments => false, :elements => ['div'])
+      @s = Sanitize.new(allow_comments: false, elements: ["div"])
     end
 
-    it 'should remove comments' do
-      _(@s.fragment('foo <!-- comment --> bar')).must_equal 'foo  bar'
-      _(@s.fragment('foo <!-- ')).must_equal 'foo '
-      _(@s.fragment('foo <!-- - -> bar')).must_equal 'foo '
-      _(@s.fragment("foo <!--\n\n\n\n-->bar")).must_equal 'foo bar'
-      _(@s.fragment("foo <!-- <!-- <!-- --> --> -->bar")).must_equal 'foo  --&gt; --&gt;bar'
-      _(@s.fragment("foo <div <!-- comment -->>bar</div>")).must_equal 'foo <div>&gt;bar</div>'
+    it "should remove comments" do
+      _(@s.fragment("foo <!-- comment --> bar")).must_equal "foo  bar"
+      _(@s.fragment("foo <!-- ")).must_equal "foo "
+      _(@s.fragment("foo <!-- - -> bar")).must_equal "foo "
+      _(@s.fragment("foo <!--\n\n\n\n-->bar")).must_equal "foo bar"
+      _(@s.fragment("foo <!-- <!-- <!-- --> --> -->bar")).must_equal "foo  --&gt; --&gt;bar"
+      _(@s.fragment("foo <div <!-- comment -->>bar</div>")).must_equal "foo <div>&gt;bar</div>"
 
       # Special case: the comment markup is inside a <script>, which makes it
       # text content and not an actual HTML comment.
-      _(@s.fragment("<script><!-- comment --></script>")).must_equal ''
+      _(@s.fragment("<script><!-- comment --></script>")).must_equal ""
 
-      _(Sanitize.fragment("<script><!-- comment --></script>", :allow_comments => false, :elements => ['script']))
-        .must_equal '<script><!-- comment --></script>'
+      _(Sanitize.fragment("<script><!-- comment --></script>", allow_comments: false, elements: ["script"]))
+        .must_equal "<script><!-- comment --></script>"
     end
   end
 
-  describe 'when :allow_comments is true' do
+  describe "when :allow_comments is true" do
     before do
-      @s = Sanitize.new(:allow_comments => true, :elements => ['div'])
+      @s = Sanitize.new(allow_comments: true, elements: ["div"])
     end
 
-    it 'should allow comments' do
-      _(@s.fragment('foo <!-- comment --> bar')).must_equal 'foo <!-- comment --> bar'
-      _(@s.fragment('foo <!-- ')).must_equal 'foo <!-- -->'
-      _(@s.fragment('foo <!-- - -> bar')).must_equal 'foo <!-- - -> bar-->'
+    it "should allow comments" do
+      _(@s.fragment("foo <!-- comment --> bar")).must_equal "foo <!-- comment --> bar"
+      _(@s.fragment("foo <!-- ")).must_equal "foo <!-- -->"
+      _(@s.fragment("foo <!-- - -> bar")).must_equal "foo <!-- - -> bar-->"
       _(@s.fragment("foo <!--\n\n\n\n-->bar")).must_equal "foo <!--\n\n\n\n-->bar"
-      _(@s.fragment("foo <!-- <!-- <!-- --> --> -->bar")).must_equal 'foo <!-- <!-- <!-- --> --&gt; --&gt;bar'
-      _(@s.fragment("foo <div <!-- comment -->>bar</div>")).must_equal 'foo <div>&gt;bar</div>'
+      _(@s.fragment("foo <!-- <!-- <!-- --> --> -->bar")).must_equal "foo <!-- <!-- <!-- --> --&gt; --&gt;bar"
+      _(@s.fragment("foo <div <!-- comment -->>bar</div>")).must_equal "foo <div>&gt;bar</div>"
 
-      _(Sanitize.fragment("<script><!-- comment --></script>", :allow_comments => true, :elements => ['script']))
-        .must_equal '<script><!-- comment --></script>'
+      _(Sanitize.fragment("<script><!-- comment --></script>", allow_comments: true, elements: ["script"]))
+        .must_equal "<script><!-- comment --></script>"
     end
   end
 end

data/test/test_clean_css.rb

@@ -1,7 +1,8 @@
-# encoding: utf-8
-require_relative 'common'
+# frozen_string_literal: true
 
-describe 'Sanitize::Transformers::CSS::CleanAttribute' do
+require_relative "common"
+
+describe "Sanitize::Transformers::CSS::CleanAttribute" do
   make_my_diffs_pretty!
   parallelize_me!
 
@@ -9,21 +10,21 @@ describe 'Sanitize::Transformers::CSS::CleanAttribute' do
     @s = Sanitize.new(Sanitize::Config::RELAXED)
   end
 
-  it 'should sanitize CSS properties in style attributes' do
+  it "should sanitize CSS properties in style attributes" do
     _(@s.fragment(%[
       <div style="color: #fff; width: expression(alert(1)); /* <-- evil! */"></div>
-    ].strip)).must_equal %[
+    ].strip)).must_equal %(
       <div style="color: #fff;  /* <-- evil! */"></div>
-    ].strip
+    ).strip
   end
 
-  it 'should remove the style attribute if the sanitized CSS is empty' do
-    _(@s.fragment('<div style="width: expression(alert(1))"></div>')).
-      must_equal '<div></div>'
+  it "should remove the style attribute if the sanitized CSS is empty" do
+    _(@s.fragment('<div style="width: expression(alert(1))"></div>'))
+      .must_equal "<div></div>"
   end
 end
 
-describe 'Sanitize::Transformers::CSS::CleanElement' do
+describe "Sanitize::Transformers::CSS::CleanElement" do
   make_my_diffs_pretty!
   parallelize_me!
 
@@ -31,7 +32,7 @@ describe 'Sanitize::Transformers::CSS::CleanElement' do
     @s = Sanitize.new(Sanitize::Config::RELAXED)
   end
 
-  it 'should sanitize CSS stylesheets in <style> elements' do
+  it "should sanitize CSS stylesheets in <style> elements" do
     html = %[
       <style>@import url(evil.css);
       /* Yay CSS! */
@@ -61,7 +62,7 @@ describe 'Sanitize::Transformers::CSS::CleanElement' do
     ].strip
   end
 
-  it 'should remove the <style> element if the sanitized CSS is empty' do
-    _(@s.fragment('<style></style>')).must_equal ''
+  it "should remove the <style> element if the sanitized CSS is empty" do
+    _(@s.fragment("<style></style>")).must_equal ""
   end
 end

data/test/test_clean_doctype.rb

@@ -1,22 +1,23 @@
-# encoding: utf-8
-require_relative 'common'
+# frozen_string_literal: true
 
-describe 'Sanitize::Transformers::CleanDoctype' do
+require_relative "common"
+
+describe "Sanitize::Transformers::CleanDoctype" do
   make_my_diffs_pretty!
   parallelize_me!
 
-  describe 'when :allow_doctype is false' do
+  describe "when :allow_doctype is false" do
     before do
-      @s = Sanitize.new(:allow_doctype => false, :elements => ['html'])
+      @s = Sanitize.new(allow_doctype: false, elements: ["html"])
     end
 
-    it 'should remove doctype declarations' do
-      _(@s.document('<!DOCTYPE html><html>foo</html>')).must_equal "<html>foo</html>"
-      _(@s.fragment('<!DOCTYPE html>foo')).must_equal 'foo'
+    it "should remove doctype declarations" do
+      _(@s.document("<!DOCTYPE html><html>foo</html>")).must_equal "<html>foo</html>"
+      _(@s.fragment("<!DOCTYPE html>foo")).must_equal "foo"
     end
 
-    it 'should not allow doctype definitions in fragments' do
-      _(@s.fragment('<!DOCTYPE html><html>foo</html>'))
+    it "should not allow doctype definitions in fragments" do
+      _(@s.fragment("<!DOCTYPE html><html>foo</html>"))
         .must_equal "foo"
 
       _(@s.fragment('<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01//EN"><html>foo</html>'))
@@ -27,13 +28,13 @@ describe 'Sanitize::Transformers::CleanDoctype' do
     end
   end
 
-  describe 'when :allow_doctype is true' do
+  describe "when :allow_doctype is true" do
     before do
-      @s = Sanitize.new(:allow_doctype => true, :elements => ['html'])
+      @s = Sanitize.new(allow_doctype: true, elements: ["html"])
     end
 
-    it 'should allow doctype declarations in documents' do
-      _(@s.document('<!DOCTYPE html><html>foo</html>'))
+    it "should allow doctype declarations in documents" do
+      _(@s.document("<!DOCTYPE html><html>foo</html>"))
         .must_equal "<!DOCTYPE html><html>foo</html>"
 
       _(@s.document('<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01//EN"><html>foo</html>'))
@@ -43,22 +44,22 @@ describe 'Sanitize::Transformers::CleanDoctype' do
         .must_equal "<!DOCTYPE html><html>foo</html>"
     end
 
-    it 'should not allow obviously invalid doctype declarations in documents' do
-      _(@s.document('<!DOCTYPE blah blah blah><html>foo</html>'))
+    it "should not allow obviously invalid doctype declarations in documents" do
+      _(@s.document("<!DOCTYPE blah blah blah><html>foo</html>"))
         .must_equal "<!DOCTYPE html><html>foo</html>"
 
-      _(@s.document('<!DOCTYPE blah><html>foo</html>'))
+      _(@s.document("<!DOCTYPE blah><html>foo</html>"))
         .must_equal "<!DOCTYPE html><html>foo</html>"
 
       _(@s.document('<!DOCTYPE html BLAH "-//W3C//DTD HTML 4.01//EN"><html>foo</html>'))
         .must_equal "<!DOCTYPE html><html>foo</html>"
 
-      _(@s.document('<!whatever><html>foo</html>'))
+      _(@s.document("<!whatever><html>foo</html>"))
         .must_equal "<html>foo</html>"
     end
 
-    it 'should not allow doctype definitions in fragments' do
-      _(@s.fragment('<!DOCTYPE html><html>foo</html>'))
+    it "should not allow doctype definitions in fragments" do
+      _(@s.fragment("<!DOCTYPE html><html>foo</html>"))
         .must_equal "foo"
 
       _(@s.fragment('<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01//EN"><html>foo</html>'))