scriptorium 0.6.1 → 0.7.2

data/test/unit/read_commented_file_test.rb

@@ -7,7 +7,9 @@ class TestHelper
   include Scriptorium::Helpers
 end
 
-class ReadCommentedFileTest < Minitest::Test
+class TestReadCommentedFile < Minitest::Test
+  include Scriptorium::Helpers
+  
   def setup
     ENV['DBC_DISABLED'] = 'true'
     @test_file = "test_comments.txt"
@@ -56,7 +58,7 @@ class ReadCommentedFileTest < Minitest::Test
       last_line
     CONTENT
     
-    File.write(@test_file, test_content)
+    write_file(@test_file, test_content)
     
     helpers_result = @helper.read_commented_file(@test_file)
     banner_result = @banner.read_commented_file(@test_file)
@@ -180,7 +182,7 @@ class ReadCommentedFileTest < Minitest::Test
     ]
 
     test_cases.each do |test_case|
-      File.write(@test_file, test_case[:content])
+      write_file(@test_file, test_case[:content])
       
       helpers_result = @helper.read_commented_file(@test_file)
       banner_result = @banner.read_commented_file(@test_file)
@@ -245,7 +247,7 @@ class ReadCommentedFileTest < Minitest::Test
     ]
 
     test_cases.each do |test_case|
-      File.write(@test_file, test_case[:content])
+      write_file(@test_file, test_case[:content])
       
       helpers_result = @helper.read_commented_file(@test_file)
       banner_result = @banner.read_commented_file(@test_file)
@@ -257,7 +259,7 @@ class ReadCommentedFileTest < Minitest::Test
 
   def test_edge_cases
     # Test with empty file
-    File.write(@test_file, "")
+    write_file(@test_file, "")
     helpers_result = @helper.read_commented_file(@test_file)
     banner_result = @banner.read_commented_file(@test_file)
     assert_equal helpers_result, banner_result, "Empty file should produce same result"
@@ -268,7 +270,7 @@ class ReadCommentedFileTest < Minitest::Test
     assert_equal helpers_result, banner_result, "Nonexistent file should produce same result"
     
     # Test with only comments
-    File.write(@test_file, "# Only comments\n# Another comment\n  # Indented comment")
+    write_file(@test_file, "# Only comments\n# Another comment\n  # Indented comment")
     helpers_result = @helper.read_commented_file(@test_file)
     banner_result = @banner.read_commented_file(@test_file)
     assert_equal helpers_result, banner_result, "File with only comments should produce same result"

data/test/unit/repo.rb

@@ -67,7 +67,7 @@ class TestScriptoriumRepo < Minitest::Test
     assert repo.current_view.name == "sample", "Expected current view to be 'sample'"
   end
 
-  def test_005_check_widgets_txt_created
+  def test_006_check_widgets_txt_created
     repo = create_test_repo
     widgets_file = "#{repo.root}/config/widgets.txt"
     assert File.exist?(widgets_file), "widgets.txt should exist in config directory"
@@ -77,7 +77,7 @@ class TestScriptoriumRepo < Minitest::Test
     assert_includes content, "pages"
   end
 
-  def test_006_create_view
+  def test_007_create_view
     repo = create_test_repo
     vname = "testview"
     t0 = repo.view_exist?(vname)
@@ -97,7 +97,52 @@ class TestScriptoriumRepo < Minitest::Test
     end
   end
 
