review 2.3.0 → 2.4.0

data/test/syntax-book/syntax.dic

@@ -0,0 +1,2 @@
+索引	さくいん
+辞書ファイル	じしょふぁいる

data/test/test_book.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 require 'book_test_helper'
 
 class BookTest < Test::Unit::TestCase
@@ -31,9 +29,7 @@ class BookTest < Test::Unit::TestCase
     test_const = "ReVIEW__BOOK__TEST__#{num}"
     begin
       Dir.mktmpdir do |dir|
-        File.open(File.join(dir, 'review-ext.rb'), 'w') do |o|
-          o.puts "#{test_const} = #{num}"
-        end
+        File.open(File.join(dir, 'review-ext.rb'), 'w') { |o| o.puts "#{test_const} = #{num}" }
         Book.update_rubyenv(dir)
         assert_equal num, (Object.class_eval { const_get(test_const) })
       end
@@ -47,95 +43,95 @@ class BookTest < Test::Unit::TestCase
     assert_equal '.re', book.ext
   end
 
-  def test_read_CHAPS
+  def test_read_chaps
     Dir.mktmpdir do |dir|
       book = Book::Base.new(dir)
-      assert_equal "", book.read_CHAPS
+      assert_equal '', book.read_chaps
 
       chaps_path = File.join(dir, 'CHAPS')
       re1_path = File.join(dir, "123#{book.ext}")
       re2_path = File.join(dir, "456#{book.ext}")
 
-      File.open(chaps_path, 'w') {|o| o.print "abc\n" }
-      File.open(re1_path, 'w') {|o| o.print "123\n" }
-      File.open(re2_path, 'w') {|o| o.print "456\n" }
+      File.open(chaps_path, 'w') { |o| o.print "abc\n" }
+      File.open(re1_path, 'w') { |o| o.print "123\n" }
+      File.open(re2_path, 'w') { |o| o.print "456\n" }
 
-      assert_equal "abc\n", book.read_CHAPS
+      assert_equal "abc\n", book.read_chaps
 
       File.unlink(chaps_path)
-      assert_equal "#{re1_path}\n#{re2_path}", book.read_CHAPS
+      assert_equal "#{re1_path}\n#{re2_path}", book.read_chaps
 
       File.unlink(re1_path)
-      assert_equal "#{re2_path}", book.read_CHAPS
+      assert_equal re2_path, book.read_chaps
 
       File.unlink(re2_path)
-      assert_equal "", book.read_CHAPS
+      assert_equal '', book.read_chaps
     end
   end
 
-  def test_read_PART
+  def test_read_part
     Dir.mktmpdir do |dir|
       book = Book::Base.new(dir)
       assert !book.part_exist?
       assert_raises Errno::ENOENT do # XXX: OK?
-        book.read_PART
+        book.read_part
       end
 
       chaps_path = File.join(dir, 'PART')
       chaps_content = "abc\n"
-      File.open(chaps_path, 'w') {|o| o.print chaps_content }
+      File.open(chaps_path, 'w') { |o| o.print chaps_content }
 
       assert book.part_exist?
-      assert_equal chaps_content, book.read_PART
+      assert_equal chaps_content, book.read_part
 
-      File.open(chaps_path, 'w') {|o| o.print "XYZ\n" }
-      assert_equal chaps_content, book.read_PART
+      File.open(chaps_path, 'w') { |o| o.print "XYZ\n" }
+      assert_equal chaps_content, book.read_part
     end
   end
 
-  def test_read_APPENDIX
+  def test_read_appendix
     Dir.mktmpdir do |dir|
       book = Book::Base.new(dir)
-      assert_equal "", book.read_APPENDIX
+      assert_equal '', book.read_appendix
 
       post_path = File.join(dir, 'POSTDEF')
       re1_path = File.join(dir, "123#{book.ext}")
       re2_path = File.join(dir, "456#{book.ext}")
 
-      File.open(post_path, 'w') {|o| o.print "abc\n" }
-      File.open(re1_path, 'w') {|o| o.print "123\n" }
-      File.open(re2_path, 'w') {|o| o.print "456\n" }
+      File.open(post_path, 'w') { |o| o.print "abc\n" }
+      File.open(re1_path, 'w') { |o| o.print "123\n" }
+      File.open(re2_path, 'w') { |o| o.print "456\n" }
 
