epub_tools 0.4.1 → 0.5.0

data/test/epub_initializer_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'test_helper'
 require_relative '../lib/epub_tools/epub_initializer'
 
@@ -16,34 +18,10 @@ class EpubInitializerTest < Minitest::Test
   def test_run_creates_basic_structure
     result = EpubTools::EpubInitializer.new(title: @title, author: @author, destination: @dest).run
 
-    # Check return value is the destination directory
     assert_equal @dest, result
-
-    # Check directories
-    assert Dir.exist?(@dest)
-    assert File.directory?(File.join(@dest, 'META-INF'))
-    assert File.directory?(File.join(@dest, 'OEBPS'))
-    # Check files
-    mimetype = File.join(@dest, 'mimetype')
-
-    assert_path_exists mimetype
-    assert_equal 'application/epub+zip', File.read(mimetype)
-    assert_path_exists File.join(@dest, 'META-INF', 'container.xml')
-    assert_path_exists File.join(@dest, 'OEBPS', 'title.xhtml')
-    assert_path_exists File.join(@dest, 'OEBPS', 'nav.xhtml')
-    assert_path_exists File.join(@dest, 'OEBPS', 'package.opf')
-    assert_path_exists File.join(@dest, 'OEBPS', 'style.css')
-    # Check content of title.xhtml
-    title_page = File.read(File.join(@dest, 'OEBPS', 'title.xhtml'))
-
-    assert_includes title_page, "<h1 class=\"title\">#{@title}</h1>"
-    assert_includes title_page, "<p class=\"author\">by #{@author}</p>"
-    # Check package.opf metadata
-    opf = File.read(File.join(@dest, 'OEBPS', 'package.opf'))
-
-    assert_includes opf, "<dc:title>#{@title}</dc:title>"
-    assert_includes opf, "<dc:creator>#{@author}</dc:creator>"
-    refute_includes opf, 'cover.xhtml'
+    verify_directory_structure
+    verify_required_files
+    verify_file_contents
   end
 
   def test_run_with_cover_image
@@ -136,4 +114,54 @@ class EpubInitializerTest < Minitest::Test
 
     refute_includes opf, 'cover-image'
   end
+
+  private
+
+  def verify_directory_structure
+    assert Dir.exist?(@dest)
+    assert File.directory?(File.join(@dest, 'META-INF'))
+    assert File.directory?(File.join(@dest, 'OEBPS'))
+  end
+
+  def verify_required_files
+    required_files = [
+      'mimetype',
+      File.join('META-INF', 'container.xml'),
+      File.join('OEBPS', 'title.xhtml'),
+      File.join('OEBPS', 'nav.xhtml'),
+      File.join('OEBPS', 'package.opf'),
+      File.join('OEBPS', 'style.css')
+    ]
+
+    required_files.each do |file|
+      assert_path_exists File.join(@dest, file)
+    end
+  end
+
+  def verify_file_contents
+    verify_mimetype_content
+    verify_title_page_content
+    verify_opf_metadata
+  end
+
+  def verify_mimetype_content
+    mimetype = File.read(File.join(@dest, 'mimetype'))
+
+    assert_equal 'application/epub+zip', mimetype
+  end
+
+  def verify_title_page_content
+    title_page = File.read(File.join(@dest, 'OEBPS', 'title.xhtml'))
+
+    assert_includes title_page, "<h1 class=\"title\">#{@title}</h1>"
+    assert_includes title_page, "<p class=\"author\">by #{@author}</p>"
+  end
+
+  def verify_opf_metadata
+    opf = File.read(File.join(@dest, 'OEBPS', 'package.opf'))
+
+    assert_includes opf, "<dc:title>#{@title}</dc:title>"
+    assert_includes opf, "<dc:creator>#{@author}</dc:creator>"
+    refute_includes opf, 'cover.xhtml'
+  end
 end