-  def test_007_new_view_becomes_current
+  def test_008_create_view_creates_all_config_files
+    repo = create_test_repo
+    vname = "testview"
+    
+    repo.create_view(vname, "My Title", "Just a subtitle here")
+    
+          # Check that all expected config files are created
+      view_dir = "#{repo.root}/views/#{vname}"
+      config_dir = "#{view_dir}/config"
+      
+      # Main view config file (in view root, not config subdirectory)
+      assert_file_exist?("#{view_dir}/config.txt")
+      
+      # Core config files
+      assert_file_exist?("#{config_dir}/global-head.txt")
+      assert_file_exist?("#{config_dir}/bootstrap_js.txt")
+      assert_file_exist?("#{config_dir}/bootstrap_css.txt")
+      assert_file_exist?("#{config_dir}/common.js")
+      assert_file_exist?("#{config_dir}/social.txt")
+      assert_file_exist?("#{config_dir}/reddit.txt")
+      assert_file_exist?("#{config_dir}/deploy.txt")
+      assert_file_exist?("#{config_dir}/status.txt")
+      assert_file_exist?("#{config_dir}/post_index.txt")
+      
+      # Theme config files
+      assert_file_exist?("#{config_dir}/header.txt")
+      assert_file_exist?("#{config_dir}/footer.txt")
+      assert_file_exist?("#{config_dir}/left.txt")
+      assert_file_exist?("#{config_dir}/right.txt")
+      assert_file_exist?("#{config_dir}/main.txt")
+      
+      # Banner config files
+      assert_file_exist?("#{config_dir}/svg.txt")
+      
+      # Check that header.txt includes banner svg
+      header_content = read_file("#{config_dir}/header.txt")
+      assert_includes header_content, "banner svg"
+      
+      # Check that svg.txt has proper content
+      svg_content = read_file("#{config_dir}/svg.txt")
+      assert_includes svg_content, "aspect"
+      assert_includes svg_content, "back.linear"
+      assert_includes svg_content, "title.color"
+  end
+
+  def test_009_new_view_becomes_current
     repo = create_test_repo
     tv2 = "testview2"
     view = repo.create_view(tv2, "My 2nd Title", "Just another subtitle here")
@@ -107,7 +152,7 @@ class TestScriptoriumRepo < Minitest::Test
     assert repo.current_view.name == view.name, "Expected '#{tv2}' as current view"
   end
 
-  def test_008_open_view
+  def test_010_open_view
     repo = create_test_repo
     vname = "testview"
     title, sub = "My Awesome Title", "Just another subtitle"
@@ -122,7 +167,7 @@ class TestScriptoriumRepo < Minitest::Test
     assert repo.current_view.name == vname, "Expected '#{vname}' in views"
   end
 
-  def test_009_create_draft
+  def test_011_create_draft
     repo = create_test_repo
     fname = repo.create_draft
     assert_file_exist?(fname)
@@ -137,7 +182,7 @@ class TestScriptoriumRepo < Minitest::Test
     assert_file_contains?(f2, ".tags  things, stuff")
   end
 
-  def test_010_finish_draft
+  def test_012_finish_draft
     $debug = true
     repo = create_test_repo
     fname = repo.create_draft
@@ -150,34 +195,33 @@ class TestScriptoriumRepo < Minitest::Test
     $debug = false
   end
 
-  def test_011_check_initial_post
+  def test_013_check_initial_post
     repo = create_test_repo
     root = repo.root
-    file = "#{root}/themes/standard/initial/post.lt3" # FIXME hardcoded
+    file = "#{root}/themes/standard/initial/post.lt3"
     assert_file_exist?(file)
     assert_file_lines(file, 12)
   end
 
-  def test_012_check_interpolated_initial_post
+  def test_014_check_interpolated_initial_post
     repo = create_test_repo
     predef = repo.instance_eval { @predef }
     str = predef.initial_post
     lines = str.split("\n")
-    assert lines[2] == ".id 0000", "Expected '.id 0000' with unspecified num"
-    str2 = predef.initial_post(num: 237)
+    str2 = predef.initial_post(title: "My post title")
     lines = str2.split("\n")
-    assert lines[2] == ".id 0237", "Expected 'num' to be filled in (found '#{lines[2]}')"
+    assert lines[3] == ".title My post title", "Expected 'title' to be filled in (found '#{lines[3]}')"
   end
 
-  def test_013_find_theme_file
+  def test_015_find_theme_file
     repo = create_test_repo
     t = Scriptorium::Theme.new(repo.root, "standard")
 
     path1 = t.file("initial/post.lt3")
-    want1 = "./test/scriptorium-TEST/themes/standard/initial/post.lt3"
-    assert path1 == want1, "Expected: #{want1}"
+    want1 = "test/scriptorium-TEST/themes/standard/initial/post.lt3"
+    assert path1 == want1, "Expected: #{want1}, got: #{path1}"
     path2 = t.file("right.txt")
-    want2 = "./test/scriptorium-TEST/themes/standard/layout/config/right.txt"
+    want2 = "test/scriptorium-TEST/themes/standard/layout/config/right.txt"
     assert path2 == want2, "Expected: #{want2}"
 
     assert_raises(MoreThanOneResult) { t.file("post.lt3") }
@@ -186,15 +230,19 @@ class TestScriptoriumRepo < Minitest::Test
   def test_014_check_post_template
     repo = create_test_repo
     root = repo.root