-      assert_equal "abc\n", book.read_APPENDIX
+      assert_equal "abc\n", book.read_appendix
 
       File.unlink(post_path)
-      assert_equal "#{re1_path}\n#{re2_path}", book.read_APPENDIX
+      assert_equal "#{re1_path}\n#{re2_path}", book.read_appendix
 
       File.unlink(re1_path)
-      assert_equal "#{re2_path}", book.read_APPENDIX
+      assert_equal re2_path, book.read_appendix
 
       File.unlink(re2_path)
-      assert_equal "", book.read_APPENDIX
+      assert_equal '', book.read_appendix
     end
   end
 
-  def test_read_POSTDEF
+  def test_read_postdef
     Dir.mktmpdir do |dir|
       book = Book::Base.new(dir)
-      assert_equal "", book.read_POSTDEF
+      assert_equal '', book.read_postdef
 
       post_path = File.join(dir, 'POSTDEF')
       re1_path = File.join(dir, "123#{book.ext}")
       re2_path = File.join(dir, "456#{book.ext}")
 
-      File.open(post_path, 'w') {|o| o.print "abc\n" }
-      File.open(re1_path, 'w') {|o| o.print "123\n" }
-      File.open(re2_path, 'w') {|o| o.print "456\n" }
+      File.open(post_path, 'w') { |o| o.print "abc\n" }
+      File.open(re1_path, 'w') { |o| o.print "123\n" }
+      File.open(re2_path, 'w') { |o| o.print "456\n" }
 
-      assert_equal "", book.read_POSTDEF
+      assert_equal '', book.read_postdef
 
       File.unlink(post_path)
-      assert_equal "", book.read_POSTDEF
+      assert_equal '', book.read_postdef
     end
   end
 
@@ -146,9 +142,8 @@ class BookTest < Test::Unit::TestCase
       assert_raises Errno::ENOENT do # XXX: OK?
         book.read_bib
       end
-
       bib_path = File.join(dir, "bib#{book.ext}")
-      File.open(bib_path, 'w') {|o| o.print "abc\n" }
+      File.open(bib_path, 'w') { |o| o.print "abc\n" }
 
       assert book.bib_exist?
       assert_equal "abc\n", book.read_bib
@@ -156,39 +151,39 @@ class BookTest < Test::Unit::TestCase
   end
 
   # backward compatible
-  def test_setParameter
+  def test_set_parameter
     book = Book::Base.new(File.dirname(__FILE__))
     book.config = :test
     assert_equal :test, book.config
   end
 
-  def test_setConfig
+  def test_set_config
     book = Book::Base.new(File.dirname(__FILE__))
     book.config = :test
     assert_equal :test, book.config
   end
 
   def test_parse_chapters
-    mktmpbookdir 'CHAPS' => '' do |dir, book, files|
+    mktmpbookdir 'CHAPS' => '' do |_dir, book, _files|
       parts = book.instance_eval { parse_chapters }
       assert_equal 0, parts.size
     end
 
-    mktmpbookdir 'CHAPS' => "chapter1.re\nchapter2\n" do |dir, book, files|
+    mktmpbookdir 'CHAPS' => "chapter1.re\nchapter2\n" do |dir, book, _files|
       parts = book.instance_eval { parse_chapters }
       assert_equal 1, parts.size
 
       assert_equal nil, parts[0].number
       assert_equal 2, parts[0].chapters.size
-      chaps = parts[0].chapters.map {|ch| [ch.number, ch.name, ch.path] }
+      chaps = parts[0].chapters.map { |ch| [ch.number, ch.name, ch.path] }
       expect = [
         [1, 'chapter1', File.join(dir, 'chapter1.re')],
-        [2, 'chapter2', File.join(dir, 'chapter2')],
+        [2, 'chapter2', File.join(dir, 'chapter2')]
       ]
       assert_equal expect, chaps
     end
 
-    mktmpbookdir 'CHAPS' => <<EOC do |dir, book, files|
+    mktmpbookdir 'CHAPS' => <<EOC do |dir, book, _files|
 part1_chapter1.re
 part1_chapter2.re
 
