shakapacker 6.5.5 → 6.5.6

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: shakapacker
 version: !ruby/object:Gem::Version
-  version: 6.5.5
+  version: 6.5.6
 platform: ruby
 authors:
 - David Heinemeier Hansson
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-12-28 00:00:00.000000000 Z
+date: 2023-02-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -122,13 +122,14 @@ files:
 - ".eslintignore"
 - ".eslintrc.js"
 - ".github/ISSUE_TEMPLATE/bug_report.md"
-- ".github/ISSUE_TEMPLATE/feature-request.md"
+- ".github/ISSUE_TEMPLATE/feature_request.md"
 - ".github/workflows/jest.yml"
 - ".github/workflows/js-lint.yml"
 - ".github/workflows/rubocop.yml"
 - ".github/workflows/ruby.yml"
 - ".gitignore"
 - ".node-version"
+- ".rspec"
 - ".rubocop.yml"
 - CHANGELOG.md
 - CONTRIBUTING.md
@@ -179,7 +180,6 @@ files:
 - lib/tasks/webpacker/verify_config.rake
 - lib/tasks/webpacker/verify_install.rake
 - lib/tasks/webpacker/yarn_install.rake
-- lib/tasks/yarn.rake
 - lib/webpacker.rb
 - lib/webpacker/base_strategy.rb
 - lib/webpacker/commands.rb
@@ -222,9 +222,13 @@ files:
 - package/esbuild/index.js
 - package/index.js
 - package/inliningCss.js
+- package/rules/__tests__/__utils__/webpack.js
+- package/rules/__tests__/babel.js
+- package/rules/__tests__/esbuild.js
 - package/rules/__tests__/file.js
 - package/rules/__tests__/index.js
 - package/rules/__tests__/raw.js
+- package/rules/__tests__/swc.js
 - package/rules/babel.js
 - package/rules/coffee.js
 - package/rules/css.js
@@ -232,6 +236,7 @@ files:
 - package/rules/esbuild.js
 - package/rules/file.js
 - package/rules/index.js
+- package/rules/jscommon.js
 - package/rules/less.js
 - package/rules/raw.js
 - package/rules/sass.js
@@ -241,103 +246,103 @@ files:
 - package/utils/get_style_rule.js
 - package/utils/helpers.js
 - rakelib/release.rake
