vite_rails 1.0.5 → 1.0.10

data/test/test_app/app/frontend/entrypoints/application.js

@@ -0,0 +1,2 @@
+/* eslint no-console:0 */
+console.log('Hello World from ViteRails')

data/test/test_app/config/vite.json

@@ -10,11 +10,9 @@
     "publicOutputDir": "vite-test"
   },
   "production": {
-    "autoBuild": false,
     "publicOutputDir": "vite-production"
   },
   "staging": {
-    "autoBuild": false,
     "publicOutputDir": "vite-staging"
   }
 }

data/test/test_app/config/vite_additional_paths.json

@@ -0,0 +1,5 @@
+{
+  "all": {
+    "watchAdditionalPaths": ["config/*"]
+  }
+}

data/test/test_app/config/vite_public_dir.json

@@ -0,0 +1,5 @@
+{
+  "all": {
+    "publicDir": "../public"
+  }
+}

data/test/test_app/public/vite-production/manifest.json

@@ -0,0 +1,22 @@
+{
+  "application.js": {
+    "file": "assets/application.a0ba047e.js",
+    "imports": [
+      "assets/example_import.8e1fddc0.js"
+    ]
+  },
+  "example_import.js": {
+    "file": "assets/example_import.8e1fddc0.js",
+    "imports": []
+  },
+  "colored.js": {
+    "file": "assets/colored.1173bfe0.js",
+    "imports": []
+  },
+  "application.css": {
+    "file": "assets/application.cccfef34.css"
+  },
+  "colored.css": {
+    "file": "assets/colored.84277fd6.css"
+  }
+}

data/test/test_helper.rb

@@ -1,34 +1,68 @@
 # frozen_string_literal: true
 
+require 'simplecov'
+SimpleCov.start {
+  add_filter '/test/'
+}
+
 require 'minitest/autorun'
 require 'rails'
 require 'rails/test_help'
-require 'byebug'
+require 'pry-byebug'
 
 require_relative 'test_app/config/environment'
 
 Rails.env = 'production'
 
-ViteRails.instance = ::ViteRails.new
+ViteRails.instance = ViteRails.new
 
-class ViteRails::Test < Minitest::Test
-private
-
-  def reloaded_config
-    ViteRails.instance.instance_variable_set(:@config, nil)
-    ViteRails.instance.instance_variable_set(:@dev_server, nil)
-    ViteRails.env = {}
-    ViteRails.config
-    ViteRails.dev_server
+module ViteRailsTestHelpers
+  def refresh_config(env_variables = ViteRails.load_env_variables)
+    ViteRails.env = env_variables
+    (ViteRails.instance = ViteRails.new).config
   end
 
   def with_rails_env(env)
     original = Rails.env
     Rails.env = ActiveSupport::StringInquirer.new(env)
-    reloaded_config
-    yield
+    yield(refresh_config)
   ensure
     Rails.env = ActiveSupport::StringInquirer.new(original)
-    reloaded_config
+    refresh_config
+  end
+
+  def test_app_path
+    File.expand_path('test_app', __dir__)
+  end
+
+  def with_dev_server_running(&block)
+    ViteRails.instance.stub(:dev_server_running?, true, &block)
+  end
+end
+
+class ViteRails::Test < Minitest::Test
+  include ViteRailsTestHelpers
+
+private
+
+  def assert_run_command(*argv, use_yarn: false, flags: [])
+    command = use_yarn ? %w[yarn vite] : ["#{ test_app_path }/node_modules/.bin/vite"]
+    cwd = Dir.pwd
+    Dir.chdir(test_app_path)
+
+    klass = ViteRails::Runner
+    instance = klass.new(argv)
+    mock = Minitest::Mock.new
+    mock.expect(:call, nil, [ViteRails.env, *command, *argv, *flags])
+
+    klass.stub(:new, instance) do
+      instance.stub(:executable_exists?, !use_yarn) do
+        Kernel.stub(:exec, mock) { ViteRails.run(argv) }
+      end
+    end
+
+    mock.verify
+  ensure
+    Dir.chdir(cwd)
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: vite_rails
 version: !ruby/object:Gem::Version
-  version: 1.0.5
+  version: 1.0.10
 platform: ruby
 authors:
 - Máximo Mussini
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-01-20 00:00:00.000000000 Z
+date: 2021-01-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -147,9 +147,9 @@ files:
 - package.json
 - package/default.vite.json
 - test/builder_test.rb
-- test/command_test.rb
-- test/configuration_test.rb
-- test/dev_server_runner_test.rb
+- test/commands_test.rb
+- test/config_test.rb
+- test/dev_server_proxy_test.rb
 - test/dev_server_test.rb
 - test/engine_rake_tasks_test.rb
 - test/helper_test.rb
@@ -164,30 +164,29 @@ files:
 - test/mounted_app/test/dummy/config/environment.rb
 - test/mounted_app/test/dummy/config/vite.json
 - test/mounted_app/test/dummy/package.json
+- test/mounted_app/test/dummy/yarn.lock
 - test/rake_tasks_test.rb
