condenser 1.2 → 1.3

data/test/preprocessor/js_analyzer_test.rb

@@ -4,8 +4,6 @@ class JSAnalyzerTest < ActiveSupport::TestCase
   
   def setup
     super
-    @env.unregister_preprocessor 'application/javascript', Condenser::BabelProcessor
-    @env.register_preprocessor 'application/javascript', Condenser::JSAnalyzer
     @env.unregister_minifier('application/javascript')
   end
   

data/test/processors/rollup_test.rb

@@ -31,15 +31,14 @@ class RollupTest < ActiveSupport::TestCase
       (function () {
         'use strict';
 
-        // This function isn't used anywhere, so
-
-        function cube(x) {
+        // This function gets included
+        function cube ( x ) {
           return x * x * x;
         }
 
-        console.log(cube(5)); // 125
+        console.log( cube( 5 ) ); // 125
 
-      }());
+      })();
     FILE
   end
   
@@ -59,13 +58,13 @@ class RollupTest < ActiveSupport::TestCase
       (function () {
         'use strict';
 
-        function cube(x) {
+        function cube ( x ) {
           return 2 * x * x;
         }
 
-        console.log(cube(5)); // 125
+        console.log( cube( 5 ) ); // 125
 
-      }());
+      })();
     FILE
   end
 
@@ -93,13 +92,13 @@ class RollupTest < ActiveSupport::TestCase
       (function () {
         'use strict';
 
-        function cube(x) {
+        function cube ( x ) {
           return x * x * x;
         }
 
-        console.log(cube(5)); // 125
+        console.log( cube( 5 ) ); // 125
 
-      }());
+      })();
     FILE
   end
 
@@ -126,17 +125,17 @@ class RollupTest < ActiveSupport::TestCase
       (function () {
         'use strict';
 
-        window.cube = function (x) {
+        window.cube = function ( x ) {
           return x * x * x;
         };
 
-        window.square = function (x) {
+        window.square = function ( x ) {
           return x * x;
         };
 
-        console.log(square(cube(5)));
+        console.log( square(cube( 5 )) );
 
-      }());
+      })();
     FILE
   end
 
@@ -168,27 +167,58 @@ class RollupTest < ActiveSupport::TestCase
       (function () {
         'use strict';
 
-        function cube(x) {
+        function cube ( x ) {
           return x * x * x;
         }
 
-        function square(x) {
+        function square ( x ) {
           return x * x;
         }
 
         var maths = [cube, square];
 
         var x = 1;
-
         for (var i = 0; i < maths.length; i++) {
           x = maths[i](x);
         }
-
         console.log(x);
 
-      }());
+      })();
     FILE
     $d = false
   end
 
+  test 'import the same file via relative require and full path' do
+    file "#{@npm_path}/module/base.js", <<~JS
+      export default class Base { };
+    JS
+    
+    file "#{@npm_path}/module/base/other.js", <<~JS
+      import Base from '../base';
+      
+      export default class Lower extends Base { };
+    JS
+    
+    file 'main.js', <<~JS
+      import Other from 'module/base/other';
+      import Base from 'module/base';
+
+      console.log( Base, Other );
+    JS
+
+
+    assert_exported_file 'main.js', 'application/javascript', <<~FILE
+      (function () {
+      	'use strict';
+
+      	class Base { }
+
+      	class Lower extends Base { }
+
+      	console.log( Base, Lower );
+
+      })();
+    FILE
+  end
+
 end

data/test/resolve_test.rb

@@ -83,7 +83,6 @@ class ResolveTest < ActiveSupport::TestCase
     @env.append_path File.join(@path, 'b')
     @env.append_path File.join(@path, 'a')
 
-    
     assert_exported_file('main.js', 'application/javascript', <<~JS)
       (function () {
         'use strict';
@@ -94,7 +93,7 @@ class ResolveTest < ActiveSupport::TestCase
 
         console.log( cube( 5 ) ); // 125
 
-      }());
+      })();
     JS
   end
   
@@ -111,15 +110,15 @@ class ResolveTest < ActiveSupport::TestCase
   end
   
   test 'resolve a file.*' do
