RubyGems - epub_tools - Versions diffs - 0.4.1 → 0.5.0 - Mend

epub_tools 0.4.1 → 0.5.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (51) hide show

checksums.yaml +4 -4
data/.github/workflows/ci.yml +3 -0
data/.rubocop.yml +10 -17
data/CLAUDE.md +124 -0
data/Gemfile +4 -4
data/Gemfile.lock +39 -34
data/Rakefile +2 -0
data/bin/epub-tools +2 -0
data/epub_tools.gemspec +3 -1
data/lib/epub_tools/add_chapters.rb +47 -29
data/lib/epub_tools/chapter_validator.rb +40 -0
data/lib/epub_tools/cli/command_options_configurator.rb +115 -0
data/lib/epub_tools/cli/command_registry.rb +2 -0
data/lib/epub_tools/cli/option_builder.rb +5 -3
data/lib/epub_tools/cli/runner.rb +59 -110
data/lib/epub_tools/cli.rb +16 -29
data/lib/epub_tools/compile_book.rb +48 -65
data/lib/epub_tools/compile_workspace.rb +40 -0
data/lib/epub_tools/epub_configuration.rb +33 -0
data/lib/epub_tools/epub_file_writer.rb +57 -0
data/lib/epub_tools/epub_initializer.rb +83 -162
data/lib/epub_tools/epub_metadata_builder.rb +92 -0
data/lib/epub_tools/loggable.rb +2 -0
data/lib/epub_tools/pack_ebook.rb +28 -14
data/lib/epub_tools/split_chapters.rb +42 -17
data/lib/epub_tools/style_finder.rb +17 -6
data/lib/epub_tools/unpack_ebook.rb +20 -10
data/lib/epub_tools/version.rb +3 -1
data/lib/epub_tools/xhtml_cleaner.rb +1 -0
data/lib/epub_tools/xhtml_extractor.rb +20 -10
data/lib/epub_tools/xhtml_generator.rb +71 -0
data/lib/epub_tools.rb +2 -0
data/test/add_chapters_test.rb +49 -25
data/test/chapter_validator_test.rb +47 -0
data/test/cli/command_registry_test.rb +2 -0
data/test/cli/option_builder_test.rb +24 -14
data/test/cli/runner_test.rb +15 -15
data/test/cli_commands_test.rb +2 -0
data/test/cli_test.rb +2 -0
data/test/cli_version_test.rb +2 -0
data/test/compile_book_test.rb +17 -102
data/test/compile_workspace_test.rb +55 -0
data/test/epub_initializer_test.rb +55 -27
data/test/pack_ebook_test.rb +33 -9
data/test/split_chapters_test.rb +27 -7
data/test/style_finder_test.rb +2 -0
data/test/test_helper.rb +2 -0
data/test/unpack_ebook_test.rb +45 -20
data/test/xhtml_cleaner_test.rb +2 -0
data/test/xhtml_extractor_test.rb +3 -1
metadata +13 -3

data/lib/epub_tools/xhtml_extractor.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 require 'zip'
 require 'fileutils'
 require_relative 'loggable'
@@ -6,6 +8,7 @@ module EpubTools
   # Extracts text .xhtml files from EPUB archives, excluding nav.xhtml
   class XHTMLExtractor
     include Loggable
     # Initializes the class
     # @param options [Hash] Configuration options
     # @option options [String] :source_dir Directory containing source .epub files (required)
@@ -39,21 +42,28 @@ module EpubTools
       epub_name = File.basename(epub_path, '.epub')
       log "Extracting from #{epub_name}.epub"
       extracted_files = []
       Zip::File.open(epub_path) do |zip_file|
-        zip_file.each do |entry|
-          next unless entry.name.downcase.end_with?('.xhtml')
-          next if File.basename(entry.name).downcase == 'nav.xhtml'
-          output_path = File.join(@target_dir, "#{epub_name}_#{File.basename(entry.name)}")
-          FileUtils.mkdir_p(File.dirname(output_path))
-          entry.extract(output_path) { true }
-          log output_path
-          extracted_files << output_path
-        end
+        zip_file.each { |entry| extract_entry_if_xhtml(entry, epub_name, extracted_files) }
       end
       extracted_files
     rescue Zip::Error => e
       warn "⚠️ Failed to process #{epub_path}: #{e.message}"
     end