+- test/runner_test.rb
 - test/test_app/Rakefile
-- test/test_app/app/javascript/entrypoints/application.js
-- test/test_app/app/javascript/entrypoints/multi_entry.css
-- test/test_app/app/javascript/entrypoints/multi_entry.js
+- test/test_app/app/frontend/entrypoints/application.js
 - test/test_app/bin/vite
 - test/test_app/config.ru
 - test/test_app/config/application.rb
 - test/test_app/config/environment.rb
 - test/test_app/config/vite.json
-- test/test_app/config/vite_public_root.yml
+- test/test_app/config/vite_additional_paths.json
+- test/test_app/config/vite_public_dir.json
 - test/test_app/package.json
-- test/test_app/public/vite/manifest.json
+- test/test_app/public/vite-production/manifest.json
 - test/test_app/some.config.js
 - test/test_app/yarn.lock
 - test/test_helper.rb
-- test/vite_runner_test.rb
-- test/webpacker_test.rb
 homepage: https://github.com/ElMassimo/vite_rails
 licenses:
 - MIT
 metadata:
-  source_code_uri: https://github.com/ElMassimo/vite_rails/tree/v1.0.5
-  changelog_uri: https://github.com/ElMassimo/vite_rails/blob/v1.0.5/CHANGELOG.md
+  source_code_uri: https://github.com/ElMassimo/vite_rails/tree/v1.0.10
+  changelog_uri: https://github.com/ElMassimo/vite_rails/blob/v1.0.10/CHANGELOG.md
 post_install_message:
 rdoc_options: []
 require_paths:
@@ -196,7 +195,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.4.0
+      version: 2.5.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -209,9 +208,9 @@ specification_version: 4
 summary: Use Vite in Rails and bring joy to your JavaScript experience
 test_files:
 - test/builder_test.rb
-- test/command_test.rb
-- test/configuration_test.rb
-- test/dev_server_runner_test.rb
+- test/commands_test.rb
+- test/config_test.rb
+- test/dev_server_proxy_test.rb
 - test/dev_server_test.rb
 - test/engine_rake_tasks_test.rb
 - test/helper_test.rb
@@ -226,21 +225,20 @@ test_files:
 - test/mounted_app/test/dummy/config/environment.rb
 - test/mounted_app/test/dummy/config/vite.json
 - test/mounted_app/test/dummy/package.json
+- test/mounted_app/test/dummy/yarn.lock
 - test/rake_tasks_test.rb
+- test/runner_test.rb
 - test/test_app/Rakefile
-- test/test_app/app/javascript/entrypoints/application.js
-- test/test_app/app/javascript/entrypoints/multi_entry.css
-- test/test_app/app/javascript/entrypoints/multi_entry.js
+- test/test_app/app/frontend/entrypoints/application.js
 - test/test_app/bin/vite
 - test/test_app/config.ru
 - test/test_app/config/application.rb
 - test/test_app/config/environment.rb
 - test/test_app/config/vite.json
-- test/test_app/config/vite_public_root.yml
+- test/test_app/config/vite_additional_paths.json
+- test/test_app/config/vite_public_dir.json
 - test/test_app/package.json
-- test/test_app/public/vite/manifest.json
+- test/test_app/public/vite-production/manifest.json
 - test/test_app/some.config.js
 - test/test_app/yarn.lock
 - test/test_helper.rb
-- test/vite_runner_test.rb
-- test/webpacker_test.rb

data/test/command_test.rb

@@ -1,35 +0,0 @@
-# frozen_string_literal: true
-
-require 'test_helper'
-
-class CommandTest < Minitest::Test
-  def test_compile_command_returns_success_status_when_stale
-    ViteRails.builder.stub :stale?, true do
-      ViteRails.builder.stub :run_vite, true do
-        assert_equal true, ViteRails.commands.compile
-      end
-    end
-  end
-
-  def test_compile_command_returns_success_status_when_fresh
-    ViteRails.builder.stub :stale?, false do
-      ViteRails.builder.stub :run_vite, true do
-        assert_equal true, ViteRails.commands.compile
-      end
-    end
-  end
-
-  def test_compile_command_returns_failure_status_when_stale
-    ViteRails.builder.stub :stale?, true do
-      ViteRails.builder.stub :run_vite, false do
-        assert_equal false, ViteRails.commands.compile
-      end
-    end
-  end
-
-  def test_clean_command_works_with_nested_hashes_and_without_any_compiled_files
-    File.stub :delete, true do
-      assert ViteRails.commands.clean
-    end
-  end
-end

data/test/configuration_test.rb

