railpack 1.2.6 → 1.2.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 30e7e6b4dbbc67e8442766655424ad14dacc36f1e5a3cc6e58211ad6ba0f1442
-  data.tar.gz: af9393a87f9030d6da531c365d69cfb9c2e8bca4798b8889e8c4e6cb2c461f99
+  metadata.gz: 26a629014375c6614d4b3c2e4f9673ff7df37152cb504cc236913d817f147123
+  data.tar.gz: ad53fdfe5e81b6cab82e71b423104979d20c6ed49df64fab037861af6a6a47ea
 SHA512:
-  metadata.gz: 503ae2cfc7ac3372396c300dae41d921c06f776d8cf3b18debaf815719db1d86483907832fe849682bcc50fa39e24496f21db3fe352cc20a3180429e58e250e5
-  data.tar.gz: b818edd2afb5d9b1c6b608227aec6e93729eb2210c0dbc597124e68103e0ce049758081e08590e66cf674540d57a307d7f61dceabad61a2d571f50b89b1ee4f5
+  metadata.gz: 706d8c94a84dc23d1b3bd7bf88a6df3ef7f075770ef168efef448e4ac136ae82255c33007b4b8aee70ffcda1321fc26bdc1b8a79b447e74349745bdab097e839
+  data.tar.gz: e3f6fb48000ad925127355df341824429091c19cb78299bcaa7caf3bcfd4c5a4a277d12a3c7c995dde3f5a8761d64303fef99a0586258e53e01839df96e4e2b3

data/CHANGELOG.md

@@ -1,13 +1,14 @@
 ## [Unreleased]
 
-## [1.2.6] - 2026-01-26
-
-- Add dedicated config_test.rb file for Config class unit tests
-- Comprehensive Config class testing with 12 focused unit tests
-- Test initialization, default values, build flags/args, environment overrides
-- Test YAML file loading, error handling, and dynamic method access
-- Improved test organization with separate test files per class
-- All 24 tests passing with 86 assertions
+## [1.2.7] - 2026-01-26
+
+- Add dedicated test files for Manager and Bundler classes
+- manager_test.rb: 13 unit tests for Manager class functionality
+- bundler_test.rb: 17 unit tests for all bundler implementations
+- Test bundler initialization, commands, inheritance, error handling
+- Test manager bundler creation, bundle size calculation, asset manifest generation
+- Improved test organization with separate test files per major class
+- All 43 tests passing with 137 assertions
 
 ## [1.2.4] - 2026-01-26
 

data/lib/railpack/bundler.rb

@@ -20,7 +20,7 @@ module Railpack
     end
 
     def name
-      self.class.name.split('::').last.sub('_bundler', '').downcase
+      self.class.name.split('::').last.sub('Bundler', '').downcase
     end
 
     def commands

data/lib/railpack/version.rb

@@ -1,3 +1,3 @@
 module Railpack
-  VERSION = "1.2.6"
+  VERSION = "1.2.7"
 end

data/test/config_test.rb

