manbook 0.0.2 → 0.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b5f2c9c00d580c00257ea87242c31b510a6fd9e4
-  data.tar.gz: f741230cd8ccd20d1a54f1aa268fae9bb16ff45f
+  metadata.gz: 85e4b2dd68f7fb7b8416653cebfa676331f0d275
+  data.tar.gz: c2618d98a3e47bed7b0c718274c600b85b4dcd4b
 SHA512:
-  metadata.gz: 093fbb8eb2f50b33d333a1405c3fb3c0074d9c5e0ad9a1a160061c6ff77b125809fe6abd5a1d595182e012eca47e89d5300472e9c98ada223bdcd130953d74a9
-  data.tar.gz: 9bd342b49bb3d9e39f5da97098abd1b6e6282b34890a75e2c1ebeffcaadea1c29bc10c8a7d2727d3acc62f65bcb01afc3b398e5571e37b2cd69ed9a424edb31d
+  metadata.gz: b72acd32f82bfcbc5eae9092720be3eb202ee0f07b2efa6e88a977e6e574061d02a0708e6693509f3e2aa53bf422e3f27f1bd8c56eee11e76eeecef8c7810123
+  data.tar.gz: 2e3620a64a80aec997379ec923b488ea5b36acc9cb55142af3a34d0e751be4dd5a6db2b4e1f12f3ecf109fd32270cf7750d7e67f3d44650be2b9f0484749fc62

data/Gemfile

@@ -9,4 +9,6 @@ group :development do
   gem 'pry'
   gem 'bundler', '~> 1.0'
   gem 'jeweler', '~> 1.8'
+  gem 'guard-minitest'#, '~> 0.5'
+  gem 'guard-bundler' #, '~> 1.0'
 end

data/Gemfile.lock

@@ -13,6 +13,8 @@ GEM
     coderay (1.0.9)
     faraday (0.8.7)
       multipart-post (~> 1.1)
+    ffi (1.9.0)
+    formatador (0.2.4)
     git (1.2.5)
     github_api (0.10.1)
       addressable
@@ -21,6 +23,17 @@ GEM
       multi_json (~> 1.4)
       nokogiri (~> 1.5.2)
       oauth2
+    guard (1.8.1)
+      formatador (>= 0.2.4)
+      listen (>= 1.0.0)
+      lumberjack (>= 1.0.2)
+      pry (>= 0.9.10)
+      thor (>= 0.14.6)
+    guard-bundler (1.0.0)
+      bundler (~> 1.0)
+      guard (~> 1.1)
+    guard-minitest (0.5.0)
+      guard (>= 0.4)
     hashie (2.0.5)
     highline (1.6.19)
     httpauth (0.2.0)
@@ -37,6 +50,11 @@ GEM
     json (1.8.0)
     jwt (0.1.8)
       multi_json (>= 1.5)
+    listen (1.2.2)
+      rb-fsevent (>= 0.9.3)
+      rb-inotify (>= 0.9)
+      rb-kqueue (>= 0.2)
+    lumberjack (1.0.4)
     method_source (0.8.1)
     minitest (4.7.5)
     multi_json (1.7.7)
@@ -57,9 +75,15 @@ GEM
       slop (~> 3.4)
     rack (1.5.2)
     rake (10.1.0)
+    rb-fsevent (0.9.3)
+    rb-inotify (0.9.0)
+      ffi (>= 0.5.0)
+    rb-kqueue (0.2.0)
+      ffi (>= 0.5.0)
     rdoc (4.0.1)
       json (~> 1.4)
     slop (3.4.5)
+    thor (0.18.1)
     thread_safe (0.1.0)
       atomic
     tzinfo (0.3.37)
@@ -70,6 +94,8 @@ PLATFORMS
 DEPENDENCIES
   activesupport (~> 4.0)
   bundler (~> 1.0)
+  guard-bundler
+  guard-minitest
   jeweler (~> 1.8)
   nokogiri (~> 1.5)
   open4 (~> 1.3)

data/Guardfile

