shakapacker 6.5.4 → 7.0.2

data/lib/tasks/webpacker/compile.rake

@@ -1,42 +1,9 @@
-$stdout.sync = true
-
-def yarn_install_available?
-  rails_major = Rails::VERSION::MAJOR
-  rails_minor = Rails::VERSION::MINOR
-
-  rails_major > 5 || (rails_major == 5 && rails_minor >= 1)
-end
-
-def enhance_assets_precompile
-  # yarn:install was added in Rails 5.1
-  deps = yarn_install_available? ? [] : ["webpacker:yarn_install"]
-  Rake::Task["assets:precompile"].enhance(deps) do |task|
-    prefix = task.name.split(/#|assets:precompile/).first
-
-    Rake::Task["#{prefix}webpacker:compile"].invoke
-  end
-end
-
 namespace :webpacker do
-  desc "Compile JavaScript packs using webpack for production with digests"
-  task compile: ["webpacker:verify_install", :environment] do
-    Webpacker.with_node_env(ENV.fetch("NODE_ENV", "production")) do
-      Webpacker.ensure_log_goes_to_stdout do
-        if Webpacker.compile
-          # Successful compilation!
-        else
-          # Failed compilation
-          exit!
-        end
-      end
-    end
-  end
-end
+  desc "DEPRECATED - Compile JavaScript packs using webpack for production with digests"
+  task :compile do |task|
+    Shakapacker.puts_rake_deprecation_message(task.name)
 
-if Webpacker.config.webpacker_precompile?
-  if Rake::Task.task_defined?("assets:precompile")
-    enhance_assets_precompile
-  else
-    Rake::Task.define_task("assets:precompile" => ["webpacker:yarn_install", "webpacker:compile"])
+    prefix = task.name.split(/#|webpacker:/).first
+    Rake::Task["#{prefix}shakapacker:compile"].invoke
   end
 end

data/lib/tasks/webpacker/info.rake

@@ -1,21 +1,9 @@
-require "webpacker/version"
-
 namespace :webpacker do
-  desc "Provide information on Webpacker's environment"
-  task :info do
-    Dir.chdir(Rails.root) do
-      $stdout.puts "Ruby: #{`ruby --version`}"
-      $stdout.puts "Rails: #{Rails.version}"
-      $stdout.puts "Webpacker: #{Webpacker::VERSION}"
-      $stdout.puts "Node: #{`node --version`}"
-      $stdout.puts "Yarn: #{`yarn --version`}"
-
-      $stdout.puts "\n"
-      $stdout.puts "shakapacker: \n#{`npm list shakapacker version`}"
+  desc "DEPRECATED - Provide information on Shakapacker's environment"
+  task :info do |task|
+    Shakapacker.puts_rake_deprecation_message(task.name)
 
-      $stdout.puts "Is bin/webpacker present?: #{File.exist? 'bin/webpacker'}"
-      $stdout.puts "Is bin/webpacker-dev-server present?: #{File.exist? 'bin/webpacker-dev-server'}"
-      $stdout.puts "Is bin/yarn present?: #{File.exist? 'bin/yarn'}"
-    end
+    prefix = task.name.split(/#|webpacker:/).first
+    Rake::Task["#{prefix}shakapacker:info"].invoke
   end
 end

data/lib/tasks/webpacker/install.rake

@@ -1,17 +1,9 @@
-install_template_path = File.expand_path("../../install/template.rb", __dir__).freeze
-bin_path = ENV["BUNDLE_BIN"] || Rails.root.join("bin")
-
 namespace :webpacker do
-  desc "Install Webpacker in this application"
-  task install: [:check_node, :check_yarn] do |task|
-    Webpacker::Configuration.installing = true
-
-    prefix = task.name.split(/#|webpacker:install/).first
+  desc "DEPRECATED - Install Shakapacker in this application"
+  task :install do |task|
+    Shakapacker.puts_rake_deprecation_message(task.name)
 
-    if Rails::VERSION::MAJOR >= 5
-      exec "#{RbConfig.ruby} #{bin_path}/rails #{prefix}app:template LOCATION='#{install_template_path}'"
-    else
-      exec "#{RbConfig.ruby} #{bin_path}/rake #{prefix}rails:template LOCATION='#{install_template_path}'"
-    end
+    prefix = task.name.split(/#|webpacker:/).first
+    Rake::Task["#{prefix}shakapacker:install"].invoke
   end
 end

data/lib/tasks/webpacker/verify_config.rake

@@ -1,14 +1,9 @@
-require "webpacker/configuration"
-
 namespace :webpacker do
-  desc "Verifies if the Webpacker config is present"
-  task :verify_config do
-    unless Webpacker.config.config_path.exist?
-      path = Webpacker.config.config_path.relative_path_from(Pathname.new(pwd)).to_s
-      $stderr.puts "Configuration #{path} file not found. \n"\
-           "Make sure webpacker:install is run successfully before " \
-           "running dependent tasks"
-      exit!
-    end
+  desc "DEPRECATED - Verifies if the Shakapacker config is present"
+  task :verify_config do |task|
+    Shakapacker.puts_rake_deprecation_message(task.name)
+
+    prefix = task.name.split(/#|webpacker:/).first
+    Rake::Task["#{prefix}shakapacker:verify_config"].invoke
   end
 end

data/lib/tasks/webpacker/verify_install.rake

@@ -1,4 +1,9 @@
 namespace :webpacker do
-  desc "Verifies if Webpacker is installed"
-  task verify_install: [:verify_config, :check_node, :check_yarn, :check_binstubs]
+  desc "DEPRECATED - Verifies if Shakapacker is installed"
+  task :verify_install do |task|
+    Shakapacker.puts_rake_deprecation_message(task.name)
+
+    prefix = task.name.split(/#|webpacker:/).first
+    Rake::Task["#{prefix}shakapacker:verify_install"].invoke
+  end
 end

data/lib/tasks/webpacker/yarn_install.rake

@@ -1,24 +1,9 @@
 namespace :webpacker do
-  desc "Support for older Rails versions. Install all JavaScript dependencies as specified via Yarn"
-  task :yarn_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
+  desc "DEPRECATED - Support for older Rails versions. Install all JavaScript dependencies as specified via Yarn"
+  task :yarn_install do |task|
+    Shakapacker.puts_rake_deprecation_message(task.name)
 
-    valid_node_envs = %w[test development production]
-    node_env = ENV.fetch("NODE_ENV") do
-      valid_node_envs.include?(Rails.env) ? Rails.env : "production"
-    end
-    Dir.chdir(Rails.root) do
-      yarn_flags =
-        if `yarn --version`.start_with?("1")
-          "--no-progress --frozen-lockfile"
-        else
-          "--immutable"
-        end
-      system({ "NODE_ENV" => node_env }, "yarn install #{yarn_flags}")
-    end
+    prefix = task.name.split(/#|webpacker:/).first
+    Rake::Task["#{prefix}shakapacker:yarn_install"].invoke
   end
 end

data/lib/tasks/webpacker.rake

@@ -1,18 +1,20 @@
 tasks = {
-  "webpacker:info"                    => "Provides information on Webpacker's environment",
-  "webpacker:install"                 => "Installs and setup webpack with Yarn",
-  "webpacker:compile"                 => "Compiles webpack bundles based on environment",
-  "webpacker:clean"                   => "Remove old compiled webpacks",
-  "webpacker:clobber"                 => "Removes the webpack compiled output directory",
-  "webpacker:check_node"              => "Verifies if Node.js is installed",
-  "webpacker:check_yarn"              => "Verifies if Yarn is installed",
-  "webpacker:check_binstubs"          => "Verifies that bin/webpacker is present",
-  "webpacker:binstubs"                => "Installs Webpacker binstubs in this application",
-  "webpacker:verify_install"          => "Verifies if Webpacker is installed",
+  "webpacker:info"                    => "DEPRECATED - Provides information on Shakapacker's environment",
+  "webpacker:install"                 => "DEPRECATED - Installs and setup webpack with Yarn",
+  "webpacker:compile"                 => "DEPRECATED - Compiles webpack bundles based on environment",
+  "webpacker:clean"                   => "DEPRECATED - Remove old compiled bundles",
+  "webpacker:clobber"                 => "DEPRECATED - Removes the webpack compiled output directory",
+  "webpacker:check_node"              => "DEPRECATED - Verifies if Node.js is installed",
+  "webpacker:check_yarn"              => "DEPRECATED - Verifies if Yarn is installed",
+  "webpacker:check_binstubs"          => "DEPRECATED - Verifies that bin/shakapacker is present",
+  "webpacker:binstubs"                => "DEPRECATED - Installs Shakapacker binstubs in this application",
+  "webpacker:verify_install"          => "DEPRECATED - Verifies if Shakapacker is installed",
 }.freeze
 
-desc "Lists all available tasks in Webpacker"
-task :webpacker do
-  puts "Available Webpacker tasks are:"
+desc "DEPRECATED - Lists all available tasks in Webpacker"
+task :webpacker do |task|
+  puts "DEPRECATED - Available Webpacker tasks are:"
   tasks.each { |task, message| puts task.ljust(30) + message }
+
+  Shakapacker.puts_rake_deprecation_message(task.name)
 end

data/lib/webpacker/dev_server_runner.rb

@@ -1,96 +1,9 @@
-require "shellwords"
-require "socket"
-require "webpacker/configuration"
-require "webpacker/dev_server"
-require "webpacker/runner"
-
-module Webpacker
-  class DevServerRunner < Webpacker::Runner
-    def run
-      load_config
-      detect_unsupported_switches!
-      detect_port!
-      execute_cmd
-    end
-
-    private
-
-      def load_config
-        app_root = Pathname.new(@app_path)
-
-        @config = Configuration.new(
-          root_path: app_root,
-          config_path: Pathname.new(@webpacker_config),
-          env: ENV["RAILS_ENV"]
-        )
-
-        dev_server = DevServer.new(@config)
-
-        @hostname          = dev_server.host
-        @port              = dev_server.port
-        @pretty            = dev_server.pretty?
-        @https             = dev_server.https?
-        @hot               = dev_server.hmr?
-
-      rescue Errno::ENOENT, NoMethodError
-        $stdout.puts "webpack dev_server configuration not found in #{@config.config_path}[#{ENV["RAILS_ENV"]}]."
-        $stdout.puts "Please run bundle exec rails webpacker:install to install Webpacker"
-        exit!
-      end
-
-      UNSUPPORTED_SWITCHES = %w[--host --port]
-      private_constant :UNSUPPORTED_SWITCHES
-      def detect_unsupported_switches!
-        unsupported_switches = UNSUPPORTED_SWITCHES & @argv
-        if unsupported_switches.any?
-          $stdout.puts "The following CLI switches are not supported by Webpacker: #{unsupported_switches.join(' ')}. Please edit your command and try again."
-          exit!
-        end
-
-        if @argv.include?("--https") && !@https
-          $stdout.puts "Please set https: true in webpacker.yml to use the --https command line flag."
-          exit!
-        end
-      end
-
-      def detect_port!
-        server = TCPServer.new(@hostname, @port)
-        server.close
-
-      rescue Errno::EADDRINUSE
-        $stdout.puts "Another program is running on port #{@port}. Set a new port in #{@config.config_path} for dev_server"
-        exit!
-      end
-
-      def execute_cmd
-        env = Webpacker::Compiler.env
-        env["WEBPACKER_CONFIG"] = @webpacker_config
-        env["WEBPACK_SERVE"] = "true"
-
-        cmd = if node_modules_bin_exist?
-          ["#{@node_modules_bin_path}/webpack", "serve"]
-        else
-          ["yarn", "webpack", "serve"]
-        end
-
-        if @argv.include?("--debug-webpacker")
-          cmd = [ "node", "--inspect-brk", "--trace-warnings" ] + cmd
-          @argv.delete "--debug-webpacker"
-        end
-
-        cmd += ["--config", @webpack_config]
-        cmd += ["--progress", "--color"] if @pretty
-
-        cmd += ["--hot"] if @hot
-        cmd += @argv
-
-        Dir.chdir(@app_path) do
-          Kernel.exec env, *cmd
-        end
-      end
-
-      def node_modules_bin_exist?
-        File.exist?("#{@node_modules_bin_path}/webpack-dev-server")
-      end
-  end
-end
+# This file exists for backward compatibility
+require "shakapacker/dev_server_runner"
+
+Shakapacker.puts_deprecation_message(
+  Shakapacker.short_deprecation_message(
+    "bin/webpacker-dev-server",
+    "bin/shakapacker-dev-server"
+  )
+)

data/lib/webpacker/webpack_runner.rb

@@ -1,58 +1,9 @@
-require "shellwords"
-require "webpacker/runner"
-
-module Webpacker
-  class WebpackRunner < Webpacker::Runner
-    WEBPACK_COMMANDS = [
-      "help",
-      "h",
-      "--help",
-      "-h",
-      "version",
-      "v",
-      "--version",
-      "-v",
-      "info",
-      "i"
-    ].freeze
-
-    def run
-      env = Webpacker::Compiler.env
-      env["WEBPACKER_CONFIG"] = @webpacker_config
-
-      cmd = if node_modules_bin_exist?
-        ["#{@node_modules_bin_path}/webpack"]
-      else
-        ["yarn", "webpack"]
-      end
-
-      if @argv.delete "--debug-webpacker"
-        cmd = ["node", "--inspect-brk"] + cmd
-      end
-
-      if @argv.delete "--trace-deprecation"
-        cmd = ["node", "--trace-deprecation"] + cmd
-      end
-
-      if @argv.delete "--no-deprecation"
-        cmd = ["node", "--no-deprecation"] + cmd
-      end
-
-      # Webpack commands are not compatible with --config option.
-      if (@argv & WEBPACK_COMMANDS).empty?
-        cmd += ["--config", @webpack_config]
-      end
-
-      cmd += @argv
-
-      Dir.chdir(@app_path) do
-        Kernel.exec env, *cmd
-      end
-    end
-
-    private
-      def node_modules_bin_exist?
-        File.exist?("#{@node_modules_bin_path}/webpack")
-      end
-  end
-end
+# This file exists for backward compatibility
+require "shakapacker/webpack_runner"
+
+Shakapacker.puts_deprecation_message(
+  Shakapacker.short_deprecation_message(
+    "bin/webpacker",
+    "bin/shakapacker"
+  )
+)

data/lib/webpacker.rb

@@ -1,48 +1,4 @@
-require "active_support/core_ext/module/attribute_accessors"
-require "active_support/core_ext/string/inquiry"
-require "active_support/logger"
-require "active_support/tagged_logging"
+# This file exists for backward compatibility
+require "shakapacker"
 
-module Webpacker
-  extend self
-
-  DEFAULT_ENV = "production".freeze
-
-  def instance=(instance)
-    @instance = instance
-  end
-
-  def instance
-    @instance ||= Webpacker::Instance.new
-  end
-
-  def with_node_env(env)
-    original = ENV["NODE_ENV"]
-    ENV["NODE_ENV"] = env
-    yield
-  ensure
-    ENV["NODE_ENV"] = original
-  end
-
-  def ensure_log_goes_to_stdout
-    old_logger = Webpacker.logger
-    Webpacker.logger = Logger.new(STDOUT)
-    yield
-  ensure
-    Webpacker.logger = old_logger
-  end
-
-  delegate :logger, :logger=, :env, :inlining_css?, to: :instance
-  delegate :config, :compiler, :manifest, :commands, :dev_server, to: :instance
-  delegate :bootstrap, :clean, :clobber, :compile, to: :commands
-end
-
-require "webpacker/instance"
-require "webpacker/env"
-require "webpacker/configuration"
-require "webpacker/manifest"
-require "webpacker/compiler"
-require "webpacker/commands"
-require "webpacker/dev_server"
-
-require "webpacker/railtie" if defined?(Rails)
+Shakapacker.puts_deprecation_message(Shakapacker::DEPRECATION_MESSAGE)

data/package/__tests__/config-bc.js

@@ -0,0 +1,27 @@
+/* global test expect, describe */
+const { resolve } = require('path')
+const { chdirWebpackerTestApp, resetEnv } = require('../utils/helpers')
+
+const rootPath = process.cwd()
+chdirWebpackerTestApp()
+
+const config = require('../config')
+
+describe('Backward Compatibility - Config', () => { 
+  beforeEach(() => jest.resetModules() && resetEnv())
+  afterAll(() => process.chdir(rootPath))
+
+  test('x public path with asset host', () => {
+    process.env.RAILS_ENV = 'development'
+    process.env.WEBPACKER_ASSET_HOST = 'http://foo.com/'
+    const config = require('../config')
+
+    expect(config.publicPath).toEqual('http://foo.com/packs/')
+  })
+
+  test('x should allow overriding manifestPath', () => {
+    process.env.WEBPACKER_CONFIG = 'config/webpacker_manifest_path.yml'
+    const config = require('../config')
+    expect(config.manifestPath).toEqual(resolve('app/packs/manifest.json'))
+  })
+})

data/package/__tests__/config.js

@@ -1,15 +1,16 @@
 /* global test expect, describe */
 
-const { chdirCwd, chdirTestApp, resetEnv } = require('../utils/helpers')
+const { chdirTestApp, resetEnv } = require('../utils/helpers')
 const { resolve } = require('path')
 
+const rootPath = process.cwd()
 chdirTestApp()
 
 const config = require('../config')
 
 describe('Config', () => {
   beforeEach(() => jest.resetModules() && resetEnv())
-  afterAll(chdirCwd)
+  afterAll(() => process.chdir(rootPath))
 
   test('public path', () => {
     process.env.RAILS_ENV = 'development'
@@ -19,7 +20,7 @@ describe('Config', () => {
 
   test('public path with asset host', () => {
     process.env.RAILS_ENV = 'development'
-    process.env.WEBPACKER_ASSET_HOST = 'http://foo.com/'
+    process.env.SHAKAPACKER_ASSET_HOST = 'http://foo.com/'
     const config = require('../config')
     expect(config.publicPath).toEqual('http://foo.com/packs/')
   })
@@ -38,8 +39,8 @@ describe('Config', () => {
   })
 
   test('should allow overriding manifestPath', () => {
-    process.env.WEBPACKER_CONFIG = 'config/webpacker_manifest_path.yml'
+    process.env.SHAKAPACKER_CONFIG = 'config/shakapacker_manifest_path.yml'
     const config = require('../config')
-    expect(config.manifestPath).toEqual(resolve('app/packs/manifest.json'))
+    expect(config.manifestPath).toEqual(resolve('app/javascript/manifest.json'))
   })
 })

data/package/__tests__/dev_server-bc.js

@@ -0,0 +1,46 @@
+/* global test expect, describe */
+
+const { chdirWebpackerTestApp } = require('../utils/helpers')
+
+const rootPath = process.cwd()
+chdirWebpackerTestApp()
+
+describe('Backward Compatibility - DevServer', () => {
+  beforeEach(() => jest.resetModules())
+  afterAll(() => process.chdir(rootPath))
+
+  test('with NODE_ENV and RAILS_ENV set to development', () => {
+    process.env.NODE_ENV = 'development'
+    process.env.RAILS_ENV = 'development'
+    process.env.WEBPACKER_DEV_SERVER_HOST = '0.0.0.0'
+    process.env.WEBPACKER_DEV_SERVER_PORT = 5000
+    process.env.WEBPACKER_DEV_SERVER_DISABLE_HOST_CHECK = false
+
+    const devServer = require('../dev_server')
+    expect(devServer).toBeDefined()
+    expect(devServer.host).toEqual('0.0.0.0')
+    expect(devServer.port).toEqual('5000')
+    expect(devServer.disable_host_check).toBe(false)
+  })
+
+  test('with custom env prefix', () => {
+    const config = require('../config')
+    config.dev_server.env_prefix = 'TEST_WEBPACKER_DEV_SERVER'
+
+    process.env.NODE_ENV = 'development'
+    process.env.RAILS_ENV = 'development'
+    process.env.TEST_WEBPACKER_DEV_SERVER_HOST = '0.0.0.0'
+    process.env.TEST_WEBPACKER_DEV_SERVER_PORT = 5000
+
+    const devServer = require('../dev_server')
+    expect(devServer).toBeDefined()
+    expect(devServer.host).toEqual('0.0.0.0')
+    expect(devServer.port).toEqual('5000')
+  })
+
+  test('with NODE_ENV and RAILS_ENV set to production', () => {
+    process.env.RAILS_ENV = 'production'
+    process.env.NODE_ENV = 'production'
+    expect(require('../dev_server')).toEqual({})
+  })
+})

data/package/__tests__/dev_server.js

@@ -1,19 +1,20 @@
 /* global test expect, describe */
 
-const { chdirTestApp, chdirCwd } = require('../utils/helpers')
+const { chdirTestApp } = require('../utils/helpers')
 
+const rootPath = process.cwd()
 chdirTestApp()
 
 describe('DevServer', () => {
   beforeEach(() => jest.resetModules())
-  afterAll(chdirCwd)
+  afterAll(() => process.chdir(rootPath))
 
   test('with NODE_ENV and RAILS_ENV set to development', () => {
     process.env.NODE_ENV = 'development'
     process.env.RAILS_ENV = 'development'
-    process.env.WEBPACKER_DEV_SERVER_HOST = '0.0.0.0'
-    process.env.WEBPACKER_DEV_SERVER_PORT = 5000
-    process.env.WEBPACKER_DEV_SERVER_DISABLE_HOST_CHECK = false
+    process.env.SHAKAPACKER_DEV_SERVER_HOST = '0.0.0.0'
+    process.env.SHAKAPACKER_DEV_SERVER_PORT = 5000
+    process.env.SHAKAPACKER_DEV_SERVER_DISABLE_HOST_CHECK = false
 
     const devServer = require('../dev_server')
     expect(devServer).toBeDefined()
@@ -24,12 +25,12 @@ describe('DevServer', () => {
 
   test('with custom env prefix', () => {
     const config = require('../config')
-    config.dev_server.env_prefix = 'TEST_WEBPACKER_DEV_SERVER'
+    config.dev_server.env_prefix = 'TEST_SHAKAPACKER_DEV_SERVER'
 
     process.env.NODE_ENV = 'development'
     process.env.RAILS_ENV = 'development'
-    process.env.TEST_WEBPACKER_DEV_SERVER_HOST = '0.0.0.0'
-    process.env.TEST_WEBPACKER_DEV_SERVER_PORT = 5000
+    process.env.TEST_SHAKAPACKER_DEV_SERVER_HOST = '0.0.0.0'
+    process.env.TEST_SHAKAPACKER_DEV_SERVER_PORT = 5000
 
     const devServer = require('../dev_server')
     expect(devServer).toBeDefined()

data/package/__tests__/development-bc.js

@@ -0,0 +1,66 @@
+/* test expect, describe, afterAll, beforeEach */
+
+const { resolve } = require('path')
+const { chdirWebpackerTestApp, resetEnv } = require('../utils/helpers')
+
+const rootPath = process.cwd()
+chdirWebpackerTestApp()
+
+describe('Backward Compatibility - Development environment', () => {
+  beforeEach(() => jest.resetModules() && resetEnv())
+  afterAll(() => process.chdir(rootPath))
+
+  describe('generateWebpackConfig', () => {
+    beforeEach(() => jest.resetModules())
+
+    test('should use development config and environment including devServer if WEBPACK_SERVE', () => {
+      process.env.RAILS_ENV = 'development'
+      process.env.NODE_ENV = 'development'
+      process.env.WEBPACK_SERVE = 'true'
+      const { generateWebpackConfig } = require('../index')
+
+      const webpackConfig = generateWebpackConfig()
+
+      expect(webpackConfig.output.path).toEqual(resolve('public', 'packs'))
+      expect(webpackConfig.output.publicPath).toEqual('/packs/')
+    })
+
+    test('should use development config and environment if WEBPACK_SERVE', () => {
+      process.env.RAILS_ENV = 'development'
+      process.env.NODE_ENV = 'development'
+      process.env.WEBPACK_SERVE = undefined
+      const { generateWebpackConfig } = require('../index')
+
+      const webpackConfig = generateWebpackConfig()
+
+      expect(webpackConfig.output.path).toEqual(resolve('public', 'packs'))
+      expect(webpackConfig.output.publicPath).toEqual('/packs/')
+      expect(webpackConfig.devServer).toEqual(undefined)
+    })
+  })
+
+  describe('globalMutableWebpackConfig', () => {
+    beforeEach(() => jest.resetModules())
+
+    test('should use development config and environment including devServer if WEBPACK_SERVE', () => {
+      process.env.RAILS_ENV = 'development'
+      process.env.NODE_ENV = 'development'
+      process.env.WEBPACK_SERVE = 'true'
+      const { globalMutableWebpackConfig: webpackConfig } = require('../index')
+
+      expect(webpackConfig.output.path).toEqual(resolve('public', 'packs'))
+      expect(webpackConfig.output.publicPath).toEqual('/packs/')
+    })
+
+    test('should use development config and environment if WEBPACK_SERVE', () => {
+      process.env.RAILS_ENV = 'development'
+      process.env.NODE_ENV = 'development'
+      process.env.WEBPACK_SERVE = undefined
+      const { globalMutableWebpackConfig: webpackConfig } = require('../index')
+
+      expect(webpackConfig.output.path).toEqual(resolve('public', 'packs'))
+      expect(webpackConfig.output.publicPath).toEqual('/packs/')
+      expect(webpackConfig.devServer).toEqual(undefined)
+    })
+  })
+})