+    def extract_entry_if_xhtml(entry, epub_name, extracted_files)
+      return unless xhtml_entry?(entry)
+      renamed = "#{epub_name}_#{File.basename(entry.name)}"
+      output_path = File.join(@target_dir, renamed)
+      FileUtils.mkdir_p(File.dirname(output_path))
+      entry.extract(renamed, destination_directory: @target_dir) { true }
+      log output_path
+      extracted_files << output_path
+    end
+    def xhtml_entry?(entry)
+      entry.name.downcase.end_with?('.xhtml') && File.basename(entry.name).downcase != 'nav.xhtml'
+    end
   end
 end

data/lib/epub_tools/xhtml_generator.rb ADDED Viewed

@@ -0,0 +1,71 @@
+# frozen_string_literal: true
+module EpubTools
+  # Generates XHTML content for EPUB files
+  class XhtmlGenerator
+    attr_accessor :cover_image_fname
+    def initialize(title:, author:)
+      @title = title
+      @author = author
+      @cover_image_fname = nil
+    end
+    # Generates title page XHTML content
+    def build_title_page
+      <<~XHTML
+        <?xml version="1.0" encoding="UTF-8"?>
+        <html xmlns="http://www.w3.org/1999/xhtml" lang="en">
+          <head>
+            <meta charset="UTF-8" />
+            <title>#{@title}</title>
+            <link rel="stylesheet" type="text/css" href="style.css"/>
+          </head>
+          <body>
+            <h1 class="title">#{@title}</h1>
+            <p class="author">by #{@author}</p>
+          </body>
+        </html>
+      XHTML
+    end
+    # Generates cover page XHTML content
+    def build_cover_page
+      <<~XHTML
+        <?xml version="1.0" encoding="UTF-8"?>
+        <html xmlns="http://www.w3.org/1999/xhtml" lang="en">
+          <head>
+            <meta charset="UTF-8" />
+            <title>Cover</title>
+            <link rel="stylesheet" type="text/css" href="style.css"/>
+          </head>
+          <body>
+            <div class="cover-image">
+              <img src="#{@cover_image_fname}" alt="Cover"/>
+            </div>
+          </body>
+        </html>
+      XHTML
+    end
+    # Generates navigation XHTML content
+    def build_nav_page
+      <<~XHTML
+        <?xml version="1.0" encoding="utf-8"?>
+        <html xmlns="http://www.w3.org/1999/xhtml" xmlns:epub="http://www.idpf.org/2007/ops" lang="en">
+          <head>
+            <title>Table of Contents</title>
+          </head>
+          <body>
+            <nav epub:type="toc" id="toc">
+              <h1>Table of Contents</h1>
+              <ol>
+                <li><a href="title.xhtml">Title Page</a></li>
+              </ol>
+            </nav>
+          </body>
+        </html>
+      XHTML
+    end
+  end
+end

data/lib/epub_tools.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 require_relative 'epub_tools/version'
 require_relative 'epub_tools/loggable'
 require_relative 'epub_tools/add_chapters'

data/test/add_chapters_test.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 require_relative 'test_helper'
 require_relative '../lib/epub_tools/add_chapters'
 require 'nokogiri'
@@ -53,47 +55,69 @@ class AddChaptersTest < Minitest::Test
   end
   def test_run_moves_files_and_updates_opf_and_nav
-    # Run the add chapters task
     result = EpubTools::AddChapters.new(chapters_dir: @chapters_dir, oebps_dir: @epub_dir).run
-    # Check return value is an array of moved file basenames
+    verify_return_value(result)
+    verify_files_moved
+    verify_opf_structure
+    verify_nav_structure
+  end
+  private
+  def verify_return_value(result)
     assert_instance_of Array, result
     assert_equal 2, result.size
     assert_includes result, 'chapter_0.xhtml'
     assert_includes result, 'chapter_1.xhtml'