@@ -0,0 +1,12 @@
+guard 'bundler' do
+  watch('Gemfile')
+  # Uncomment next line if Gemfile contain `gemspec' command
+  # watch(/^.+\.gemspec/)
+end
+
+guard 'minitest' do
+  # with Minitest::Unit
+  watch(%r|^test/(.*)\/?test_(.*)\.rb|)
+  watch(%r|^lib/(.*)([^/]+)\.rb|)     { |m| "test/#{m[1]}test_#{m[2]}.rb" }
+  watch(%r|^test/test_helper\.rb|)    { "test" }
+end

data/TODO

@@ -1,2 +1,2 @@
 * Do not overwrite existing files unless --force was given
-* Prevent the cover image attribution in about.html if no or a custom cover is used
+* Prevent the cover image attribution in about.html if no or a custom cover is used

data/VERSION

@@ -1 +1 @@
-0.0.2
+0.0.3

data/bin/mktoc

@@ -36,6 +36,7 @@ GENERATOR = "#{File.basename(__FILE__)} v#{ManBook::VERSION}"
 @book_title  = ManBook::TITLE_DEFAULT
 @cover_image = ManBook::COVER_IMAGE_DEFAULT
 @book_author = ManBook::AUTHOR_DEFAULT
+@order       = ManBook::ORDER_DEFAULT
 
 opts = OptionParser.new do |opts|
   opts.banner = help
@@ -75,6 +76,11 @@ opts = OptionParser.new do |opts|
   opts.on("--no-cover-image", "Do not reference a cover image") do
     @cover_image = nil
   end
+
+  opts.on("--sort-by ORDER", "Speficy what attribute is used to determine the sort order of pages") do |order|
+    ManBook.logger.info "Chapters will be ordered by page #{order}"
+    @order = order.to_sym
+  end
 end
 
 def render(template, my_binding = binding)
@@ -113,9 +119,14 @@ begin
   }
 
   # find pages
-  @pages = Dir.glob(File.join(src_dir, "*.html")).reject{|f| INDEX_HTML == File.basename(f)}.collect{|html_file|
-    ManBook::Parser.parse(html_file)
-  }.sort{|l,r| l[:title] <=> r[:title]}
+  begin
+    @pages = Dir.glob(File.join(src_dir, "*.html")).reject{|f| INDEX_HTML == File.basename(f)}.collect{|html_file|
+      ManBook::Parser.parse(html_file)
+    }.sort{|l,r| l.send(@order) <=> r.send(@order)}
+  rescue NoMethodError
+    attributes = (ManBook::Page.new.public_methods - Object.public_methods).reject{|m| m.match(/=$/)}
+    raise "#{@order} is not a valid sort order attribute. Value values are #{attributes.join(', ')}."
+  end
 
   # index.html
   File.open(File.join(src_dir, INDEX_HTML), 'w') {|f|

data/lib/manbook.rb

@@ -10,15 +10,17 @@ require 'nokogiri'
 require 'manbook/formatter'
 require 'manbook/html_formatter'
 require 'manbook/parser'
+require 'manbook/page'
 
 require 'manbook/errors'
 require 'manbook/log_formatter'
 
 module ManBook
   VERSION = File.read(File.join(File.dirname(__FILE__), *%w[.. VERSION])).chomp
-  TITLE_DEFAULT = "ManBook - a book of selected man pages"
-  AUTHOR_DEFAULT = "Multiple Authors"
+  TITLE_DEFAULT       = 'ManBook - a book of selected man pages'
+  AUTHOR_DEFAULT      = 'Multiple Authors'
   COVER_IMAGE_DEFAULT = File.join(File.dirname(__FILE__), '..', 'templates', 'library_books.jpg')
+  ORDER_DEFAULT       = :title
 
   mattr_accessor :logger
 

data/lib/manbook/page.rb

@@ -0,0 +1,5 @@
+module ManBook
+  class Page
+    attr_accessor :file_name, :title, :author
+  end
+end

data/lib/manbook/parser.rb

@@ -25,7 +25,11 @@ module ManBook
           author = doc.xpath("//h2[text() = 'AUTHORS']/following-sibling::p[1]/descendant-or-self::text()").to_s
         end
 
-        {:file_name => File.basename(html_file), :title => title.split("\n").join(' '), :author => author.split("\n").join(' ')}
+        Page.new.tap do |page|
+          page.file_name = File.basename(html_file)
+          page.title     = title.split("\n").join(' ')
+          page.author    = author.split("\n").join(' ')
+        end
       end
     end
   end

data/manbook.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = "manbook"
-  s.version = "0.0.2"
+  s.version = "0.0.3"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Nicholas E. Rabenau"]
-  s.date = "2013-07-22"
+  s.date = "2013-07-26"
   s.description = "The manbook command can be used to produce an eBook from one or more man pages."
   s.email = "nerab@gmx.net"
   s.executables = ["manbook", "mktoc", "manbook"]
@@ -22,6 +22,7 @@ Gem::Specification.new do |s|
     ".document",
     "Gemfile",
     "Gemfile.lock",
+    "Guardfile",
     "LICENSE.txt",
     "README.md",
     "Rakefile",
@@ -34,6 +35,7 @@ Gem::Specification.new do |s|
     "lib/manbook/formatter.rb",
     "lib/manbook/html_formatter.rb",
     "lib/manbook/log_formatter.rb",
+    "lib/manbook/page.rb",
     "lib/manbook/parser.rb",
     "manbook.gemspec",
     "templates/_page.html.erb",
@@ -43,6 +45,7 @@ Gem::Specification.new do |s|
     "templates/library_books.jpg",
     "templates/manbook.ncx.erb",
     "templates/manbook.opf.erb",
+    "test/fixtures/about.html",
     "test/fixtures/alt-cover.jpg",
     "test/fixtures/bash.html",
     "test/fixtures/cat.html",
@@ -53,7 +56,8 @@ Gem::Specification.new do |s|
     "test/fixtures/man.html",
     "test/helper.rb",
     "test/unit/test_manbook.rb",
-    "test/unit/test_mktoc.rb"
+    "test/unit/test_mktoc.rb",
+    "test/unit/test_page.rb"
   ]
   s.homepage = "http://github.com/nerab/manbook"
   s.licenses = ["MIT"]
