commonmarker 0.19.0 → 0.21.1

data/lib/commonmarker/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module CommonMarker
-  VERSION = '0.19.0'.freeze
+  VERSION = '0.21.1'
 end

data/test/benchmark.rb

@@ -13,16 +13,12 @@ end
 
 benchinput = File.open('test/benchinput.md', 'r').read
 
-printf("input size = %d bytes\n\n", benchinput.bytesize)
+printf("input size = %<bytes>d bytes\n\n", benchinput.bytesize)
 
 dobench('redcarpet') do
   Redcarpet::Markdown.new(Redcarpet::Render::HTML, autolink: false, tables: false).render(benchinput)
 end
 
-dobench('github-markdown') do
-  GitHub::Markdown.render(benchinput)
-end
-
 dobench('commonmarker with to_html') do
   CommonMarker.render_html(benchinput)
 end

data/test/fixtures/strong.md

@@ -0,0 +1 @@
+I am **strong**

data/test/fixtures/table.md

@@ -0,0 +1,10 @@
+One extension:
+
+| a   | b   |
+| --- | --- |
+| c   | d   |
+| **x** | |
+
+Another extension:
+
+~~hi~~

data/test/test_attributes.rb

@@ -4,7 +4,7 @@ require 'test_helper'
 
 class TestAttributes < Minitest::Test
   def setup
-    contents = File.read(File.join(FIXTURES_DIR, 'dingus.md'))
+    contents = fixtures_file('dingus.md')
     @doc = CommonMarker.render_doc(contents.strip)
   end
 
@@ -15,9 +15,9 @@ class TestAttributes < Minitest::Test
       sourcepos << node.sourcepos
     end
 
-    sourcepos.delete_if { |h| h.values.all? { |v| v == 0 } }
+    sourcepos.delete_if { |h| h.values.all?(&:zero?) }
 
-    result = [{start_line: 1, start_column: 1, end_line: 10, end_column: 12}, {start_line: 1, start_column: 1, end_line: 1, end_column: 17}, {start_line: 1, start_column: 4, end_line: 1, end_column: 17}, {start_line: 3, start_column: 1, end_line: 5, end_column: 36}, {start_line: 3, start_column: 1, end_line: 3, end_column: 55}, {start_line: 4, start_column: 1, end_line: 4, end_column: 53}, {start_line: 4, start_column: 2, end_line: 4, end_column: 14}, {start_line: 4, start_column: 54, end_line: 4, end_column: 58}, {start_line: 5, start_column: 1, end_line: 5, end_column: 36}, {start_line: 7, start_column: 1, end_line: 10, end_column: 12}, {start_line: 7, start_column: 1, end_line: 7, end_column: 11}, {start_line: 7, start_column: 4, end_line: 7, end_column: 11}, {start_line: 7, start_column: 4, end_line: 7, end_column: 11}, {start_line: 8, start_column: 1, end_line: 10, end_column: 12}, {start_line: 8, start_column: 4, end_line: 8, end_column: 11}, {start_line: 8, start_column: 4, end_line: 8, end_column: 11}, {start_line: 9, start_column: 4, end_line: 10, end_column: 12}, {start_line: 9, start_column: 4, end_line: 9, end_column: 12}, {start_line: 9, start_column: 6, end_line: 9, end_column: 12}, {start_line: 9, start_column: 6, end_line: 9, end_column: 12}, {start_line: 10, start_column: 4, end_line: 10, end_column: 12}, {start_line: 10, start_column: 6, end_line: 10, end_column: 12}, {start_line: 10, start_column: 6, end_line: 10, end_column: 12}]
+    result = [{ start_line: 1, start_column: 1, end_line: 10, end_column: 12 }, { start_line: 1, start_column: 1, end_line: 1, end_column: 17 }, { start_line: 1, start_column: 4, end_line: 1, end_column: 17 }, { start_line: 3, start_column: 1, end_line: 5, end_column: 36 }, { start_line: 3, start_column: 1, end_line: 3, end_column: 55 }, { start_line: 4, start_column: 1, end_line: 4, end_column: 53 }, { start_line: 4, start_column: 2, end_line: 4, end_column: 14 }, { start_line: 4, start_column: 54, end_line: 4, end_column: 58 }, { start_line: 5, start_column: 1, end_line: 5, end_column: 36 }, { start_line: 7, start_column: 1, end_line: 10, end_column: 12 }, { start_line: 7, start_column: 1, end_line: 7, end_column: 11 }, { start_line: 7, start_column: 4, end_line: 7, end_column: 11 }, { start_line: 7, start_column: 4, end_line: 7, end_column: 11 }, { start_line: 8, start_column: 1, end_line: 10, end_column: 12 }, { start_line: 8, start_column: 4, end_line: 8, end_column: 11 }, { start_line: 8, start_column: 4, end_line: 8, end_column: 11 }, { start_line: 9, start_column: 4, end_line: 10, end_column: 12 }, { start_line: 9, start_column: 4, end_line: 9, end_column: 12 }, { start_line: 9, start_column: 6, end_line: 9, end_column: 12 }, { start_line: 9, start_column: 6, end_line: 9, end_column: 12 }, { start_line: 10, start_column: 4, end_line: 10, end_column: 12 }, { start_line: 10, start_column: 6, end_line: 10, end_column: 12 }, { start_line: 10, start_column: 6, end_line: 10, end_column: 12 }]
 
     assert_equal result, sourcepos
   end