@@ -0,0 +1,178 @@
+# frozen_string_literal: true
+
+require 'minitest/autorun'
+require 'tempfile'
+require 'fileutils'
+require 'pathname'
+require 'railpack'
+
+class ConfigTest < Minitest::Test
+  def setup
+    @temp_dir = Dir.mktmpdir
+    @original_rails_root = nil
+    if defined?(Rails) && Rails.respond_to?(:root)
+      @original_rails_root = Rails.singleton_class.instance_method(:root)
+    end
+  end
+
+  def teardown
+    FileUtils.remove_entry(@temp_dir) if @temp_dir && Dir.exist?(@temp_dir)
+    # Restore original Rails.root method if it was overridden
+    if @original_rails_root && defined?(Rails)
+      Rails.singleton_class.define_method(:root, @original_rails_root)
+    end
+  end
+
+  def test_config_initialization
+    config = Railpack::Config.new
+    assert_instance_of Railpack::Config, config
+    assert_respond_to config, :bundler
+    assert_respond_to config, :entrypoint
+  end
+
+  def test_config_default_values
+    config = Railpack::Config.new
+    assert_equal 'bun', config.bundler
+    assert_equal './app/javascript/application.js', config.entrypoint
+    assert_equal 'app/assets/builds', config.outdir
+  end
+
+  def test_config_build_flags_unit
+    config = Railpack::Config.new
+
+    # Test build_flags with different environments
+    dev_flags = config.build_flags('development')
+    prod_flags = config.build_flags('production')
+
+    # Development should have sourcemap
+    assert_includes dev_flags, '--sourcemap'
+    # Production should have minify
+    assert_includes prod_flags, '--minify'
+  end
+
+  def test_config_build_args_unit
+    config = Railpack::Config.new
+
+    args = config.build_args('development')
+    assert_includes args, './app/javascript/application.js'
+    assert_includes args, '--outdir=app/assets/builds'
+    assert_includes args, '--target=browser'
+    assert_includes args, '--format=esm'
+  end
+
+  def test_config_bundler_method
+    config = Railpack::Config.new
+
+    # Test bundler method with different environments
+    assert_equal 'bun', config.bundler('development')
+    assert_equal 'bun', config.bundler('production')
+  end
+
+  def test_config_for_environment_unit
+    config = Railpack::Config.new
+
+    dev_config = config.for_environment('development')
+    prod_config = config.for_environment('production')
+
+    # Check that environment-specific settings are applied
+    assert_equal true, dev_config['sourcemap']
+    assert_equal true, prod_config['minify']
+    assert_equal false, prod_config['sourcemap']
+  end
+
+  def test_config_yaml_file_loading
+    # Test that Railpack can load config from a YAML file
+    config_dir = File.join(@temp_dir, 'config')
+    FileUtils.mkdir_p(config_dir)
+    config_file = File.join(config_dir, 'railpack.yml')
+
+    File.write(config_file, <<~YAML)
+      default:
+        bundler: rollup
+        target: node
+        format: cjs
+        entrypoint: "./server.js"
+        outdir: "dist"
+        minify: false
+      development:
+        sourcemap: true
+      production:
+        minify: true
+        sourcemap: false
+    YAML
+
+    # Verify the file was created
+    assert File.exist?(config_file), "Config file should exist at #{config_file}"
+
+    # Mock Rails.root to point to our temp directory
+    mock_rails_root(@temp_dir)
+
+    # Create a fresh config instance to test file loading
+    config = Railpack::Config.new
+
+    # Test that the YAML file was loaded correctly
+    assert_equal 'rollup', config.bundler
+    assert_equal './server.js', config.entrypoint
+    assert_equal 'dist', config.outdir
+    assert_equal 'node', config.target
+    assert_equal 'cjs', config.format
+    assert_equal false, config.minify
+
+    # Test environment overrides
+    dev_config = config.for_environment('development')
+    prod_config = config.for_environment('production')
+
+    assert_equal true, dev_config['sourcemap']
+    assert_equal true, prod_config['minify']
+    assert_equal false, prod_config['sourcemap']
+  end
+
+  def test_config_environment_overrides
+    # Test environment override logic with default config
+    config = Railpack::Config.new
+    dev_config = config.for_environment('development')
+    prod_config = config.for_environment('production')
+
+    # Development should have sourcemap enabled by default
+    assert_equal true, dev_config['sourcemap']
+    # Production should have minify enabled by default
+    assert_equal true, prod_config['minify']
+  end
+
+  def test_config_error_handling
+    # Test config error handling for invalid YAML
+    config_dir = File.join(@temp_dir, 'config')
+    FileUtils.mkdir_p(config_dir)
+    config_file = File.join(config_dir, 'railpack.yml')
+    File.write(config_file, "invalid: yaml: content: [")
+
+    mock_rails_root(@temp_dir)
+
+    # This should raise an error for invalid YAML
+    assert_raises(Railpack::Config::Error) do
+      Railpack::Config.new
+    end
+  end
+
+  def test_config_method_missing
+    config = Railpack::Config.new
+
+    # Test that config responds to dynamic methods
+    assert_respond_to config, :target
+    assert_respond_to config, :format
+    assert_equal 'browser', config.target
+    assert_equal 'esm', config.format
+  end
+
+  private
+
+  def mock_rails_root(path)
+    rails_module = if defined?(Rails)
+                     Rails
+                   else
+                     Object.const_set(:Rails, Module.new)
+                   end
+
+    rails_module.define_singleton_method(:root) { Pathname.new(path) }
+  end
+end

data/test/railpack_test.rb

@@ -38,104 +38,7 @@ class RailpackTest < Minitest::Test
     assert_instance_of Railpack::Manager, Railpack.manager
   end
 