@@ -72,6 +76,8 @@ Gem::Specification.new do |s|
       s.add_development_dependency(%q<pry>, [">= 0"])
       s.add_development_dependency(%q<bundler>, ["~> 1.0"])
       s.add_development_dependency(%q<jeweler>, ["~> 1.8"])
+      s.add_development_dependency(%q<guard-minitest>, [">= 0"])
+      s.add_development_dependency(%q<guard-bundler>, [">= 0"])
     else
       s.add_dependency(%q<activesupport>, ["~> 4.0"])
       s.add_dependency(%q<open4>, ["~> 1.3"])
@@ -80,6 +86,8 @@ Gem::Specification.new do |s|
       s.add_dependency(%q<pry>, [">= 0"])
       s.add_dependency(%q<bundler>, ["~> 1.0"])
       s.add_dependency(%q<jeweler>, ["~> 1.8"])
+      s.add_dependency(%q<guard-minitest>, [">= 0"])
+      s.add_dependency(%q<guard-bundler>, [">= 0"])
     end
   else
     s.add_dependency(%q<activesupport>, ["~> 4.0"])
@@ -89,6 +97,8 @@ Gem::Specification.new do |s|
     s.add_dependency(%q<pry>, [">= 0"])
     s.add_dependency(%q<bundler>, ["~> 1.0"])
     s.add_dependency(%q<jeweler>, ["~> 1.8"])
+    s.add_dependency(%q<guard-minitest>, [">= 0"])
+    s.add_dependency(%q<guard-bundler>, [">= 0"])
   end
 end
 

data/templates/_page.html.erb

@@ -1 +1 @@
-<li><a href="<%= page[:file_name] %>"><%= page[:title] %></a></li>
+<li><a href="<%= page.file_name %>"><%= page.title %></a></li>

data/templates/manbook.ncx.erb

@@ -13,9 +13,9 @@
   <navMap>
     <% i = 0 %>
     <% @pages.each_with_index{|page, i| %>
-      <navPoint id="<%= page[:file_name] %>" playOrder="<%= i + 1 %>">
-        <navLabel><text><%= page[:title] %></text></navLabel>
-        <content src="<%= page[:file_name] %>" />
+      <navPoint id="<%= page.file_name %>" playOrder="<%= i + 1 %>">
+        <navLabel><text><%= page.title %></text></navLabel>
+        <content src="<%= page.file_name %>" />
       </navPoint>
     <% } %>
 <% unless @cover_image.nil? %>