@@ -204,28 +199,28 @@ EOC
 
       assert_equal nil, parts[0].number
       assert_equal 2, parts[0].chapters.size
-      chaps = parts[0].chapters.map {|ch| [ch.number, ch.name, ch.path] }
+      chaps = parts[0].chapters.map { |ch| [ch.number, ch.name, ch.path] }
       expect = [
         [1, 'part1_chapter1', File.join(dir, 'part1_chapter1.re')],
-        [2, 'part1_chapter2', File.join(dir, 'part1_chapter2.re')],
+        [2, 'part1_chapter2', File.join(dir, 'part1_chapter2.re')]
       ]
       assert_equal expect, chaps
 
       assert_equal nil, parts[1].number
       assert_equal 3, parts[1].chapters.size
-      chaps = parts[1].chapters.map {|ch| [ch.number, ch.name, ch.path] }
+      chaps = parts[1].chapters.map { |ch| [ch.number, ch.name, ch.path] }
       expect = [
         [3, 'part2_chapter1', File.join(dir, 'part2_chapter1.re')],
         [4, 'part2_chapter2', File.join(dir, 'part2_chapter2.re')],
-        [5, 'part2_chapter3', File.join(dir, 'part2_chapter3.re')],
+        [5, 'part2_chapter3', File.join(dir, 'part2_chapter3.re')]
       ]
       assert_equal expect, chaps
 
       assert_equal nil, parts[2].number
       assert_equal 1, parts[2].chapters.size
-      chaps = parts[2].chapters.map {|ch| [ch.number, ch.name, ch.path] }
+      chaps = parts[2].chapters.map { |ch| [ch.number, ch.name, ch.path] }
       expect = [
-        [6, 'part3_chapter1', File.join(dir, 'part3_chapter1.re')],
+        [6, 'part3_chapter1', File.join(dir, 'part3_chapter1.re')]
       ]
       assert_equal expect, chaps
     end
@@ -239,103 +234,103 @@ EOC
         2,
         "part1_chapter1.re\n\npart2_chpater1.re\n",
         "part1\npart2\npart3\n",
-        %w(part1 part2),
+        %w[part1 part2]
       ],
       [
         3,
         "part1_chapter1.re\n\npart2_chapter1.re\n\npart3_chapter1.re",
         "part1\n",
         [
-          "part1",
-          "", # XXX: OK?
-          ""
-        ],
+          'part1',
+          '', # XXX: OK?
+          ''
+        ]
       ],
       [
         1,
         "part1_chapter1.re\n",
-        "",
+        '',
         [
-          "", # XXX: OK?
-        ],
+          '', # XXX: OK?
+        ]
       ],
       [
         1,
         "part1_chapter1.re\n",
         nil,
         [
-          "",
-        ],
-      ],
+          ''
+        ]
+      ]
     ].each do |n_parts, chaps_text, parts_text, part_names|
       n_test += 1
       Dir.mktmpdir do |dir|
         book = Book::Base.new(dir)
         chaps_path = File.join(dir, 'CHAPS')
-        File.open(chaps_path, 'w') {|o| o.print chaps_text }
-        unless parts_text.nil?
+        File.open(chaps_path, 'w') { |o| o.print chaps_text }
+        if parts_text
           parts_path = File.join(dir, 'PART')
-          File.open(parts_path, 'w') {|o| o.print parts_text }
+          File.open(parts_path, 'w') { |o| o.print parts_text }
         end
 
         parts = book.instance_eval { parse_chapters }
-        assert_equal n_parts, parts.size, "\##{n_test}"
-        assert_equal part_names, parts.map {|p| p.name }, "\##{n_test}"
+        assert_equal n_parts, parts.size, "##{n_test}"
+        assert_equal part_names, parts.map(&:name), "##{n_test}"
       end
     end
   end
 
   def test_prefaces
-    mktmpbookdir do |dir, book, files|
+    mktmpbookdir do |_dir, book, _files|
       assert_equal nil, book.prefaces
     end
 
