condenser 1.2 → 1.4

data/lib/condenser/transformers/svg_transformer/template.rb

@@ -0,0 +1,151 @@
+class Condenser::SVGTransformer::Template
+  
+  include Condenser::ParseHelpers
+
+  attr_accessor :source
+
+  START_TAGS = ['<']
+  CLOSE_TAGS = ['/>', '>']
+  VOID_ELEMENTS = ['!DOCTYPE', '?xml']
+  
+  def initialize(source)
+    @source = source.strip
+    process
+  end
+
+  def process
+    seek(0)
+    @tree =   [Condenser::SVGTransformer::Base.new]
+    @stack =  [:str]
+    
+    while !eos?
+      case @stack.last
+      when :str
+        scan_until(Regexp.new("(#{START_TAGS.map{|s| Regexp.escape(s) }.join('|')}|\\z)"))
+        if !matched.nil? && START_TAGS.include?(matched)
+          @stack << :tag
+        end
+      when :tag
+        scan_until(Regexp.new("(\\/|[^\\s>]+)"))
+        if matched == '/'
+          @stack.pop
+          @stack << :close_tag
+        else
+          @tree << Condenser::SVGTransformer::Tag.new(matched)
+          @stack << :tag_attr_key
+        end
+      when :close_tag
+        scan_until(Regexp.new("([^\\s>]+)"))
+
+        el = @tree.pop
+        if el.tag_name != matched
+          raise Condenser::SVGTransformer::TemplateError.new("Expected to close #{el.tag_name.inspect} tag, instead closed #{matched.inspect}\n#{cursor}")
+        end
+        if !['!DOCTYPE', '?xml'].include?(el.tag_name)
+          @tree.last.children << el
+          scan_until(Regexp.new("(#{CLOSE_TAGS.map{|s| Regexp.escape(s) }.join('|')})"))
+          @stack.pop
+        end
+      when :tag_attr_key
+        scan_until(Regexp.new("(#{CLOSE_TAGS.map{|s| Regexp.escape(s) }.join('|')}|[^\\s=>]+)"))
+        if CLOSE_TAGS.include?(matched)
+          if matched == '/>' || VOID_ELEMENTS.include?(@tree.last.tag_name)
+            el = @tree.pop
+            @tree.last.children << el
+            @stack.pop
+            @stack.pop
+          else
+            @stack << :str
+          end
+        else
+          key = if matched.start_with?('"') && matched.end_with?('"')
+            matched[1..-2]
+          elsif matched.start_with?('"') && matched.end_with?('"')
+            matched[1..-2]
+          else
+            matched
+          end
+          @tree.last.attrs << key
+          @stack << :tag_attr_value_tx
+        end
+      when :tag_attr_value_tx
+        scan_until(Regexp.new("(#{(CLOSE_TAGS).map{|s| Regexp.escape(s) }.join('|')}|=|\\S)"))
+        tag_key = @tree.last.attrs.pop
+        if CLOSE_TAGS.include?(matched)
+          el = @tree.last
+          el.attrs << tag_key
+          if VOID_ELEMENTS.include?(el.tag_name)
+            @tree.pop
+            @tree.last.children << el
+          end
+          @stack.pop
+          @stack.pop
+          @stack.pop
+        elsif matched == '='
+          @stack.pop
+          @tree.last.attrs << tag_key
+          @stack << :tag_attr_value
+        else
+          @stack.pop
+          @tree.last.attrs << tag_key
+          rewind(1)
+        end
+
+      when :tag_attr_value
+        scan_until(Regexp.new("(#{CLOSE_TAGS.map{|s| Regexp.escape(s) }.join('|')}|'|\"|\\S+)"))
+
+        if matched == '"'
+          @stack.pop
+          @stack << :tag_attr_value_double_quoted
+        elsif matched == "'"
+          @stack.pop
+          @stack << :tag_attr_value_single_quoted
+        else
+          @stack.pop
+          key = @tree.last.attrs.pop
+          @tree.last.namespace = matched if key == 'xmlns'
+          @tree.last.attrs << { key => matched }
+        end
+      when :tag_attr_value_double_quoted
+        quoted_value = ''
+        scan_until(/"/)
+        quoted_value << pre_match if !pre_match.strip.empty?
+        rewind(1)
+
+        quoted_value = Condenser::SVGTransformer::Value.new(quoted_value)
+
+        key = @tree.last.attrs.pop
+        @tree.last.namespace = quoted_value if key == 'xmlns'
+        if @tree.last.attrs.last.is_a?(Hash) && !@tree.last.attrs.last.has_key?(key)
+          @tree.last.attrs.last[key] = quoted_value
+        else
+          @tree.last.attrs << { key => quoted_value }
+        end
+        scan_until(/\"/)
+        @stack.pop
+      when :tag_attr_value_single_quoted
+        quoted_value = ''
+        scan_until(/(')/)
+        quoted_value << pre_match if !pre_match.strip.empty?
+        rewind(1)
+
+        quoted_value = Condenser::SVGTransformer::Value.new(quoted_value)
+
+        key = @tree.last.attrs.pop
+        @tree.last.namespace = quoted_value if key == 'xmlns'
+        if @tree.last.attrs.last.is_a?(Hash) && !@tree.last.attrs.last.has_key?(key)
+          @tree.last.attrs.last[key] = quoted_value
+        else
+          @tree.last.attrs << { key => quoted_value }
+        end
+        scan_until(/\'/)
+        @stack.pop
+      end
+    end
+  end
+
+  def to_module
+    @tree.first.to_module
+  end
+
+end

data/lib/condenser/transformers/svg_transformer/template_error.rb

@@ -0,0 +1,2 @@
+class Condenser::SVGTransformer::TemplateError < StandardError
+end

data/lib/condenser/transformers/svg_transformer/value.rb

@@ -0,0 +1,13 @@
+require 'json'
+
+class Condenser::SVGTransformer::Value
+
+  def initialize(value)
+    @value = value
+  end
+
+  def to_js
+    JSON.generate(@value)
+  end
+  
+end

data/lib/condenser/transformers/svg_transformer/var_generator.rb

@@ -0,0 +1,10 @@
+class Condenser::SVGTransformer::VarGenerator
+  def initialize
+    @current = nil
+  end
+  
+  def next
+    @current = @current.nil? ? '__a' : @current.next
+    @current
+  end
+end

data/lib/condenser/transformers/svg_transformer.rb

@@ -0,0 +1,19 @@
+class Condenser::SVGTransformer
+  
+  autoload :Base, File.expand_path('../svg_transformer/base', __FILE__)
+  autoload :Tag, File.expand_path('../svg_transformer/tag', __FILE__)
+  autoload :Template, File.expand_path('../svg_transformer/template', __FILE__)
+  autoload :TemplateError, File.expand_path('../svg_transformer/template_error', __FILE__)
+  autoload :Value, File.expand_path('../svg_transformer/value', __FILE__)
+  autoload :VarGenerator, File.expand_path('../svg_transformer/var_generator', __FILE__)
+  
+  def self.setup(env)
+  end
+
+  def self.call(environment, input)
+    input[:source] = Condenser::SVGTransformer::Template.new(input[:source]).to_module
+  end
+
+end
+
+

data/lib/condenser/version.rb

@@ -1,3 +1,3 @@
 class Condenser
-  VERSION = '1.2'
+  VERSION = '1.4'
 end

data/lib/condenser.rb

@@ -19,19 +19,25 @@ class Condenser
   autoload :RollupProcessor,  'condenser/processors/rollup_processor'
   autoload :JSAnalyzer,       'condenser/processors/js_analyzer'
   autoload :PurgeCSSProcessor,'condenser/processors/purgecss_processor'
+  autoload :CSSMediaCombinerProcessor,'condenser/processors/css_media_combiner_processor'
   autoload :NodeProcessor,    'condenser/processors/node_processor'
   autoload :UglifyMinifier,   'condenser/minifiers/uglify_minifier'
   autoload :TerserMinifier,   'condenser/minifiers/terser_minifier'
   autoload :Erubi,            'condenser/templating_engine/erb'
+  autoload :Sass,             'condenser/transformers/sass'
   autoload :SassMinifier,     'condenser/minifiers/sass_minifier'
-  autoload :SassTransformer,  'condenser/transformers/sass_transformer'
+  autoload :DartSassTransformer, 'condenser/transformers/dart_sass_transformer'
+  autoload :DartScssTransformer, 'condenser/transformers/dart_sass_transformer'
+  autoload :SassTransformer,     'condenser/transformers/sass_transformer'
   autoload :ScssTransformer,  'condenser/transformers/sass_transformer'
-  autoload :EjsTemplare,      'condenser/templating_engine/ejs'
+  autoload :EjsTemplate,      'condenser/templating_engine/ejs'
   autoload :JstTransformer,   'condenser/transformers/jst_transformer'
+  autoload :SVGTransformer,   'condenser/transformers/svg_transformer'
   autoload :FileWriter,       'condenser/writers/file_writer'
   autoload :ZlibWriter,       'condenser/writers/zlib_writer'
   autoload :BrotliWriter,     'condenser/writers/brotli_writer'
   autoload :BuildCache,       'condenser/build_cache'
+  autoload :ParseHelpers,     'condenser/helpers/parse_helpers'
   
   def self.configure(&block)
     instance_eval(&block)
@@ -56,8 +62,8 @@ class Condenser
       configure(&block)
     elsif pipeline != false
       self.configure do
-        # register_preprocessor 'application/javascript', Condenser::JSAnalyzer
-        register_preprocessor 'application/javascript', Condenser::BabelProcessor
+        register_preprocessor 'application/javascript', Condenser::JSAnalyzer
+        # register_preprocessor 'application/javascript', Condenser::BabelProcessor
         register_exporter     'application/javascript', Condenser::RollupProcessor
         register_minifier     'application/javascript', Condenser::UglifyMinifier
         
@@ -132,6 +138,9 @@ Condenser.configure do
   # CSS
   register_mime_type 'text/css', extension: '.css', charset: :css
   
+  # PDF
+  register_mime_type 'application/pdf', extensions: %w(.pdf)
+  
   # SASS
   register_mime_type 'text/sass', extensions: %w(.sass .css.sass)
   # register_transformer  'text/sass', 'text/css', SassProcessor
@@ -145,12 +154,15 @@ Condenser.configure do
   
   # EJS
   register_mime_type 'application/ejs', extensions: '.ejs', charset: :unicode
-  register_template  'application/ejs', Condenser::EjsTemplare
+  register_template  'application/ejs', Condenser::EjsTemplate
   
   # JST
   register_mime_type    'application/jst', extensions: '.jst', charset: :unicode
   register_transformer  'application/jst', 'application/javascript', Condenser::JstTransformer
   
+  # SVG
+  register_transformer  'image/svg+xml', 'application/javascript', Condenser::SVGTransformer
+  
   # Writers
   register_mime_type 'application/gzip',    extensions: %w(.gz .gzip)
   register_mime_type 'application/brotli',  extension: %w(.br)

data/test/cache_test.rb

@@ -60,6 +60,19 @@ class CacheTest < ActiveSupport::TestCase
     CSS
   end
 
+  test 'changing a source file on calls needs_reprocessing! and needs_reexporting! once' do
+    file 'test.txt.erb', "1<%= 1 + 1 %>3\n"
+    assert_file 'test.txt', 'text/plain', <<~CSS
+    123
+    CSS
+
+    asset = @env.find('test.txt')
+
+    asset.expects(:needs_reprocessing!).once
+    asset.expects(:needs_reexporting!).once
+    file 'test.txt.erb', "1<%= 1 + 2 %>5\n"
+  end
+
   test 'changing a js dependency reflects in the next call' do
     file 'main.js', <<-JS
       import { cube } from 'math';
@@ -74,11 +87,11 @@ class CacheTest < ActiveSupport::TestCase
     JS
 
     assert_exported_file 'main.js', 'application/javascript', <<~CSS
-      !function(){"use strict";var o;console.log((o=5)*o)}();
+      !function(){var o;console.log((o=5)*o)}();
     CSS
 
     assert_exported_file 'main.js', 'application/javascript', <<~CSS
-      !function(){"use strict";var o;console.log((o=5)*o)}();
+      !function(){var o;console.log((o=5)*o)}();
     CSS
 
     file 'math.js', <<-JS
@@ -88,7 +101,7 @@ class CacheTest < ActiveSupport::TestCase
     JS
 
     assert_exported_file 'main.js', 'application/javascript', <<~CSS
-      !function(){"use strict";var o;console.log((o=5)*o*o)}();
+      !function(){var o;console.log((o=5)*o*o)}();
     CSS
   end
 
@@ -123,10 +136,10 @@ class CacheTest < ActiveSupport::TestCase
     JS
 
     assert_exported_file 'a.js', 'application/javascript', <<~JS
-      !function(){"use strict";console.log(5)}();
+      console.log(5);
     JS
     assert_exported_file 'b.js', 'application/javascript', <<~JS
-      !function(){"use strict";console.log(5)}();
+      console.log(5);
     JS
 
     file 'dep.js', <<-JS
@@ -134,15 +147,14 @@ class CacheTest < ActiveSupport::TestCase
     JS
 
     assert_exported_file 'a.js', 'application/javascript', <<~JS
-      !function(){"use strict";console.log(10)}();
+      console.log(10);
     JS
     assert_exported_file 'b.js', 'application/javascript', <<~JS
-      !function(){"use strict";console.log(10)}();
+      console.log(10);
     JS
-
   end
 
-  test 'a dependency is supurceeded by a new file' do
+  test 'a dependency is superceeded by a new file' do
     Dir.mkdir(File.join(@path, 'a'))
     Dir.mkdir(File.join(@path, 'b'))
     @env.clear_path
@@ -158,7 +170,7 @@ class CacheTest < ActiveSupport::TestCase
     JS
 
     assert_exported_file 'a.js', 'application/javascript', <<~JS
-      !function(){"use strict";console.log(5)}();
+      console.log(5);
     JS
 
     file 'a/dep.js', <<-JS
@@ -166,7 +178,7 @@ class CacheTest < ActiveSupport::TestCase
     JS
 
     assert_exported_file 'a.js', 'application/javascript', <<~JS
-      !function(){"use strict";console.log(10)}();
+      console.log(10);
     JS
   end
 
@@ -186,7 +198,7 @@ class CacheTest < ActiveSupport::TestCase
     JS
 
     assert_exported_file 'a.js', 'application/javascript', <<~JS
-      !function(){"use strict";console.log(5)}();
+      console.log(5);
     JS
 
     file 'a/deps/dep.js', <<-JS
@@ -194,7 +206,7 @@ class CacheTest < ActiveSupport::TestCase
     JS
 
     assert_exported_file 'a.js', 'application/javascript', <<~JS
-      !function(){"use strict";console.log(10)}();
+      console.log(10);
     JS
 
     file 'a/deps/dep.js', <<-JS
@@ -202,7 +214,7 @@ class CacheTest < ActiveSupport::TestCase
     JS
 
     assert_exported_file 'a.js', 'application/javascript', <<~JS
-      !function(){"use strict";console.log(20)}();
+      console.log(20);
     JS
   end
   
@@ -221,6 +233,21 @@ class CacheTest < ActiveSupport::TestCase
     CSS
   end
 
+  test '2a new dependency for a glob call is reflected in the next call' do
+    file "dir/a.svg", "<svg>test</svg>"
+    file 'test.scss', 'body { background: image-url("dir/a.svg") }'
+
+    assert_exported_file 'test.css', 'text/css', <<~CSS
+      body{background:url(/assets/dir/a-01a4bd3cb9faa518c5df2d2fcc8e6cd0ba24cfc3e9438dd01455ab1e59a39068.svg)}
+    CSS
+
+    file "dir/a.svg", "<svg>tests</svg>"
+
+    assert_exported_file 'test.css', 'text/css', <<~CSS
+      body{background:url(/assets/dir/a-1d7d038c7ace080963e116cbb962075a38d5e5dc68c6ff688e42d213dd432256.svg)}
+    CSS
+  end
+  
   test 'a dependency is removed for a glob call when one of it dependencies is delted' do
     file "css/a.scss", "body { color: blue; }"
     file "css/b.scss", "body { color: green; }"
@@ -237,14 +264,14 @@ class CacheTest < ActiveSupport::TestCase
     CSS
   end
 
-  test 'a dependency is added then changed should flush the parent' do
+  test 'a dependency is added then changed should flush the parent (JS)' do
     file 'a.js', "console.log('a');\n"
     file 'b.js', <<~JS
       export default function b () { console.log('b'); }
     JS
 
     assert_exported_file 'a.js', 'application/javascript', <<~JS
-      !function(){"use strict";console.log("a")}();
+    console.log("a");
     JS
 
     file 'a.js', <<~JS
@@ -254,7 +281,7 @@ class CacheTest < ActiveSupport::TestCase
     JS
 
     assert_exported_file 'a.js', 'application/javascript', <<~JS
-      !function(){"use strict";console.log("a"),console.log("b")}();
+      console.log("a"),console.log("b");
     JS
 
     file 'b.js', <<~JS
@@ -262,7 +289,119 @@ class CacheTest < ActiveSupport::TestCase
     JS
 
     assert_exported_file 'a.js', 'application/javascript', <<~JS
-      !function(){"use strict";console.log("a"),console.log("c")}();
+      console.log("a"),console.log("c");
     JS
   end
+
+  test 'a dependency is added then changed should flush the parent (CSS)' do
+    file 'a.scss', "body { background: aqua; }"
+    file 'b.scss', <<~JS
+      body { background: blue; }
+    JS
+
+    assert_exported_file 'a.css', 'text/css', <<~JS
+      body{background:aqua}
+    JS
+
+    file 'a.scss', <<~JS
+      @import "b";
+      body { background: aqua; }
+    JS
+
+    assert_exported_file 'a.css', 'text/css', <<~JS
+      body{background:blue}body{background:aqua}
+    JS
+
+    file 'b.scss', <<~JS
+      body { background: green; }
+    JS
+
+    assert_exported_file 'a.css', 'text/css', <<~JS
+      body{background:green}body{background:aqua}
+    JS
+  end
+  
+  test 'ensure the build cache only walks the dependency tree once' do
+    # a
+    # | |
+    # b c
+    #   |
+    #   d
+      
+    file 'd.js', "export default function d () { console.log('d'); }\n"
+    file 'b.js', "export default function b () { console.log('b'); }\n"
+    file 'c.js', <<~JS
+      import d from 'd';
+
+      export default function c () { console.log('c'); d(); }
+    JS
+    file 'a.js', <<~JS
+      import b from 'b';
+      import c from 'c';
+      
+      console.log('a'); b(); c();
+    JS
+
+    assert_exported_file 'a.js', 'application/javascript', <<~JS
+      console.log("a"),console.log("b"),console.log("c"),console.log("d");
+    JS
+
+    file 'd.js', "export default function e () { console.log('e'); }\n"
+
+    pd = @env.build_cache.instance_variable_get(:@process_dependencies)
+    pd["#{@path}/a.js"] ||= Set.new
+    pd["#{@path}/b.js"] ||= Set.new
+    pd["#{@path}/c.js"] ||= Set.new
+    pd["#{@path}/d.js"] ||= Set.new
+    pd["#{@path}/a.js"].expects(:<<).with { |a| a.source_file == "#{@path}/a.js" }.once
+    pd["#{@path}/b.js"].expects(:<<).with { |a| a.source_file == "#{@path}/b.js" }.never
+    pd["#{@path}/c.js"].expects(:<<).with { |a| a.source_file == "#{@path}/c.js" }.never
+    pd["#{@path}/d.js"].expects(:<<).with { |a| a.source_file == "#{@path}/d.js" }.once
+
+    assert_exported_file 'a.js', 'application/javascript', <<~JS
+      console.log("a"),console.log("b"),console.log("c"),console.log("e");
+    JS
+  end
+  
+  test 'same files in diffrent dirs sharing a cache doesnt poison the cache (ie capistrano deploys)' do
+    cachepath = Dir.mktmpdir
+    
+    dir = File.realpath(Dir.mktmpdir)
+    base1 = File.join(dir, 'a')
+    base2 = File.join(dir, 'b')
+    base3 = File.join(dir, 'c')
+    Dir.mkdir(base1)
+    Dir.mkdir(base2)
+
+    [base1, base2, base3].each do |b|
+      file 'test.js', "export default function c () { console.log('t'); }\n", base: b
+      file 'test/b.js', <<~JS, base: b
+        import c from './c';
+        export default function b () { console.log('b'); c(); }
+      JS
+      file 'test/a.js', <<~JS, base: b
+        import t from 'test';
+        import b from './b';
+
+        console.log('a');
+        b();
+      JS
+    end
+
+    file 'test/c.js', "export default function c () { console.log('c'); }\n", base: base1
+    file 'test/c.js', "export default function c () { console.log('d'); }\n", base: base2
+    file 'test/c.js', "export default function c () { console.log('e'); }\n", base: base3
+
+    # Set the cache
+    env1 = Condenser.new(base1, logger: Logger.new(STDOUT, level: :debug), base: base1, npm_path: @npm_dir, cache: Condenser::Cache::FileStore.new(cachepath))
+    assert_equal 'console.log("a"),console.log("b"),console.log("c");', env1.find('test/a.js').export.source
+
+    # Poison the cache
+    env2 = Condenser.new(base2, logger: Logger.new(STDOUT, level: :debug), base: base2, npm_path: @npm_dir, cache: Condenser::Cache::FileStore.new(cachepath))
+    assert_equal 'console.log("a"),console.log("b"),console.log("d");', env2.find('test/a.js').export.source
+
+    # Fails to find dependency change because cache is missing the dependency
+    env3 = Condenser.new(base3, logger: Logger.new(STDOUT, level: :debug), base: base3, npm_path: @npm_dir, cache: Condenser::Cache::FileStore.new(cachepath))
+    assert_equal 'console.log("a"),console.log("b"),console.log("e");', env3.find('test/a.js').export.source
+  end
 end

data/test/dependency_test.rb

@@ -17,7 +17,7 @@ class DependencyTest < ActiveSupport::TestCase
     JS
 
     asset = @env.find('name.js')
-    assert_equal asset.instance_variable_get(:@process_dependencies), ["models/*.js","helpers/*.js"]
+    assert_equal asset.instance_variable_get(:@process_dependencies).to_a, [["models/*", ["application/javascript"]],["helpers/*", ["application/javascript"]]]
 
 
     assert_file 'name.js', 'application/javascript', <<~JS
@@ -54,7 +54,7 @@ class DependencyTest < ActiveSupport::TestCase
     JS
 
     asset = @env.find('name.js')
-    assert_equal asset.instance_variable_get(:@process_dependencies), ["models/*.js","helpers/*.js"]
+    assert_equal asset.instance_variable_get(:@process_dependencies).to_a, [["models/*", ["application/javascript"]],["helpers/*", ["application/javascript"]]]
 
 
     assert_file 'name.js', 'application/javascript', <<~JS
@@ -74,4 +74,53 @@ class DependencyTest < ActiveSupport::TestCase
     JS
   end
 
+  test 'js depending on another file type with JSAnalzyer' do
+    @env.unregister_preprocessor 'application/javascript', Condenser::BabelProcessor
+    @env.register_preprocessor 'application/javascript', Condenser::JSAnalyzer
+    @env.unregister_minifier('application/javascript')
+
+    file 'a.js', ''
+    file 'b.rb', ''
+    file 'models/a.js', ''
+    file 'models/b.rb', ''
+
+    file 'name.js.erb', <<~JS
+      // depends_on **/*.rb
+
+      console.log([<%= Dir.children("#{@path}").sort.map(&:inspect).join(', ') %>]);
+    JS
+
+    asset = @env.find('name.js')
+    assert_equal asset.instance_variable_get(:@process_dependencies).to_a, [["**/*", ["application/ruby"]]]
+    assert_equal asset.process_dependencies.map(&:source_file), ["#{@path}/b.rb", "#{@path}/models/b.rb"]
+  end
+
+  test 'relative imports with JSAnalzyer' do
+    @env.unregister_preprocessor 'application/javascript', Condenser::BabelProcessor
+    @env.register_preprocessor 'application/javascript', Condenser::JSAnalyzer
+    @env.unregister_minifier('application/javascript')
+    
+    file 'a/a.js', <<~JS
+      export decault function () { console.log("a/a"); }
+    JS
+    file 'b/a.js', <<~JS
+      export decault function () { console.log("a/a"); }
+    JS
+
+    file 'a/b.js', <<~JS
+      import fn from './a';
+      a();
+      console.log("a/b");
+    JS
+    file 'b/b.js', <<~JS
+      import fn from './a';
+      a();
+      console.log("b/b");
+    JS
+    
+    asset = @env.find('a/b.js')
+    assert_equal asset.instance_variable_get(:@export_dependencies).to_a, [["#{@path}/a/a", ["application/javascript"]]]
+    assert_equal asset.export_dependencies.map(&:source_file), ["#{@path}/a/a.js"]
+  end
+  
 end

data/test/manifest_test.rb

@@ -66,6 +66,40 @@ class ManifestTest < ActiveSupport::TestCase
     assert_equal asset.path, data['application.js']['path']
   end
 
+  test "compile asset dependencies includes the dependencies" do
+    file 'foo.svg', <<-SVG
+      <svg xmlns="http://www.w3.org/2000/svg" fill="none" viewBox="0 0 24 24" stroke="currentColor">
+        <path d="M6 18L18 6M6 6l12 12" />
+      </svg>
+    SVG
+    
+    file 'test.scss', <<-SCSS
+      body {
+        background: asset-url("foo.svg");
+      }
+    SCSS
+    
+    manifest = Condenser::Manifest.new(@env, File.join(@dir, 'manifest.json'))
+
+    main_digest_path = @env['test.css'].path
+    dep_digest_path = @env['foo.svg'].path
+
+    assert !File.exist?("#{@dir}/#{main_digest_path}")
+    assert !File.exist?("#{@dir}/#{dep_digest_path}")
+
+    manifest.compile('test.css')
+    assert File.directory?(manifest.dir)
+    assert File.file?(manifest.filename)
+
+    assert File.exist?("#{@dir}/manifest.json")
+    assert File.exist?("#{@dir}/#{main_digest_path}")
+    assert File.exist?("#{@dir}/#{dep_digest_path}")
+
+    data = JSON.parse(File.read(manifest.filename))
+    assert_equal main_digest_path, data['test.css']['path']
+    assert_equal dep_digest_path, data['foo.svg']['path']
+  end
+  
   # TODO:
   # test "compile asset with aliased index links" do
   #   manifest = Sprockets::Manifest.new(@env, File.join(@dir, 'manifest.json'))

data/test/minifiers/terser_minifier_test.rb

@@ -4,7 +4,6 @@ class TerserMinifierTest < ActiveSupport::TestCase
 
   def setup
     super
-    @env.unregister_preprocessor('application/javascript', Condenser::BabelProcessor)
     @env.unregister_exporter('application/javascript', Condenser::RollupProcessor)
     @env.register_minifier('application/javascript', Condenser::TerserMinifier)
   end