Merge pull request #338 from jsilvestri/nil-compression

Allow no compression for all environments

lib/sass/rails/railtie.rb

@@ -72,7 +72,9 @@ class Railtie < ::Rails::Railtie
 
     initializer :setup_compression, group: :all do |app|
       unless Rails.env.development?
-        app.config.assets.css_compressor ||= :sass
+        # config.assets.css_compressor may be set to nil in non-dev environments.
+        # otherwise, the default is sass compression.
+        app.config.assets.css_compressor = :sass unless app.config.assets.has_key?(:css_compressor)
       else
         # Use expanded output instead of the sass default of :nested unless specified
         app.config.sass.style ||= :expanded

test/fixtures/alternate_config_project/config/environments/test.rb

@@ -39,4 +39,7 @@
 
   # Print deprecation notices to the stderr
   config.active_support.deprecation = :stderr
+
+  # Decide not to use sass compression
+  config.assets.css_compressor = nil
 end

test/sass_rails_test.rb

@@ -76,6 +76,16 @@ class SassRailsTest < Sass::Rails::TestCase
     end
   end
 
+  test 'sass allows compressor override in test mode' do
+    within_rails_app 'alternate_config_project' do
+      runner 'test' do
+        "puts Rails.application.config.assets.css_compressor.nil?"
+      end
+
+      assert_equal 'true', $last_output.chomp
+    end
+  end
+
   test 'sass defines compressor by default in production mode' do
     within_rails_app 'scss_project' do
       runner 'production' do