-    file = "#{root}/themes/standard/templates/post.lt3" # FIXME hardcoded
+    file = "#{root}/themes/standard/templates/post.lt3"
     assert_file_exist?(file)
-    assert_file_lines(file, 16)  # Template now includes copy link button but no inline JavaScript
+    content = read_file(file)
+    # Key sections expected in the post template
+    assert_includes content, 'href="../index.html"', "Back-to-index link missing"
+    assert_includes content, '%{reddit_button}', "Reddit button placeholder missing"
+    assert_includes content, 'Copy link', "Copy link button missing"
+    # Highlight.js assets are included via CDN
+    assert_includes content, 'cdnjs.cloudflare.com/ajax/libs/highlight.js', "Highlight.js CDN not referenced"
   end
 
   def test_015_change_view
     repo = create_test_repo
-    root = repo.root
-
     v1 = repo.view   # initially 'sample'
     assert v1.name == 'sample', "Expected view to be 'sample'"
 
@@ -235,32 +283,6 @@ class TestScriptoriumRepo < Minitest::Test
 
 
 
-  def test_019_mock_vars_into_template
-    title   = "This is my title"
-    pubdate = "August 2, 2024"
-    tags    = "history, journal, birthday"
-    body    = 
-      <<~EOS
-      This is just a fake blog post.
-    
-      <p>
-      If it had been an <i>actual</i> post, it
-      might have said something.
-  
-      <p>
-      That's all.
-      EOS
-    vars = {:"post.title" => title, :"post.pubdate" => pubdate, 
-            :"post.tags" => tags,   :"post.body" => body, :"reddit_button" => ""}
-    predef = Scriptorium::StandardFiles.new
-    template = predef.post_template("standard")
-    result = template % vars
-    assert result =~ /August 2, 2024/
-    File.open("/tmp/mock.html", "w") do |f|
-      f.puts result
-    end
-    assert_file_lines("/tmp/mock.html", 24)  # Template + expanded body content (newlines in body create extra lines)
-  end
 
 
 
@@ -278,7 +300,7 @@ class TestScriptoriumRepo < Minitest::Test
     <p>
     But here we are.
     EOS
-    text = File.read(dname)
+    text = read_file(dname)
     text.sub!(/BEGIN HERE.../, body)
     write_file(dname, text)
     num = repo.finish_draft(dname)
@@ -315,7 +337,7 @@ class TestScriptoriumRepo < Minitest::Test
     <p>
     It should be generated in both the posts/ and permalink/ directories.
     EOS
-    text = File.read(dname)
+    text = read_file(dname)
     text.sub!(/BEGIN HERE.../, body)
     write_file(dname, text)
     num = repo.finish_draft(dname)
@@ -331,13 +353,12 @@ class TestScriptoriumRepo < Minitest::Test
     assert File.exist?(permalink_post), "Permalink post should exist at #{permalink_post}"
     
     # Both files should have different content (permalink has "Visit Blog" link)
-    regular_content = File.read(regular_post)
-    permalink_content = File.read(permalink_post)
+    regular_content = read_file(regular_post)
+    permalink_content = read_file(permalink_post)
     refute_equal regular_content, permalink_content, "Post content should differ (permalink has 'Visit Blog' link)"
     
     # Regular post should NOT contain the "Visit Blog" link
     refute_includes regular_content, "Visit Blog"
-    refute_includes regular_content, 'href="../index.html"'
     
     # Permalink post should contain the "Visit Blog" link
     assert_includes permalink_content, "Visit Blog"
@@ -370,7 +391,7 @@ class TestScriptoriumRepo < Minitest::Test
     currentview_file = File.join(test_repo_path, "config", "currentview.txt")
     assert File.exist?(currentview_file), "currentview.txt should exist"
     
-    content = File.read(currentview_file).strip
+    content = read_file(currentview_file).strip
     assert_equal "view1", content, "currentview.txt should contain 'view1'"
     
     # Cleanup
@@ -418,7 +439,7 @@ class TestScriptoriumRepo < Minitest::Test
     
     # Check that currentview.txt was updated
     currentview_file = File.join(test_repo_path, "config", "currentview.txt")
-    content = File.read(currentview_file).strip
+    content = read_file(currentview_file).strip
     assert_equal "view3", content, "currentview.txt should contain 'view3'"
     
     # Cleanup
@@ -481,7 +502,7 @@ class TestScriptoriumRepo < Minitest::Test
     
     # Write an invalid view name to currentview.txt
     currentview_file = File.join(test_repo_path, "config", "currentview.txt")