+  end
-    # Original chapter files should be moved
+  def verify_files_moved
     assert_empty Dir.glob(File.join(@chapters_dir, '*.xhtml'))
     assert_path_exists File.join(@epub_dir, 'chapter_0.xhtml')
     assert_path_exists File.join(@epub_dir, 'chapter_1.xhtml')
+  end
-    # package.opf should include manifest items and spine refs
-    doc = Nokogiri::XML(File.read(@opf_file)) { |cfg| cfg.default_xml.noblanks }
-    items   = doc.xpath('//xmlns:manifest/xmlns:item')
-    idrefs  = doc.xpath('//xmlns:spine/xmlns:itemref')
-    hrefs   = items.map { |i| i['href'] }
-    ids     = items.map { |i| i['id'] }
-    refs    = idrefs.map { |ir| ir['idref'] }
-    assert_includes hrefs, 'chapter_0.xhtml'
-    assert_includes hrefs, 'chapter_1.xhtml'
-    assert_includes ids,   'chap0'
-    assert_includes ids,   'chap1'
-    assert_includes refs,  'chap0'
-    assert_includes refs,  'chap1'
-    # nav.xhtml should have list entries for each chapter
-    nav_doc = Nokogiri::XML(File.read(@nav_file))
-    # strip namespaces for easy querying
-    nav_doc.remove_namespaces!
-    links = nav_doc.xpath('//nav/ol/li/a')
+  def verify_opf_structure
+    doc = parse_opf_document
+    opf_data = extract_opf_data(doc)
+    assert_includes opf_data[:hrefs], 'chapter_0.xhtml'
+    assert_includes opf_data[:hrefs], 'chapter_1.xhtml'
+    assert_includes opf_data[:ids], 'chap0'
+    assert_includes opf_data[:ids], 'chap1'
+    assert_includes opf_data[:refs], 'chap0'
+    assert_includes opf_data[:refs], 'chap1'
+  end
+  def verify_nav_structure
+    links = extract_nav_links
     assert_equal 2, links.size
-    # First is Prologue (chapter_0)
     assert_equal 'chapter_0.xhtml', links[0]['href']
     assert_equal 'Prologue', links[0].text
-    # Second is Chapter 1
     assert_equal 'chapter_1.xhtml', links[1]['href']
     assert_equal 'Chapter 1', links[1].text
   end
+  def parse_opf_document
+    Nokogiri::XML(File.read(@opf_file)) { |cfg| cfg.default_xml.noblanks }
+  end
+  def extract_opf_data(doc)
+    items = doc.xpath('//xmlns:manifest/xmlns:item')
+    idrefs = doc.xpath('//xmlns:spine/xmlns:itemref')
+    {
+      hrefs: items.map { |i| i['href'] },
+      ids: items.map { |i| i['id'] },
+      refs: idrefs.map { |ir| ir['idref'] }
+    }
+  end
+  def extract_nav_links
+    nav_doc = Nokogiri::XML(File.read(@nav_file))
+    nav_doc.remove_namespaces!
+    nav_doc.xpath('//nav/ol/li/a')
+  end
 end

data/test/chapter_validator_test.rb ADDED Viewed

@@ -0,0 +1,47 @@
+# frozen_string_literal: true
+require_relative 'test_helper'
+require_relative '../lib/epub_tools/chapter_validator'
+class ChapterValidatorTest < Minitest::Test
+  def setup
+    @tmp = Dir.mktmpdir
+    @validator = EpubTools::ChapterValidator.new(chapters_dir: @tmp)
+  end
+  def teardown
+    FileUtils.rm_rf(@tmp)
+  end
+  def test_validates_complete_sequence
+    create_chapter_files([1, 2, 3, 4, 5])
+    assert_silent { @validator.validate }
+  end
+  def test_raises_on_missing_chapters
+    create_chapter_files([1, 2, 4, 5]) # Missing 3
+    error = assert_raises(RuntimeError) { @validator.validate }
+    assert_match(/Missing chapter numbers: 3/, error.message)
+  end
+  def test_raises_on_no_chapters
+    error = assert_raises(RuntimeError) { @validator.validate }
+    assert_match(/No chapter files found/, error.message)
+  end
+  def test_handles_non_sequential_start
+    create_chapter_files([5, 6, 7, 8])
+    assert_silent { @validator.validate }
+  end
+  private
+  def create_chapter_files(numbers)
+    numbers.each do |num|
+      File.write(File.join(@tmp, "chapter_#{num}.xhtml"), '<html></html>')
+    end
+  end
+end