data/templates/manbook.opf.erb

@@ -17,7 +17,7 @@
     <item id="index" href="index.html" media-type="text/html" />
     <item id="toc" href="index.ncx" media-type="application/x-dtbncx+xml"/>
     <% @pages.each{|page| %>
-    <item id="<%= page[:file_name] %>" href="<%= page[:file_name] %>" media-type="text/html" />
+    <item id="<%= page.file_name %>" href="<%= page.file_name %>" media-type="text/html" />
     <% } %>
   </manifest>
   <spine toc="toc">
@@ -26,7 +26,7 @@
 <% end %>
     <itemref idref="index" />
     <% @pages.each{|page| %>
-    <itemref idref="<%= page[:file_name] %>" />
+    <itemref idref="<%= page.file_name %>" />
     <% } %>
   </spine>
   <guide>

data/test/fixtures/about.html

@@ -0,0 +1,27 @@
+<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
+<html>
+<head>
+  <meta name="generator" content="mktoc v0.0.2">
+  <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
+  <style type="text/css">
+    h1 {font-size: 1.5em; margin-bottom: 1em;}
+    h2 {font-size: 1.375em; margin: 1.25em 0 0.25em 0;}
+    h3 {font-size: 1.25em; margin: 1.25em 0 0.25em 0;}
+    h4 {font-size: 1.125em; margin: 1.25em 0 0.25em 0;}
+    h5 {font-size: 1em; font-weight: bold; margin: 1.25em 0 0.25em 0;}
+    h6 {font-size: 1.375em; font-weight: bold; margin: 1.25em 0 0.25em 0;}
+    table {margin-top: 0.75em;}
+    p {margin: .125em 0 .5em 0; text-indent: 0;}
+    ul, ol, blockquote, pre, dl, hr {margin-top: .125em; margin-bottom: .5em;}
+    .toc ul {list-style: none; padding-left: 0; margin-left: 20px;}
+  </style>
+  <title>About this book</title>
+</head>
+<body>
+<h1 style="font-size: 2em; margin-top: 2em;" align="center">ManBook - a book of selected man pages</h1>
+
+<p>This book was created using the <a href='http://github.com/nerab/manbook'>mktoc v0.0.2</a> tool.</p>
+
+<p>The cover image was published by CCAC North Library under CC BY 2.0. It can be found at <a href='http://www.flickr.com/photos/ccacnorthlib/4131076795/'>flickr</a>.</p>
+</body>
+</html>

data/test/helper.rb

@@ -2,6 +2,7 @@ require 'bundler'
 require 'tmpdir'
 require 'fileutils'
 require 'active_support/core_ext/string'
+require 'active_support/core_ext/hash/reverse_merge'
 
 begin
   Bundler.setup(:default, :development)

data/test/unit/test_mktoc.rb

@@ -37,6 +37,8 @@ module ManBookTest
 
     COVER_IMAGE_ALT = File.join(FIXTURES_DIR, 'alt-cover.jpg')
 
+    SPECIAL_TITLE_FIXTURES = ['about.html', 'bash.html', 'git.html', 'gunzip.html', 'less.html', 'man.html']
+
     def setup
       super
       fixtures = Dir.glob(File.join(FIXTURES_DIR, '*.html'))
@@ -50,38 +52,53 @@ module ManBookTest
 
     def test_defaults
       assert_exec("#{app_script} #{output_dir}")
-      test_all_workproducts(ManBook::TITLE_DEFAULT, ManBook::AUTHOR_DEFAULT, File.basename(ManBook::COVER_IMAGE_DEFAULT))
+      test_all_workproducts(ManBook::TITLE_DEFAULT, ManBook::AUTHOR_DEFAULT, :cover_image => File.basename(ManBook::COVER_IMAGE_DEFAULT))
     end
 
     def test_overridden_title
       title = "Foo42Bar"
       assert_exec("#{app_script} #{output_dir} --title \"#{title}\"")