-  def test_bundler_support
-    assert_includes Railpack::Manager::BUNDLERS.keys, 'bun'
-    assert_includes Railpack::Manager::BUNDLERS.keys, 'esbuild'
-    assert_includes Railpack::Manager::BUNDLERS.keys, 'rollup'
-    assert_includes Railpack::Manager::BUNDLERS.keys, 'webpack'
-    assert_equal Railpack::BunBundler, Railpack::Manager::BUNDLERS['bun']
-    assert_equal Railpack::EsbuildBundler, Railpack::Manager::BUNDLERS['esbuild']
-    assert_equal Railpack::RollupBundler, Railpack::Manager::BUNDLERS['rollup']
-    assert_equal Railpack::WebpackBundler, Railpack::Manager::BUNDLERS['webpack']
-  end
-
-
-
-  def test_manager_bundle_size_calculation
-    # Create test output directory with fake assets
-    outdir = File.join(@temp_dir, 'builds')
-    FileUtils.mkdir_p(outdir)
-
-    # Create fake JS and CSS files
-    File.write(File.join(outdir, 'app.js'), 'console.log("test");')
-    File.write(File.join(outdir, 'app.css'), 'body { color: red; }')
-    File.write(File.join(outdir, 'app.js.map'), '{"version":3}')
-
-    config = { 'outdir' => outdir }
-    manager = Railpack::Manager.new
-
-    size = manager.send(:calculate_bundle_size, config)
-    assert size.is_a?(Float)
-    assert size > 0
-  end
-
-  def test_manager_asset_manifest_generation
-    outdir = File.join(@temp_dir, 'builds')
-    FileUtils.mkdir_p(outdir)
-
-    # Create fake built assets
-    File.write(File.join(outdir, 'application.js'), 'console.log("app");')
-    File.write(File.join(outdir, 'application.css'), 'body { color: blue; }')
-
-    config = { 'outdir' => outdir }
-    manager = Railpack::Manager.new
-
-    manager.send(:generate_asset_manifest, config)
 
-    manifest_path = File.join(outdir, '.manifest.json')
-    assert File.exist?(manifest_path)
-
-    manifest = JSON.parse(File.read(manifest_path))
-    assert manifest.key?('application.js')
-    assert manifest.key?('application.css')
-    assert manifest['application.js']['logical_path'] == 'application.js'
-  end
-
-  def test_bundler_base_class
-    bundler = Railpack::Bundler.new({})
-
-    assert_respond_to bundler, :build!
-    assert_respond_to bundler, :watch
-    assert_respond_to bundler, :install!
-    assert_respond_to bundler, :name
-    assert_respond_to bundler, :commands
-  end
-
-  def test_bun_bundler_commands
-    bundler = Railpack::BunBundler.new({})
-    commands = bundler.send(:commands)
-
-    assert_equal 'bun run build', commands[:build]
-    assert_equal 'bun run watch', commands[:watch]
-    assert_equal 'bun install', commands[:install]
-  end
-
-  def test_esbuild_bundler_commands
-    bundler = Railpack::EsbuildBundler.new({})
-    commands = bundler.send(:commands)
-
-    assert_equal 'esbuild', commands[:build]
-    assert_equal 'esbuild --watch', commands[:watch]
-    assert_equal 'npm install', commands[:install]
-  end
-
-  def test_rollup_bundler_commands
-    bundler = Railpack::RollupBundler.new({})
-    commands = bundler.send(:commands)
-
-    assert_equal 'rollup', commands[:build]
-    assert_equal 'rollup --watch', commands[:watch]
-    assert_equal 'npm install', commands[:install]
-  end
-
-  def test_webpack_bundler_commands
-    bundler = Railpack::WebpackBundler.new({})
-    commands = bundler.send(:commands)
-
-    assert_equal 'webpack', commands[:build]
-    assert_equal 'webpack --watch', commands[:watch]
-    assert_equal 'npm install', commands[:install]
-  end
 
   def test_event_hooks
     events = []

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: railpack
 version: !ruby/object:Gem::Version
-  version: 1.2.6
+  version: 1.2.7
 platform: ruby
 authors:
 - 21tycoons LLC
@@ -48,6 +48,7 @@ files:
 - lib/railpack/version.rb
 - lib/tasks/railpack.rake
 - sig/railpack.rbs
+- test/config_test.rb
 - test/railpack_test.rb
 homepage: https://github.com/21tycoons/railpack
 licenses: