refactor(JSX::Transformer) use Transformer class; allow asset_path config

Author: rmosolgo

lib/react/jsx.rb

@@ -1,36 +1,21 @@
 require 'execjs'
 require 'react/jsx/template'
+require 'react/jsx/transformer'
 require 'rails'
 
 module React
   module JSX
     mattr_accessor :transform_options
+    def self.transform(code)
+      transformer.transform(code)
+    end
 
-    def self.context
+    def self.transformer
       # lazily loaded during first request and reloaded every time when in dev or test
-      unless @context && ::Rails.env.production?
-        contents =
-          # If execjs uses therubyracer, there is no 'global'. Make sure
-          # we have it so JSX script can work properly.
-          'var global = global || this;' +
-
-          # search for transformer file using sprockets - allows user to override
-          # this file in his own application
-          File.read(::Rails.application.assets.resolve('JSXTransformer.js'))
-
-        @context = ExecJS.compile(contents)
+      if @transformer.nil? || !::Rails.env.production?
+        @transformer = Transformer.new(transform_options)
       end
-
-      @context
-    end
-
-    def self.transform(code, options={})
-      js_options = {
-        stripTypes: options[:strip_types],
-        harmony: options[:harmony],
-      }
-      result = context.call('JSXTransformer.transform', code, js_options)
-      return result['code']
+      @transformer
     end
   end
 end

lib/react/jsx/template.rb

@@ -10,7 +10,7 @@ def prepare
       end
 
       def evaluate(scope, locals, &block)
-        @output ||= JSX::transform(data, JSX.transform_options)
+        @output ||= JSX::transform(data)
       end
     end
   end

lib/react/jsx/transformer.rb

@@ -0,0 +1,30 @@
+module React
+  module JSX
+    class Transformer
+      DEFAULT_ASSET_PATH = 'JSXTransformer.js'
+
+      def initialize(options)
+        @strip_types =  options.fetch(:strip_types, false)
+        @harmony =      options.fetch(:harmony, false)
+        @asset_path =    options.fetch(:asset_path, DEFAULT_ASSET_PATH)
+
+        # If execjs uses therubyracer, there is no 'global'. Make sure
+        # we have it so JSX script can work properly.
+        js_code = 'var global = global || this;' + jsx_transform_code
+        @context = ExecJS.compile(js_code)
+      end
+
+
+      def transform(code)
+        result = @context.call('JSXTransformer.transform', code, {stripTypes: @strip_types, harmony: @harmony})
+        result["code"]
+      end
+
+      def jsx_transform_code
+        # search for transformer file using sprockets - allows user to override
+        # this file in his own application
+        ::Rails.application.assets[@asset_path].to_s
+      end
+    end
+  end
+end

test/jsxtransform_test.rb renamed to test/react/jsx_test.rb

@@ -21,6 +21,8 @@
 eos
 
 class JSXTransformTest < ActionDispatch::IntegrationTest
+  setup { clear_sprockets_cache }
+  teardown { clear_sprockets_cache }
 
   test 'asset pipeline should transform JSX' do
     get '/assets/example.js'
@@ -40,14 +42,12 @@ class JSXTransformTest < ActionDispatch::IntegrationTest
   end
 
   test 'can use dropped-in version of JSX transformer' do
-    hidden_path = File.expand_path("../dummy/vendor/assets/react/JSXTransformer__.js",  __FILE__)
-    replacing_path = File.expand_path("../dummy/vendor/assets/react/JSXTransformer.js",  __FILE__)
+    hidden_path =     Rails.root.join("vendor/assets/react/JSXTransformer__.js")
+    replacing_path =  Rails.root.join("vendor/assets/react/JSXTransformer.js")
 
-    FileUtils.mv hidden_path, replacing_path
+    FileUtils.cp hidden_path, replacing_path
     get '/assets/example3.js'
-
-    FileUtils.mv replacing_path, hidden_path
-    FileUtils.rm_r CACHE_PATH if CACHE_PATH.exist?
+    FileUtils.rm replacing_path
 
     assert_response :success
     assert_equal 'test_confirmation_token_jsx_transformed;', @response.body
@@ -69,4 +69,20 @@ class JSXTransformTest < ActionDispatch::IntegrationTest
     assert_response :success
     assert_match(/\(i\s*,\s*name\s*\)\s*\{/, @response.body, "type annotations are removed")
   end
+
+  test 'accepts asset_path: option' do
+    hidden_path =     Rails.root.join("vendor/assets/react/JSXTransformer__.js")
+    custom_path =     Rails.root.join("vendor/assets/react/custom")
+    replacing_path =  custom_path.join("CustomTransformer.js")
+
+    React::JSX.transform_options = {asset_path: "custom/CustomTransformer.js"}
+
+    FileUtils.mkdir_p(custom_path)
+    FileUtils.cp(hidden_path, replacing_path)
+    get '/assets/example3.js'
+
+    FileUtils.rm_rf custom_path
+    assert_response :success
+    assert_equal 'test_confirmation_token_jsx_transformed;', @response.body
+  end
 end

test/react_test.rb

@@ -1,12 +1,11 @@
 require 'test_helper'
 class ReactTest < ActionDispatch::IntegrationTest
   setup do
-    FileUtils.rm_r(CACHE_PATH) if CACHE_PATH.exist?
-
+    clear_sprockets_cache
   end
 
   teardown do
-    FileUtils.rm_r(CACHE_PATH) if CACHE_PATH.exist?
+    clear_sprockets_cache
   end
 
   test 'asset pipeline should deliver drop-in react file replacement' do
@@ -25,7 +24,6 @@ def react_asset.fresh?(env); false; end
     get '/assets/react.js'
 
     File.unlink(app_react_file_path)
-    FileUtils.rm_r(CACHE_PATH) if CACHE_PATH.exist?
 
     assert_response :success
     assert_equal react_file_token.length, react_asset.to_s.length, "The asset pipeline serves the drop-in file"

test/test_helper.rb

@@ -11,10 +11,12 @@
 
 Rails.backtrace_cleaner.remove_silencers!
 
-# Remove cached files
-Rails.root.join('tmp/cache').tap do |tmp|
-  tmp.rmtree if tmp.exist?
-  tmp.mkpath
+def clear_sprockets_cache
+  # Remove cached files
+  Rails.root.join('tmp/cache').tap do |tmp|
+    tmp.rmtree if tmp.exist?
+    tmp.mkpath
+  end
 end
 
 # Load support files