review 3.0.0 → 4.2.0

data/test/test_logger.rb

@@ -14,7 +14,20 @@ class LoggerTest < Test::Unit::TestCase
       msg = r.readline
       STDERR.reopen(old_stderr)
 
-      assert_equal "WARN: test\n", msg
+      assert_equal "WARN --: test\n", msg
+    end
+  end
+
+  def test_logging_with_progname
+    old_stderr = STDERR.dup
+    IO.pipe do |r, w|
+      STDERR.reopen(w)
+      @logger = ReVIEW::Logger.new(STDERR, progname: 'review-dummy-cmd')
+      @logger.warn('test')
+      msg = r.readline
+      STDERR.reopen(old_stderr)
+
+      assert_equal "WARN review-dummy-cmd: test\n", msg
     end
   end
 end

data/test/test_makerhelper.rb

@@ -12,8 +12,8 @@ class MakerHelperTest < Test::Unit::TestCase
   end
 
   def teardown
-    FileUtils.rm_rf @tmpdir1
-    FileUtils.rm_rf @tmpdir2
+    FileUtils.rm_rf(@tmpdir1)
+    FileUtils.rm_rf(@tmpdir2)
   end
 
   def test_copy_images_to_dir
@@ -34,7 +34,7 @@ class MakerHelperTest < Test::Unit::TestCase
 
   def test_copy_images_to_dir_convert
     if /mswin|mingw|cygwin/ !~ RUBY_PLATFORM && (`convert -version` rescue nil) && (`gs --version` rescue nil)
-      FileUtils.cp File.join(assets_dir, 'black.eps'), File.join(@tmpdir1, 'foo.eps')
+      FileUtils.cp(File.join(assets_dir, 'black.eps'), File.join(@tmpdir1, 'foo.eps'))
 
       image_files = MakerHelper.copy_images_to_dir(@tmpdir1, @tmpdir2,
                                                    convert: { eps: :png })

data/test/test_markdownbuilder.rb

@@ -9,10 +9,7 @@ class MARKDOWNBuilderTest < Test::Unit::TestCase
 
   def setup
     @builder = MARKDOWNBuilder.new
-    @config = {
-      'secnolevel' => 2,
-      'stylesheet' => nil
-    }
+    @config = ReVIEW::Configure.values
     @book = Book::Base.new('.')
     @book.config = @config
     @compiler = ReVIEW::Compiler.new(@builder)
@@ -23,17 +20,58 @@ class MARKDOWNBuilderTest < Test::Unit::TestCase
 
   def test_quote
     actual = compile_block("//quote{\nfoo\nbar\n\nbuz\n//}\n")
-    assert_equal %Q(\n> foobar\n> \n> buz\n\n), actual
+    expected = <<-EOS
+
+> foobar
+> 
+> buz
+
+EOS
+    assert_equal expected, actual
+
+    @book.config['join_lines_by_lang'] = true
+    actual = compile_block("//quote{\nfoo\nbar\n\nbuz\n//}\n")
+    expected = <<-EOS
+
+> foo bar
+> 
+> buz
+
+EOS
+    assert_equal expected, actual
   end
 
   def test_memo
     actual = compile_block("//memo[this is @<b>{test}<&>_]{\ntest1\n\ntest@<i>{2}\n//}\n")
-    assert_equal %Q(<div class="memo">\n<p class="caption">this is **test**<&>_</p>\ntest1\ntest*2*\n</div>\n), actual
+    expected = <<-EOS
+<div class="memo">
+<p class="caption">this is **test**<&>_</p>
+test1
+test*2*
+</div>
+EOS
+    assert_equal expected, actual
   end
 
   def test_noindent
     actual = compile_block("//noindent\nfoo\nbar\n\nfoo2\nbar2\n")
-    assert_equal %Q(<p class="noindent">foobar</p>\n\nfoo2bar2\n\n), actual
+    expected = <<-EOS
+<p class="noindent">foobar</p>
+
+foo2bar2
+
+EOS
+    assert_equal expected, actual
+
+    @book.config['join_lines_by_lang'] = true
+    actual = compile_block("//noindent\nfoo\nbar\n\nfoo2\nbar2\n")
+    expected = <<-EOS
+<p class="noindent">foo bar</p>
+
+foo2 bar2
+
+EOS
+    assert_equal expected, actual
   end
 
   def test_inline_em
@@ -78,23 +116,69 @@ EOS
 
   def test_cmd
     actual = compile_block("//cmd{\nlineA\nlineB\n//}\n")