data/test/test_commands.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+require 'test_helper'
+
+class TestCommands < Minitest::Test
+  def test_basic
+    out = make_bin('strong.md')
+    assert_equal out, '<p>I am <strong>strong</strong></p>'
+  end
+
+  def test_does_not_have_extensions
+    out = make_bin('table.md')
+    assert out.include?('| a')
+    refute out.include?('<p><del>hi</del>')
+    refute out.include?('<table> <tr> <th> a </th> <td> c </td>')
+  end
+
+  def test_understands_extensions
+    out = make_bin('table.md', '--extension=table')
+    refute out.include?('| a')
+    refute out.include?('<p><del>hi</del>')
+    %w[<table> <tr> <th> a </th> <td> c </td>].each { |html| assert out.include?(html) }
+  end
+
+  def test_understands_multiple_extensions
+    out = make_bin('table.md', '--extension=table,strikethrough')
+    refute out.include?('| a')
+    assert out.include?('<p><del>hi</del>')
+    %w[<table> <tr> <th> a </th> <td> c </td>].each { |html| assert out.include?(html) }
+  end
+end

data/test/test_commonmark.rb

@@ -3,22 +3,22 @@
 require 'test_helper'
 
 class TestCommonmark < Minitest::Test
-  HTML_COMMENT = %r[<!--.*?-->\s?]
+  HTML_COMMENT = /<!--.*?-->\s?/.freeze
 
   def setup
-    @markdown = <<-MD
-Hi *there*!
+    @markdown = <<~MD
+      Hi *there*!
 
-1. I am a numeric list.
-2. I continue the list.
-* Suddenly, an unordered list!
-* What fun!
+      1. I am a numeric list.
+      2. I continue the list.
+      * Suddenly, an unordered list!
+      * What fun!
 
-Okay, _enough_.
+      Okay, _enough_.
 
-| a   | b   |
-| --- | --- |
-| c   | d   |
+      | a   | b   |
+      | --- | --- |
+      | c   | d   |
     MD
   end
 

data/test/test_encoding.rb

@@ -1,4 +1,3 @@
-# coding: utf-8
 # frozen_string_literal: true
 
 require 'test_helper'
@@ -6,7 +5,7 @@ require 'test_helper'
 class TestEncoding < Minitest::Test
   # see http://git.io/vq4FR
   def test_encoding
-    contents = File.read(File.join(FIXTURES_DIR, 'curly.md'), encoding: 'utf-8')
+    contents = fixtures_file('curly.md')
     doc = CommonMarker.render_doc(contents, :SMART)
     render = doc.to_html
     assert_equal render.rstrip, '<p>This curly quote “makes commonmarker throw an exception”.</p>'

data/test/test_extensions.rb

@@ -4,18 +4,7 @@ require 'test_helper'
 
 class TestExtensions < Minitest::Test
   def setup