-    file 'file.js', 'console.log(1);'
-    file 'file.scss', 'body { background: red; }'
-    file 'test/file.scss', 'body { background: green; }'
+    file 'foo.js', 'console.log(1);'
+    file 'foo.scss', 'body { background: red; }'
+    file 'test/foo.scss', 'body { background: green; }'
 
-    assert_equal %w(file.js file.scss), @env.resolve('file.*').map(&:filename)
-    assert_equal %w(file.css file.js),  @env.resolve('file.*', accept: ['text/css', 'application/javascript']).map(&:filename)
+    assert_equal %w(foo.js foo.scss), @env.resolve('foo.*').map(&:filename)
+    assert_equal %w(foo.css foo.js),  @env.resolve('foo.*', accept: ['text/css', 'application/javascript']).map(&:filename)
 
-    assert_equal %w(file.js file.scss test/file.scss), @env.resolve('**/file.*').map(&:filename)
-    assert_equal %w(file.css file.js test/file.css), @env.resolve('**/file.*', accept: ['text/css', 'application/javascript']).map(&:filename)
+    assert_equal %w(foo.js foo.scss test/foo.scss), @env.resolve('**/foo.*').map(&:filename)
+    assert_equal %w(foo.css foo.js test/foo.css), @env.resolve('**/foo.*', accept: ['text/css', 'application/javascript']).map(&:filename)
   end
   
 end

data/test/server_test.rb

@@ -224,7 +224,12 @@ class ServerTest < ActiveSupport::TestCase
     assert_equal "pass", last_response.headers['X-Cascade']
   end
 
-  test "re-throw JS exceptions in the browser" do
+  test "re-throw JS exceptions in the browser when using babel" do
+    @env.unregister_preprocessor('application/javascript', Condenser::JSAnalyzer)
+    @env.register_preprocessor 'application/javascript', Condenser::BabelProcessor.new(@path,
+      presets: [ ['@babel/preset-env', { modules: false, targets: { browsers: 'firefox > 41' } }] ]
+    )
+    
     file 'error.js', "var error = {;"
 
     get "/assets/error.js"

data/test/templates/ejs_test.rb

@@ -4,7 +4,6 @@ class CondenserEJSTest < ActiveSupport::TestCase
 
   def setup
     super
-    @env.unregister_preprocessor('application/javascript', Condenser::BabelProcessor)
     @env.unregister_exporter('application/javascript', Condenser::RollupProcessor)
   end
   
@@ -15,14 +14,10 @@ class CondenserEJSTest < ActiveSupport::TestCase
       import { escape } from 'ejs';
       export default function (locals) {
         var __output = [],
-            __append = __output.push.bind(__output);
-      
+          __append = __output.push.bind(__output);
         __append("1");
-      
         __append(escape(1 + 1));
-      
         __append("3\\n");
-      
         return __output.join("");
       }
     JS
@@ -35,14 +30,10 @@ class CondenserEJSTest < ActiveSupport::TestCase
       import { escape } from 'ejs';
       export default function (locals) {
         var __output = [],
-            __append = __output.push.bind(__output);
-      
+          __append = __output.push.bind(__output);
         __append("1");
-      
         __append(escape(locals.input));
-      
         __append("3\\n");
-      
         return __output.join("");
       }
     JS

data/test/templates/erb_test.rb

@@ -2,11 +2,6 @@ require 'test_helper'
 
 class CondenserErubiTest < ActiveSupport::TestCase
 
-  def setup
-    super
-    @env.unregister_preprocessor('application/javascript', Condenser::BabelProcessor)
-  end
-  
   test 'find' do
     file 'test.js.erb', "1<%= 1 + 1 %>3\n"
     

data/test/test_helper.rb

@@ -39,6 +39,7 @@ class ActiveSupport::TestCase
     @env.unregister_writer(Condenser::BrotliWriter)
     @env.context_class.class_eval do
       def asset_path(path, options = {})
+        path = environment.find!(path, options).path
         "/assets/#{path}"
       end
     end
@@ -53,7 +54,7 @@ class ActiveSupport::TestCase
   end
   
   def file(name, source)