data/test/cli/command_registry_test.rb CHANGED Viewed

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

data/test/cli/option_builder_test.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 require_relative '../test_helper'
 require_relative '../../lib/epub_tools'
 require 'stringio'
@@ -41,17 +43,13 @@ class OptionBuilderTest < Minitest::Test
   def test_with_input_file
     @builder.with_input_file('Test input')
-    assert_includes @builder.parser.to_s, '-i, --input-file FILE'
-    assert_includes @builder.parser.to_s, 'Test input (required)'
+    verify_parser_option_format('-i, --input-file FILE', 'Test input (required)')
-    # Test with required=false
     builder2 = EpubTools::CLI::OptionBuilder.new
-    builder2.with_input_file('Test input', false)
+    builder2.with_input_file('Test input', required: false)
-    assert_includes builder2.parser.to_s, 'Test input'
-    refute_includes builder2.parser.to_s, 'Test input (required)'
+    verify_optional_parser_format(builder2, 'Test input')
-    # Test with actual parsing
     @builder.parse(['-i', 'file.txt'])
     assert_equal 'file.txt', @builder.options[:input_file]
@@ -69,20 +67,16 @@ class OptionBuilderTest < Minitest::Test
   end
   def test_with_output_dir
-    # Test with default value
     @builder.with_output_dir('Test output dir', 'default/path')
-    assert_includes @builder.parser.to_s, '-o, --output-dir DIR'
-    assert_includes @builder.parser.to_s, 'Test output dir (default: default/path)'
+    verify_parser_option_format('-o, --output-dir DIR', 'Test output dir (default: default/path)')
     assert_equal 'default/path', @builder.options[:output_dir]
-    # Test without default
     builder2 = EpubTools::CLI::OptionBuilder.new
     builder2.with_output_dir('Test output dir')
+    verify_required_parser_format(builder2, 'Test output dir (required)')
-    assert_includes builder2.parser.to_s, 'Test output dir (required)'
-    # Test actual parsing
     @builder.parse(['-o', 'new/path'])
     assert_equal 'new/path', @builder.options[:output_dir]
@@ -170,4 +164,20 @@ class OptionBuilderTest < Minitest::Test
     assert_equal @builder, result
   end
+  private
+  def verify_parser_option_format(option_format, description)
+    assert_includes @builder.parser.to_s, option_format
+    assert_includes @builder.parser.to_s, description
+  end
+  def verify_optional_parser_format(builder, description)
+    assert_includes builder.parser.to_s, description
+    refute_includes builder.parser.to_s, "#{description} (required)"
+  end
+  def verify_required_parser_format(builder, description)
+    assert_includes builder.parser.to_s, description
+  end
 end

data/test/cli/runner_test.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 require_relative '../test_helper'
 require_relative '../../lib/epub_tools'
 require 'stringio'
@@ -33,15 +35,18 @@ class RunnerTest < Minitest::Test
   end
   def test_handle_command
-    assert_output(/Usage: test-program test-cmd/) do
-      assert_raises(SystemExit) { @runner.handle_command('test-cmd', ['-h']) }
+    @runner.registry.register('add', TestCommand)
+    assert_output(/Usage: test-program add/) do
+      assert_raises(SystemExit) { @runner.handle_command('add', ['-h']) }
     end
   end
   def test_handle_command_with_required_args
     runner = EpubTools::CLI::Runner.new('test-program')
-    runner.registry.register('test-cmd', TestCommand)
-    assert_output("Called!\n") { assert runner.handle_command('test-cmd') }
+    runner.registry.register('add', TestCommand)
+    assert_output("Called!\n") { assert runner.handle_command('add') }
   end
   def test_handle_nonexistent_command
@@ -69,18 +74,13 @@ class RunnerTest < Minitest::Test
   end
   def test_configure_command_options