-    @markdown = <<-MD
-One extension:
-
-| a   | b   |
-| --- | --- |
-| c   | d   |
-| **x** | |
-
-Another extension:
-
-~~hi~~
-    MD
+    @markdown = fixtures_file('table.md')
   end
 
   def test_uses_specified_extensions
@@ -27,7 +16,7 @@ Another extension:
 
     CommonMarker.render_html(@markdown, :DEFAULT, %i[table]).tap do |out|
       refute out.include?('| a')
-      %w(<table> <tr> <th> a </th> <td> c </td> <strong>x</strong>).each { |html| assert out.include?(html) }
+      %w[<table> <tr> <th> a </th> <td> c </td> <strong>x</strong>].each { |html| assert out.include?(html) }
       assert out.include?('~~hi~~')
     end
 
@@ -45,7 +34,6 @@ Another extension:
       refute out.include?('| <strong>x</strong>')
       refute out.include?('~~hi~~')
     end
-
   end
 
   def test_extensions_with_renderers
@@ -53,13 +41,13 @@ Another extension:
 
     doc.to_html.tap do |out|
       refute out.include?('| a')
-      %w(<table> <tr> <th> a </th> <td> c </td> <strong>x</strong>).each { |html| assert out.include?(html) }
+      %w[<table> <tr> <th> a </th> <td> c </td> <strong>x</strong>].each { |html| assert out.include?(html) }
       assert out.include?('~~hi~~')
     end
 
     HtmlRenderer.new.render(doc).tap do |out|
       refute out.include?('| a')
-      %w(<table> <tr> <th> a </th> <td> c </td> <strong>x</strong>).each { |html| assert out.include?(html) }
+      %w[<table> <tr> <th> a </th> <td> c </td> <strong>x</strong>].each { |html| assert out.include?(html) }
       assert out.include?('~~hi~~')
     end
 
@@ -75,51 +63,51 @@ Another extension:
 
   def test_comments_are_kept_as_expected
     assert_equal "<!--hello--> <blah> &lt;xmp>\n",
-      CommonMarker.render_html("<!--hello--> <blah> <xmp>\n", :UNSAFE, %i[tagfilter])
+                 CommonMarker.render_html("<!--hello--> <blah> <xmp>\n", :UNSAFE, %i[tagfilter])
   end
 
   def test_table_prefer_style_attributes
-      assert_equal(<<-HTML, CommonMarker.render_html(<<-MD, :TABLE_PREFER_STYLE_ATTRIBUTES, %i[table]))
-<table>
-<thead>
-<tr>
-<th style="text-align: left">aaa</th>
-<th>bbb</th>
-<th style="text-align: center">ccc</th>
-<th>ddd</th>
-<th style="text-align: right">eee</th>
-</tr>
-</thead>
-<tbody>
-<tr>
-<td style="text-align: left">fff</td>
-<td>ggg</td>
-<td style="text-align: center">hhh</td>
-<td>iii</td>
-<td style="text-align: right">jjj</td>
-</tr>
-</tbody>
-</table>
-      HTML
-aaa | bbb | ccc | ddd | eee
-:-- | --- | :-: | --- | --:
-fff | ggg | hhh | iii | jjj
-      MD
+    assert_equal(<<~HTML, CommonMarker.render_html(<<~MD, :TABLE_PREFER_STYLE_ATTRIBUTES, %i[table]))
+      <table>
+      <thead>
+      <tr>
+      <th style="text-align: left">aaa</th>
+      <th>bbb</th>
+      <th style="text-align: center">ccc</th>
+      <th>ddd</th>
+      <th style="text-align: right">eee</th>
+      </tr>
+      </thead>
+      <tbody>
+      <tr>
+      <td style="text-align: left">fff</td>
+      <td>ggg</td>
+      <td style="text-align: center">hhh</td>
+      <td>iii</td>
+      <td style="text-align: right">jjj</td>
+      </tr>
+      </tbody>
+      </table>
+    HTML
+      aaa | bbb | ccc | ddd | eee
+      :-- | --- | :-: | --- | --:
+      fff | ggg | hhh | iii | jjj
+    MD
   end
 
   def test_plaintext