-    mktmpbookdir 'PREDEF' => '' do |dir, book, files|
+    mktmpbookdir 'PREDEF' => '' do |_dir, book, _files|
       assert_equal nil, book.prefaces # XXX: OK?
     end
 
     mktmpbookdir 'PREDEF' => 'chapter1',
-                 'chapter1.re' => '' do |dir, book, files|
+                 'chapter1.re' => '' do |_dir, book, files|
       assert_kind_of Book::Part, book.prefaces
       assert_equal '', book.prefaces.name
       assert_equal 1, book.prefaces.chapters.size
-      assert_equal "chapter1", book.prefaces.chapters.first.name
+      assert_equal 'chapter1', book.prefaces.chapters.first.name
       assert_equal files['chapter1.re'], book.prefaces.chapters.first.path
     end
 
     mktmpbookdir 'PREDEF' => "chapter1\n\nchapter2",
-                 'chapter1.re' => '', 'chapter2.re' => '' do |dir, book, files|
+                 'chapter1.re' => '', 'chapter2.re' => '' do |_dir, book, files|
       assert_kind_of Book::Part, book.prefaces
       assert_equal '', book.prefaces.name
       assert_equal 2, book.prefaces.chapters.size
-      assert_equal "chapter1", book.prefaces.chapters.first.name
+      assert_equal 'chapter1', book.prefaces.chapters.first.name
       assert_equal files['chapter1.re'], book.prefaces.chapters.first.path
-      assert_equal "chapter2", book.prefaces.chapters.last.name
+      assert_equal 'chapter2', book.prefaces.chapters.last.name
       assert_equal files['chapter2.re'], book.prefaces.chapters.last.path
     end
 
-    mktmpbookdir 'PREDEF' => "chapter1 chapter2",
-                 'chapter1.re' => '', 'chapter2.re' => '' do |dir, book, files|
+    mktmpbookdir 'PREDEF' => 'chapter1 chapter2',
+                 'chapter1.re' => '', 'chapter2.re' => '' do |_dir, book, _files|
       assert_kind_of Book::Part, book.prefaces
       assert_equal '', book.prefaces.name
       assert_equal 2, book.prefaces.chapters.size # XXX: OK?
     end
 
-    mktmpbookdir 'PREDEF' => 'not_exist' do |dir, book, files|
+    mktmpbookdir 'PREDEF' => 'not_exist' do |_dir, book, _files|
       assert_raises FileNotFound do
         assert_equal nil, book.prefaces
       end
     end
 
     mktmpbookdir 'PREDEF' => 'chapter1.re',
-                 'chapter1.re' => '' do |dir, book, files|
+                 'chapter1.re' => '' do |_dir, book, _files|
       assert_kind_of Book::Part, book.prefaces
       assert_equal '', book.prefaces.name
       assert_equal 1, book.prefaces.chapters.size
     end
 
     mktmpbookdir 'PREDEF' => 'chapter1.txt',
-                 'chapter1.txt' => '' do |dir, book, files|
+                 'chapter1.txt' => '' do |_dir, book, _files|
       assert_kind_of Book::Part, book.prefaces
       assert_equal '', book.prefaces.name
       assert_equal 1, book.prefaces.chapters.size
@@ -343,39 +338,39 @@ EOC
   end
 
   def test_appendix
-    mktmpbookdir do |dir, book, files|
+    mktmpbookdir do |_dir, book, _files|
       assert_equal nil, book.appendix
     end
 
-    mktmpbookdir 'POSTDEF' => '' do |dir, book, files|
+    mktmpbookdir 'POSTDEF' => '' do |_dir, book, _files|
       assert_equal nil, book.appendix
     end
 
     mktmpbookdir 'POSTDEF' => 'chapter1',
-                 'chapter1.re' => '' do |dir, book, files|
+                 'chapter1.re' => '' do |_dir, book, files|
       assert_kind_of Book::Part, book.appendix
       assert_equal '', book.appendix.name
       assert_equal 1, book.appendix.chapters.size
-      assert_equal "chapter1", book.appendix.chapters.first.name
+      assert_equal 'chapter1', book.appendix.chapters.first.name
       assert_equal files['chapter1.re'], book.appendix.chapters.first.path
       assert_equal 1, book.appendix.chapters.first.number
     end
 
     mktmpbookdir 'POSTDEF' => "chapter1\n\nchapter2",