-    dir = File.join(@path, File.dirname(name))
+    dir = name.include?('/') ? File.join(@path, File.dirname(name)) : @path
     path = File.join(@path, name)
     
     FileUtils.mkdir_p(dir)
@@ -62,6 +63,7 @@ class ActiveSupport::TestCase
       sleep(1 - stat) if stat < 1
     end
     File.write(path, source)
+    sleep 0.25 if @env.build_cache.listening
   end
   
   def rm(name)

data/test/transformers/dart_scss_test.rb

@@ -0,0 +1,139 @@
+require 'test_helper'
+
+class CondenserDartSCSSTest < ActiveSupport::TestCase
+
+  def setup
+    super
+    @env.clear_pipeline
+    @env.register_transformer 'text/scss', 'text/css', Condenser::DartScssTransformer
+  end
+
+  test 'find' do
+    file 'test.scss', <<~SCSS
+    body {
+      background-color: green;
+
+      &:hover {
+        background-color: blue;
+      }
+    }
+    SCSS
+
+    assert_file 'test.css', 'text/css', <<~CSS
+    body {
+      background-color: green;
+    }
+    body:hover {
+      background-color: blue;
+    }
+    CSS
+  end
+
+  test 'scss import globing' do
+    file "c/dir/a.scss", "body { color: blue; }"
+    file "c/dir/b.scss", "body { color: green; }"
+
+    file 'c/test.scss', '@import "./dir/*"'
+
+    assert_file 'c/test.css', 'text/css', <<~CSS
+    body {
+      color: blue;
+    }
+
+    body {
+      color: green;
+    }
+    CSS
+
+    file 'c/test2.scss', '@import "c/dir/*"'
+
+    assert_file 'c/test2.css', 'text/css', <<~CSS
+    body {
+      color: blue;
+    }
+
+    body {
+      color: green;
+    }
+    CSS
+  end
+
+  test "url functions" do
+    file 'a.scss', <<~SCSS
+      body {
+        color: green; }
+    SCSS
+
+    file 'test.scss', <<~SCSS
+    @import 'a';
+
+    div {
+       url: asset-url("foo.svg");
+       url: image-url("foo.png");
+       url: video-url("foo.mov");
+       url: audio-url("foo.mp3");
+       url: font-url("foo.woff2");
+       url: font-url("foo.woff");
+       url: javascript-url("foo.js");
+       url: stylesheet-url("foo.css");
+    }
+    SCSS
+
+    file 'foo.svg', ''
+    file 'foo.png', ''
+    file 'foo.mov', ''
+    file 'foo.mp3', ''
+    file 'foo.woff2', ''
+    file 'foo.woff', ''
+    file 'foo.js', ''
+    file 'foo.css', ''
+
+    assert_file 'test.css', 'text/css', <<~CSS
+    body {
+      color: green;
+    }
+
+    div {
+      url: url(/assets/foo-e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855.svg);
+      url: url(/assets/foo-e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855.png);
+      url: url(/assets/foo-e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855.mov);
+      url: url(/assets/foo-e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855.mp3);
+      url: url(/assets/foo-e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855.woff2);
+      url: url(/assets/foo-e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855.woff);
+      url: url(/assets/foo-e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855.js);
+      url: url(/assets/foo-e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855.css);
+    }
+    CSS
+
+    asset = @env.find('test.css')
+    assert_equal ["a.scss", "foo.svg", "foo.png", "foo.mov", "foo.mp3", "foo.woff2", "foo.woff", "foo.js", "foo.css"], asset.process_dependencies.map(&:filename)
+    assert_equal ["a.scss", "foo.svg", "foo.png", "foo.mov", "foo.mp3", "foo.woff2", "foo.woff", "foo.js", "foo.css"], asset.export_dependencies.map(&:filename)
+  end
+
+  test "sass dependencies" do
+    file 'd.scss', <<~SCSS
+      $secondary-color: #444;
+    SCSS
+
+    file 'a.scss', <<~SCSS
+      @import 'd';
+      $primary-color: #333;
+    SCSS
+
+    file 'b.scss', <<~SCSS
+      body {
+        color: $primary-color;
+      }
+    SCSS
+
+    file 'c.scss', <<~SCSS
+      @import 'a';
+      @import 'b';
+    SCSS
+
+    asset = @env.find('c.css')
+    assert_equal ["a.scss", "d.scss", "b.scss"], asset.process_dependencies.map(&:filename)
+    assert_equal ["a.scss", "d.scss", "b.scss"], asset.export_dependencies.map(&:filename)
+  end
+  
+end