data/test/pack_ebook_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'test_helper'
 require_relative '../lib/epub_tools/pack_ebook'
 require 'zip'
@@ -14,37 +16,59 @@ class PackEbookTest < Minitest::Test
   end
 
   def test_run_creates_epub_with_expected_entries
-    # Create minimal EPUB directory
+    create_minimal_epub_structure
+    output = File.join(@tmp, 'out.epub')
+    result = EpubTools::PackEbook.new(input_dir: @epub_dir, output_file: output).run
+
+    verify_return_value(output, result)
+    verify_epub_entries(output)
+  end
+
+  private
+
+  def create_minimal_epub_structure
     File.write(File.join(@epub_dir, 'mimetype'), 'application/epub+zip')
     FileUtils.mkdir_p(File.join(@epub_dir, 'META-INF'))
     File.write(File.join(@epub_dir, 'META-INF', 'container.xml'), '<container/>')
     FileUtils.mkdir_p(File.join(@epub_dir, 'OEBPS'))
     File.write(File.join(@epub_dir, 'OEBPS', 'title.xhtml'), '<html/>')
+  end
 
-    output = File.join(@tmp, 'out.epub')
-    result = EpubTools::PackEbook.new(input_dir: @epub_dir, output_file: output).run
+  def verify_return_value(expected_output, result)
+    assert_equal expected_output, result
+    assert_path_exists expected_output, 'Expected output EPUB to exist'
+  end
+
+  def verify_epub_entries(output_file)
+    entries = extract_zip_entries(output_file)
+    verify_mimetype_entry(entries)
+    verify_expected_files(entries)
+  end
 
-    # Check return value is the output file path
-    assert_equal output, result
-    assert_path_exists output, 'Expected output EPUB to exist'
+  def extract_zip_entries(output_file)
     entries = []
-    Zip::File.open(output) do |zip|
+    Zip::File.open(output_file) do |zip|
       zip.each do |entry|
         entries << { name: entry.name, compression: entry.compression_method }
       end
     end
+    entries
+  end
 
-    # First entry should be mimetype, stored without compression
+  def verify_mimetype_entry(entries)
     assert_equal 'mimetype', entries.first[:name]
     assert_equal 0, entries.first[:compression]
+  end
 
-    # Check presence of other files
+  def verify_expected_files(entries)
     names = entries.map { |e| e[:name] }
 
     assert_includes names, 'META-INF/container.xml'
     assert_includes names, 'OEBPS/title.xhtml'
   end
 
+  public
+
   def test_missing_input_dir_raises_error
     assert_raises(ArgumentError) do
       EpubTools::PackEbook.new(input_dir: File.join(@tmp, 'nonexistent'), output_file: 'out.epub').run

data/test/split_chapters_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'test_helper'
 require_relative '../lib/epub_tools/split_chapters'
 
@@ -30,10 +32,19 @@ class SplitChaptersTest < Minitest::Test
     result = EpubTools::SplitChapters.new(input_file: @input, book_title: 'BookTitle', output_dir: @out,
                                           output_prefix: 'chap').run
 
-    # Check return value is an array of chapter file paths
+    verify_return_value(result)
+    verify_generated_files_exist(result)
+    verify_chapter_contents
+  end
+
+  private
+
+  def verify_return_value(result)
     assert_instance_of Array, result
     assert_equal 3, result.size
+  end
 
