Add test to check that files are recompiled if dependencies and sub dependencies change

Author: brendon

lib/less/rails/import_processor.rb

@@ -4,13 +4,13 @@ class ImportProcessor
 
  IMPORT_SCANNER = /@import\s*['"]([^'"]+)['"]\s*;/.freeze
 
- PATHNAME_FINDER = Proc.new { |scope, path|
+ PATHNAME_FINDER = Proc.new do |scope, path|
  begin
  scope.resolve(path)
  rescue Sprockets::FileNotFound
  nil
  end
- }
+ end
 
  def initialize(filename, &block)
  @filename = filename
@@ -38,7 +38,7 @@ def self.call(input)
  def self.depend_on(scope, source, base=File.dirname(scope.logical_path))
  import_paths = source.scan(IMPORT_SCANNER).flatten.compact.uniq
  import_paths.each do |path|
- pathname = PATHNAME_FINDER.call(scope,path) || PATHNAME_FINDER.call(scope, File.join(base, path))
+ pathname = PATHNAME_FINDER.call(scope, path) || PATHNAME_FINDER.call(scope, File.join(base, path))
  scope.depend_on(pathname) if pathname && pathname.to_s.ends_with?('.less')
  depend_on scope, File.read(pathname), File.dirname(path) if pathname
  end

test/cases/basics_spec.rb

@@ -1,36 +1,54 @@
 require 'spec_helper'
 
 class BasicsSpec < Less::Rails::Spec
- 
+
  it 'must render variables' do
  basics.must_match %r{#test-variable\{color:#4d926f\}}
  end
- 
+
  it 'must render mixins' do
  basics.must_match %r{#test-mixin span\{border:1px solid black\}}
  end
- 
+
  it 'must be able to use vendored less files' do
  basics.must_match %r{#test-vendored\{border-radius:10px\}}
  end
- 
+
  describe 'less import dependency hooks' do
- 
+
  it 'must update when imported file changes' do
  basics.must_match %r{#test-radiused\{border-radius:5px\}}, 'default is 5px'
  safely_edit(:mixins) do |data, asset|
- data.gsub! '5px', '10px'
+ data.gsub! '5px', '8px'
  File.open(asset.pathname,'w') { |f| f.write(data) }
- basics.must_match %r{#test-radiused\{border-radius:10px\}}, 'mixins.less should be a sprockets context dependency'
+ basics.must_match %r{#test-radiused\{border-radius:8px\}}, 'mixins.less should be a sprockets context dependency'
+
+ data.gsub! '8px', '15px'
+ File.open(asset.pathname,'w') { |f| f.write(data) }
+
+ # Force a recompile
+ # https://github.com/rails/sprockets/blob/master/test/shared_sass_tests.rb#L164
+ mtime = Time.now + 1
+ File.utime(mtime, mtime, asset.pathname)
+
+ basics.must_match %r{#test-radiused\{border-radius:15px\}}, 'mixins.less should be a sprockets context dependency'
  end
  end
- 
+
  it 'must update when an imported file of another imported file changes' do
  basics.must_match %r{#test-variable-colored\{color:#424242\}}, 'default is #424242'
  safely_edit(:variables) do |data, asset|
  data.gsub! '424242', '666666'
  File.open(asset.pathname,'w') { |f| f.write(data) }
  basics.must_match %r{#test-variable-colored\{color:#666\}}, 'variables.less should be a sprockets context dependency'
+
+ data.gsub! '666666', '888888'
+ File.open(asset.pathname,'w') { |f| f.write(data) }
+
+ mtime = Time.now + 1
+ File.utime(mtime, mtime, asset.pathname)
+
+ basics.must_match %r{#test-variable-colored\{color:#888\}}, 'variables.less should be a sprockets context dependency'
  end
  end
 
@@ -61,15 +79,15 @@ class BasicsSpec < Less::Rails::Spec
  end
 
  protected
- 
+
  def basics
  dummy_asset 'basics'
  end
- 
+
  def mixins_asset
  dummy_assets['frameworks/bootstrap/mixins.less']
  end
- 
+
  def variables_asset
  dummy_assets['frameworks/bootstrap/variables.less']
  end