data/test/transformers/jst_test.rb

@@ -2,23 +2,23 @@ require 'test_helper'
 
 class JSTTransformerTest < ActiveSupport::TestCase
 
-  test 'jst transoformation' do
+  test 'jst transformation' do
     file 'test.jst', <<~SCSS
       import {escape} from 'ejs';
       export default function (locals) {
           var __output = [], __append = __output.push.bind(__output);
               __append("<div class=\\"uniformLoader\\n");
-               if(typeof transparent != "undefined") { 
+               if(typeof transparent != "undefined") {
               __append(" -transparent");
-               } 
+               }
               __append("\\n");
-               if(typeof cover != "undefined") { 
+               if(typeof cover != "undefined") {
               __append(" -cover");
-               } 
+               }
               __append("\\n");
-               if(typeof light != "undefined") { 
+               if(typeof light != "undefined") {
               __append(" -light");
-               } 
+               }
               __append(" ");
               __append( klass );
               __append("\\">\\n    <div class=\\"uniformLoader-container\\">\\n        <span></span>\\n        <span></span>\\n        <span></span>\\n    </div>\\n</div>");
@@ -27,41 +27,185 @@ class JSTTransformerTest < ActiveSupport::TestCase
     SCSS
 
     assert_file 'test.js', 'application/javascript', <<~JS
-    import _bindInstanceProperty from "@babel/runtime-corejs3/core-js-stable/instance/bind";
     import { escape } from 'ejs';
     export default function (locals) {
-      var _context;
-
       var __output = [],
-          __append = _bindInstanceProperty(_context = __output.push).call(_context, __output);
-
+        __append = __output.push.bind(__output);
       __append("<div class=\\"uniformLoader\\n");
-
       if (typeof locals.transparent != "undefined") {
         __append(" -transparent");
       }
-
       __append("\\n");
-
       if (typeof locals.cover != "undefined") {
         __append(" -cover");
       }
-
       __append("\\n");
-
       if (typeof locals.light != "undefined") {
         __append(" -light");
       }
-
       __append(" ");
-
       __append(locals.klass);
-
       __append("\\">\\n    <div class=\\"uniformLoader-container\\">\\n        <span></span>\\n        <span></span>\\n        <span></span>\\n    </div>\\n</div>");
-
       return __output.join("");
     }
     JS
   end
 
+  test 'jst transformation with object' do
+    file 'test.jst', <<~SCSS
+      import {append as __ejx_append} from 'ejx';
+      export default function (locals) {
+          var __output = [];
+          __ejx_append(avatarTemplate({ account: account }), __output);
+          return __output;
+      }
+    SCSS
+
+    assert_file 'test.js', 'application/javascript', <<~JS
+      import { append as __ejx_append } from 'ejx';
+      export default function (locals) {
+        var __output = [];
+        __ejx_append(locals.avatarTemplate({
+          account: locals.account
+        }), __output);
+        return __output;
+      }
+    JS
+  end
+
+  test 'jst with transformation scope test of a defined function' do
+    file 'test.jst', <<~SCSS
+      import {append as __ejx_append} from 'ejx';
+      export default function (locals) {
+          var __output = [];
+          
+          function x(files) {
+            return files.test();
+          }
+          
+          class B {
+          }
+          
+          models.map(m => m.name)
+          
+          __ejx_append(avatarTemplate({ account: x(files), klass: B }), __output);
+          return __output;
+      }
+    SCSS
+
+    assert_file 'test.js', 'application/javascript', <<~JS
+      import { append as __ejx_append } from 'ejx';
+      export default function (locals) {
+        var __output = [];
+        function x(files) {
+          return files.test();
+        }
+        class B {}
+        locals.models.map(m => m.name);
+        __ejx_append(locals.avatarTemplate({
+          account: x(locals.files),
+          klass: B
+        }), __output);
+        return __output;
+      }
+    JS
+  end
+
+  test 'jst with transformation shadow variable example' do
+    file 'test.jst', <<~JS
+    import {append as __ejx_append} from 'ejx';
+    export default async function (locals) {
+        function f (items, template) {
+                return items.map((file) => {
+                    const row = template(file);
+                    const bar = row.querySelector('.progress-bar');
+                    file.onprogress = (n) => { row.style.width = "" + n + "%"; }
+            
+                    return row;
+                });
+            }
+        __ejx_append(f(items, (f) => { return __v; }));
+        return __output;
+    }
+    JS
+
+    assert_file 'test.js', 'application/javascript', <<~JS
+    import { append as __ejx_append } from 'ejx';
+    export default async function (locals) {
+      function f(items, template) {
+        return items.map(file => {
+          const row = template(file);
+          const bar = row.querySelector('.progress-bar');
+          file.onprogress = n => {
+            row.style.width = "" + n + "%";
+          };
+          return row;
+        });
+      }
+      __ejx_append(f(locals.items, f => {
+        return locals.__v;
+      }));
+      return locals.__output;
+    }
+    JS
+  end
+  
+  test 'try/catch example' do
+    file 'test.jst', <<~JS
+      import {append as __ejx_append} from 'ejx';
+      export default function (locals) {
+          var __output = [];
+          try {
+            __ejx_append(avatarTemplate({ account: account }), __output);
+          } catch (e) {
+            console.error(e);
+          }
+          
+          console.error(e);
+          
+          return __output;
+      }
+    JS
+
+    assert_file 'test.js', 'application/javascript', <<~JS
+      import { append as __ejx_append } from 'ejx';
+      export default function (locals) {
+        var __output = [];
+        try {
+          __ejx_append(locals.avatarTemplate({
+            account: locals.account
+          }), __output);
+        } catch (e) {
+          console.error(e);
+        }
+        console.error(locals.e);
+        return __output;
+      }
+    JS
+  end
+  
+  test 'splats' do
+    file 'test.jst', <<~JS
+      import {append as __ejx_append} from 'ejx';
+      export default function (locals) {
+          var __output = [];
+          
+          var x = ((...args) => { return y(...args); })(z, y);
+          
+          return __output;
+      }
+    JS
+
+    assert_file 'test.js', 'application/javascript', <<~JS
+      import { append as __ejx_append } from 'ejx';
+      export default function (locals) {
+        var __output = [];
+        var x = ((...args) => {
+          return locals.y(...args);
+        })(locals.z, locals.y);
+        return __output;
+      }
+    JS
+  end
+
 end

data/test/transformers/scss_test.rb

@@ -47,7 +47,14 @@ class CondenserSCSSTest < ActiveSupport::TestCase
   end
 
   test "url functions" do
+    file 'a.scss', <<~SCSS
+      body {
+        color: green; }
+    SCSS
+    
     file 'test.scss', <<~SCSS
+    @import 'a';
+    
     div {
        url: asset-url("foo.svg");
        url: image-url("foo.png");
@@ -70,6 +77,9 @@ class CondenserSCSSTest < ActiveSupport::TestCase
     file 'foo.css', ''
 
     assert_file 'test.css', 'text/css', <<~CSS
+    body {
+      color: green; }
+    
     div {
       url: url(/assets/foo-e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855.svg);
       url: url(/assets/foo-e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855.png);
@@ -80,6 +90,10 @@ class CondenserSCSSTest < ActiveSupport::TestCase
       url: url(/assets/foo-e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855.js);
       url: url(/assets/foo-e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855.css); }
     CSS
+    
+    asset = @env.find('test.css')
+    assert_equal ["a.scss", "foo.svg", "foo.png", "foo.mov", "foo.mp3", "foo.woff2", "foo.woff", "foo.js", "foo.css"], asset.process_dependencies.map(&:filename)
+    assert_equal ["a.scss", "foo.svg", "foo.png", "foo.mov", "foo.mp3", "foo.woff2", "foo.woff", "foo.js", "foo.css"], asset.export_dependencies.map(&:filename)
   end
   
   test "sass dependencies" do