shakapacker 6.6.0 → 7.0.0.rc.0

data/package/webpackDevServerConfig.js

@@ -0,0 +1,68 @@
+const shakapackerDevServerYamlConfig = require('./dev_server')
+const snakeToCamelCase = require('./utils/snakeToCamelCase')
+const { outputPath: contentBase, publicPath } = require('./config')
+
+const webpackDevServerMappedKeys = new Set([
+  // client, server, liveReload, devMiddleware are handled separately
+  'allowedHosts',
+  'bonjour',
+  'compress',
+  'headers',
+  'historyApiFallback',
+  'host',
+  'hot',
+  'http2',
+  'https',
+  'ipc',
+  'magicHtml',
+  'onAfterSetupMiddleware',
+  'onBeforeSetupMiddleware',
+  'open',
+  'port',
+  'proxy',
+  'server',
+  'setupExitSignals',
+  'setupMiddlewares',
+  'watchFiles',
+  'webSocketServer'
+])
+
+function createDevServerConfig() {
+  const devServerYamlConfig = { ...shakapackerDevServerYamlConfig }
+  const liveReload = devServerYamlConfig.live_reload !== undefined ? devServerYamlConfig.live_reload : !devServerYamlConfig.hmr
+  delete devServerYamlConfig.live_reload
+
+  const config = {
+    devMiddleware: {
+      publicPath
+    },
+    liveReload,
+    historyApiFallback: {
+      disableDotRule: true
+    },
+    static: {
+      publicPath: contentBase
+    }
+  }
+
+  if (devServerYamlConfig.static) {
+    config.static = { ...config.static, ...devServerYamlConfig.static }
+    delete devServerYamlConfig.static
+  }
+
+  if (devServerYamlConfig.client) {
+    config.client = devServerYamlConfig.client
+    delete devServerYamlConfig.client
+  }
+
+  Object.keys(devServerYamlConfig).forEach((yamlKey) => {
+    const camelYamlKey = snakeToCamelCase(yamlKey)
+    if (webpackDevServerMappedKeys.has(camelYamlKey)) {
+      config[camelYamlKey] = devServerYamlConfig[yamlKey]
+    }
+  })
+
+  return config
+}
+
+module.exports = createDevServerConfig

data/package.json