-      test_all_workproducts(title, ManBook::AUTHOR_DEFAULT, File.basename(ManBook::COVER_IMAGE_DEFAULT))
+      test_all_workproducts(title, ManBook::AUTHOR_DEFAULT, :cover_image => File.basename(ManBook::COVER_IMAGE_DEFAULT))
     end
 
     def test_overridden_author
       author = "James Born"
       assert_exec("#{app_script} #{output_dir} --author \"#{author}\"")
-      test_all_workproducts(ManBook::TITLE_DEFAULT, author, File.basename(ManBook::COVER_IMAGE_DEFAULT))
+      test_all_workproducts(ManBook::TITLE_DEFAULT, author, :cover_image => File.basename(ManBook::COVER_IMAGE_DEFAULT))
     end
 
     def test_overridden_title_and_author
       title = "Foo42Bar42"
       author = "James F. Born"
       assert_exec("#{app_script} #{output_dir} --title \"#{title}\" --author \"#{author}\"")
-      test_all_workproducts(title, author, File.basename(ManBook::COVER_IMAGE_DEFAULT))
+      test_all_workproducts(title, author, :cover_image => File.basename(ManBook::COVER_IMAGE_DEFAULT))
     end
 
     def test_no_cover_image
       assert_exec("#{app_script} #{output_dir} --no-cover-image")
-      test_all_workproducts(ManBook::TITLE_DEFAULT, ManBook::AUTHOR_DEFAULT, nil)
+      test_all_workproducts(ManBook::TITLE_DEFAULT, ManBook::AUTHOR_DEFAULT)
+    end
+
+    def test_order_by_title
+      assert_exec("#{app_script} #{output_dir} --no-cover-image --sort-by=title")
+      test_all_workproducts(ManBook::TITLE_DEFAULT, ManBook::AUTHOR_DEFAULT, :order => :title)
+    end
+
+    def test_order_by_author
+      assert_exec("#{app_script} #{output_dir} --no-cover-image --sort-by=author")
+      test_all_workproducts(ManBook::TITLE_DEFAULT, ManBook::AUTHOR_DEFAULT, :order => :author)
+    end
+
+    def test_order_by_unknown_attribute
+      order = "DOES_NOT_EXIST"
+      assert_exec("#{app_script} #{output_dir} --no-cover-image --sort-by=#{order}", false, nil, /ERROR: #{order} is not a valid sort order attribute/)
     end
 
     def test_alt_cover_image
       cover_image = COVER_IMAGE_ALT
       assert(File.exist?(cover_image))
       assert_exec("#{app_script} #{output_dir} --cover-image #{cover_image}")
-      test_all_workproducts(ManBook::TITLE_DEFAULT, ManBook::AUTHOR_DEFAULT, File.basename(cover_image))
+      test_all_workproducts(ManBook::TITLE_DEFAULT, ManBook::AUTHOR_DEFAULT, :cover_image => File.basename(cover_image))
     end
 
     def test_alt_cover_image_not_found
@@ -91,11 +108,13 @@ module ManBookTest
     end
 
   private
-    def test_all_workproducts(title, author, cover_image = nil)
-      if cover_image.nil?
+    def test_all_workproducts(title, author, options = {})
+      options.reverse_merge!({:cover_image => nil, :order => ManBook::ORDER_DEFAULT})
+
+      if options[:cover_image].nil?
         workproducts = WORKPRODUCTS
       else
-        workproducts = WORKPRODUCTS.merge({:cover => cover_image})
+        workproducts = WORKPRODUCTS.merge({:cover => options[:cover_image]})
       end
 
       expected = @fixtures + workproducts.values
@@ -109,7 +128,7 @@ module ManBookTest
         # dispatch to test that is specific to the work product
         wp_test = WORK_PRODUCT_TESTS[k]
         raise "No test defined for work product #{k}" if wp_test.nil?
-        send(wp_test, title, author, cover_image)
+        send(wp_test, title, author, options)
       }
     end
 
@@ -143,9 +162,12 @@ module ManBookTest
       "#{APP_SCRIPT}"
     end
 
-    def test_workproduct_html(title, author, cover_image = nil)
+    def test_workproduct_html(title, author, options)
+      options.reverse_merge!({:cover_image => nil, :order => ManBook::ORDER_DEFAULT})
+
       doc = Nokogiri::HTML(File.read(File.join(output_dir, 'index.html')))