-    assert_equal(<<-HTML, CommonMarker.render_doc(<<-MD, :DEFAULT, %i[table strikethrough]).to_plaintext)
-Hello ~there~.
+    assert_equal(<<~HTML, CommonMarker.render_doc(<<~MD, :DEFAULT, %i[table strikethrough]).to_plaintext)
+      Hello ~there~.
 
-| a |
-| --- |
-| b |
+      | a |
+      | --- |
+      | b |
     HTML
-Hello ~~there~~.
+      Hello ~~there~~.
 
-| a |
-| - |
-| b |
+      | a |
+      | - |
+      | b |
     MD
   end
 end

data/test/test_footnotes.rb

@@ -5,15 +5,15 @@ require 'test_helper'
 class TestFootnotes < Minitest::Test
   def setup
     @doc = CommonMarker.render_doc("Hello[^hi].\n\n[^hi]: Hey!\n", :FOOTNOTES)
-    @expected = <<-HTML
-<p>Hello<sup class="footnote-ref"><a href="#fn1" id="fnref1">1</a></sup>.</p>
-<section class="footnotes">
-<ol>
-<li id="fn1">
-<p>Hey! <a href="#fnref1" class="footnote-backref">↩</a></p>
-</li>
-</ol>
-</section>
+    @expected = <<~HTML
+      <p>Hello<sup class="footnote-ref"><a href="#fn1" id="fnref1">1</a></sup>.</p>
+      <section class="footnotes">
+      <ol>
+      <li id="fn1">
+      <p>Hey! <a href="#fnref1" class="footnote-backref">↩</a></p>
+      </li>
+      </ol>
+      </section>
     HTML
   end
 

data/test/test_gc.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+# rubocop:disable Lint/UselessAssignment
 require 'test_helper'
 
 class TestNode < Minitest::Test
@@ -43,3 +44,4 @@ class TestNode < Minitest::Test
     # free.
   end
 end
+# rubocop:enable Lint/UselessAssignment

data/test/test_helper.rb

@@ -1,4 +1,3 @@
-# coding: utf-8
 # frozen_string_literal: true
 
 require 'commonmarker'
@@ -6,10 +5,18 @@ require 'minitest/autorun'
 require 'minitest/pride'
 require 'minitest/focus'
 
-include CommonMarker
+include CommonMarker # rubocop:disable Style/MixinUsage
 
 FIXTURES_DIR = File.join(File.dirname(__FILE__), 'fixtures')
 
+def fixtures_file(file)
+  File.read(File.join(FIXTURES_DIR, file), encoding: 'utf-8')
+end
+
+def make_bin(file, args = '')
+  `ruby bin/commonmarker #{File.join(FIXTURES_DIR, file)} #{args}`.chomp
+end
+
 def open_spec_file(filename)
   line_number = 0
   start_line = 0
@@ -31,7 +38,7 @@ def open_spec_file(filename)
     l = line.strip
     if l =~ /^`{32} example(.*)$/
       state = 1
-      extensions = $1.split
+      extensions = Regexp.last_match(1).split
     elsif l == '`' * 32
       state = 0
       example_number += 1
@@ -43,7 +50,7 @@ def open_spec_file(filename)
         start_line: start_line,
         end_line: end_line,
         section: headertext,
-        extensions: extensions.map(&:to_sym),
+        extensions: extensions.map(&:to_sym)
       }
       start_line = 0
       markdown_lines = []
@@ -51,11 +58,11 @@ def open_spec_file(filename)
     elsif l == '.'
       state = 2
     elsif state == 1
-      start_line = line_number - 1 if start_line == 0
-      markdown_lines << "#{line}"
+      start_line = line_number - 1 if start_line.zero?
+      markdown_lines << line.to_s
     elsif state == 2
-      html_lines << "#{line}"
-    elsif state == 0 && header_re.match(line)
+      html_lines << line.to_s
+    elsif state.zero? && header_re.match(line)
       headertext = line.sub(header_re, '').strip
     end
   end

data/test/test_maliciousness.rb

@@ -2,264 +2,266 @@
 
 require 'test_helper'
 
-class CommonMarker::TestMaliciousness < Minitest::Test
-  def setup
-    @doc = CommonMarker.render_doc('Hi *there*')
-  end
-
-  def test_init_with_bad_type
-    assert_raises TypeError do
-      Node.new(123)
+module CommonMarker
+  class TestMaliciousness < Minitest::Test
+    def setup
+      @doc = CommonMarker.render_doc('Hi *there*')
     end
 
-    assert_raises NodeError do
-      Node.new(:totes_fake)
-    end
+    def test_init_with_bad_type
+      assert_raises TypeError do
+        Node.new(123)
+      end
 
-    assert_raises TypeError do
-      Node.new([])
-    end
+      assert_raises NodeError do
+        Node.new(:totes_fake)
+      end
 
-    assert_raises TypeError do
-      Node.new([23])
-    end
+      assert_raises TypeError do
+        Node.new([])
+      end
 
-    assert_raises TypeError do
-      Node.new(nil)
-    end
-  end
+      assert_raises TypeError do
+        Node.new([23])
+      end
 
-  def test_rendering_with_bad_type
-    assert_raises TypeError do
-      CommonMarker.render_html("foo \n baz", 123)
+      assert_raises TypeError do
+        Node.new(nil)
+      end
     end
 
-    assert_raises TypeError do
-      CommonMarker.render_html("foo \n baz", :totes_fake)
-    end
+    def test_rendering_with_bad_type
+      assert_raises TypeError do
+        CommonMarker.render_html("foo \n baz", 123)
+      end
 
-    assert_raises TypeError do
-      CommonMarker.render_html("foo \n baz", [])
-    end
+      assert_raises TypeError do
+        CommonMarker.render_html("foo \n baz", :totes_fake)
+      end
 
-    assert_raises TypeError do
-      CommonMarker.render_html("foo \n baz", [23])
-    end
+      assert_raises TypeError do
+        CommonMarker.render_html("foo \n baz", [])
+      end
 
-    assert_raises TypeError do
-      CommonMarker.render_html("foo \n baz", nil)
-    end
+      assert_raises TypeError do
+        CommonMarker.render_html("foo \n baz", [23])
+      end
 
-    assert_raises TypeError do
-      CommonMarker.render_html("foo \n baz", [:SMART, 'totes_fake'])
-    end
+      assert_raises TypeError do
+        CommonMarker.render_html("foo \n baz", nil)
+      end
 
-    assert_raises TypeError do
-      CommonMarker.render_html(123)
-    end
+      assert_raises TypeError do
+        CommonMarker.render_html("foo \n baz", [:SMART, 'totes_fake'])
+      end
 
-    assert_raises TypeError do
-      CommonMarker.render_html([123])
-    end
+      assert_raises TypeError do
+        CommonMarker.render_html(123)
+      end
 
-    assert_raises TypeError do
-      CommonMarker.render_html(nil)
-    end
+      assert_raises TypeError do
+        CommonMarker.render_html([123])
+      end
 
-    err = assert_raises TypeError do
-      CommonMarker.render_html("foo \n baz", [:SMART])
-    end
-    assert_equal err.message, 'option \':SMART\' does not exist for CommonMarker::Config::Render'
+      assert_raises TypeError do
+        CommonMarker.render_html(nil)
+      end
 
-    assert_raises TypeError do
-      CommonMarker.render_doc("foo \n baz", 123)
-    end
+      err = assert_raises TypeError do
+        CommonMarker.render_html("foo \n baz", [:SMART])
+      end
+      assert_equal err.message, 'option \':SMART\' does not exist for CommonMarker::Config::Render'
 
-    err = assert_raises TypeError do
-      CommonMarker.render_doc("foo \n baz", :safe)
-    end
-    assert_equal err.message, 'option \':safe\' does not exist for CommonMarker::Config::Parse'
+      assert_raises TypeError do
+        CommonMarker.render_doc("foo \n baz", 123)
+      end
 
-    assert_raises TypeError do
-      CommonMarker.render_doc("foo \n baz", :totes_fake)
-    end
+      err = assert_raises TypeError do
+        CommonMarker.render_doc("foo \n baz", :safe)
+      end
+      assert_equal err.message, 'option \':safe\' does not exist for CommonMarker::Config::Parse'
 
-    assert_raises TypeError do
-      CommonMarker.render_doc("foo \n baz", [])
-    end
+      assert_raises TypeError do
+        CommonMarker.render_doc("foo \n baz", :totes_fake)
+      end
 
-    assert_raises TypeError do
-      CommonMarker.render_doc("foo \n baz", [23])
-    end
+      assert_raises TypeError do
+        CommonMarker.render_doc("foo \n baz", [])
+      end
 
-    assert_raises TypeError do
-      CommonMarker.render_doc("foo \n baz", nil)
-    end
+      assert_raises TypeError do
+        CommonMarker.render_doc("foo \n baz", [23])
+      end
 
-    assert_raises TypeError do
-      CommonMarker.render_doc("foo \n baz", [:SMART, 'totes_fake'])
-    end
+      assert_raises TypeError do
+        CommonMarker.render_doc("foo \n baz", nil)
+      end
 
-    assert_raises TypeError do
-      CommonMarker.render_doc(123)
-    end
+      assert_raises TypeError do
+        CommonMarker.render_doc("foo \n baz", [:SMART, 'totes_fake'])
+      end
 
-    assert_raises TypeError do
-      CommonMarker.render_doc([123])
-    end
+      assert_raises TypeError do
+        CommonMarker.render_doc(123)
+      end
 
-    assert_raises TypeError do
-      CommonMarker.render_doc(nil)
-    end
-  end
+      assert_raises TypeError do
+        CommonMarker.render_doc([123])
+      end
 
-  def test_bad_set_string_content
-    assert_raises TypeError do
-      @doc.string_content = 123
+      assert_raises TypeError do
+        CommonMarker.render_doc(nil)
+      end
     end
-  end
-
-  def test_bad_walking
-    assert_nil @doc.parent
-    assert_nil @doc.previous
-  end
 
-  def test_bad_insertion
-    code = Node.new(:code)
-    assert_raises NodeError do
-      @doc.insert_before(code)
+    def test_bad_set_string_content
+      assert_raises TypeError do
+        @doc.string_content = 123
+      end
     end
 
-    paragraph = Node.new(:paragraph)
-    assert_raises NodeError do
-      @doc.insert_after(paragraph)
+    def test_bad_walking
+      assert_nil @doc.parent
+      assert_nil @doc.previous
     end
 
-    document = Node.new(:document)
-    assert_raises NodeError do
-      @doc.prepend_child(document)
-    end
+    def test_bad_insertion
+      code = Node.new(:code)
+      assert_raises NodeError do
+        @doc.insert_before(code)
+      end
 
-    assert_raises NodeError do
-      @doc.append_child(document)
-    end
-  end
+      paragraph = Node.new(:paragraph)
+      assert_raises NodeError do
+        @doc.insert_after(paragraph)
+      end
 
-  def test_bad_url_get
-    assert_raises NodeError do
-      @doc.url
-    end
-  end
+      document = Node.new(:document)
+      assert_raises NodeError do
+        @doc.prepend_child(document)
+      end
 
-  def test_bad_url_set
-    assert_raises NodeError do
-      @doc.url = '123'
+      assert_raises NodeError do
+        @doc.append_child(document)
+      end
     end
 
-    link = CommonMarker.render_doc('[GitHub](https://www.github.com)').first_child.first_child
-    assert_raises TypeError do
-      link.url = 123
+    def test_bad_url_get
+      assert_raises NodeError do
+        @doc.url
+      end
     end
-  end
 
-  def test_bad_title_get
-    assert_raises NodeError do
-      @doc.title
-    end
-  end
+    def test_bad_url_set
+      assert_raises NodeError do
+        @doc.url = '123'
+      end
 
-  def test_bad_title_set
-    assert_raises NodeError do
-      @doc.title = '123'
+      link = CommonMarker.render_doc('[GitHub](https://www.github.com)').first_child.first_child
+      assert_raises TypeError do
+        link.url = 123
+      end
     end
 
-    image = CommonMarker.render_doc('![alt text](https://github.com/favicon.ico "Favicon")')
-    image = image.first_child.first_child
-    assert_raises TypeError do
-      image.title = 123
+    def test_bad_title_get
+      assert_raises NodeError do
+        @doc.title
+      end
     end