@@ -1,11 +1,11 @@
 {
   "name": "shakapacker",
-  "version": "6.6.0",
+  "version": "7.0.0-rc.0",
   "description": "Use webpack to manage app-like JavaScript modules in Rails",
   "main": "package/index.js",
   "files": [
     "package",
-    "lib/install/config/webpacker.yml"
+    "lib/install/config/shakapacker.yml"
   ],
   "engines": {
     "node": "^12.13.0 || ^14 || >=16",

data/{webpacker.gemspec → shakapacker.gemspec}

@@ -1,16 +1,16 @@
 $:.push File.expand_path("../lib", __FILE__)
-require "webpacker/version"
+require "shakapacker/version"
 
 Gem::Specification.new do |s|
   s.name     = "shakapacker"
-  s.version  = Webpacker::VERSION
+  s.version  = Shakapacker::VERSION
   s.authors  = [ "David Heinemeier Hansson", "Gaurav Tiwari", "Justin Gordon" ]
   s.email    = [ "david@basecamp.com", "gaurav@gauravtiwari.co.uk", "justin@shakacode.com" ]
   s.summary  = "Use webpack to manage app-like JavaScript modules in Rails"
   s.homepage = "https://github.com/shakacode/shakapacker"
   s.license  = "MIT"
 
-  npm_version = Webpacker::VERSION.gsub(".rc", "-rc")
+  npm_version = Shakapacker::VERSION.gsub(".rc", "-rc")
   s.metadata = {
     "source_code_uri" => "https://github.com/shakacode/shakapacker/tree/v#{npm_version}",
   }

data/spec/backward_compatibility_specs/command_spec_bc.rb

@@ -0,0 +1,116 @@
+require_relative "spec_helper_initializer"
+
+describe "Command" do
+  before do
+    allow(Webpacker.logger).to receive(:info)
+  end
+
+  describe "#compile" do
+    it "returns success status when stale" do
+      expect(Webpacker.compiler).to receive(:stale?).and_return(true)
+      expect(Webpacker.compiler).to receive(:run_webpack).and_return(true)
+
+      expect(Webpacker.commands.compile).to be true
+    end
+
+    it "returns success status when fresh" do
+      expect(Webpacker.compiler).to receive(:stale?).and_return(false)
+
+      expect(Webpacker.commands.compile).to be true
+    end
+
+    it "returns failure status when stale" do
+      expect(Webpacker.compiler).to receive(:stale?).and_return(true)
+      expect(Webpacker.compiler).to receive(:run_webpack).and_return(false)
+
+      expect(Webpacker.commands.compile).to be false
+    end
+  end
+
+  describe "#clean" do
+    let(:now) { Time.parse("2021-01-01 12:34:56 UTC") }
+    let(:prev_files) do
+      # Test assets to be kept and deleted, path and mtime
+      {
+        # 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}" }
+    end
+
+    let(:expired_files) do
+      {
+        # 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}" }
+    end
+
+    let(:all_files) { prev_files.merge(expired_files) }
+
+    let(:file_delete_mock) { double("File Delete") }
+    let(:file_mtime_stub) { Proc.new { |longpath| all_files[longpath] } }
+    let(:file_delete_stub) { Proc.new { |longpath| file_delete_mock.delete(longpath) } }
+
+    before :context do
+      @dir_glob_stub = Proc.new { |arg|
+        case arg
+        when "#{Webpacker.config.public_output_path}/**/*"
+          all_files.keys
+        else
+          []
+        end
+      }
+    end
+
+    it "works with nested hashes and without any compiled files" do
+      allow(File).to receive(:delete).and_return(true)
+      expect(Webpacker.commands.clean).to be true
+    end
+
+    it "deletes only and only expired versioned files if no parameter passed" do
+      all_files.keys.each do |longpath|
+        allow(file_delete_mock).to receive(:delete).with(longpath)
+      end
+
+      with_time_dir_and_files_stub do
+        expect(Webpacker.commands.clean).to be true
+
+        # Verify that only and only expired files are deleted
+        all_files.keys.each do |longpath|
+          if expired_files.has_key? longpath
+            expect(file_delete_mock).to have_received(:delete).with(longpath)
+          else
+            expect(file_delete_mock).to_not have_received(:delete).with(longpath)
+          end
+        end
+      end
+    end
+
+    private
+
+      def with_time_dir_and_files_stub(&proc)
+        allow(Time).to receive(:now).and_return(now)
+        allow(Dir).to receive(:glob) { |arg| @dir_glob_stub.call(arg) }
+        allow(File).to receive(:directory?).and_return(false)
+        allow(File).to receive(:file?).and_return(true)
+        allow(File).to receive(:mtime) { |arg| file_mtime_stub.call(arg) }
+        allow(File).to receive(:delete) { |arg| file_delete_stub.call(arg) }
+
+        yield proc
+      end
+  end
+end

data/spec/backward_compatibility_specs/compiler_spec_bc.rb

@@ -0,0 +1,59 @@
+require_relative "spec_helper_initializer"
+
+describe "Webpacker::Compiler" do
+  it "accepts custom environment variables" do
+    expect(Webpacker.compiler.send(:webpack_env)["FOO"]).to be nil
+
+    Webpacker.compiler.env["FOO"] = "BAR"
+    expect(Webpacker.compiler.send(:webpack_env)["FOO"]).to eq "BAR"
+  ensure
+    Webpacker.compiler.env = {}
+  end
+
+  it "returns true when fresh" do
+    mocked_strategy = double("Strategy")
+    expect(mocked_strategy).to receive(:stale?).and_return(false)
+
+    expect(Webpacker.compiler).to receive(:strategy).and_return(mocked_strategy)
+
+    expect(Webpacker.compiler.compile).to be true
+  end
+
+  it "returns true and calls after_compile_hook on successful compile" do
+    mocked_strategy = spy("Strategy")
+    expect(mocked_strategy).to receive(:stale?).and_return(true)
+
+    allow(Webpacker.compiler).to receive(:strategy).and_return(mocked_strategy)
+
+    status = OpenStruct.new(success?: true)
+    allow(Open3).to receive(:capture3).and_return([:sterr, :stdout, status])
+
+    expect(Webpacker.compiler.compile).to be true
+    expect(mocked_strategy).to have_received(:after_compile_hook)
+  end
+
+  it "returns false and calls after_compile_hook on failed compile" do
+    mocked_strategy = spy("Strategy")
+    allow(mocked_strategy).to receive(:stale?).and_return(true)
+    allow(mocked_strategy).to receive(:after_compile_hook)
+
+    allow(Webpacker.compiler).to receive(:strategy).and_return(mocked_strategy)
+
+    status = OpenStruct.new(success?: false)
+    allow(Open3).to receive(:capture3).and_return([:sterr, :stdout, status])
+
+    expect(Webpacker.compiler.compile).to be false
+    expect(mocked_strategy).to have_received(:after_compile_hook)
+  end
+
+  it "accepts external env variables" do
+    expect(Webpacker.compiler.send(:webpack_env)["SHAKAPACKER_ASSET_HOST"]).to be nil
+    expect(Webpacker.compiler.send(:webpack_env)["SHAKAPACKER_RELATIVE_URL_ROOT"]).to be nil
+
+    ENV["WEBPACKER_ASSET_HOST"] = "foo.bar"
+    ENV["WEBPACKER_RELATIVE_URL_ROOT"] = "/baz"
+
+    expect(Webpacker.compiler.send(:webpack_env)["SHAKAPACKER_ASSET_HOST"]).to eq "foo.bar"
+    expect(Webpacker.compiler.send(:webpack_env)["SHAKAPACKER_RELATIVE_URL_ROOT"]).to eq "/baz"
+  end
+end

data/spec/backward_compatibility_specs/compiler_strategy_spec_bc.rb

@@ -0,0 +1,22 @@
+require_relative "spec_helper_initializer"
+
+describe "Webpacker::CompilerStrategy" do
+  describe "#from_config" do
+    it "returns and instance of MtimeStrategy when compiler_strategy is set to mtime" do
+      allow(Webpacker.config).to receive(:compiler_strategy).and_return("mtime")
+      expect(Webpacker::CompilerStrategy.from_config).to be_an_instance_of(Webpacker::MtimeStrategy)
+    end
+
+    it "returns and instance of DigestStrategy when compiler_strategy is set to digest" do
+      allow(Webpacker.config).to receive(:compiler_strategy).and_return("digest")
+      expect(Webpacker::CompilerStrategy.from_config).to be_an_instance_of(Webpacker::DigestStrategy)
+    end
+
+    it "raise exception for unknown compiler_strategy in the config file" do
+      expected_error_message = "Unknown strategy 'other'. Available options are 'mtime' and 'digest'."
+      allow(Webpacker.config).to receive(:compiler_strategy).and_return("other")
+
+      expect { Webpacker::CompilerStrategy.from_config }.to raise_error(expected_error_message)
+    end
+  end
+end

data/spec/backward_compatibility_specs/configuration_spec_bc.rb

@@ -0,0 +1,286 @@
+require_relative "spec_helper_initializer"
+
+describe "Webpacker::Configuration" do
+  ROOT_PATH = Pathname.new(File.expand_path("webpacker_test_app", __dir__))
+
+  context "with standard webpacker.yml" do
+    let(:config) do
+      Webpacker::Configuration.new(
+        root_path: ROOT_PATH,
+        config_path: Pathname.new(File.expand_path("./webpacker_test_app/config/webpacker.yml", __dir__)),
+        env: "production"
+      )
+    end
+
+    it "#source_path returns correct path" do
+      source_path = File.expand_path File.join(File.dirname(__FILE__), "webpacker_test_app/app/packs").to_s
+      expect(config.source_path.to_s).to eq source_path
+    end
+
+    it "#source_entry_path returns correct path" do
+      source_entry_path = File.expand_path File.join(File.dirname(__FILE__), "webpacker_test_app/app/packs", "entrypoints").to_s
+      expect(config.source_entry_path.to_s).to eq source_entry_path
+    end
+
+    it "#public_root_path returns correct path" do
+      public_root_path = File.expand_path File.join(File.dirname(__FILE__), "webpacker_test_app/public").to_s
+      expect(config.public_path.to_s).to eq public_root_path
+    end
+
+    it "#public_output_path returns correct path" do
+      public_output_path = File.expand_path File.join(File.dirname(__FILE__), "webpacker_test_app/public/packs").to_s
+      expect(config.public_output_path.to_s).to eq public_output_path
+    end
+
+    it "#public_manifest_path returns correct path" do
+      public_manifest_path = File.expand_path File.join(File.dirname(__FILE__), "webpacker_test_app/public/packs", "manifest.json").to_s
+      expect(config.public_manifest_path.to_s).to eq public_manifest_path
+    end
+
+    it "#manifest_path returns correct path" do
+      manifest_path = File.expand_path File.join(File.dirname(__FILE__), "webpacker_test_app/public/packs", "manifest.json").to_s
+      expect(config.manifest_path.to_s).to eq manifest_path
+    end
+
+    it "#cache_path returns correct path" do
+      cache_path = File.expand_path File.join(File.dirname(__FILE__), "webpacker_test_app/tmp/webpacker").to_s
+      expect(config.cache_path.to_s).to eq cache_path
+    end
+
+    it "#additional_paths returns correct path" do
+      expect(config.additional_paths).to eq ["app/assets", "/etc/yarn", "some.config.js", "app/elm"]
+    end
+
+    describe "#cache_manifest?" do
+      it "returns true in production environment" do
+        expect(config.cache_manifest?).to be true
+      end
+
+      it "returns false in developemnt environemnt" do
+        with_rails_env("development") do
+          expect(Webpacker.config.cache_manifest?).to be false
+        end
+      end
+
+      it "returns false in test environment" do
+        with_rails_env("test") do
+          expect(Webpacker.config.cache_manifest?).to be false
+        end
+      end
+    end
+
+    describe "#compile?" do
+      it "returns false in production environment" do
+        expect(config.compile?).to be false
+      end
+
+      it "returns true in developemnt environemnt" do
+        with_rails_env("development") do
+          expect(Webpacker.config.compile?).to be true
+        end
+      end
+
+      it "returns true in test environemnt" do
+        with_rails_env("test") do
+          expect(Webpacker.config.compile?).to be true
+        end
+      end
+    end
+
+    describe "#nested_entries?" do
+      it "returns false in production environment" do
+        expect(config.nested_entries?).to be false
+      end
+
+      it "returns false in development environment" do
+        with_rails_env("development") do
+          expect(Webpacker.config.nested_entries?).to be false
+        end
+      end
+
+      it "returns false in test environment" do
+        with_rails_env("test") do
+          expect(Webpacker.config.nested_entries?).to be false
+        end
+      end
+    end
+
+    describe "#ensure_consistent_versioning?" do
+      it "returns false in production environment" do
+        expect(config.ensure_consistent_versioning?).to be false
+      end
+
+      it "returns true in development environment" do
+        with_rails_env("development") do
+          expect(Webpacker.config.ensure_consistent_versioning?).to be true
+        end
+      end
+
+      it "returns false in test environment" do
+        with_rails_env("test") do
+          expect(Webpacker.config.ensure_consistent_versioning?).to be false
+        end
+      end
+    end
+
+    describe "#webpacker_precompile?" do
+      before :each do
+        ENV.delete("SHAKAPACKER_PRECOMPILE")
+        ENV.delete("WEBPACKER_PRECOMPILE")
+      end
+
+      subject { config.webpacker_precompile? }
+
+      it "returns true when WEBPACKER_PRECOMPILE is unset" do
+        is_expected.to be true
+      end
+
+      it "returns false when WEBPACKER_PRECOMPILE sets to no" do
+        ENV["WEBPACKER_PRECOMPILE"] = "no"
+        is_expected.to be false
+      end
+
+      it "returns true when WEBPACKER_PRECOMPILE sets to yes" do
+        ENV["WEBPACKER_PRECOMPILE"] = "yes"
+        is_expected.to be true
+      end
+
+      it "returns false when WEBPACKER_PRECOMPILE sets to false" do
+        ENV["WEBPACKER_PRECOMPILE"] = "false"
+        is_expected.to be false
+      end
+
+      it "returns true when WEBPACKER_PRECOMPILE sets to true" do
+        ENV["WEBPACKER_PRECOMPILE"] = "true"
+        is_expected.to be true
+      end
+
+      it "returns false when WEBPACKER_PRECOMPILE sets to n" do
+        ENV["WEBPACKER_PRECOMPILE"] = "n"
+        is_expected.to be false
+      end
+
+      it "returns true when WEBPACKER_PRECOMPILE sets to y" do
+        ENV["WEBPACKER_PRECOMPILE"] = "y"
+        is_expected.to be true
+      end
+
+      it "returns false when WEBPACKER_PRECOMPILE sets to f" do
+        ENV["WEBPACKER_PRECOMPILE"] = "f"
+        is_expected.to be false
+      end
+
+      it "returns true when WEBPACKER_PRECOMPILE sets to t" do
+        ENV["WEBPACKER_PRECOMPILE"] = "t"
+        is_expected.to be true
+      end
+    end
+  end
+
+  context "with webpacker config file containing public_output_path entry" do
+    config = Webpacker::Configuration.new(
+      root_path: ROOT_PATH,
+      config_path: Pathname.new(File.expand_path("./webpacker_test_app/config/webpacker_public_root.yml", __dir__)),
+      env: "production"
+    )
+
+    it "#public_output_path returns correct path" do
+      expected_public_output_path = File.expand_path File.join(File.dirname(__FILE__), "public/packs").to_s
+      expect(config.public_output_path.to_s).to eq expected_public_output_path
+    end
+  end
+
+  context "with webpacker config file containing manifext_path entry" do
+    config = Webpacker::Configuration.new(
+      root_path: ROOT_PATH,
+      config_path: Pathname.new(File.expand_path("./webpacker_test_app/config/webpacker_manifest_path.yml", __dir__)),
+      env: "production"
+    )
+
+    it "#manifest_path returns correct expected value" do
+      expected_manifest_path = File.expand_path File.join(File.dirname(__FILE__), "webpacker_test_app/app/packs", "manifest.json").to_s
+      expect(config.manifest_path.to_s).to eq expected_manifest_path
+    end
+  end
+
+  context "with webpacker_precompile entry set to false" do
+    describe "#webpacker_precompile?" do
+      before :each do
+        ENV.delete("WEBPACKER_PRECOMPILE")
+        ENV.delete("SHAKAPACKER_PRECOMPILE")
+      end
+
+      let(:config) {
+        Webpacker::Configuration.new(
+          root_path: ROOT_PATH,
+          config_path: Pathname.new(File.expand_path("./webpacker_test_app/config/webpacker_no_precompile.yml", __dir__)),
+          env: "production"
+        )
+      }
+
+      subject { config.webpacker_precompile? }
+
+      it "returns false with unset WEBPACKER_PRECOMPILE" do
+        expect(subject).to be false
+      end
+
+      it "returns true with WEBPACKER_PRECOMPILE set to true" do
+        ENV["WEBPACKER_PRECOMPILE"] = "true"
+        expect(subject).to be true
+      end
+
+      it "returns false with WEBPACKER_PRECOMPILE set to falsy value" do
+        # ENV["WEBPACKER_PRECOMPILE"] = "no"
+        ENV.delete("WEBPACKER_PRECOMPILE")
+        expect(subject).to be false
+      end
+    end
+  end
+
+  context "with webpacker config file containing invalid path" do
+    config = Webpacker::Configuration.new(
+      root_path: ROOT_PATH,
+      config_path: Pathname.new(File.expand_path("./webpacker_test_app/config/invalid_path.yml", __dir__)),
+      env: "default"
+    )
+
+    it "#webpacker_precompile? returns false" do
+      expect(config.webpacker_precompile?).to be false
+    end
+  end
+
+  context "with webpacker config file with defaults fallback" do
+    let(:config) do
+      Webpacker::Configuration.new(
+        root_path: ROOT_PATH,
+        config_path: Pathname.new(File.expand_path("./webpacker_test_app/config/webpacker_defaults_fallback.yml", __dir__)),
+        env: "default"
+      )
+    end
+
+    it "#cache_manifest? falls back to 'default' config from bundled file" do
+      expect(config.cache_manifest?).to be false
+    end
+
+    it "#webpacker_precompile? uses 'default' config from custom file" do
+      expect(config.webpacker_precompile?).to be false
+    end
+  end
+
+  context "falls back to bundled production config for custom environments" do
+    let(:config) do
+      Webpacker::Configuration.new(
+        root_path: ROOT_PATH,
+        config_path: Pathname.new(File.expand_path("./webpacker_test_app/config/webpacker_defaults_fallback.yml", __dir__)),
+        env: "staging"
+      )
+    end
+
+    it "#cache_manifest? fall back to 'production' config from bundled file" do
+      expect(config.cache_manifest?).to be true
+    end
+    it "#webpacker_precompile? use 'staging' config from custom file" do
+      expect(config.webpacker_precompile?).to be false
+    end
+  end
+end

data/spec/backward_compatibility_specs/dev_server_runner_spec_bc.rb

@@ -0,0 +1,79 @@
+require_relative "spec_helper_initializer"
+# Requiring from webpacker directory to ensure old ./bin/webpacker-dev-server works fine
+require "webpacker/dev_server_runner"
+
+describe "DevServerRunner" do
+  before do
+    @original_node_env, ENV["NODE_ENV"] = ENV["NODE_ENV"], "development"
+    @original_rails_env, ENV["RAILS_ENV"] = ENV["RAILS_ENV"], "development"
+    @original_webpacker_config = ENV["WEBPACKER_CONFIG"]
+  end
+
+  after do
+    ENV["NODE_ENV"] = @original_node_env
+    ENV["RAILS_ENV"] = @original_rails_env
+    ENV["WEBPACKER_CONFIG"] = @original_webpacker_config
+  end
+
+  let(:test_app_path) { File.expand_path("webpacker_test_app", __dir__) }
+
+  it "run cmd via node modules" do
+    cmd = ["#{test_app_path}/node_modules/.bin/webpack", "serve", "--config", "#{test_app_path}/config/webpack/webpack.config.js"]
+    verify_command(cmd, use_node_modules: true)
+  end
+
+  it "run cmd via yarn" do
+    cmd = ["yarn", "webpack", "serve", "--config", "#{test_app_path}/config/webpack/webpack.config.js"]
+    verify_command(cmd, use_node_modules: false)
+  end
+
+  it "run cmd argv" do
+    cmd = ["#{test_app_path}/node_modules/.bin/webpack", "serve", "--config", "#{test_app_path}/config/webpack/webpack.config.js", "--quiet"]
+    verify_command(cmd, argv: (["--quiet"]))
+  end
+
+  it "run cmd argv with https" do
+    cmd = ["#{test_app_path}/node_modules/.bin/webpack", "serve", "--config", "#{test_app_path}/config/webpack/webpack.config.js", "--https"]
+
+    dev_server = double()
+    allow(dev_server).to receive(:host).and_return("localhost")
+    allow(dev_server).to receive(:port).and_return("3035")
+    allow(dev_server).to receive(:pretty?).and_return(false)
+    allow(dev_server).to receive(:https?).and_return(true)
+    allow(dev_server).to receive(:hmr?).and_return(false)
+
+    allow(Webpacker::DevServer).to receive(:new) do
+      verify_command(cmd, argv: (["--https"]))
+    end.and_return(dev_server)
+  end
+
+  it "accepts environment variables" do
+    cmd = ["#{test_app_path}/node_modules/.bin/webpack", "serve", "--config", "#{test_app_path}/config/webpack/webpack.config.js"]
+    env = Webpacker::Compiler.env.dup
+
+    # ENV["WEBPACKER_CONFIG"] is the interface and env["SHAKAPACKER_CONFIG"] is internal
+    ENV["WEBPACKER_CONFIG"] = env["SHAKAPACKER_CONFIG"] = "#{test_app_path}/config/webpacker_other_location.yml"
+    env["WEBPACK_SERVE"] = "true"
+    verify_command(cmd, env: env)
+  end
+
+  private
+
+    def verify_command(cmd, use_node_modules: true, argv: [], env: Webpacker::Compiler.env)
+      cwd = Dir.pwd
+      Dir.chdir(test_app_path)
+      klass = Webpacker::DevServerRunner
+      instance = klass.new(argv)
+
+      allow(klass).to receive(:new).and_return(instance)
+      allow(instance).to receive(:node_modules_bin_exist?).and_return(use_node_modules)
+      allow(Kernel).to receive(:exec).with(env, *cmd)
+
+      klass.run(argv)
+
+      expect(Kernel).to have_received(:exec).with(env, *cmd)
+
+      ensure
+        Dir.chdir(cwd)
+    end
+end