-- test/command_test.rb
-- test/compiler_strategy_test.rb
-- test/compiler_test.rb
-- test/configuration_test.rb
-- test/dev_server_runner_test.rb
-- test/dev_server_test.rb
-- test/digest_strategy_test.rb
-- test/engine_rake_tasks_test.rb
-- test/env_test.rb
-- test/fixtures/beta_package-lock.v1.json
-- test/fixtures/beta_package-lock.v2.json
-- test/fixtures/beta_package.json
-- test/fixtures/beta_yarn.v1.lock
-- test/fixtures/beta_yarn.v2.lock
-- test/fixtures/git_url_package-lock.v1.json
-- test/fixtures/git_url_package-lock.v2.json
-- test/fixtures/git_url_package.json
-- test/fixtures/git_url_yarn.v1.lock
-- test/fixtures/git_url_yarn.v2.lock
-- test/fixtures/github_url_package-lock.v1.json
-- test/fixtures/github_url_package-lock.v2.json
-- test/fixtures/github_url_package.json
-- test/fixtures/github_url_yarn.v1.lock
-- test/fixtures/github_url_yarn.v2.lock
-- test/fixtures/relative_path_package-lock.v1.json
-- test/fixtures/relative_path_package-lock.v2.json
-- test/fixtures/relative_path_package.json
-- test/fixtures/relative_path_yarn.v1.lock
-- test/fixtures/relative_path_yarn.v2.lock
-- test/fixtures/semver_caret_package-lock.v1.json
-- test/fixtures/semver_caret_package-lock.v2.json
-- test/fixtures/semver_caret_package.json
-- test/fixtures/semver_caret_yarn.v1.lock
-- test/fixtures/semver_caret_yarn.v2.lock
-- test/fixtures/semver_exact_package-lock.v1.json
-- test/fixtures/semver_exact_package-lock.v2.json
-- test/fixtures/semver_exact_package.json
-- test/fixtures/semver_exact_yarn.v1.lock
-- test/fixtures/semver_exact_yarn.v2.lock
-- test/fixtures/semver_tilde_package-lock.v1.json
-- test/fixtures/semver_tilde_package-lock.v2.json
-- test/fixtures/semver_tilde_package.json
-- test/fixtures/semver_tilde_yarn.v1.lock
-- test/fixtures/semver_tilde_yarn.v2.lock
-- test/fixtures/without_package-lock.v1.json
-- test/fixtures/without_package-lock.v2.json
-- test/fixtures/without_package.json
-- test/fixtures/without_yarn.v1.lock
-- test/fixtures/without_yarn.v2.lock
-- test/helper_test.rb
-- test/manifest_test.rb
-- test/mounted_app/Rakefile
-- test/mounted_app/test/dummy/Rakefile
-- test/mounted_app/test/dummy/bin/rails
-- test/mounted_app/test/dummy/bin/rake
-- test/mounted_app/test/dummy/config.ru
-- test/mounted_app/test/dummy/config/application.rb
-- test/mounted_app/test/dummy/config/environment.rb
-- test/mounted_app/test/dummy/config/webpacker.yml
-- test/mounted_app/test/dummy/package.json
-- test/mtime_strategy_test.rb
-- test/rake_tasks_test.rb
-- test/test_app/Rakefile
-- test/test_app/app/packs/entrypoints/application.js
-- test/test_app/app/packs/entrypoints/generated/something.js
-- test/test_app/app/packs/entrypoints/multi_entry.css
-- test/test_app/app/packs/entrypoints/multi_entry.js
-- test/test_app/bin/webpacker
-- test/test_app/bin/webpacker-dev-server
-- test/test_app/config.ru
-- test/test_app/config/application.rb
-- test/test_app/config/environment.rb
-- test/test_app/config/initializers/inspect_autoload_paths.rb
-- test/test_app/config/webpack/webpack.config.js
-- test/test_app/config/webpacker.yml
-- test/test_app/config/webpacker_css_extract_ignore_order_warnings.yml
-- test/test_app/config/webpacker_defaults_fallback.yml
-- test/test_app/config/webpacker_manifest_path.yml
-- test/test_app/config/webpacker_nested_entries.yml
-- test/test_app/config/webpacker_no_precompile.yml
-- test/test_app/config/webpacker_other_location.yml
-- test/test_app/config/webpacker_public_root.yml
-- test/test_app/package.json
-- test/test_app/public/packs/manifest.json
-- test/test_app/some.config.js
-- test/test_app/yarn.lock
-- test/test_helper.rb
-- test/version_checker_test.rb
-- test/webpack_runner_test.rb
-- test/webpacker_test.rb
+- spec/command_spec.rb
+- spec/compiler_spec.rb
+- spec/compiler_strategy_spec.rb
+- spec/configuration_spec.rb
+- spec/dev_server_runner_spec.rb
+- spec/dev_server_spec.rb
+- spec/digest_strategy_spec.rb
+- spec/engine_rake_tasks_spec.rb
+- spec/env_spec.rb
+- spec/fixtures/beta_package-lock.v1.json
+- spec/fixtures/beta_package-lock.v2.json
+- spec/fixtures/beta_package.json
+- spec/fixtures/beta_yarn.v1.lock
+- spec/fixtures/beta_yarn.v2.lock
+- spec/fixtures/git_url_package-lock.v1.json
+- spec/fixtures/git_url_package-lock.v2.json
+- spec/fixtures/git_url_package.json
+- spec/fixtures/git_url_yarn.v1.lock
+- spec/fixtures/git_url_yarn.v2.lock
+- spec/fixtures/github_url_package-lock.v1.json
+- spec/fixtures/github_url_package-lock.v2.json
+- spec/fixtures/github_url_package.json
+- spec/fixtures/github_url_yarn.v1.lock
+- spec/fixtures/github_url_yarn.v2.lock
+- spec/fixtures/relative_path_package-lock.v1.json
+- spec/fixtures/relative_path_package-lock.v2.json
+- spec/fixtures/relative_path_package.json
+- spec/fixtures/relative_path_yarn.v1.lock
+- spec/fixtures/relative_path_yarn.v2.lock
+- spec/fixtures/semver_caret_package-lock.v1.json
+- spec/fixtures/semver_caret_package-lock.v2.json
+- spec/fixtures/semver_caret_package.json
+- spec/fixtures/semver_caret_yarn.v1.lock
+- spec/fixtures/semver_caret_yarn.v2.lock
+- spec/fixtures/semver_exact_package-lock.v1.json
+- spec/fixtures/semver_exact_package-lock.v2.json
+- spec/fixtures/semver_exact_package.json
+- spec/fixtures/semver_exact_yarn.v1.lock
+- spec/fixtures/semver_exact_yarn.v2.lock
+- spec/fixtures/semver_tilde_package-lock.v1.json
+- spec/fixtures/semver_tilde_package-lock.v2.json
+- spec/fixtures/semver_tilde_package.json
+- spec/fixtures/semver_tilde_yarn.v1.lock
+- spec/fixtures/semver_tilde_yarn.v2.lock
+- spec/fixtures/without_package-lock.v1.json
+- spec/fixtures/without_package-lock.v2.json
+- spec/fixtures/without_package.json
+- spec/fixtures/without_yarn.v1.lock
+- spec/fixtures/without_yarn.v2.lock
+- spec/helper_spec.rb
+- spec/manifest_spec.rb
+- spec/mounted_app/Rakefile
+- spec/mounted_app/test/dummy/Rakefile
+- spec/mounted_app/test/dummy/bin/rails
+- spec/mounted_app/test/dummy/bin/rake
+- spec/mounted_app/test/dummy/config.ru
+- spec/mounted_app/test/dummy/config/application.rb
+- spec/mounted_app/test/dummy/config/environment.rb
+- spec/mounted_app/test/dummy/config/webpacker.yml
+- spec/mounted_app/test/dummy/package.json
+- spec/mtime_strategy_spec.rb
+- spec/rake_tasks_spec.rb
+- spec/spec_helper.rb
+- spec/test_app/Rakefile
+- spec/test_app/app/packs/entrypoints/application.js
+- spec/test_app/app/packs/entrypoints/generated/something.js
+- spec/test_app/app/packs/entrypoints/multi_entry.css
+- spec/test_app/app/packs/entrypoints/multi_entry.js
+- spec/test_app/bin/webpacker
+- spec/test_app/bin/webpacker-dev-server
+- spec/test_app/config.ru
+- spec/test_app/config/application.rb
+- spec/test_app/config/environment.rb
+- spec/test_app/config/initializers/inspect_autoload_paths.rb
+- spec/test_app/config/webpack/webpack.config.js
+- spec/test_app/config/webpacker.yml
+- spec/test_app/config/webpacker_css_extract_ignore_order_warnings.yml
+- spec/test_app/config/webpacker_defaults_fallback.yml
+- spec/test_app/config/webpacker_manifest_path.yml
+- spec/test_app/config/webpacker_nested_entries.yml
+- spec/test_app/config/webpacker_no_precompile.yml
+- spec/test_app/config/webpacker_other_location.yml
+- spec/test_app/config/webpacker_public_root.yml
+- spec/test_app/package.json
+- spec/test_app/public/packs/manifest.json
+- spec/test_app/some.config.js
+- spec/test_app/yarn.lock
+- spec/version_checker_spec.rb
+- spec/webpack_runner_spec.rb
+- spec/webpacker_spec.rb
 - webpacker.gemspec
 - yarn.lock
 homepage: https://github.com/shakacode/shakapacker
 licenses:
 - MIT
 metadata:
-  source_code_uri: https://github.com/shakacode/shakapacker/tree/v6.5.5
+  source_code_uri: https://github.com/shakacode/shakapacker/tree/v6.5.6
 post_install_message:
 rdoc_options: []
 require_paths:
@@ -353,98 +358,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.32
+rubygems_version: 3.4.6
 signing_key:
 specification_version: 4
 summary: Use webpack to manage app-like JavaScript modules in Rails
-test_files:
-- test/command_test.rb
-- test/compiler_strategy_test.rb
-- test/compiler_test.rb
-- test/configuration_test.rb
-- test/dev_server_runner_test.rb
-- test/dev_server_test.rb
-- test/digest_strategy_test.rb
-- test/engine_rake_tasks_test.rb
-- test/env_test.rb
-- test/fixtures/beta_package-lock.v1.json
-- test/fixtures/beta_package-lock.v2.json
-- test/fixtures/beta_package.json
-- test/fixtures/beta_yarn.v1.lock
-- test/fixtures/beta_yarn.v2.lock
-- test/fixtures/git_url_package-lock.v1.json
-- test/fixtures/git_url_package-lock.v2.json
-- test/fixtures/git_url_package.json
-- test/fixtures/git_url_yarn.v1.lock
-- test/fixtures/git_url_yarn.v2.lock
-- test/fixtures/github_url_package-lock.v1.json
-- test/fixtures/github_url_package-lock.v2.json
-- test/fixtures/github_url_package.json
-- test/fixtures/github_url_yarn.v1.lock
-- test/fixtures/github_url_yarn.v2.lock
-- test/fixtures/relative_path_package-lock.v1.json
-- test/fixtures/relative_path_package-lock.v2.json
-- test/fixtures/relative_path_package.json
-- test/fixtures/relative_path_yarn.v1.lock
-- test/fixtures/relative_path_yarn.v2.lock
-- test/fixtures/semver_caret_package-lock.v1.json
-- test/fixtures/semver_caret_package-lock.v2.json
-- test/fixtures/semver_caret_package.json
-- test/fixtures/semver_caret_yarn.v1.lock
-- test/fixtures/semver_caret_yarn.v2.lock
-- test/fixtures/semver_exact_package-lock.v1.json
-- test/fixtures/semver_exact_package-lock.v2.json
-- test/fixtures/semver_exact_package.json
-- test/fixtures/semver_exact_yarn.v1.lock
-- test/fixtures/semver_exact_yarn.v2.lock
-- test/fixtures/semver_tilde_package-lock.v1.json
-- test/fixtures/semver_tilde_package-lock.v2.json
-- test/fixtures/semver_tilde_package.json
-- test/fixtures/semver_tilde_yarn.v1.lock
-- test/fixtures/semver_tilde_yarn.v2.lock
-- test/fixtures/without_package-lock.v1.json
-- test/fixtures/without_package-lock.v2.json
-- test/fixtures/without_package.json
-- test/fixtures/without_yarn.v1.lock
-- test/fixtures/without_yarn.v2.lock
-- test/helper_test.rb
-- test/manifest_test.rb
-- test/mounted_app/Rakefile
-- test/mounted_app/test/dummy/Rakefile
-- test/mounted_app/test/dummy/bin/rails
-- test/mounted_app/test/dummy/bin/rake
-- test/mounted_app/test/dummy/config.ru
-- test/mounted_app/test/dummy/config/application.rb
-- test/mounted_app/test/dummy/config/environment.rb
-- test/mounted_app/test/dummy/config/webpacker.yml
-- test/mounted_app/test/dummy/package.json
-- test/mtime_strategy_test.rb
-- test/rake_tasks_test.rb
-- test/test_app/Rakefile
-- test/test_app/app/packs/entrypoints/application.js
-- test/test_app/app/packs/entrypoints/generated/something.js
-- test/test_app/app/packs/entrypoints/multi_entry.css
-- test/test_app/app/packs/entrypoints/multi_entry.js
-- test/test_app/bin/webpacker
-- test/test_app/bin/webpacker-dev-server
-- test/test_app/config.ru
-- test/test_app/config/application.rb
-- test/test_app/config/environment.rb
-- test/test_app/config/initializers/inspect_autoload_paths.rb
-- test/test_app/config/webpack/webpack.config.js
-- test/test_app/config/webpacker.yml
-- test/test_app/config/webpacker_css_extract_ignore_order_warnings.yml
-- test/test_app/config/webpacker_defaults_fallback.yml
-- test/test_app/config/webpacker_manifest_path.yml
-- test/test_app/config/webpacker_nested_entries.yml
-- test/test_app/config/webpacker_no_precompile.yml
-- test/test_app/config/webpacker_other_location.yml
-- test/test_app/config/webpacker_public_root.yml
-- test/test_app/package.json
-- test/test_app/public/packs/manifest.json
-- test/test_app/some.config.js
-- test/test_app/yarn.lock
-- test/test_helper.rb
-- test/version_checker_test.rb
-- test/webpack_runner_test.rb
-- test/webpacker_test.rb
+test_files: []