+  def verify_generated_files_exist(result)
     expected_paths = [
       File.join(@out, 'chap_0.xhtml'),
       File.join(@out, 'chap_1.xhtml'),
@@ -47,25 +58,34 @@ class SplitChaptersTest < Minitest::Test
 
     files = Dir.children(@out)
 
-    assert_includes files, 'chap_0.xhtml'
-    assert_includes files, 'chap_1.xhtml'
-    assert_includes files, 'chap_2.xhtml'
+    ['chap_0.xhtml', 'chap_1.xhtml', 'chap_2.xhtml'].each do |file|
+      assert_includes files, file
+    end
+  end
+
+  def verify_chapter_contents
+    verify_prologue_content
+    verify_chapter_one_content
+    verify_chapter_two_content
+  end
 
-    # Prologue
+  def verify_prologue_content
     prologue = File.read(File.join(@out, 'chap_0.xhtml'))
 
     assert_includes prologue, '<h1>Prologue</h1>'
     assert_includes prologue, 'Intro text'
     refute_includes prologue, 'Chapter 1'
+  end
 
-    # Chapter 1
+  def verify_chapter_one_content
     ch1 = File.read(File.join(@out, 'chap_1.xhtml'))
 
     assert_includes ch1, '<h1>Chapter 1</h1>'
     assert_includes ch1, 'First paragraph'
     refute_includes ch1, 'Chapter 2'
+  end
 
-    # Chapter 2
+  def verify_chapter_two_content
     ch2 = File.read(File.join(@out, 'chap_2.xhtml'))
 
     assert_includes ch2, '<h1>Chapter 2</h1>'

data/test/style_finder_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'yaml'
 require_relative 'test_helper'
 require_relative '../lib/epub_tools/style_finder'

data/test/test_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'simplecov'
 SimpleCov.start do
   add_filter '/test/'

data/test/unpack_ebook_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'test_helper'
 require_relative '../lib/epub_tools/unpack_ebook'
 require 'zip'
@@ -5,37 +7,60 @@ require 'zip'
 class UnpackEbookTest < Minitest::Test
   def setup
     @tmp = Dir.mktmpdir
-    # Build a minimal EPUB directory for zipping
     @build_dir = File.join(@tmp, 'build')
+    @epub_file = File.join(@tmp, 'test.epub')
+    @dest_dir = File.join(@tmp, 'output')
+
+    create_epub_build_directory
+    create_zip_file_from_build_dir
+  end
+
+  private
+
+  def create_epub_build_directory
     FileUtils.mkdir_p(File.join(@build_dir, 'META-INF'))
     FileUtils.mkdir_p(File.join(@build_dir, 'OEBPS'))
+
+    write_epub_files
+  end
+
+  def write_epub_files
     File.write(File.join(@build_dir, 'mimetype'), 'application/epub+zip')
     File.write(File.join(@build_dir, 'META-INF', 'container.xml'), '<container/>')
     File.write(File.join(@build_dir, 'OEBPS', 'title.xhtml'), '<html/>')
+  end
 
-    # Create .epub zip file
-    @epub_file = File.join(@tmp, 'test.epub')
-    # Create .epub zip file with absolute src paths to avoid cwd issues
-    Zip::File.open(@epub_file, Zip::File::CREATE) do |zip|
-      # Add mimetype first, uncompressed
-      mime_src = File.join(@build_dir, 'mimetype')
-      zip.add_stored('mimetype', mime_src)
-      # Add directories and files
-      Dir.glob(File.join(@build_dir, '**', '*'), File::FNM_DOTMATCH).sort.each do |src_path|
-        rel_path = src_path.sub(%r{^#{Regexp.escape(@build_dir)}/?}, '')
-        next if rel_path.empty? || rel_path == 'mimetype'
-
-        if File.directory?(src_path)
-          zip.mkdir(rel_path)
-        else
-          zip.add(rel_path, src_path)
-        end
-      end
+  def create_zip_file_from_build_dir
+    Zip::File.open(@epub_file, create: true) do |zip|
+      add_mimetype_to_zip(zip)
+      add_remaining_files_to_zip(zip)
     end
+  end
 
-    @dest_dir = File.join(@tmp, 'output')
+  def add_mimetype_to_zip(zip)
+    mime_src = File.join(@build_dir, 'mimetype')
+    zip.add_stored('mimetype', mime_src)
+  end
+
+  def add_remaining_files_to_zip(zip)
+    Dir.glob(File.join(@build_dir, '**', '*'), File::FNM_DOTMATCH).sort.each do |src_path|
+      add_file_or_directory_to_zip(zip, src_path)
+    end
   end
 
+  def add_file_or_directory_to_zip(zip, src_path)
+    rel_path = src_path.sub(%r{^#{Regexp.escape(@build_dir)}/?}, '')
+    return if rel_path.empty? || rel_path == 'mimetype'
+
+    if File.directory?(src_path)
+      zip.mkdir(rel_path)
+    else
+      zip.add(rel_path, src_path)
+    end
+  end
+
+  public
+
   def teardown
     FileUtils.remove_entry(@tmp)
   end

data/test/xhtml_cleaner_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'yaml'
 require_relative 'test_helper'
 require_relative '../lib/epub_tools/xhtml_cleaner'

data/test/xhtml_extractor_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'test_helper'
 require 'zip'
 require_relative '../lib/epub_tools/xhtml_extractor'
@@ -9,7 +11,7 @@ class XHTMLExtractorTest < Minitest::Test
     @tgt = File.join(@tmp, 'tgt')
     Dir.mkdir(@src)
     @file = File.join(@src, 'sample.epub')
-    Zip::File.open(@file, Zip::File::CREATE) do |zip|
+    Zip::File.open(@file, create: true) do |zip|
       zip.get_output_stream('chapter1.xhtml') { |f| f.write '<html><body><p>One</p></body></html>' }
       zip.get_output_stream('nav.xhtml') { |f| f.write '<html><body>Nav</body></html>' }
       zip.get_output_stream('folder/ch2.xhtml') { |f| f.write '<html><body><p>Two</p></body></html>' }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: epub_tools
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.5.0
 platform: ruby
 authors:
 - Jaime Rodas
@@ -43,14 +43,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.4'
+        version: '3.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.4'
+        version: '3.2'
 email:
 - rodas@hey.com
 executables:
@@ -64,6 +64,7 @@ files:
 - ".nova/Configuration.json"
 - ".rubocop.yml"
 - ".tool-versions"
+- CLAUDE.md
 - Gemfile
 - Gemfile.lock
 - LICENSE
@@ -73,12 +74,18 @@ files:
 - epub_tools.gemspec
 - lib/epub_tools.rb
 - lib/epub_tools/add_chapters.rb
+- lib/epub_tools/chapter_validator.rb
 - lib/epub_tools/cli.rb
+- lib/epub_tools/cli/command_options_configurator.rb
 - lib/epub_tools/cli/command_registry.rb
 - lib/epub_tools/cli/option_builder.rb
 - lib/epub_tools/cli/runner.rb
 - lib/epub_tools/compile_book.rb
+- lib/epub_tools/compile_workspace.rb
+- lib/epub_tools/epub_configuration.rb
+- lib/epub_tools/epub_file_writer.rb
 - lib/epub_tools/epub_initializer.rb
+- lib/epub_tools/epub_metadata_builder.rb
 - lib/epub_tools/loggable.rb
 - lib/epub_tools/pack_ebook.rb
 - lib/epub_tools/split_chapters.rb
@@ -87,8 +94,10 @@ files:
 - lib/epub_tools/version.rb
 - lib/epub_tools/xhtml_cleaner.rb
 - lib/epub_tools/xhtml_extractor.rb
+- lib/epub_tools/xhtml_generator.rb
 - style.css
 - test/add_chapters_test.rb
+- test/chapter_validator_test.rb
 - test/cli/command_registry_test.rb
 - test/cli/option_builder_test.rb
 - test/cli/runner_test.rb
@@ -96,6 +105,7 @@ files:
 - test/cli_test.rb
 - test/cli_version_test.rb
 - test/compile_book_test.rb
+- test/compile_workspace_test.rb
 - test/epub_initializer_test.rb
 - test/pack_ebook_test.rb
 - test/split_chapters_test.rb