-    # This is testing a private method, which is generally not recommended,
-    # but it's useful to ensure all command configurations work
+    # Test command configuration through public interface
+    @runner.registry.register('add', TestCommand)
-    # Use send to access private method
-    builder = EpubTools::CLI::OptionBuilder.new
-    # Test each command configuration - we'll just check one example
-    @runner.send(:configure_command_options, 'add', builder)
-    # Check add command options were added
-    assert_includes builder.parser.to_s, '--chapters-dir DIR'
-    assert_includes builder.parser.to_s, '--oebps-dir DIR'
+    # Should show help without errors
+    assert_output(/Usage: test-program add/) do
+      assert_raises(SystemExit) { @runner.handle_command('add', ['-h']) }
+    end
   end
   def test_run_with_unknown_command

data/test/cli_commands_test.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 require_relative 'test_helper'
 require 'fileutils'
 require 'tempfile'

data/test/cli_test.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 require_relative 'test_helper'
 require_relative 'cli/command_registry_test'
 require_relative 'cli/option_builder_test'

data/test/cli_version_test.rb CHANGED Viewed

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

data/test/compile_book_test.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 require_relative 'test_helper'
 require_relative '../lib/epub_tools/compile_book'
@@ -11,7 +13,7 @@ class CompileBookTest < Minitest::Test
   end
   def teardown
-    FileUtils.remove_entry(@tmp)
+    FileUtils.rm_rf(@tmp)
   end
   def test_default_output_file
@@ -67,39 +69,6 @@ class CompileBookTest < Minitest::Test
     refute cb.verbose
   end