-      assert_workproduct(['about.html'].concat(@fixtures), doc, '/html/body/ul/li', 'a/@href')
+
+      assert_workproduct(@fixtures, doc, '/html/body/ul/li', 'a/@href')
 
       # index.html does not use the book title, but "Table Of Contents"
       assert_equal("Table Of Contents", doc.xpath('/html/head/title/text()').to_s)
@@ -153,36 +175,49 @@ module ManBookTest
 
       assert_equal(GENERATOR, doc.xpath("/html/head/meta[@name='generator']/@content").to_s)
       assert_equal("About this book", doc.xpath('/html/body/ul/li[1]/a/text()').to_s)
+
+      sorted_pages = pages.sort{|l,r| l.send(options[:order]) <=> r.send(options[:order])}
+
+      # Test href in content and in order
+      expected_files = sorted_pages.map{|p| p.file_name}
+      actual_files   = doc.xpath('/html/body/ul/li/a/@href').map{|a| a.value}
+      assert_equal(expected_files, actual_files)
+
+      # Test titles in content and in order
+      expected_titles = sorted_pages.map{|p| p.title.upcase}
+      actual_titles   = doc.xpath("/html/body/ul/li").map{|li| li.inner_text.upcase}
+      assert_equal(expected_titles, actual_titles)
     end
 
-    def test_workproduct_ncx(title, author, cover_image = nil)
+    def test_workproduct_ncx(title, author, options)
+      options.reverse_merge!({:cover_image => nil, :order => ManBook::ORDER_DEFAULT})
+
       doc = Nokogiri::XML(File.read(File.join(output_dir, 'index.ncx')))
 
-      fixtures = ['about.html'].concat(@fixtures)
-      fixtures << cover_image unless cover_image.nil?
-      assert_workproduct(fixtures, doc, '/xmlns:ncx/xmlns:navMap/xmlns:navPoint', 'xmlns:content/@src')
+      @fixtures << options[:cover_image] unless options[:cover_image].nil?
+      assert_workproduct(@fixtures, doc, '/xmlns:ncx/xmlns:navMap/xmlns:navPoint', 'xmlns:content/@src')
 
       assert_equal(title, doc.xpath("/xmlns:ncx/xmlns:head/xmlns:meta[@name='dtb:title']/@content").to_s)
       assert_equal(title, doc.xpath("/xmlns:ncx/xmlns:docTitle/xmlns:text/text()").to_s)
       assert_equal(GENERATOR, doc.xpath("/xmlns:ncx/xmlns:head/xmlns:meta[@name='dtb:generator']/@content").to_s)
 
-      # TODO id and order
+      # TODO id and order of members as specified in playOrder
       # navPoint
       #   @id="bash.html"
       #   @playOrder="0"
     end
 
-    def test_workproduct_opf(title, author, cover_image = nil)
+    def test_workproduct_opf(title, author, options)
+      options.reverse_merge!({:cover_image => nil, :order => ManBook::ORDER_DEFAULT})
       doc = Nokogiri::XML(File.read(File.join(output_dir, 'index.opf')))
 
       # the opf must include links to index.html and index.ncx
-      item_fixtures = ['index.html', 'index.ncx', 'about.html'].concat(@fixtures)
-      item_fixtures << cover_image unless cover_image.nil?
+      item_fixtures = ['index.html', 'index.ncx'].concat(@fixtures)
       assert_workproduct(item_fixtures, doc, '//xmlns:manifest/xmlns:item', '@href')
 
-      # cross-references within the document
-      xref_fixtures = ['index', 'about.html'].concat(@fixtures)
-      xref_fixtures << 'cover-image' unless cover_image.nil?
+      # cross-references within the document, but only include *.html fixtures
+      xref_fixtures = ['index'].concat(@fixtures.select{|f| f.match(/.*\.html/)})
+      xref_fixtures << 'cover-image' unless options[:cover_image].nil?
       assert_workproduct(xref_fixtures, doc, '//xmlns:spine/xmlns:itemref', '@idref')
 
       assert_equal(title, doc.xpath('/xmlns:package/xmlns:metadata/dc:title/text()',
@@ -194,16 +229,25 @@ module ManBookTest
                                         'xmlns' => 'http://www.idpf.org/2007/opf'}).first.to_s)
 
       # reference to cover image in meta data. The other two references were already tested above