-    assert_equal "```shell-session\nlineA\nlineB\n```\n", actual
+    expected = <<-EOS
+```shell-session
+lineA
+lineB
+```
+EOS
+    assert_equal expected, actual
   end
 
   def test_dlist
-    actual = compile_block(": foo\n  foo.\n  bar.\n")
-    assert_equal %Q(<dl>\n<dt>foo</dt>\n<dd>foo.bar.</dd>\n</dl>\n), actual
+    actual = compile_block(" : foo\n  foo.\n  bar.\n")
+    expected = <<-EOS
+<dl>
+<dt>foo</dt>
+<dd>foo.bar.</dd>
+</dl>
+EOS
+    assert_equal expected, actual
+
+    @book.config['join_lines_by_lang'] = true
+    actual = compile_block(" : foo\n  foo.\n  bar.\n")
+    expected = <<-EOS
+<dl>
+<dt>foo</dt>
+<dd>foo. bar.</dd>
+</dl>
+EOS
+    assert_equal expected, actual
   end
 
   def test_dlist_with_bracket
-    actual = compile_block(": foo[bar]\n    foo.\n    bar.\n")
-    assert_equal %Q(<dl>\n<dt>foo[bar]</dt>\n<dd>foo.bar.</dd>\n</dl>\n), actual
+    actual = compile_block(" : foo[bar]\n    foo.\n    bar.\n")
+    expected = <<-EOS
+<dl>
+<dt>foo[bar]</dt>
+<dd>foo.bar.</dd>
+</dl>
+EOS
+    assert_equal expected, actual
+
+    @book.config['join_lines_by_lang'] = true
+    actual = compile_block(" : foo[bar]\n    foo.\n    bar.\n")
+    expected = <<-EOS
+<dl>
+<dt>foo[bar]</dt>
+<dd>foo. bar.</dd>
+</dl>
+EOS
+    assert_equal expected, actual
   end
 
   def test_dlist_with_comment
-    source = ": title\n  body\n\#@ comment\n\#@ comment\n: title2\n  body2\n"
+    source = " : title\n  body\n\#@ comment\n\#@ comment\n: title2\n  body2\n"
     actual = compile_block(source)
-    assert_equal %Q(<dl>\n<dt>title</dt>\n<dd>body</dd>\n<dt>title2</dt>\n<dd>body2</dd>\n</dl>\n), actual
+    expected = <<-EOS
+<dl>
+<dt>title</dt>
+<dd>body</dd>
+<dt>title2</dt>
+<dd>body2</dd>
+</dl>
+EOS
+    assert_equal expected, actual
   end
 
   def test_comment
@@ -144,6 +228,23 @@ BBB
     EOS
   end
 
+  def test_listnum
+    def @chapter.list(_id)
+      Book::Index::Item.new('test', 1)
+    end
+    actual = compile_block("//listnum[test][this is @<b>{test}<&>_]{\nfoo\nbar\n\tbuz\n//}\n")
+    expected = <<-EOS
+リスト1.1 this is **test**<&>_
+
+```
+ 1: foo
+ 2: bar
+ 3:         buz
+```
+EOS
+    assert_equal expected, actual
+  end
+
   def test_emlist_lang
     actual = compile_block(<<-EOS)
 //emlist[caption][ruby]{
@@ -166,11 +267,28 @@ BBB
 
   def test_table
     actual = compile_block("//table{\ntestA\ttestB\n------------\ncontentA\tcontentB\n//}\n")
-    assert_equal "|testA|testB|\n|:--|:--|\n|contentA|contentB|\n\n", actual
+    expected = <<-EOS
+|testA|testB|
+|:--|:--|
+|contentA|contentB|
+
+EOS
+    assert_equal expected, actual
+
+    actual = compile_block("//table[foo][FOO]{\ntestA\ttestB\n------------\ncontentA\tcontentB\n//}\n")
+    expected = <<-EOS
+表1.1: FOO
+
+|testA|testB|
+|:--|:--|
+|contentA|contentB|
+
+EOS
+    assert_equal expected, actual
   end
 
   def test_ruby
     actual = compile_block('@<ruby>{謳,うた}い文句')
-    assert_equal "<ruby><rb>謳</rb><rp>（</rp><rt>うた</rt><rp>）</rp></ruby>い文句\n\n", actual
+    assert_equal "<ruby>謳<rp>（</rp><rt>うた</rt><rp>）</rp></ruby>い文句\n\n", actual
   end
 end

data/test/test_md2inaobuilder.rb

