sassc-rails 0.0.6 → 0.0.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 607d6ebf5f495a80e3d4cada4f6ee28939db8221
-  data.tar.gz: 846873d730c106fd751f70fe94db8adff1864465
+  metadata.gz: 078582cf7ffcfded8c2ad89c4b79eb42edd7d36f
+  data.tar.gz: 73a3aa78f26ff4458327c91f8cfd26344c9969d6
 SHA512:
-  metadata.gz: 9d7bcd45f41f36c9cbbce561dba0746ac67ae3f233a86c6ff3165f665f4e0fa47bca4198302313a4fb6acb6e2a156bfbddd4efbaaa89e8c765388165244e7fcf
-  data.tar.gz: 2f6b7d28784bd732876b831d75959b285968b4b26b585e8edc55e5ceefdcb7c5551fbebd8b0ff34a75a1b9518dcafeb5855a3d9f31d01b128a6498b43aef469f
+  metadata.gz: 898c33b217fadacde7a691e9eeb863af815f41babd9e384db38096a4043d312fff175dc2b1b1b22df882edf18905a9f4676b7d64e56c56218cc52e82b23bf2f7
+  data.tar.gz: 1548c14632589db9b8124806e46b25e4882d624f40589446ddb4923b39015051b5b6b2193e392e3dd7648c851dd5ef7a8c193faa1fac4b92a590c48f3f2672ee

data/lib/sassc/rails/importer.rb

@@ -10,7 +10,7 @@ module SassC
           @postfix = postfix
         end
 
-        def import_for(original_path, parent_path, full_path, options)
+        def import_for(full_path, parent_dir, options)
           SassC::Importer::Import.new(full_path)
         end
       end
@@ -20,7 +20,7 @@ module SassC
           ".css"
         end
 
-        def import_for(original_path, parent_path, full_path, options)
+        def import_for(full_path, parent_dir, options)
           import_path = full_path.gsub(/\.css$/,"")
           SassC::Importer::Import.new(import_path)
         end
@@ -31,7 +31,7 @@ module SassC
           ".css.scss"
         end
 
-        def import_for(original_path, parent_path, full_path, options)
+        def import_for(full_path, parent_dir, options)
           source = File.open(full_path, 'rb') { |f| f.read }
           SassC::Importer::Import.new(full_path, source: source)
         end
@@ -42,7 +42,7 @@ module SassC
           ".css.sass"
         end
 
-        def import_for(original_path, parent_path, full_path, options)
+        def import_for(full_path, parent_dir, options)
           sass = File.open(full_path, 'rb') { |f| f.read }
           parsed_scss = SassC::Sass2Scss.convert(sass)
           SassC::Importer::Import.new(full_path, source: parsed_scss)
@@ -54,7 +54,7 @@ module SassC
           ".sass.erb"
         end
 
-        def import_for(original_path, parent_path, full_path, options)
+        def import_for(full_path, parent_dir, options)
           template = Tilt::ERBTemplate.new(full_path)
           parsed_erb = template.render(options[:sprockets][:context], {})
           parsed_scss = SassC::Sass2Scss.convert(parsed_erb)
@@ -63,7 +63,7 @@ module SassC
       end
 
       class ERBExtension < Extension
-        def import_for(original_path, parent_path, full_path, options)
+        def import_for(full_path, parent_dir, options)
           template = Tilt::ERBTemplate.new(full_path)
           parsed_erb = template.render(options[:sprockets][:context], {})
           SassC::Importer::Import.new(full_path, source: parsed_erb)
@@ -71,22 +71,29 @@ module SassC
       end
 
       EXTENSIONS = [
+        CssScssExtension.new,
+        CssSassExtension.new,
         Extension.new(".scss"),
         Extension.new(".sass"),
         CSSExtension.new,
         ERBExtension.new(".scss.erb"),
         ERBExtension.new(".css.erb"),
-        SassERBExtension.new,
-        CssScssExtension.new,
-        CssSassExtension.new
+        SassERBExtension.new
       ]
 
       PREFIXS = [ "", "_" ]
+      GLOB = /(\A|\/)(\*|\*\*\/\*)\z/
 
       def imports(path, parent_path)
         parent_dir, _ = File.split(parent_path)
         specified_dir, specified_file = File.split(path)
 
+        if m = path.match(GLOB)
+          path = path.sub(m[0], "")
+          base = File.expand_path(path, File.dirname(parent_path))
+          return glob_imports(base, m[2], parent_path)
+        end
+
         search_paths = ([parent_dir] + load_paths).uniq
 
         if specified_dir != "."
@@ -103,17 +110,24 @@ module SassC
               try_path = File.join(search_path, file_name + extension.postfix)
               if File.exists?(try_path)
                 record_import_as_dependency try_path
-                return extension.import_for(path, parent_path, try_path, options)
+                return extension.import_for(try_path, parent_dir, options)
               end
             end
           end
         end
 
-        Import.new(path)
+        # TODO: Raise an error from SassC here
+        raise ArgumentError.new("file not found: #{path}")
       end
 
       private
 
+      def extension_for_file(file)
+        EXTENSIONS.detect do |extension|
+          file.include? extension.postfix
+        end
+      end
+
       def record_import_as_dependency(path)
         context.depend_on path
       end
@@ -125,6 +139,39 @@ module SassC
       def load_paths
         options[:load_paths]
       end
+
+      def glob_imports(base, glob, current_file)
+        files = globbed_files(base, glob)
+        files = files.reject { |f| f == current_file }
+
+        files.map do |filename|
+          record_import_as_dependency(filename)
+          extension = extension_for_file(filename)
+          extension.import_for(filename, base, options)
+        end
+      end
+
+      def globbed_files(base, glob)
+        # TODO: Raise an error from SassC here
+        raise ArgumentError unless glob == "*" || glob == "**/*"
+
+        extensions = EXTENSIONS.map(&:postfix)
+        exts = extensions.map { |ext| Regexp.escape("#{ext}") }.join("|")
+        sass_re = Regexp.compile("(#{exts})$")
+
+        record_import_as_dependency(base)
+
+        files = Dir["#{base}/#{glob}"].sort.map do |path|
+          if File.directory?(path)
+            record_import_as_dependency(path)
+            nil
+          elsif sass_re =~ path
+            path
+          end
+        end
+
+        files.compact
+      end
     end
   end
 end

data/lib/sassc/rails/version.rb

@@ -1,5 +1,5 @@
 module SassC
   module Rails
-    VERSION = "0.0.6"
+    VERSION = "0.0.7"
   end
 end

data/sassc-rails.gemspec

@@ -26,7 +26,7 @@ Gem::Specification.new do |spec|
   # unfortunately we require sass for now, so that we can
   # reuse portions of the sprockets template
   spec.add_dependency 'sass'
-  spec.add_dependency "sassc", "1.1.1"
+  spec.add_dependency "sassc", "~> 1.1.2"
 
   spec.add_dependency "tilt"
 

data/test/dummy/app/assets/stylesheets/glob_test.scss

@@ -0,0 +1 @@
+@import "globbed/**/*";

data/test/dummy/app/assets/stylesheets/imports_test.scss

@@ -1,7 +1,7 @@
 @import "partials/css_sass_import";
 @import "partials/sass_import";
 @import "partials/scss_import";
-// //@import "globbed/**/*";
+@import "globbed/**/*";
 @import "subfolder/plain";
 @import "subfolder/second_level";
 @import "partials/without_css_ext";

data/test/sassc_rails_test.rb

@@ -133,9 +133,8 @@ class SassRailsTest < MiniTest::Unit::TestCase
 
     assert_match /default-old-css/,          css_output
 
-    # skip for now
-    # assert_match /globbed/,                  css_output
-    # assert_match /nested-glob/,              css_output
+    assert_match /globbed/,                  css_output
+    assert_match /nested-glob/,              css_output
   end
 
   def test_style_config_item_is_defaulted_to_expanded_in_development_mode
@@ -198,41 +197,50 @@ class SassRailsTest < MiniTest::Unit::TestCase
   #  assert_match /\.import-css-application/, css_output
   #end
 
-  #test 'globbed imports work when new file is added' do
-  #  skip
+  def test_globbed_imports_work_when_globbed_file_is_changed
+    skip "This seems to work in practice, possible test setup problem"
 