-      unless cover_image.nil?
+      unless options[:cover_image].nil?
         assert_equal('cover-image', doc.xpath("/xmlns:package/xmlns:metadata/xmlns:meta[@name='cover']/@content").to_s)
       end
+
+      # TODO id and href. Order does not matter.
+      # <item id="less.html" href="less.html" media-type="text/html" />
+
+      # TODO Assert than members present in the manifest are also present in the spine
     end
 
-    def test_workproduct_about(title, author, cover_image = nil)
+    def test_workproduct_about(title, author, options)
+      options.reverse_merge!({:cover_image => nil, :order => ManBook::ORDER_DEFAULT})
+
       # no further tests
     end
 
-    def test_workproduct_cover(title, author, cover_image = nil)
+    def test_workproduct_cover(title, author, options)
+      options.reverse_merge!({:cover_image => nil, :order => ManBook::ORDER_DEFAULT})
+
       # no further tests
     end
 
@@ -229,5 +273,11 @@ module ManBookTest
         assert(fixtures.include?(li), "Could not find '#{li}' in #{fixtures.inspect}")
       }
     end
+
+    def pages
+      @fixtures.map do |fixture|
+        ManBook::Parser.parse(File.join(FIXTURES_DIR, fixture))
+      end
+    end
   end
 end

data/test/unit/test_page.rb

@@ -0,0 +1,38 @@
+require 'helper'
+
+module ManBookTest
+  class TestPage < ManBookTest::TestCase
+    def test_about
+      file_name = 'about.html'
+      page = ManBook::Parser.parse(fixture(file_name))
+      refute_nil(page)
+      assert_equal('About this book', page.title)
+      assert_equal('', page.author)
+      assert_equal(file_name, page.file_name)
+    end
+
+    def test_cat
+      file_name = 'cat.html'
+      page = ManBook::Parser.parse(fixture(file_name))
+      refute_nil(page)
+      assert_equal('cat — concatenate and print files', page.title)
+      assert_equal('', page.author)
+      assert_equal(file_name, page.file_name)
+    end
+
+    def test_git
+      file_name = 'git.html'
+      page = ManBook::Parser.parse(fixture(file_name))
+      refute_nil(page)
+      assert_equal('git − the stupid content tracker', page.title)
+      assert_match(/Linus Torvalds/, page.author)
+      assert_equal(file_name, page.file_name)
+    end
+
+    private
+
+    def fixture(basename)
+      File.join(FIXTURES_DIR, basename)
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: manbook
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
 platform: ruby
 authors:
 - Nicholas E. Rabenau
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-07-22 00:00:00.000000000 Z
+date: 2013-07-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -108,6 +108,34 @@ dependencies:
     - - ~>
       - !ruby/object:Gem::Version
         version: '1.8'
+- !ruby/object:Gem::Dependency
+  name: guard-minitest
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: guard-bundler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: The manbook command can be used to produce an eBook from one or more
   man pages.
 email: nerab@gmx.net
@@ -123,6 +151,7 @@ files:
 - .document
 - Gemfile
 - Gemfile.lock
+- Guardfile
 - LICENSE.txt
 - README.md
 - Rakefile
@@ -135,6 +164,7 @@ files:
 - lib/manbook/formatter.rb
 - lib/manbook/html_formatter.rb
 - lib/manbook/log_formatter.rb
+- lib/manbook/page.rb
 - lib/manbook/parser.rb
 - manbook.gemspec
 - templates/_page.html.erb
@@ -144,6 +174,7 @@ files:
 - templates/library_books.jpg
 - templates/manbook.ncx.erb
 - templates/manbook.opf.erb
+- test/fixtures/about.html
 - test/fixtures/alt-cover.jpg
 - test/fixtures/bash.html
 - test/fixtures/cat.html
@@ -155,6 +186,7 @@ files:
 - test/helper.rb
 - test/unit/test_manbook.rb
 - test/unit/test_mktoc.rb
+- test/unit/test_page.rb
 homepage: http://github.com/nerab/manbook
 licenses:
 - MIT