@@ -9,10 +9,7 @@ class MD2INAOBuilderTest < Test::Unit::TestCase
 
   def setup
     @builder = MD2INAOBuilder.new
-    @config = {
-      'secnolevel' => 2,
-      'stylesheet' => nil
-    }
+    @config = ReVIEW::Configure.values
     @book = Book::Base.new('.')
     @book.config = @config
     @compiler = ReVIEW::Compiler.new(@builder)
@@ -24,17 +21,43 @@ class MD2INAOBuilderTest < Test::Unit::TestCase
 
   def test_paragraph
     actual = compile_block('Hello, world!')
-    assert_equal "　Hello, world!\n\n", actual
+    expected = <<-EOS
+　Hello, world!
+
+EOS
+    assert_equal expected, actual
   end
 
   def test_cmd
     actual = compile_block("//cmd{\nlineA\nlineB\n//}\n")
-    assert_equal "!!! cmd\nlineA\nlineB\n\n", actual
+    expected = <<-EOS
+!!! cmd
+lineA
+lineB
+
+EOS
+    assert_equal expected, actual
   end
 
   def test_dlist
-    actual = compile_block(": foo\n  foo.\n  bar.\n")
-    assert_equal "<dl>\n<dt>foo</dt>\n<dd>foo.bar.</dd>\n</dl>\n", actual
+    actual = compile_block(" : foo\n  foo.\n  bar.\n")
+    expected = <<-EOS
+<dl>
+<dt>foo</dt>
+<dd>foo.bar.</dd>
+</dl>
+EOS
+    assert_equal expected, actual
+
+    @book.config['join_lines_by_lang'] = true
+    actual = compile_block(" : foo\n  foo.\n  bar.\n")
+    expected = <<-EOS
+<dl>
+<dt>foo</dt>
+<dd>foo. bar.</dd>
+</dl>
+EOS
+    assert_equal expected, actual
   end
 
   def test_list
@@ -46,9 +69,9 @@ BBB
     EOS
 
     assert_equal <<-EOS, actual
