Ensure manifest_path is read from config properly by Assembly (#203)

rails · Sep 8, 2024 · f1761f2 · f1761f2
1 parent f06b3f9
commit f1761f2
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/lib/propshaft/assembly.rb b/lib/propshaft/assembly.rb
@@ -32,7 +32,7 @@ def server
 
   def processor
     Propshaft::Processor.new \
-      load_path: load_path, output_path: config.output_path, compilers: compilers, manifest_path: config.manifest_path.manifest_path
+      load_path: load_path, output_path: config.output_path, compilers: compilers, manifest_path: config.manifest_path
   end
 
   def compilers

diff --git a/test/propshaft/assembly_test.rb b/test/propshaft/assembly_test.rb
@@ -33,4 +33,14 @@ class Propshaft::AssemblyTest < ActiveSupport::TestCase
     assert_equal assembly.resolver.object_id, assembly.resolver.object_id
     assert_equal assembly.load_path.object_id, assembly.load_path.object_id
   end
+
+  test "instantiates a valid processor" do
+    assembly = Propshaft::Assembly.new(ActiveSupport::OrderedOptions.new.tap { |config|
+      config.output_path = Pathname.new("#{__dir__}/../fixtures/assets")
+      config.manifest_path = config.output_path.join(".manifest.json")
+      config.prefix = "/assets"
+    })
+
+    assert assembly.processor.is_a?(Propshaft::Processor)
+  end
 end