@@ -1,80 +0,0 @@
-# frozen_string_literal: true
-
-require 'test_helper'
-
-class ConfigurationTest < ViteRails::Test
-  def setup
-    @config = ViteRails::Configuration.new(
-      root_path: Pathname.new(File.expand_path('test_app', __dir__)),
-      config_path: Pathname.new(File.expand_path('./test_app/config/vite.yml', __dir__)),
-      env: 'production',
-    )
-  end
-
-  def test_source_path
-    source_path = File.expand_path File.join(File.dirname(__FILE__), 'test_app/app/javascript').to_s
-    assert_equal source_path, @config.source_path.to_s
-  end
-
-  def test_source_entry_path
-    source_entry_path = File.expand_path File.join(File.dirname(__FILE__), 'test_app/app/javascript', 'packs').to_s
-    assert_equal @config.source_entry_path.to_s, source_entry_path
-  end
-
-  def test_public_root_path
-    public_root_path = File.expand_path File.join(File.dirname(__FILE__), 'test_app/public').to_s
-    assert_equal @config.public_path.to_s, public_root_path
-  end
-
-  def test_public_output_path
-    public_output_path = File.expand_path File.join(File.dirname(__FILE__), 'test_app/public/packs').to_s
-    assert_equal @config.public_output_path.to_s, public_output_path
-
-    @config = ViteRails::Configuration.new(
-      root_path: @config.root_path,
-      config_path: Pathname.new(File.expand_path('./test_app/config/vite_public_root.yml', __dir__)),
-      env: 'production',
-    )
-
-    public_output_path = File.expand_path File.join(File.dirname(__FILE__), 'public/packs').to_s
-    assert_equal @config.public_output_path.to_s, public_output_path
-  end
-
-  def test_public_manifest_path
-    public_manifest_path = File.expand_path File.join(File.dirname(__FILE__), 'test_app/public/packs', 'manifest.json').to_s
-    assert_equal @config.public_manifest_path.to_s, public_manifest_path
-  end
-
-  def test_cache_path
-    cache_path = File.expand_path File.join(File.dirname(__FILE__), 'test_app/tmp/cache/vite').to_s
-    assert_equal @config.cache_path.to_s, cache_path
-  end
-
-  def test_additional_paths
-    assert_equal @config.additional_paths, ['app/assets', '/etc/yarn', 'some.config.js', 'app/elm']
-  end
-
-  def test_cache_manifest?
-    assert @config.cache_manifest?
-
-    with_rails_env('development') do
-      refute ViteRails.config.cache_manifest?
-    end
-
-    with_rails_env('test') do
-      refute ViteRails.config.cache_manifest?
-    end
-  end
-
-  def test_compile?
-    refute @config.compile?
-
-    with_rails_env('development') do
-      assert ViteRails.config.compile?
-    end
-
-    with_rails_env('test') do
-      assert ViteRails.config.compile?
-    end
-  end
-end

data/test/dev_server_runner_test.rb

@@ -1,83 +0,0 @@
-# frozen_string_literal: true
-
-require 'test_helper'
-
-class DevServerRunnerTest < ViteRails::Test
-  def setup
-    @original_node_env, ENV['NODE_ENV'] = ENV['NODE_ENV'], 'development'
-    @original_rails_env, ENV['RAILS_ENV'] = ENV['RAILS_ENV'], 'development'
-  end
-
-  def teardown
-    ENV['NODE_ENV'] = @original_node_env
-    ENV['RAILS_ENV'] = @original_rails_env
-  end
-
-  def test_run_cmd_via_node_modules
-    cmd = ["#{ test_app_path }/node_modules/.bin/vite", '--mode', 'development']
-
-    verify_command(cmd, use_node_modules: true)
-  end
-
-  def test_run_cmd_via_yarn
-    cmd = ['yarn', 'vite', '--mode', 'development']
-
-    verify_command(cmd, use_node_modules: false)
-  end
-
-  def test_run_cmd_argv
-    cmd = ["#{ test_app_path }/node_modules/.bin/vite", '--quiet', '--mode', 'development']
-
-    verify_command(cmd, argv: ['--quiet'])
-  end
-
-  def test_run_cmd_argv_with_https
-    cmd = ["#{ test_app_path }/node_modules/.bin/vite", '--https', '--mode', 'development']
-
-    dev_server = ViteRails::DevServer.new({})
-    def dev_server.host
-      'localhost'
-    end
-
-    def dev_server.port
-      '3035'
-    end
-
-    def dev_server.pretty?
-      false
-    end
-
-    def dev_server.https?
-      true
-    end
-    ViteRails::DevServer.stub(:new, dev_server) do
-      verify_command(cmd, argv: ['--https'])
-    end
-  end
-
-private
-
-  def test_app_path
-    File.expand_path('test_app', __dir__)
-  end
-
-  def verify_command(cmd, use_node_modules: true, argv: [])
-    cwd = Dir.pwd
-    Dir.chdir(test_app_path)
-
-    klass = ViteRails::Runner
-    instance = klass.new(argv)
-    mock = Minitest::Mock.new
-    mock.expect(:call, nil, [ViteRails.env, *cmd])
-
-    klass.stub(:new, instance) do
-      instance.stub(:executable_exists?, use_node_modules) do
-        Kernel.stub(:exec, mock) { ViteRails.run(argv) }
-      end
-    end
-
-    mock.verify
-  ensure
-    Dir.chdir(cwd)
-  end
-end