-                 'chapter1.re' => '', 'chapter2.re' => '' do |dir, book, files|
+                 'chapter1.re' => '', 'chapter2.re' => '' do |_dir, book, files|
       assert_kind_of Book::Part, book.appendix
       assert_equal '', book.appendix.name
       assert_equal 2, book.appendix.chapters.size
-      assert_equal "chapter1", book.appendix.chapters.first.name
+      assert_equal 'chapter1', book.appendix.chapters.first.name
       assert_equal files['chapter1.re'], book.appendix.chapters.first.path
-      assert_equal "chapter2", book.appendix.chapters.last.name
+      assert_equal 'chapter2', book.appendix.chapters.last.name
       assert_equal files['chapter2.re'], book.appendix.chapters.last.path
       assert_equal 1, book.appendix.chapters.first.number
       assert_equal 2, book.appendix.chapters.last.number
     end
 
-    mktmpbookdir 'POSTDEF' => "chapter1 chapter2",
-                 'chapter1.re' => '', 'chapter2.re' => '' do |dir, book, files|
+    mktmpbookdir 'POSTDEF' => 'chapter1 chapter2',
+                 'chapter1.re' => '', 'chapter2.re' => '' do |_dir, book, _files|
       assert_kind_of Book::Part, book.appendix
       assert_equal '', book.appendix.name
       assert_equal 2, book.appendix.chapters.size # XXX: OK?
@@ -383,14 +378,14 @@ EOC
       assert_equal 2, book.appendix.chapters.last.number
     end
 
-    mktmpbookdir 'POSTDEF' => 'not_exist' do |dir, book, files|
+    mktmpbookdir 'POSTDEF' => 'not_exist' do |_dir, book, _files|
       assert_raises FileNotFound do
         assert_equal nil, book.appendix
       end
     end
 
     mktmpbookdir 'catalog.yml' => "APPENDIX:\n  - p01.re",
-                 'p01.re' => '= appendix' do |dir, book, files|
+                 'p01.re' => '= appendix' do |_dir, book, _files|
       assert_equal 'appendix', book.appendix.chapters.first.title
       assert_equal 1, book.appendix.chapters.first.number
     end
@@ -398,7 +393,7 @@ EOC
 
   def test_postscripts
     mktmpbookdir 'catalog.yml' => "POSTDEF:\n  - b01.re",
-                 'b01.re' => '= back' do |dir, book, files|
+                 'b01.re' => '= back' do |_dir, book, _files|
       assert_kind_of Book::Part, book.postscripts
       assert_equal 1, book.postscripts.chapters.size
       assert_equal 'back', book.postscripts.chapters.first.title
@@ -407,7 +402,7 @@ EOC
   end
 
   def test_parts
-    mktmpbookdir do |dir, book, files|
+    mktmpbookdir do |_dir, book, _files|
       assert book.parts.empty?
       assert !book.part(0)
       assert !book.part(1)
@@ -417,28 +412,28 @@ EOC
       assert tmp.empty?
     end
 
-    mktmpbookdir 'CHAPS' => "ch1\nch2\n\nch3", 'PART' => "foo\nbar\n" do |dir, book, files|
+    mktmpbookdir 'CHAPS' => "ch1\nch2\n\nch3", 'PART' => "foo\nbar\n" do |_dir, book, _files|
       parts = book.parts
       assert_equal 2, parts.size
       assert !book.part(0)
-      assert_equal "foo", book.part(1).name
-      assert_equal "bar", book.part(2).name
+      assert_equal 'foo', book.part(1).name
+      assert_equal 'bar', book.part(2).name
       assert !book.part(3)
 
       tmp = []
-      book.each_part {|p| tmp << p.number }
+      book.each_part { |p| tmp << p.number }
       assert_equal [1, 2], tmp
     end
   end
 
   def test_chapters
-    mktmpbookdir 'CHAPS' => "ch1\nch2\n\nch3" do |dir, book, files|
+    mktmpbookdir 'CHAPS' => "ch1\nch2\n\nch3" do |_dir, book, _files|
       chapters = book.chapters
       assert_equal 3, chapters.size
 