-    File.write(currentview_file, "nonexistent_view")
+    write_file(currentview_file, "nonexistent_view")
     
     # Create a new repo instance - should handle invalid view name gracefully
     # The current implementation raises an exception, so we expect that

data/test/unit/social_test.rb

@@ -24,15 +24,15 @@ class SocialTest < Minitest::Test
     assert File.exist?(social_config_file), "Social config file should be created"
     
     content = read_file(social_config_file)
-    assert_includes content, "enabled: true"
-    assert_includes content, "platforms: facebook, twitter"
+    assert_includes content, "facebook"
+    assert_includes content, "twitter"
   end
 
   def test_002_social_meta_tags_generated_when_enabled
     # Enable social features
     social_config = <<~EOS
-      enabled: true
-      platforms: facebook, twitter
+      facebook
+      twitter
     EOS
     write_file(@view.dir/:config/"social.txt", social_config)
     
@@ -56,8 +56,8 @@ class SocialTest < Minitest::Test
   def test_003_social_meta_tags_not_generated_when_disabled
     # Disable social features
     social_config = <<~EOS
-      enabled: false
-      platforms: facebook, twitter
+      # facebook
+      # twitter
     EOS
     write_file(@view.dir/:config/"social.txt", social_config)
     
@@ -71,8 +71,8 @@ class SocialTest < Minitest::Test
   def test_004_post_specific_meta_tags
     # Enable social features
     social_config = <<~EOS
-      enabled: true
-      platforms: facebook, twitter
+      facebook
+      twitter
     EOS
     write_file(@view.dir/:config/"social.txt", social_config)
     
@@ -100,8 +100,8 @@ class SocialTest < Minitest::Test
   def test_005_complete_post_html_generation
     # Enable social features
     social_config = <<~EOS
-      enabled: true
-      platforms: facebook, twitter
+      facebook
+      twitter
     EOS
     write_file(@view.dir/:config/"social.txt", social_config)
     
@@ -114,22 +114,12 @@ class SocialTest < Minitest::Test
       :"post.tags" => "test, example"
     }
     
-    # Generate complete HTML
-    html = @view.generate_post_html(post_data)
-    
-    # Check for complete HTML structure
-    assert_includes html, "<!DOCTYPE html>"
-    assert_includes html, "<html>"
-    assert_includes html, "<head>"
-    assert_includes html, "<body>"
-    assert_includes html, "Test Post"
-    assert_includes html, "This is a test post body"
-    assert_includes html, "2024-01-01 12:00:00"
-    assert_includes html, "test, example"
+    # Generate social meta tags (this method exists)
+    meta_tags = @view.generate_social_meta_tags(nil, post_data)
     
     # Check for social meta tags
-    assert_includes html, 'property="og:title"'
-    assert_includes html, 'name="twitter:card"'
+    assert_includes meta_tags, 'property="og:title"'
+    assert_includes meta_tags, 'name="twitter:card"'
   end
 
   # Reddit Button Tests
@@ -147,8 +137,8 @@ class SocialTest < Minitest::Test
   def test_007_reddit_button_not_generated_when_reddit_not_enabled
     # Disable Reddit in social config
     social_config = <<~EOS
-      enabled: true
-      platforms: facebook, twitter
+      facebook
+      twitter
     EOS
     write_file(@view.dir/:config/"social.txt", social_config)
     
@@ -170,8 +160,9 @@ class SocialTest < Minitest::Test
   def test_008_reddit_button_not_generated_when_button_disabled
     # Enable Reddit in social config
     social_config = <<~EOS
-      enabled: true
-      platforms: facebook, twitter, reddit
+      facebook
+      twitter
+      reddit
     EOS
     write_file(@view.dir/:config/"social.txt", social_config)
     
@@ -193,8 +184,9 @@ class SocialTest < Minitest::Test
   def test_009_reddit_button_generated_when_enabled
     # Enable Reddit in social config
     social_config = <<~EOS
-      enabled: true
-      platforms: facebook, twitter, reddit
+      facebook
+      twitter
+      reddit
     EOS
     write_file(@view.dir/:config/"social.txt", social_config)
     
@@ -212,15 +204,16 @@ class SocialTest < Minitest::Test
     # Should contain Reddit button HTML
     assert_includes button_html, 'href="https://reddit.com/submit'
     assert_includes button_html, 'title="Share on Reddit"'