data/.github/ISSUE_TEMPLATE/feature-request.md

@@ -1,18 +0,0 @@
----
-name: Feature request
-about: Create a request for new functionality
-title: ''
-labels: enhancement
-assignees: ''
-
----
-
-A feature request describes a suggested improvement. If you have a debugging or troubleshooting question, please open a discussion on the [Discussions tab](https://github.com/shakacode/shakapacker/discussions).
-
-Ruby version:
-Rails version:
-Shakapacker version:
-
-Desired behavior:
-
-Actual behavior:

data/lib/tasks/yarn.rake

@@ -1,44 +0,0 @@
-# frozen_string_literal: true
-
-# Duplicate of the yarn tasks still present in Rails until Webpacker <5 have been deprecated
-
-namespace :yarn do
-  desc "Install all JavaScript dependencies as specified via Yarn"
-  task :install do
-    warn <<~MSG.strip
-      Shakapacker - Automatic installation of yarn packages is deprecated
-      Automatic installation of yarn packages when assets are precompiled is deprecated and will be removed in Shakapacker v7.
-      Please ensure you are installing yarn packages explicitly before the asset compilation.
-    MSG
-
-    begin
-      # Install only production deps when for not usual envs.
-      valid_node_envs = %w[test development production]
-      node_env = ENV.fetch("NODE_ENV") do
-        valid_node_envs.include?(Rails.env) ? Rails.env : "production"
-      end
-
-      yarn_flags =
-        if `#{RbConfig.ruby} "#{Rails.root}/bin/yarn" --version`.start_with?("1")
-          "--no-progress --frozen-lockfile"
-        else
-          "--immutable"
-        end
-
-      system(
-        { "NODE_ENV" => node_env },
-        "#{RbConfig.ruby} \"#{Rails.root}/bin/yarn\" install #{yarn_flags}",
-        exception: true
-      )
-    rescue Errno::ENOENT
-      $stderr.puts "bin/yarn was not found."
-      $stderr.puts "Please run `bundle exec rails app:update:bin` to create it."
-      exit 1
-    end
-  end
-end
-
-# Run Yarn prior to Sprockets assets precompilation, so dependencies are available for use.
-if Rake::Task.task_defined?("assets:precompile") && File.exist?(Rails.root.join("bin", "yarn"))
-  Rake::Task["assets:precompile"].enhance [ "yarn:install" ]
-end

data/test/command_test.rb

@@ -1,109 +0,0 @@
-require "test_helper"
-
-class CommandTest < Minitest::Test
-  def test_compile_command_returns_success_status_when_stale
-    Webpacker.compiler.stub :stale?, true do
-      Webpacker.compiler.stub :run_webpack, true do
-        assert_equal true, Webpacker.commands.compile
-      end
-    end
-  end
-
-  def test_compile_command_returns_success_status_when_fresh
-    Webpacker.compiler.stub :stale?, false do
-      Webpacker.compiler.stub :run_webpack, true do
-        assert_equal true, Webpacker.commands.compile
-      end
-    end
-  end
-
-  def test_compile_command_returns_failure_status_when_stale
-    Webpacker.compiler.stub :stale?, true do
-      Webpacker.compiler.stub :run_webpack, false do
-        assert_equal false, Webpacker.commands.compile
-      end
-    end
-  end
-
-  def test_clean_command_works_with_nested_hashes_and_without_any_compiled_files
-    File.stub :delete, true do
-      assert Webpacker.commands.clean
-    end
-  end
-end
-
-class ClearCommandVersioningTest < Minitest::Test
-  def setup
-    @now = Time.parse("2021-01-01 12:34:56 UTC")
-    # Test assets to be kept and deleted, path and mtime
-    @prev_files = {
-      # recent versions to be kept with Webpacker.commands.clean(count = 2)
-      "js/application-deadbeef.js" => @now - 4000,
-      "js/common-deadbeee.js" => @now - 4002,
-      "css/common-deadbeed.css" => @now - 4004,
-      "media/images/logo-deadbeeb.css" => @now - 4006,
-      "js/application-1eadbeef.js" => @now - 8000,
-      "js/common-1eadbeee.js" => @now - 8002,
-      "css/common-1eadbeed.css" => @now - 8004,
-      "media/images/logo-1eadbeeb.css" => @now - 8006,
-      # new files to be kept with Webpacker.commands.clean(age = 3600)
-      "js/brandnew-0001.js" => @now,
-      "js/brandnew-0002.js" => @now - 10,
-      "js/brandnew-0003.js" => @now - 20,
-      "js/brandnew-0004.js" => @now - 40,
-    }.transform_keys { |path| "#{Webpacker.config.public_output_path}/#{path}" }
-    @expired_files = {
-      # old files that are outside count = 2 or age = 3600 and to be deleted
-      "js/application-0eadbeef.js" => @now - 9000,
-      "js/common-0eadbeee.js" => @now - 9002,
-      "css/common-0eadbeed.css" => @now - 9004,
-      "js/brandnew-0005.js" => @now - 3640,
-    }.transform_keys { |path| "#{Webpacker.config.public_output_path}/#{path}" }
-    @all_files = @prev_files.merge(@expired_files)
-    @dir_glob_stub = Proc.new { |arg|
-      case arg
-      when "#{Webpacker.config.public_output_path}/**/*"
-        @all_files.keys
-      else
-        []
-      end
-    }
-    @file_mtime_stub = Proc.new { |longpath|
-      @all_files[longpath]
-    }
-    @file_delete_mock = Minitest::Mock.new
-    @expired_files.keys.each do |longpath|
-      @file_delete_mock.expect(:delete, 1, [longpath])
-    end
-    @file_delete_stub = Proc.new { |longpath|
-      if @prev_files.has_key?(longpath)
-        flunk "#{longpath} should not be deleted"
-      else
-        @file_delete_mock.delete(longpath)
-      end
-    }
-  end
-
-  def time_and_files_stub(&proc)
-    Time.stub :now, @now do
-      Dir.stub :glob, @dir_glob_stub do
-        File.stub :directory?, false do
-          File.stub :file?, true do
-            File.stub :mtime, @file_mtime_stub do
-              File.stub :delete, @file_delete_stub do
-                yield proc
-              end
-            end
-          end
-        end
-      end
-    end
-    @file_delete_mock.verify
-  end
-
-  def test_clean_command_with_versioned_files
-    time_and_files_stub do
-      assert Webpacker.commands.clean
-    end
-  end
-end

data/test/compiler_strategy_test.rb

@@ -1,27 +0,0 @@
-require "test_helper"
-
-class CompilerStrategyTest < Minitest::Test
-  def test_mtime_strategy_returned
-    Webpacker.config.stub :compiler_strategy, "mtime" do
-      assert_instance_of Webpacker::MtimeStrategy, Webpacker::CompilerStrategy.from_config
-    end
-  end
-
-  def test_digest_strategy_returned
-    Webpacker.config.stub :compiler_strategy, "digest" do
-      assert_instance_of Webpacker::DigestStrategy, Webpacker::CompilerStrategy.from_config
-    end
-  end
-
-  def test_raise_on_unknown_strategy
-    Webpacker.config.stub :compiler_strategy, "other" do
-      error = assert_raises do
-        Webpacker::CompilerStrategy.from_config
-      end
-
-      assert_equal \
-        "Unknown strategy 'other'. Available options are 'mtime' and 'digest'.",
-        error.message
-    end
-  end
-end

data/test/compiler_test.rb

@@ -1,60 +0,0 @@
-require "test_helper"
-
-class CompilerTest < Minitest::Test
-  def test_custom_environment_variables
-    assert_nil Webpacker.compiler.send(:webpack_env)["FOO"]
-    Webpacker.compiler.env["FOO"] = "BAR"
-    assert Webpacker.compiler.send(:webpack_env)["FOO"] == "BAR"
-  ensure
-    Webpacker.compiler.env = {}
-  end
-
-  def test_compile_true_when_fresh
-    mock = Minitest::Mock.new
-    mock.expect(:stale?, false)
-    Webpacker.compiler.stub(:strategy, mock) do
-      assert Webpacker.compiler.compile
-    end
-    assert_mock mock
-  end
-
-  def test_after_compile_hook_called_on_success
-    mock = Minitest::Mock.new
-    mock.expect(:stale?, true)
-    mock.expect(:after_compile_hook, nil)
-
-    status = OpenStruct.new(success?: true)
-
-    Webpacker.compiler.stub(:strategy, mock) do
-      Open3.stub :capture3, [:sterr, :stdout, status] do
-        Webpacker.compiler.compile
-      end
-    end
-    assert_mock mock
-  end
-
-  def test_after_compile_hook_called_on_failure
-    mock = Minitest::Mock.new
-    mock.expect(:stale?, true)
-    mock.expect(:after_compile_hook, nil)
-
-    status = OpenStruct.new(success?: false)
-
-    Webpacker.compiler.stub(:strategy, mock) do
-      Open3.stub :capture3, [:sterr, :stdout, status] do
-        Webpacker.compiler.compile
-      end
-    end
-    assert_mock mock
-  end
-
-  def test_external_env_variables
-    assert_nil Webpacker.compiler.send(:webpack_env)["WEBPACKER_ASSET_HOST"]
-    assert_nil Webpacker.compiler.send(:webpack_env)["WEBPACKER_RELATIVE_URL_ROOT"]
-
-    ENV["WEBPACKER_ASSET_HOST"] = "foo.bar"
-    ENV["WEBPACKER_RELATIVE_URL_ROOT"] = "/baz"
-    assert_equal Webpacker.compiler.send(:webpack_env)["WEBPACKER_ASSET_HOST"], "foo.bar"
-    assert_equal Webpacker.compiler.send(:webpack_env)["WEBPACKER_RELATIVE_URL_ROOT"], "/baz"
-  end
-end