-  #  project = 'scss_project'
-  #  filename = 'application.scss'
+    begin
+      initialize!
 
-  #  within_rails_app(project) do |tmpdir|
-  #    asset_output(filename)
+      new_file = File.join(File.dirname(__FILE__), 'dummy', 'app', 'assets', 'stylesheets', 'globbed', 'new_glob.scss')
 
-  #    new_file = File.join(tmpdir, 'app', 'assets', 'stylesheets', 'globbed', 'new.scss')
-  #    File.open(new_file, 'w') do |file|
-  #      file.puts '.new-file-test { color: #000; }'
-  #    end
+      File.open(new_file, 'w') do |file|
+        file.puts '.new-file-test { color: #000; }'
+      end
 
-  #    css_output = asset_output(filename)
-  #    assert_match /new-file-test/, css_output
-  #  end
-  #end
+      css_output = render_asset("glob_test.scss")
+      assert_match /new-file-test/, css_output
 
-  #test 'globbed imports work when globbed file is changed' do
-  #  skip
+      File.open(new_file, 'w') do |file|
+        file.puts '.changed-file-test { color: #000; }'
+      end
 
-  #  project = 'scss_project'
-  #  filename = 'application.scss'
+      new_css_output = render_asset("glob_test.scss")
+      assert_match /changed-file-test/, new_css_output
+      refute_equal css_output, new_css_output
+    ensure
+      File.delete(new_file)
+    end
+  end
 
-  #  within_rails_app(project) do |tmpdir|
-  #    asset_output(filename)
+  def test_globbed_imports_work_when_globbed_file_is_added
+    begin
+      initialize!
 
-  #    new_file = File.join(tmpdir, 'app', 'assets', 'stylesheets', 'globbed', 'globbed.scss')
-  #    File.open(new_file, 'w') do |file|
-  #      file.puts '.changed-file-test { color: #000; }'
-  #    end
+      css_output = render_asset("glob_test.scss")
+      refute_match /changed-file-test/, css_output
+      new_file = File.join(File.dirname(__FILE__), 'dummy', 'app', 'assets', 'stylesheets', 'globbed', 'new_glob.scss')
 
-  #    css_output = asset_output(filename)
-  #    assert_match /changed-file-test/, css_output
-  #  end
-  #end
+      File.open(new_file, 'w') do |file|
+        file.puts '.changed-file-test { color: #000; }'
+      end
+
+      new_css_output = render_asset("glob_test.scss")
+      assert_match /changed-file-test/, new_css_output
+      refute_equal css_output, new_css_output
+    ensure
+      File.delete(new_file)
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sassc-rails
 version: !ruby/object:Gem::Version
-  version: 0.0.6
+  version: 0.0.7
 platform: ruby
 authors:
 - Ryan Boland
@@ -84,16 +84,16 @@ dependencies:
   name: sassc
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.1.1
+        version: 1.1.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.1.1
+        version: 1.1.2
 - !ruby/object:Gem::Dependency
   name: tilt
   requirement: !ruby/object:Gem::Requirement
@@ -176,6 +176,7 @@ files:
 - test/dummy/app/assets/stylesheets/css_scss_erb_handler.scss.erb
 - test/dummy/app/assets/stylesheets/css_scss_handler.css.scss
 - test/dummy/app/assets/stylesheets/erb_render_with_context.css.erb
+- test/dummy/app/assets/stylesheets/glob_test.scss
 - test/dummy/app/assets/stylesheets/globbed/globbed.scss
 - test/dummy/app/assets/stylesheets/globbed/nested/nested_glob.scss
 - test/dummy/app/assets/stylesheets/helpers_test.scss
@@ -447,6 +448,7 @@ test_files:
 - test/dummy/app/assets/stylesheets/css_scss_erb_handler.scss.erb
 - test/dummy/app/assets/stylesheets/css_scss_handler.css.scss
 - test/dummy/app/assets/stylesheets/erb_render_with_context.css.erb
+- test/dummy/app/assets/stylesheets/glob_test.scss
 - test/dummy/app/assets/stylesheets/globbed/globbed.scss
 - test/dummy/app/assets/stylesheets/globbed/nested/nested_glob.scss
 - test/dummy/app/assets/stylesheets/helpers_test.scss