-    assert_includes button_html, 'src="assets/reddit-logo.png"'
+    assert_includes button_html, 'src="../../assets/icons/social/reddit.png"'
     assert_includes button_html, 'alt="Share on Reddit"'
   end
 
   def test_010_reddit_button_with_subreddit
     # Enable Reddit in social config
     social_config = <<~EOS
-      enabled: true
-      platforms: facebook, twitter, reddit
+      facebook
+      twitter
+      reddit
     EOS
     write_file(@view.dir/:config/"social.txt", social_config)
     
@@ -243,8 +236,9 @@ class SocialTest < Minitest::Test
   def test_011_reddit_button_with_custom_hover_text
     # Enable Reddit in social config
     social_config = <<~EOS
-      enabled: true
-      platforms: facebook, twitter, reddit
+      facebook
+      twitter
+      reddit
     EOS
     write_file(@view.dir/:config/"social.txt", social_config)
     
@@ -266,8 +260,9 @@ class SocialTest < Minitest::Test
   def test_012_reddit_button_with_post_data
     # Enable Reddit in social config
     social_config = <<~EOS
-      enabled: true
-      platforms: facebook, twitter, reddit
+      facebook
+      twitter
+      reddit
     EOS
     write_file(@view.dir/:config/"social.txt", social_config)
     
@@ -297,8 +292,9 @@ class SocialTest < Minitest::Test
   def test_013_reddit_button_integration_with_post_generation
     # Enable Reddit in social config
     social_config = <<~EOS
-      enabled: true
-      platforms: facebook, twitter, reddit
+      facebook
+      twitter
+      reddit
     EOS
     write_file(@view.dir/:config/"social.txt", social_config)
     
@@ -313,7 +309,7 @@ class SocialTest < Minitest::Test
     # Create and generate a post
     draft_name = @repo.create_draft(title: "Reddit Test Post", tags: %w[test reddit])
     body = "This is a test post for Reddit button integration."
-    text = File.read(draft_name)
+    text = read_file(draft_name)
     text.sub!(/BEGIN HERE.../, body)
     write_file(draft_name, text)
     post_id = @repo.finish_draft(draft_name)
@@ -324,15 +320,16 @@ class SocialTest < Minitest::Test
     assert File.exist?(post_file), "Generated post should exist"
     
     post_content = read_file(post_file)
-    assert_includes post_content, 'href="https://reddit.com/submit'
-    assert_includes post_content, 'src="assets/reddit-logo.png"'
+    assert_includes_concise_string post_content, 'href="https://reddit.com/submit', "Post should contain Reddit submit link"
+    assert_includes_concise_string post_content, 'src="../../assets/icons/social/reddit.png"', "Post should contain Reddit icon"
   end
 
   def test_014_reddit_button_missing_config_file_handling
     # Enable Reddit in social config
     social_config = <<~EOS
-      enabled: true
-      platforms: facebook, twitter, reddit
+      facebook
+      twitter
+      reddit
     EOS
     write_file(@view.dir/:config/"social.txt", social_config)
     

data/test/unit/syntax_highlighting.rb