-  def test_clean_build_dir_removes_directory
-    build = File.join(@tmp, 'build')
-    FileUtils.mkdir_p(build)
-    File.write(File.join(build, 'foo'), 'bar')
-    cb = EpubTools::CompileBook.new(
-      title: @title,
-      author: @author,
-      source_dir: @source,
-      build_dir: build
-    )
-    assert Dir.exist?(build)
-    cb.send(:clean_build_dir)
-    refute Dir.exist?(build)
-  end
-  def test_prepare_dirs_creates_xhtml_and_chapters_directories
-    build = File.join(@tmp, 'build')
-    cb = EpubTools::CompileBook.new(
-      title: @title,
-      author: @author,
-      source_dir: @source,
-      build_dir: build
-    )
-    cb.send(:prepare_dirs)
-    xhtml_dir = cb.send(:xhtml_dir)
-    chapters_dir = cb.send(:chapters_dir)
-    assert Dir.exist?(xhtml_dir)
-    assert Dir.exist?(chapters_dir)
-  end
   def test_log_outputs_when_verbose
     cb = EpubTools::CompileBook.new(
       title: @title,
@@ -122,81 +91,27 @@ class CompileBookTest < Minitest::Test
     assert_silent { cb.send(:log, 'hello') }
   end
-  def test_validate_sequence_raises_when_no_chapters
-    cb = EpubTools::CompileBook.new(
-      title: @title,
-      author: @author,
-      source_dir: @source,
-      build_dir: @tmp
-    )
-    FileUtils.mkdir_p(cb.send(:chapters_dir))
-    err = assert_raises(RuntimeError) { cb.send(:validate_sequence) }
-    assert_match(/No chapter files found/, err.message)
-  end
-  def test_validate_sequence_raises_on_missing_chapters
-    build = File.join(@tmp, 'build')
+  def test_run_completes_workflow
+    # Test that run method executes the complete workflow
     cb = EpubTools::CompileBook.new(
       title: @title,
       author: @author,
       source_dir: @source,
-      build_dir: build
+      build_dir: @tmp,
+      output_file: 'test.epub'
     )
-    chapters = cb.send(:chapters_dir)
-    FileUtils.mkdir_p(chapters)
-    File.write(File.join(chapters, 'chap_1.xhtml'), '')
-    File.write(File.join(chapters, 'chap_3.xhtml'), '')
-    err = assert_raises(RuntimeError) { cb.send(:validate_sequence) }
-    assert_match(/Missing chapter numbers: 2/, err.message)
-  end
-  def test_validate_sequence_succeeds_on_complete_sequence
-    build = File.join(@tmp, 'build')
-    cb = EpubTools::CompileBook.new(
-      title: @title,
-      author: @author,
-      source_dir: @source,
-      build_dir: build
-    )
-    chapters = cb.send(:chapters_dir)
-    FileUtils.mkdir_p(chapters)
-    File.write(File.join(chapters, 'chap_1.xhtml'), '')
-    File.write(File.join(chapters, 'chap_2.xhtml'), '')
-    File.write(File.join(chapters, 'chap_3.xhtml'), '')
+    # Mock the workflow methods to avoid complex file setup
+    def cb.extract_xhtmls; end
+    def cb.split_xhtmls; end
+    def cb.validate_chapters; end
+    def cb.initialize_epub; end
+    def cb.add_chapters; end
+    def cb.pack_epub; end
-    assert_nil cb.send(:validate_sequence)
-  end
+    # Should complete without error
+    result = cb.run
-  def test_run_calls_all_steps_in_order
-    cb = EpubTools::CompileBook.new(
-      title: @title,
-      author: @author,
-      source_dir: @source,
-      build_dir: @tmp,
-      output_file: 'o.epub'
-    )
-    seq = []
-    cb.define_singleton_method(:clean_build_dir)   { seq << :clean }
-    cb.define_singleton_method(:prepare_dirs)      { seq << :prepare }
-    cb.define_singleton_method(:extract_xhtmls)    { seq << :extract }
-    cb.define_singleton_method(:split_xhtmls)      { seq << :split }
-    cb.define_singleton_method(:validate_sequence)  { seq << :validate }
-    cb.define_singleton_method(:initialize_epub)    { seq << :init }
-    cb.define_singleton_method(:add_chapters)      { seq << :add }
-    cb.define_singleton_method(:pack_epub)         { seq << :pack }
-    cb.define_singleton_method(:log)               { |msg| seq << [:log, msg] }
-    cb.run
-    expected = [
-      :clean, :prepare, :extract, :split,
-      :validate, :init, :add, :pack,
-      [:log, /Done\. Output EPUB: .*o\.epub/],
-      :clean
-    ]
-    assert_equal expected[0..7], seq[0..7]
-    assert_kind_of Array, seq[8]
-    assert_equal :log, seq[8][0]
-    assert_match expected[8][1], seq[8][1]
-    assert_equal expected[9], seq[9]
+    assert_equal 'test.epub', result
   end
 end

data/test/compile_workspace_test.rb ADDED Viewed

@@ -0,0 +1,55 @@
+# frozen_string_literal: true
+require_relative 'test_helper'
+require_relative '../lib/epub_tools/compile_workspace'
+class CompileWorkspaceTest < Minitest::Test
+  def setup
+    @tmp = Dir.mktmpdir
+    @workspace = EpubTools::CompileWorkspace.new(@tmp)
+  end
+  def teardown
+    FileUtils.rm_rf(@tmp)
+  end
+  def test_initialize_sets_build_dir
+    assert_equal @tmp, @workspace.build_dir
+  end
+  def test_prepare_directories_creates_required_dirs
+    @workspace.prepare_directories
+    assert Dir.exist?(@workspace.xhtml_dir)
+    assert Dir.exist?(@workspace.chapters_dir)
+  end
+  def test_directory_paths_are_correct
+    assert_equal File.join(@tmp, 'xhtml'), @workspace.xhtml_dir
+    assert_equal File.join(@tmp, 'chapters'), @workspace.chapters_dir
+    assert_equal File.join(@tmp, 'epub'), @workspace.epub_dir
+  end
+  def test_directory_paths_are_memoized
+    # First call creates the path
+    path1 = @workspace.xhtml_dir
+    # Second call should return the same object
+    path2 = @workspace.xhtml_dir
+    assert_equal path1, path2
+  end
+  def test_clean_removes_build_directory
+    # Create the directory and some content
+    FileUtils.mkdir_p(@tmp)
+    test_file = File.join(@tmp, 'test.txt')
+    File.write(test_file, 'test content')
+    assert Dir.exist?(@tmp)
+    assert_path_exists test_file
+    @workspace.clean
+    refute Dir.exist?(@tmp)
+  end
+end