cleanup tests

Author: pftg

test/support/hugo_helpers.rb

@@ -10,6 +10,7 @@ def initialize(path: nil, port: nil)
   end
 
   HUGO_OPTIONS = %w[
+    hugo
     --environment test
     --buildDrafts
     --logLevel warn
@@ -25,14 +26,12 @@ def precompile(port:)
 
     port ||= @port
 
-    base_url_option = port ? "--baseURL=\"http://localhost:#{port}\"" : ""
-
-    options = HUGO_OPTIONS.join(" ")
-    hugo_build_cmd = "hugo #{options} #{base_url_option} --destination=\"#{destination}\"".strip
-
-    warn "Hugo: #{hugo_build_cmd}" if ENV["DEBUG"]
-
-    system(hugo_build_cmd, exception: true)
+    # Build hugo command using argv array to avoid shell quoting issues
+    args = HUGO_OPTIONS.dup
+    args += %W[--baseURL http://localhost:#{port}] if port
+    args += %W[--destination #{destination}]
+    warn "Hugo: #{args.join(' ')}" if ENV["DEBUG"]
+    system(*args, exception: true)
     self
   end
 

test/unit/asset_url_validation_test.rb

@@ -1,6 +1,6 @@
 require_relative "base_page_test"
 
-class AssetUrlValidationTest < BasePageTest
+class AssetUrlValidationTest < BasePageTestCase
   DEFAULT_TEST_HTML_FILE = '404.html'
 
 

test/unit/base_page_test.rb test/unit/base_page_test_case.rbtest/unit/base_page_test.rb renamed to test/unit/base_page_test_case.rb

@@ -6,7 +6,7 @@
 # Build Hugo without baseURL for unit tests (they don't need specific ports)
 Hugo.new.precompile(port: nil)
 
-class BasePageTest < Minitest::Test
+class BasePageTestCase < Minitest::Test
   private
 
   def root_path

test/unit/diagram_rendering_test.rb

@@ -1,6 +1,6 @@
 require_relative "base_page_test"
 
-class DiagramRenderingTest < BasePageTest
+class DiagramRenderingTest < BasePageTestCase
   # Unit tests for diagram rendering functionality (Mermaid support)
   # Tests rendered HTML output from public-test directory (no server required)
 

test/unit/hugo_asset_validation_test.rb

@@ -1,6 +1,6 @@
 require_relative "base_page_test"
 
-class HugoAssetValidationTest < BasePageTest
+class HugoAssetValidationTest < BasePageTestCase
   EXPECTED_HOST = "localhost"
 
   # Asset pattern constants
@@ -104,4 +104,4 @@ def test_no_hardcoded_production_urls
     assert_empty hardcoded_assets.map { |a| a["href"] || a["src"] },
       "Should not have hardcoded production asset URLs"
   end
-end
+end

test/unit/hugo_partials_test.rb

@@ -1,6 +1,6 @@
 require_relative "base_page_test"
 
-class HugoPartialsTest < BasePageTest
+class HugoPartialsTest < BasePageTestCase
   # Unit tests for Hugo partial templates functionality and rendering
   # Tests rendered HTML output from public-test directory (no server required)
 

test/unit/meta_tags/article_schema_test.rb

@@ -1,6 +1,6 @@
 require_relative "../base_page_test"
 
-class ArticleSchemaTest < BasePageTest
+class ArticleSchemaTest < BasePageTestCase
   def test_blog_post_has_article_schema
     doc = parse_html_file("blog/devtools-for-ai-agents/index.html")
     schemas = find_schemas_by_type(doc, "Article")

test/unit/meta_tags/breadcrumb_schema_test.rb

@@ -1,6 +1,6 @@
 require_relative "../base_page_test"
 
-class BreadcrumbSchemaTest < BasePageTest
+class BreadcrumbSchemaTest < BasePageTestCase
   def test_blog_post_has_breadcrumb_list_schema
     doc = parse_html_file("blog/4-lines-speed-up-your-rails-test-suite-on-circleci/index.html")
     schemas = find_schemas_by_type(doc, "BreadcrumbList")

test/unit/meta_tags/meta_tags_test.rb

@@ -1,6 +1,6 @@
 require_relative "../base_page_test"
 
-class MetaTagsTest < BasePageTest
+class MetaTagsTest < BasePageTestCase
   # Unit tests for meta tags functionality and SEO validation
   # Tests rendered HTML output from public-test directory (no server required)
 

test/unit/meta_tags/organization_schema_test.rb

@@ -1,6 +1,6 @@
 require_relative "../base_page_test"
 
-class OrganizationSchemaTest < BasePageTest
+class OrganizationSchemaTest < BasePageTestCase
   def test_organization_schema_exists
     doc = parse_html_file("about-us/index.html")
     schemas = find_schemas_by_type(doc, "Organization")