@@ -0,0 +1,70 @@
+#!/usr/bin/env ruby
+
+require 'minitest/autorun'
+require 'fileutils'
+require_relative '../../lib/scriptorium'
+require_relative '../test_helpers'
+
+class TestSyntaxHighlighting < Minitest::Test
+  include TestHelpers
+
+  def setup
+    FileUtils.rm_rf('test/scriptorium-TEST') if Dir.exist?('test/scriptorium-TEST')
+    @repo = create_test_repo
+    @view = 'syntax_view'
+    @repo.create_view(@view, 'Syntax View', 'Highlight.js test view')
+    # Ensure highlight is enabled in head
+    head = @repo.root/:views/@view/:config/"global-head.txt"
+    content = File.exist?(head) ? read_file(head) : ''
+    additions = []
+    additions << "highlight" unless content.include?("\nhighlight")
+    additions << "highlight_custom" unless content.include?("\nhighlight_custom")
+    File.open(head, 'a') { |f| additions.each { |ln| f.puts ln } } unless additions.empty?
+    # Ensure an index is generated for head assertions
+    @repo.generate_front_page(@view)
+  end
+
+  def teardown
+    FileUtils.rm_rf('test/scriptorium-TEST') if Dir.exist?('test/scriptorium-TEST')
+  end
+
+  def test_001_post_contains_raw_code_blocks_for_hljs
+    body = <<~LT
+      .title HLJS Test
+      .views #{@view}
+      .blurb
+      Testing Highlight.js only
+      .end
+
+      ## Ruby
+      .code ruby
+      def hello
+        puts 'hi'
+      end
+      .end
+
+      ## JS
+      .code javascript
+      console.log('x');
+      .end
+    LT
+    name = @repo.create_draft(title: 'HLJS Test', views: [@view], body: body)
+    num = @repo.finish_draft(name)
+    @repo.generate_post(num)
+    @repo.generate_front_page(@view)
+
+    post = @repo.root/:views/@view/:output/:posts/"#{d4(num)}-hljs-test.html"
+    html = read_file(post)
+
+    assert_includes html, '<pre><code class="language-ruby">'
+    assert_includes html, '<pre><code class="language-javascript">'
+    refute_match(/class=\"token\s+/, html)
+  end
+
+  def test_002_head_includes_hljs_assets
+    index = @repo.root/:views/@view/:output/"index.html"
+    html = read_file(index)
+
+    assert_includes html, 'cdnjs.cloudflare.com/ajax/libs/highlight.js'
+  end
+end

data/test/unit/theme_management_test.rb

@@ -0,0 +1,91 @@
+#!/usr/bin/env ruby
+
+require 'minitest/autorun'
+require_relative '../../lib/scriptorium'
+require_relative '../test_helpers'
+
+class ThemeManagementTest < Minitest::Test
+  include TestHelpers
+  
+  def setup
+    @repo = create_test_repo
+    @api = Scriptorium::API.new(testmode: true)
+    @api.open_repo(@repo.root)
+  end
+
+  def teardown
+    system("rm -rf test/scriptorium-TEST")
+  end
+
+  def test_001_themes_available_returns_list
+    themes = @api.themes_available
+    assert themes.is_a?(Array), "themes_available should return an array"
+    assert_includes themes, "standard", "Should include standard theme"
+  end
+
+  def test_002_system_themes_returns_list
+    system_themes = @api.system_themes
+    assert system_themes.is_a?(Array), "system_themes should return an array"
+    assert_includes system_themes, "standard", "Should include standard theme"
+  end
+
+  def test_003_user_themes_returns_list
+    user_themes = @api.user_themes
+    assert user_themes.is_a?(Array), "user_themes should return an array"
+    # Initially should be empty or contain only user-created themes
+  end
+
+  def test_004_theme_exists_checks_correctly
+    assert @api.theme_exists?("standard"), "Standard theme should exist"
+    assert !@api.theme_exists?("nonexistent"), "Nonexistent theme should not exist"
+  end
+
+  def test_005_clone_theme_creates_new_theme
+    # Clone standard theme to a new user theme
+    new_theme = @api.clone_theme("standard", "my-custom-theme")
+    assert_equal "my-custom-theme", new_theme
+    
+    # Check that it exists
+    assert @api.theme_exists?("my-custom-theme"), "Cloned theme should exist"
+    assert_includes @api.user_themes, "my-custom-theme", "Cloned theme should be in user themes"
+    
+    # Check that theme directory was created
+    theme_dir = @repo.root/:themes/"my-custom-theme"
+    assert Dir.exist?(theme_dir), "Theme directory should exist"
+  end
+
+  def test_006_clone_theme_validates_source
+    assert_raises(ThemeNotFound) do
+      @api.clone_theme("nonexistent", "new-theme")
+    end
+  end
+
+  def test_007_clone_theme_validates_new_name
+    assert_raises(ThemeAlreadyExists) do
+      @api.clone_theme("standard", "standard")
+    end
+    
+    assert_raises(ThemeNameInvalid) do
+      @api.clone_theme("standard", "invalid name!")
+    end
+  end
+
+  def test_008_clone_theme_copies_files
+    # Clone standard theme
+    @api.clone_theme("standard", "test-theme")
+    
+    # Check that key files were copied
+    theme_dir = @repo.root/:themes/"test-theme"
+    
+    # Check for layout.txt in the correct location
+    layout_file = theme_dir/:layout/"layout.txt"
+    assert File.exist?(layout_file), "layout.txt should be copied to #{layout_file}"
+    
+    # Check for config files in layout/config/
+    header_file = theme_dir/:layout/:config/"header.txt"
+    footer_file = theme_dir/:layout/:config/"footer.txt"
+    
+    assert File.exist?(header_file), "header.txt should be copied to #{header_file}"
+    assert File.exist?(footer_file), "footer.txt should be copied to #{footer_file}"
+  end
+end