-  end
 
-  def test_bad_header_level_get
-    assert_raises NodeError do
-      @doc.header_level
-    end
-  end
+    def test_bad_title_set
+      assert_raises NodeError do
+        @doc.title = '123'
+      end
 
-  def test_bad_header_level_set
-    assert_raises NodeError do
-      @doc.header_level = 1
+      image = CommonMarker.render_doc('![alt text](https://github.com/favicon.ico "Favicon")')
+      image = image.first_child.first_child
+      assert_raises TypeError do
+        image.title = 123
+      end
     end
 
-    header = CommonMarker.render_doc('### Header Three').first_child
-    assert_raises TypeError do
-      header.header_level = '123'
+    def test_bad_header_level_get
+      assert_raises NodeError do
+        @doc.header_level
+      end
     end
-  end
 
-  def test_bad_list_type_get
-    assert_raises NodeError do
-      @doc.list_type
-    end
-  end
+    def test_bad_header_level_set
+      assert_raises NodeError do
+        @doc.header_level = 1
+      end
 
-  def test_bad_list_type_set
-    assert_raises NodeError do
-      @doc.list_type = :bullet_list
+      header = CommonMarker.render_doc('### Header Three').first_child
+      assert_raises TypeError do
+        header.header_level = '123'
+      end
     end
 
-    ul_list = CommonMarker.render_doc("* Bullet\n*Bullet").first_child
-    assert_raises NodeError do
-      ul_list.list_type = :fake
-    end
-    assert_raises TypeError do
-      ul_list.list_type = 1234
+    def test_bad_list_type_get
+      assert_raises NodeError do
+        @doc.list_type
+      end
     end
-  end
 
-  def test_bad_list_start_get
-    assert_raises NodeError do
-      @doc.list_start
-    end
-  end
+    def test_bad_list_type_set
+      assert_raises NodeError do
+        @doc.list_type = :bullet_list
+      end
 
-  def test_bad_list_start_set
-    assert_raises NodeError do
-      @doc.list_start = 12
+      ul_list = CommonMarker.render_doc("* Bullet\n*Bullet").first_child
+      assert_raises NodeError do
+        ul_list.list_type = :fake
+      end
+      assert_raises TypeError do
+        ul_list.list_type = 1234
+      end
     end
 
-    ol_list = CommonMarker.render_doc("1. One\n2. Two").first_child
-    assert_raises TypeError do
-      ol_list.list_start = :fake
+    def test_bad_list_start_get
+      assert_raises NodeError do
+        @doc.list_start
+      end
     end
-  end
 
-  def test_bad_list_tight_get
-    assert_raises NodeError do
-      @doc.list_tight
+    def test_bad_list_start_set
+      assert_raises NodeError do
+        @doc.list_start = 12
+      end
+
+      ol_list = CommonMarker.render_doc("1. One\n2. Two").first_child
+      assert_raises TypeError do
+        ol_list.list_start = :fake
+      end
     end
-  end
 
-  def test_bad_list_tight_set
-    assert_raises NodeError do
-      @doc.list_tight = false
+    def test_bad_list_tight_get
+      assert_raises NodeError do
+        @doc.list_tight
+      end
     end
-  end
 
-  def test_bad_fence_info_get
-    assert_raises NodeError do
-      @doc.fence_info
+    def test_bad_list_tight_set
+      assert_raises NodeError do
+        @doc.list_tight = false
+      end
     end
-  end
 
-  def test_bad_fence_info_set
-    assert_raises NodeError do
-      @doc.fence_info = 'ruby'
+    def test_bad_fence_info_get
+      assert_raises NodeError do
+        @doc.fence_info
+      end
     end
 
-    fence = CommonMarker.render_doc("``` ruby\nputs 'wow'\n```").first_child
-    assert_raises TypeError do
-      fence.fence_info = 123
+    def test_bad_fence_info_set
+      assert_raises NodeError do
+        @doc.fence_info = 'ruby'
+      end
+
+      fence = CommonMarker.render_doc("``` ruby\nputs 'wow'\n```").first_child
+      assert_raises TypeError do
+        fence.fence_info = 123
+      end
     end
   end
 end