-      ch_names = %w(ch1 ch2 ch3)
+      ch_names = %w[ch1 ch2 ch3]
       tmp = []
-      book.each_chapter {|ch| tmp << ch.name }
+      book.each_chapter { |ch| tmp << ch.name }
       assert_equal ch_names, tmp
 
       ch_names.each do |name|
@@ -451,13 +446,13 @@ EOC
       end
     end
 
-    mktmpbookdir 'CHAPS' => "ch1.txt\nch2.txt\n\nch3.txt" do |dir, book, files|
+    mktmpbookdir 'CHAPS' => "ch1.txt\nch2.txt\n\nch3.txt" do |_dir, book, _files|
       chapters = book.chapters
       assert_equal 3, chapters.size
 
-      ch_names = %w(ch1 ch2 ch3)
+      ch_names = %w[ch1 ch2 ch3]
       tmp = []
-      book.each_chapter {|ch| tmp << ch.name }
+      book.each_chapter { |ch| tmp << ch.name }
       assert_equal ch_names, tmp
 
       ch_names.each do |name|
@@ -472,7 +467,7 @@ EOC
   end
 
   def test_next_chapter
-    mktmpbookdir 'CHAPS' => "ch1\nch2" do |dir, book, files|
+    mktmpbookdir 'CHAPS' => "ch1\nch2" do |_dir, book, _files|
       chapter = book.chapter('ch1')
       assert_equal book.chapter('ch2'), book.next_chapter(chapter)
 
@@ -482,7 +477,7 @@ EOC
   end
 
   def test_prev_chapter
-    mktmpbookdir 'CHAPS' => "ch1\nch2" do |dir, book, files|
+    mktmpbookdir 'CHAPS' => "ch1\nch2" do |_dir, book, _files|
       chapter = book.chapter('ch2')
       assert_equal book.chapter('ch1'), book.prev_chapter(chapter)
 
@@ -492,21 +487,21 @@ EOC
   end
 
   def test_volume
-    mktmpbookdir do |dir, book, files|
+    mktmpbookdir do |_dir, book, _files|
       assert book.volume
       assert_equal 0, book.volume.bytes
       assert_equal 0, book.volume.chars
       assert_equal 0, book.volume.lines
     end
 
-    mktmpbookdir 'CHAPS' => 'chapter1.re', 'chapter1.re' => '12345' do |dir, book, files|
+    mktmpbookdir 'CHAPS' => 'chapter1.re', 'chapter1.re' => '12345' do |_dir, book, _files|
       assert book.volume
       assert book.volume.bytes > 0
       assert book.volume.chars > 0
       assert book.volume.lines > 0
     end
 
-    mktmpbookdir 'preface.re' => '12345' do |dir, book, files|
+    mktmpbookdir 'preface.re' => '12345' do |dir, _book, _files|
       Dir.chdir(dir) do
         book2 = Book::Base.new('.')
         assert book2.volume
@@ -532,18 +527,18 @@ EOC
   end
 
   def test_page_metric_config
-    mktmpbookdir('config.yml'=>"bookname: book\npage_metric: B5\n") do |dir, book, files|
+    mktmpbookdir('config.yml' => "bookname: book\npage_metric: B5\n") do |dir, _book, _files|
       book = Book::Base.new(dir)
-      config_file = File.join(dir,"config.yml")
+      config_file = File.join(dir, 'config.yml')
       book.load_config(config_file)
       assert_equal ReVIEW::Book::PageMetric::B5, book.page_metric
     end
   end
 
   def test_page_metric_config_array
-    mktmpbookdir('config.yml'=>"bookname: book\npage_metric: [46, 80, 30, 74, 2]\n") do |dir, book, files|
+    mktmpbookdir('config.yml' => "bookname: book\npage_metric: [46, 80, 30, 74, 2]\n") do |dir, _book, _files|
       book = Book::Base.new(dir)
-      config_file = File.join(dir,"config.yml")
+      config_file = File.join(dir, 'config.yml')
       book.load_config(config_file)
       assert_equal ReVIEW::Book::PageMetric::B5, book.page_metric
     end