i18n-js 3.0.0.rc8 → 3.0.0.rc9

data/app/assets/javascripts/i18n/filtered.js.erb

@@ -1,14 +1,12 @@
 <%# encoding: utf-8 %>
 
 ;(function(factory) {
-  if (typeof module !== 'undefined') {
+  if (typeof module !== 'undefined' && module.exports) {
     // Node/CommonJS
     factory(require('i18n'));
-
   } else if (typeof define === 'function' && define.amd) {
     // AMD
     define(['i18n'], factory);
-
   } else {
     // Browser globals
     factory(this.I18n);

data/gemfiles/i18n_0_7.gemfile

@@ -0,0 +1,7 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "i18n", "0.7.0"
+
+gemspec :path => "../"

data/i18n-js.gemspec

@@ -23,7 +23,6 @@ Gem::Specification.new do |s|
   s.add_development_dependency "activesupport", ">= 3"
   s.add_development_dependency "rspec", "~> 3.0"
   s.add_development_dependency "rake"
-  s.add_development_dependency "pry-meta"
   s.add_development_dependency "gem-release", ">= 0.7"
 
   s.required_ruby_version = ">= 1.9.3"

data/lib/i18n/js.rb

@@ -1,12 +1,13 @@
+require "yaml"
 require "i18n"
 require "fileutils"
-
 require "i18n/js/utils"
 
 module I18n
   module JS
     require "i18n/js/dependencies"
     require "i18n/js/fallback_locales"
+    require "i18n/js/segment"
     if JS::Dependencies.rails?
       require "i18n/js/middleware"
       require "i18n/js/engine"
@@ -20,6 +21,7 @@ module I18n
     def self.config_file_path
       @config_file_path ||= DEFAULT_CONFIG_PATH
     end
+
     def self.config_file_path=(new_path)
       @config_file_path = new_path
     end
@@ -29,48 +31,51 @@ module I18n
     def self.export
       export_i18n_js
 
-      translation_segments.each do |filename, translations|
-        save(translations, filename)
-      end
+      translation_segments.each(&:save!)
     end
 
-    def self.segments_per_locale(pattern, scope)
-      I18n.available_locales.each_with_object({}) do |locale, segments|
+    def self.segments_per_locale(pattern, scope, exceptions, options)
+      I18n.available_locales.each_with_object([]) do |locale, segments|
         scope = [scope] unless scope.respond_to?(:each)
-        result = scoped_translations(scope.collect{|s| "#{locale}.#{s}"})
-        merge_with_fallbacks!(result, locale, scope) if use_fallbacks?
+        result = scoped_translations(scope.collect{|s| "#{locale}.#{s}"}, exceptions)
+        merge_with_fallbacks!(result, locale, scope, exceptions) if use_fallbacks?
 
         next if result.empty?
-
-        segment_name = ::I18n.interpolate(pattern,{:locale => locale})
-        segments[segment_name] = result
+        segments << Segment.new(::I18n.interpolate(pattern, {:locale => locale}), result, options)
       end
     end
 
-    def self.segment_for_scope(scope)
+    def self.segment_for_scope(scope, exceptions)
       if scope == "*"
-        translations
+        exclude(translations, exceptions)
       else
-        scoped_translations(scope)
+        scoped_translations(scope, exceptions)
       end
     end
 
     def self.configured_segments
-      config[:translations].each_with_object({}) do |options, segments|
-        options.reverse_merge!(:only => "*")
-        if options[:file] =~ ::I18n::INTERPOLATION_PATTERN
-          segments.merge!(segments_per_locale(options[:file], options[:only]))
+      config[:translations].inject([]) do |segments, options|
+        file = options[:file]
+        only = options[:only] || '*'
+        exceptions = [options[:except] || []].flatten
+
+        segment_options = options.slice(:namespace, :pretty_print)
+
+        if file =~ ::I18n::INTERPOLATION_PATTERN
+          segments += segments_per_locale(file, only, exceptions, segment_options)
         else
-          result = segment_for_scope(options[:only])
-          segments[options[:file]] = result unless result.empty?
+          result = segment_for_scope(only, exceptions)
+          segments << Segment.new(file, result, segment_options) unless result.empty?
         end
+
+        segments
       end
     end
 
     def self.filtered_translations
       {}.tap do |result|
-        translation_segments.each do |filename, translations|
-          Utils.deep_merge!(result, translations)
+        translation_segments.each do |segment|
+          Utils.deep_merge!(result, segment.translations)
         end
       end
     end
@@ -79,7 +84,7 @@ module I18n
       if config? && config[:translations]
         configured_segments
       else
-        {"#{DEFAULT_EXPORT_DIR_PATH}/translations.js" => translations}
+        [Segment.new("#{DEFAULT_EXPORT_DIR_PATH}/translations.js", translations)]
       end
     end
 
@@ -99,28 +104,30 @@ module I18n
       File.file? config_file_path
     end
 
-    # Convert translations to JSON string and save file.
-    def self.save(translations, file)
-      FileUtils.mkdir_p File.dirname(file)
-
-      File.open(file, "w+") do |f|
-        f << %(I18n.translations || (I18n.translations = {});\n)
-        Utils.strip_keys_with_nil_values(translations).each do |locale, translations_for_locale|
-          f << %(I18n.translations["#{locale}"] = #{translations_for_locale.to_json};\n);
-        end
-      end
-    end
-
-    def self.scoped_translations(scopes) # :nodoc:
+    def self.scoped_translations(scopes, exceptions = []) # :nodoc:
       result = {}
 
       [scopes].flatten.each do |scope|
-        Utils.deep_merge! result, filter(translations, scope)
+        translations_without_exceptions = exclude(translations, exceptions)
+        filtered_translations = filter(translations_without_exceptions, scope)
+
+        Utils.deep_merge! result, filtered_translations
       end
 
       result
     end
 
+    # Exclude keys from translations listed in the `except:` section in the config file
+    def self.exclude(translations, exceptions)
+      return translations if exceptions.empty?
+
+      exceptions.inject(translations) do |memo, exception|
+        Utils.deep_reject(memo) do |key, value|
+          key.to_s == exception.to_s
+        end
+      end
+    end
+
     # Filter translations according to the specified scope.
     def self.filter(translations, scopes)
       scopes = scopes.split(".") if scopes.is_a?(String)
@@ -160,35 +167,38 @@ module I18n
     end
 
     # deep_merge! given result with result for fallback locale
-    def self.merge_with_fallbacks!(result, locale, scope)
+    def self.merge_with_fallbacks!(result, locale, scope, exceptions)
       result[locale] ||= {}
       fallback_locales = FallbackLocales.new(fallbacks, locale)
 
       fallback_locales.each do |fallback_locale|
-        fallback_result = scoped_translations(scope.collect{|s| "#{fallback_locale}.#{s}"}) # NOTE: Duplicated code here
+        fallback_result = scoped_translations(scope.collect{|s| "#{fallback_locale}.#{s}"}, exceptions) # NOTE: Duplicated code here
         result[locale] = Utils.deep_merge(fallback_result[fallback_locale], result[locale])
       end
     end
 
-
     ### Export i18n.js
     begin
+
       # Copy i18n.js
       def self.export_i18n_js
-        return if export_i18n_js_dir_path.nil?
+        return unless export_i18n_js_dir_path.is_a? String
 
         FileUtils.mkdir_p(export_i18n_js_dir_path)
 
         i18n_js_path = File.expand_path('../../../app/assets/javascripts/i18n.js', __FILE__)
         FileUtils.cp(i18n_js_path, export_i18n_js_dir_path)
       end
-      def self.export_i18n_js_dir_path
-        return @export_i18n_js_dir_path if defined?(@export_i18n_js_dir_path)
 
-        @export_i18n_js_dir_path = DEFAULT_EXPORT_DIR_PATH
+      def self.export_i18n_js_dir_path
+        @export_i18n_js_dir_path ||= (config[:export_i18n_js] || :none) if config.has_key?(:export_i18n_js)
+        @export_i18n_js_dir_path ||= DEFAULT_EXPORT_DIR_PATH
+        @export_i18n_js_dir_path
       end
+
       # Setting this to nil would disable i18n.js exporting
       def self.export_i18n_js_dir_path=(new_path)
+        new_path = :none unless new_path.is_a? String
         @export_i18n_js_dir_path = new_path
       end
     end

data/lib/i18n/js/dependencies.rb

@@ -1,13 +1,15 @@
 module I18n
   module JS
+    # When using `safe_gem_check` to check for a pre-release version of gem,
+    # we need to specify pre-release version suffix in version constraint
     module Dependencies
       class << self
         def rails3?
-          safe_gem_check("rails", "~> 3") && running_rails3?
+          safe_gem_check("rails", "~> 3.0") && running_rails3?
         end
 
         def rails4?
-          safe_gem_check("rails", "~> 4") && running_rails4?
+          safe_gem_check("rails", "~> 4.0", ">= 4.0.0.beta1") && running_rails4?
         end
 
         def sprockets_supports_register_preprocessor?
@@ -19,7 +21,7 @@ module I18n
         end
 
         def rails_available?
-          safe_gem_check("rails", '>= 3')
+          safe_gem_check("rails", '>= 3.0.0.beta')
         end
 
         def using_asset_pipeline?
@@ -49,11 +51,11 @@ module I18n
           defined?(Rails) && Rails.respond_to?(:version)
         end
 
-        def safe_gem_check(gem_name, version_string)
+        def safe_gem_check(*args)
           if Gem::Specification.respond_to?(:find_by_name)
-            Gem::Specification.find_by_name(gem_name, version_string)
+            Gem::Specification.find_by_name(*args)
           elsif Gem.respond_to?(:available?)
-            Gem.available?(gem_name, version_string)
+            Gem.available?(*args)
           end
         rescue Gem::LoadError
           false

data/lib/i18n/js/segment.rb

@@ -0,0 +1,39 @@
+module I18n
+  module JS
+
+    # Class which enscapulates a translations hash and outputs a single JSON translation file
+    class Segment
+      attr_accessor :file, :translations, :namespace, :pretty_print
+
+      def initialize(file, translations, options = {})
+        @file         = file
+        @translations = translations
+        @namespace    = options[:namespace] || 'I18n'
+        @pretty_print = !!options[:pretty_print]
+      end
+
+      # Saves JSON file containing translations
+      def save!
+        FileUtils.mkdir_p File.dirname(self.file)
+
+        File.open(self.file, "w+") do |f|
+          f << %(#{self.namespace}.translations || (#{self.namespace}.translations = {});\n)
+          self.translations.each do |locale, translations|
+            f << %(#{self.namespace}.translations["#{locale}"] = #{print_json(translations)};\n);
+          end
+        end
+      end
+
+      protected
+
+      # Outputs pretty or ugly JSON depending on :pretty_print option
+      def print_json(translations)
+        if pretty_print
+          JSON.pretty_generate(translations)
+        else
+          translations.to_json
+        end
+      end
+    end
+  end
+end

data/lib/i18n/js/utils.rb

@@ -5,11 +5,11 @@ module I18n
       MERGER = proc do |key, v1, v2|
         Hash === v1 && Hash === v2 ? v1.merge(v2, &MERGER) : v2
       end
+
       HASH_NIL_VALUE_CLEANER_PROC = proc do |k, v|
-         v.kind_of?(Hash) ? (v.delete_if(&HASH_NIL_VALUE_CLEANER_PROC); false) : v.nil?
+        v.kind_of?(Hash) ? (v.delete_if(&HASH_NIL_VALUE_CLEANER_PROC); false) : v.nil?
       end
 
-
       def self.strip_keys_with_nil_values(hash)
         hash.dup.delete_if(&HASH_NIL_VALUE_CLEANER_PROC)
       end
@@ -21,6 +21,14 @@ module I18n
       def self.deep_merge!(target_hash, hash) # :nodoc:
         target_hash.merge!(hash, &MERGER)
       end
+
+      def self.deep_reject(hash, &block)
+        hash.each_with_object({}) do |(k, v), memo|
+          unless block.call(k, v)
+            memo[k] = v.kind_of?(Hash) ? deep_reject(v, &block) : v
+          end
+        end
+      end
     end
   end
 end

data/lib/i18n/js/version.rb

@@ -4,7 +4,7 @@ module I18n
       MAJOR = 3
       MINOR = 0
       PATCH = 0
-      STRING = "#{MAJOR}.#{MINOR}.#{PATCH}.rc8"
+      STRING = "#{MAJOR}.#{MINOR}.#{PATCH}.rc9"
     end
   end
 end

data/package.json

@@ -7,5 +7,9 @@
   "main": "app/assets/javascripts/i18n.js",
   "scripts": {
     "test": "./node_modules/.bin/jasmine-node spec/js"
+  },
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/fnando/i18n-js.git"
   }
 }

data/spec/fixtures/except_condition.yml

@@ -0,0 +1,5 @@
+translations:
+  - file: "tmp/i18n-js/trimmed.js"
+    except:
+      - "active_admin"
+      - "mailers"

data/spec/fixtures/js_export_dir_custom.yml

@@ -0,0 +1,6 @@
+
+export_i18n_js: 'tmp/i18n-js/foo'
+
+translations:
+  - file: "tmp/i18n-js/%{locale}.js"
+    only: '*'

data/spec/fixtures/js_export_dir_none.yml

@@ -0,0 +1,6 @@
+
+export_i18n_js: false
+
+translations:
+  - file: "tmp/i18n-js/%{locale}.js"
+    only: '*'

data/spec/fixtures/js_file_with_namespace_and_pretty_print.yml

@@ -0,0 +1,5 @@
+translations:
+  - file: "tmp/i18n-js/%{locale}.js"
+    only: '*'
+    namespace: "Foo"
+    pretty_print: true

data/spec/i18n_js_spec.rb

@@ -29,13 +29,15 @@ describe I18n::JS do
 
     it "exports messages using custom output path" do
       set_config "custom_path.yml"
-      I18n::JS.should_receive(:save).with(translations, "tmp/i18n-js/all.js")
+      I18n::JS::Segment.should_receive(:new).with("tmp/i18n-js/all.js", translations, {}).and_call_original
+      I18n::JS::Segment.any_instance.should_receive(:save!).with(no_args)
       I18n::JS.export
     end
 
     it "sets default scope to * when not specified" do
       set_config "no_scope.yml"
-      I18n::JS.should_receive(:save).with(translations, "tmp/i18n-js/no_scope.js")
+      I18n::JS::Segment.should_receive(:new).with("tmp/i18n-js/no_scope.js", translations, {}).and_call_original
+      I18n::JS::Segment.any_instance.should_receive(:save!).with(no_args)
       I18n::JS.export
     end
 
@@ -59,6 +61,7 @@ describe I18n::JS do
       I18n::JS.export
 
       file_should_exist "en.js"
+      file_should_exist "fr.js"
     end
 
     it "exports with multiple conditions" do
@@ -73,18 +76,23 @@ describe I18n::JS do
 
       set_config "multiple_conditions_per_locale.yml"
 
-      expected_locales = %w(en fr)
-
       result = I18n::JS.translation_segments
-      expected_files = expected_locales.map { |locale| "tmp/i18n-js/bits.#{locale}.js" }
-      result.keys.should eql(expected_files)
+      result.map(&:file).should eql(["tmp/i18n-js/bits.en.js", "tmp/i18n-js/bits.fr.js"])
 
-      expected_locales.each do |lang|
-        result["tmp/i18n-js/bits.#{lang}.js"].keys.should eql([lang.to_sym])
-        result["tmp/i18n-js/bits.#{lang}.js"][lang.to_sym].keys.sort.should eql([:date, :number])
+      %w(en fr).each do |lang|
+        segment = result.select{|s| s.file == "tmp/i18n-js/bits.#{lang}.js"}.first
+        segment.translations.keys.should eql([lang.to_sym])
+        segment.translations[lang.to_sym].keys.sort.should eql([:date, :number])
       end
     end
 
+    it "exports with :except condition" do
+      set_config "except_condition.yml"
+      I18n::JS.export
+
+      file_should_exist "trimmed.js"
+    end
+
     it "calls .export_i18n_js" do
       allow(described_class).to receive(:export_i18n_js)
       I18n::JS.export
@@ -132,33 +140,67 @@ describe I18n::JS do
     end
   end
 
+  context "exceptions" do
+    it "does not include a key listed in the exceptions list" do
+      result = I18n::JS.scoped_translations("*", ['admin'])
+
+      result[:en][:admin].should be_nil
+      result[:fr][:admin].should be_nil
+    end
+
+    it "does not include multiple keys listed in the exceptions list" do
+      result = I18n::JS.scoped_translations("*", ['title', 'note'])
+
+      result[:en][:admin][:show].should be_empty
+      result[:en][:admin][:edit].should be_empty
+
+      result[:fr][:admin][:show].should be_empty
+      result[:fr][:admin][:show].should be_empty
+      result[:fr][:admin][:edit].should be_empty
+    end
+
+    it "does not include a key listed in the exceptions list and respecs the 'only' option" do
+      result = I18n::JS.scoped_translations("fr.*", ['date', 'time', 'number', 'show'])
+
+      result[:en].should be_nil
+      result[:de].should be_nil
+      result[:ja].should be_nil
+
+      result[:fr][:date].should be_nil
+      result[:fr][:time].should be_nil
+      result[:fr][:number].should be_nil
+      result[:fr][:admin][:show].should be_nil
+
+      result[:fr][:admin][:edit][:title].should be_a(String)
+    end
+  end
+
   context "fallbacks" do
+    subject do
+      I18n::JS.translation_segments.inject({}) do |hash, segment|
+        hash[segment.file] = segment.translations
+        hash
+      end
+    end
+
     it "exports without fallback when disabled" do
       set_config "js_file_per_locale_without_fallbacks.yml"
-
-      result = I18n::JS.translation_segments
-      result["tmp/i18n-js/fr.js"][:fr][:fallback_test].should eql(nil)
+      subject["tmp/i18n-js/fr.js"][:fr][:fallback_test].should eql(nil)
     end
 
     it "exports with default_locale as fallback when enabled" do
       set_config "js_file_per_locale_with_fallbacks_enabled.yml"
-
-      result = I18n::JS.translation_segments
-      result["tmp/i18n-js/fr.js"][:fr][:fallback_test].should eql("Success")
+      subject["tmp/i18n-js/fr.js"][:fr][:fallback_test].should eql("Success")
     end
 
     it "exports with default_locale as fallback when enabled with :default_locale" do
       set_config "js_file_per_locale_with_fallbacks_as_default_locale_symbol.yml"
-
-      result = I18n::JS.translation_segments
-      result["tmp/i18n-js/fr.js"][:fr][:fallback_test].should eql("Success")
+      subject["tmp/i18n-js/fr.js"][:fr][:fallback_test].should eql("Success")
     end
 
     it "exports with given locale as fallback" do
       set_config "js_file_per_locale_with_fallbacks_as_locale.yml"
-
-      result = I18n::JS.translation_segments
-      result["tmp/i18n-js/fr.js"][:fr][:fallback_test].should eql("Erfolg")
+      subject["tmp/i18n-js/fr.js"][:fr][:fallback_test].should eql("Erfolg")
     end
 
     context "with I18n::Fallbacks enabled" do
@@ -173,27 +215,53 @@ describe I18n::JS do
 
       it "exports with defined locale as fallback when enabled" do
         set_config "js_file_per_locale_with_fallbacks_enabled.yml"
-
-        result = I18n::JS.translation_segments
-        result["tmp/i18n-js/fr.js"][:fr][:fallback_test].should eql("Erfolg")
+        subject["tmp/i18n-js/fr.js"][:fr][:fallback_test].should eql("Erfolg")
       end
 
       it "exports with defined locale as fallback when enabled with :default_locale" do
         set_config "js_file_per_locale_with_fallbacks_as_default_locale_symbol.yml"
-
-        result = I18n::JS.translation_segments
-        result["tmp/i18n-js/fr.js"][:fr][:fallback_test].should eql("Success")
+        subject["tmp/i18n-js/fr.js"][:fr][:fallback_test].should eql("Success")
       end
 
       it "exports with Fallbacks as Hash" do
         set_config "js_file_per_locale_with_fallbacks_as_hash.yml"
-
-        result = I18n::JS.translation_segments
-        result["tmp/i18n-js/fr.js"][:fr][:fallback_test].should eql("Erfolg")
+        subject["tmp/i18n-js/fr.js"][:fr][:fallback_test].should eql("Erfolg")
       end
     end
   end
 
+  context "namespace and pretty_print options" do
+
+    before do
+      stub_const('I18n::JS::DEFAULT_EXPORT_DIR_PATH', temp_path)
+      set_config "js_file_with_namespace_and_pretty_print.yml"
+    end
+
+    it "exports with defined locale as fallback when enabled" do
+      I18n::JS.export
+      file_should_exist "en.js"
+      output = File.read(File.join(I18n::JS.export_i18n_js_dir_path, "en.js"))
+      expect(output).to match(/^#{
+<<EOS
+Foo.translations || (Foo.translations = {});
+Foo.translations["en"] = {
+  "number": {
+      "format": {
+EOS
+}.+#{
+<<EOS
+    "edit": {
+      "title": "Edit"
+    }
+  },
+  "foo": "Foo",
+  "fallback_test": "Success"
+};
+EOS
+}$/)
+    end
+  end
+
   context "I18n.available_locales" do
     context "when I18n.available_locales is not set" do
       it "should allow all locales" do
@@ -236,9 +304,10 @@ describe I18n::JS do
     end
   end
 
-
   describe "i18n.js exporting" do
-    describe ".export_i18n_js" do
+    after { begin described_class.send(:remove_instance_variable, :@export_i18n_js_dir_path); rescue; end }
+
+    describe ".export_i18n_js with global variable" do
       before do
         allow(FileUtils).to receive(:mkdir_p).and_call_original
         allow(FileUtils).to receive(:cp).and_call_original
@@ -273,11 +342,47 @@ describe I18n::JS do
       end
     end
 
+    describe ".export_i18n_js with config" do
+
+      let(:export_action) do
+        allow(FileUtils).to receive(:mkdir_p).and_call_original
+        allow(FileUtils).to receive(:cp).and_call_original
+        I18n::JS.export_i18n_js
+      end
+
+      context 'when :export_i18n_js set in config' do
+        before { set_config "js_export_dir_custom.yml"; export_action }
+        let(:export_i18n_js_dir_path) { temp_path }
+        let(:config_export_path) { "tmp/i18n-js/foo" }
+
+        it "does create the folder before copying" do
+          expect(FileUtils).to have_received(:mkdir_p).with(config_export_path).once
+        end
+        it "does copy the file with FileUtils.cp" do
+          expect(FileUtils).to have_received(:cp).once
+        end
+        it "exports the file" do
+          File.should be_file(File.join(config_export_path, "i18n.js"))
+        end
+      end
+
+      context 'when .export_i18n_js_dir_path is set to false' do
+        before { set_config "js_export_dir_none.yml"; export_action }
+
+        it "does NOT create the folder before copying" do
+          expect(FileUtils).to_not have_received(:mkdir_p)
+        end
+
+        it "does NOT copy the file with FileUtils.cp" do
+          expect(FileUtils).to_not have_received(:cp)
+        end
+      end
+    end
 
     describe '.export_i18n_js_dir_path' do
       let(:default_path) { I18n::JS::DEFAULT_EXPORT_DIR_PATH }
       let(:new_path) { File.join("tmp", default_path) }
-      before { described_class.send(:remove_instance_variable, :@export_i18n_js_dir_path) }
+      after { described_class.send(:remove_instance_variable, :@export_i18n_js_dir_path) }
 
       subject { described_class.export_i18n_js_dir_path }
 
@@ -292,92 +397,8 @@ describe I18n::JS do
       context "when it is set to nil already" do
         before { described_class.export_i18n_js_dir_path = nil }
 
-        it { should be_nil }
-      end
-    end
-  end
-end
-
-describe I18n::JS::Dependencies, ".sprockets_supports_register_preprocessor?" do
-
-  subject { described_class.sprockets_supports_register_preprocessor? }
-
-  context 'when Sprockets is available to register preprocessors' do
-    let!(:sprockets_double) do
-      class_double('Sprockets').as_stubbed_const(register_processor: true).tap do |double|
-        allow(double).to receive(:respond_to?).with(:register_preprocessor).and_return(true)
+        it { should eq :none }
       end
     end
-
-    it { is_expected.to be_truthy }
-    it 'calls respond_to? with register_preprocessor on Sprockets' do
-      expect(sprockets_double).to receive(:respond_to?).with(:register_preprocessor).and_return(true)
-      subject
-    end
   end
-
-  context 'when Sprockets is NOT available to register preprocessors' do
-    let!(:sprockets_double) do
-      class_double('Sprockets').as_stubbed_const(register_processor: true).tap do |double|
-        allow(double).to receive(:respond_to?).with(:register_preprocessor).and_return(false)
-      end
-    end
-
-    it { is_expected.to be_falsy }
-    it 'calls respond_to? with register_preprocessor on Sprockets' do
-      expect(sprockets_double).to receive(:respond_to?).with(:register_preprocessor).and_return(false)
-      subject
-    end
-  end
-
-  context 'when Sprockets is missing' do
-    before do
-      hide_const('Sprockets')
-      expect { Sprockets }.to raise_error(NameError)
-    end
-
-    it { is_expected.to be_falsy }
-  end
-
-end
-
-describe I18n::JS::Utils do
-
-  describe ".strip_keys_with_nil_values" do
-    subject { described_class.strip_keys_with_nil_values(input_hash) }
-
-    context 'when input_hash does NOT contain nil value' do
-      let(:input_hash) { {a: 1, b: { c: 2 }} }
-      let(:expected_hash) { input_hash }
-
-      it 'returns the original input' do
-        is_expected.to eq expected_hash
-      end
-    end
-    context 'when input_hash does contain nil value' do
-      let(:input_hash) { {a: 1, b: { c: 2, d: nil }, e: { f: nil }} }
-      let(:expected_hash) { {a: 1, b: { c: 2 }, e: {}} }
-
-      it 'returns the original input with nil values removed' do
-        is_expected.to eq expected_hash
-      end
-    end
-  end
-
-  context "hash merging" do
-    it "performs a deep merge" do
-      target = {:a => {:b => 1}}
-      result = described_class.deep_merge(target, {:a => {:c => 2}})
-
-      result[:a].should eql({:b => 1, :c => 2})
-    end
-
-    it "performs a banged deep merge" do
-      target = {:a => {:b => 1}}
-      described_class.deep_merge!(target, {:a => {:c => 2}})
-
-      target[:a].should eql({:b => 1, :c => 2})
-    end
-  end
-
 end