-```
 ●リスト1::caption
 
+```
 AAA
 BBB
 ```

data/test/test_pdfmaker.rb

@@ -49,7 +49,7 @@ class PDFMakerTest < Test::Unit::TestCase
     begin
       assert_equal(path, 'sample-pdf')
     ensure
-      FileUtils.remove_entry_secure path
+      FileUtils.remove_entry_secure(path)
     end
   end
 
@@ -149,6 +149,23 @@ class PDFMakerTest < Test::Unit::TestCase
     end
   end
 
+  def test_template_content_with_localconfig
+    @config['mycustom'] = { 'value' => '#_TEST_' }
+    Dir.mktmpdir do |dir|
+      Dir.chdir(dir) do
+        Dir.mkdir('layouts')
+        File.write(File.join('layouts', 'config-local.tex.erb'), %q(\def\customvalue{<%= escape(@config['mycustom']['value']) %>}) + "\n")
+        @maker.basedir = Dir.pwd
+        @maker.erb_config
+        tmpl = @maker.template_content
+        expect = File.read(File.join(assets_dir, 'test_template.tex'))
+        expect.gsub!(/\\def\\review@reviewversion{[^\}]+}/, "\\def\\review@reviewversion{#{ReVIEW::VERSION}}")
+        expect.sub!("\\makeatother\n", '\&' + "%% BEGIN: config-local.tex.erb\n\\def\\customvalue{\\#\\textunderscore{}TEST\\textunderscore{}}\n%% END: config-local.tex.erb\n")
+        assert_equal(expect, tmpl)
+      end
+    end
+  end
+
   def test_gettemplate_with_backmatter
     @config.merge!(
       'backcover' => 'backcover.tex',

data/test/test_pdfmaker_cmd.rb

@@ -3,6 +3,7 @@ require 'tmpdir'
 require 'fileutils'
 require 'yaml'
 require 'rbconfig'
+require 'open3'
 
 REVIEW_PDFMAKER = File.expand_path('../bin/review-pdfmaker', File.dirname(__FILE__))
 
@@ -15,22 +16,115 @@ class PDFMakerCmdTest < Test::Unit::TestCase
   end
 
   def teardown
-    FileUtils.rm_rf @tmpdir1
+    FileUtils.rm_rf(@tmpdir1)
     ENV['RUBYLIB'] = @old_rubylib
   end
 
-  def test_pdfmaker_cmd
+  def common_buildpdf(bookdir, templatedir, configfile, targetpdffile, option = nil)
     if /mswin|mingw|cygwin/ !~ RUBY_PLATFORM
-      config = prepare_samplebook(@tmpdir1)
+      config = prepare_samplebook(@tmpdir1, bookdir, templatedir, configfile)
       builddir = File.join(@tmpdir1, config['bookname'] + '-pdf')
-      # assert !File.exist?(builddir)
+      assert !File.exist?(builddir)
 
       ruby_cmd = File.join(RbConfig::CONFIG['bindir'], RbConfig::CONFIG['ruby_install_name'])
       Dir.chdir(@tmpdir1) do
-        system("#{ruby_cmd} -S #{REVIEW_PDFMAKER} config.yml 1>/dev/null 2>/dev/null")
+        _o, e, s = Open3.capture3("#{ruby_cmd} -S #{REVIEW_PDFMAKER} #{option} #{configfile}")
+        if !e.empty? && !s.success?
+          STDERR.puts e
+        end
+        assert s.success?
       end
+      assert File.exist?(File.join(@tmpdir1, targetpdffile))
+    end
+  end
+
+  def test_pdfmaker_cmd_sample_jsbook_print
+    begin
+      `uplatex -v`
+    rescue
+      $stderr.puts 'skip test_pdfmaker_cmd_sample_jsbook_print'
+      return true
+    end
+    common_buildpdf('sample-book/src', 'review-jsbook', 'config.yml', 'book.pdf')
+  end
+
+  def test_pdfmaker_cmd_sample_jsbook_ebook
+    begin
+      `uplatex -v`
+    rescue
+      $stderr.puts 'skip test_pdfmaker_cmd_sample_jsbook_ebook'
+      return true
+    end
+    common_buildpdf('sample-book/src', 'review-jsbook', 'config-ebook.yml', 'book.pdf')
+  end
+
+  def test_pdfmaker_cmd_sample_jlreq_print
+    begin
+      `uplatex -v`
+    rescue
+      $stderr.puts 'skip test_pdfmaker_cmd_sample_jlreq_print'
+      return true
+    end
+    common_buildpdf('sample-book/src', 'review-jlreq', 'config-jlreq.yml', 'book.pdf')
+  end
+
+  def test_pdfmaker_cmd_sample_jlreq_ebook
+    begin
+      `uplatex -v`
+    rescue
+      $stderr.puts 'skip test_pdfmaker_cmd_sample_jlreq_ebook'
+      return true
+    end
+    common_buildpdf('sample-book/src', 'review-jlreq', 'config-jlreq-ebook.yml', 'book.pdf')
+  end
+
+  def test_pdfmaker_cmd_syntax_jsbook_print
+    begin
+      `uplatex -v`
+    rescue
+      $stderr.puts 'skip test_pdfmaker_cmd_syntax_jsbook_print'
+      return true
+    end
+    common_buildpdf('syntax-book', 'review-jsbook', 'config-print.yml', 'syntax-book.pdf')
+  end
+
+  def test_pdfmaker_cmd_syntax_jsbook_print_buildonly
+    begin
+      `uplatex -v`
+    rescue
+      $stderr.puts 'skip test_pdfmaker_cmd_syntax_jsbook_print_buildonly'
+      return true
+    end
+    common_buildpdf('syntax-book', 'review-jsbook', 'config-print.yml', 'syntax-book.pdf', '-y ch01')
+  end
+
+  def test_pdfmaker_cmd_syntax_jsbook_ebook
+    begin
+      `uplatex -v`
+    rescue
+      $stderr.puts 'skip test_pdfmaker_cmd_syntax_jsbook_ebook'
+      return true
+    end
+    common_buildpdf('syntax-book', 'review-jsbook', 'config.yml', 'syntax-book.pdf')
+  end
+
+  def test_pdfmaker_cmd_syntax_jlreq_ebook
+    begin
+      `uplatex -v`
+    rescue
+      $stderr.puts 'skip test_pdfmaker_cmd_syntax_jlreq_ebook'
+      return true
+    end
+    common_buildpdf('syntax-book', 'review-jlreq', 'config-jlreq.yml', 'syntax-book.pdf')
+  end
 
-      assert File.exist?(builddir)
+  def test_pdfmaker_cmd_syntax_jlreq_ebook_lualatex
+    begin
+      `lualatex -v`
+    rescue
+      $stderr.puts 'skip test_pdfmaker_cmd_syntax_jlreq_ebook_lualatex'
+      return true
     end
+    common_buildpdf('syntax-book', 'review-jlreq', 'config-jlreq-lualatex.yml', 'syntax-book.pdf')
   end
 end