aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJosé Valim <jose.valim@gmail.com>2011-12-12 14:53:25 +0100
committerJosé Valim <jose.valim@gmail.com>2011-12-12 14:57:09 +0100
commit57e0c038d627fe0c5d85883a0a29c17257f9c4e5 (patch)
tree95c7364ea3a843314b9935534fd3dffdc00ba983
parent5c234ab8ed4659a74a6c47ce8337055a62f75eff (diff)
downloadrails-57e0c038d627fe0c5d85883a0a29c17257f9c4e5.tar.gz
rails-57e0c038d627fe0c5d85883a0a29c17257f9c4e5.tar.bz2
rails-57e0c038d627fe0c5d85883a0a29c17257f9c4e5.zip
Allow FileUpdateChecker to work with globs.
-rw-r--r--activesupport/lib/active_support/file_update_checker.rb4
-rw-r--r--activesupport/test/file_update_checker_test.rb24
2 files changed, 22 insertions, 6 deletions
diff --git a/activesupport/lib/active_support/file_update_checker.rb b/activesupport/lib/active_support/file_update_checker.rb
index f76ddff038..9d617b39d4 100644
--- a/activesupport/lib/active_support/file_update_checker.rb
+++ b/activesupport/lib/active_support/file_update_checker.rb
@@ -22,7 +22,9 @@ module ActiveSupport
end
def updated_at
- paths.map { |path| File.mtime(path) }.max
+ # TODO: Use Enumerable check once we get rid of 1.8.7
+ all = paths.is_a?(Array) ? paths : Dir[paths]
+ all.map { |path| File.mtime(path) }.max
end
def execute_if_updated
diff --git a/activesupport/test/file_update_checker_test.rb b/activesupport/test/file_update_checker_test.rb
index b65bb1d024..425931f49a 100644
--- a/activesupport/test/file_update_checker_test.rb
+++ b/activesupport/test/file_update_checker_test.rb
@@ -4,7 +4,7 @@ require 'fileutils'
MTIME_FIXTURES_PATH = File.expand_path("../fixtures", __FILE__)
-class FileUpdateCheckerTest < Test::Unit::TestCase
+module FileUpdateCheckerSuite
FILES = %w(1.txt 2.txt 3.txt)
def setup
@@ -15,6 +15,10 @@ class FileUpdateCheckerTest < Test::Unit::TestCase
FileUtils.rm(FILES)
end
+ def args
+ raise NotImplementedError
+ end
+
def test_should_not_execute_the_block_if_no_paths_are_given
i = 0
checker = ActiveSupport::FileUpdateChecker.new([]){ i += 1 }
@@ -24,28 +28,28 @@ class FileUpdateCheckerTest < Test::Unit::TestCase
def test_should_invoke_the_block_on_first_call_if_it_does_not_calculate_last_updated_at_on_load
i = 0
- checker = ActiveSupport::FileUpdateChecker.new(FILES){ i += 1 }
+ checker = ActiveSupport::FileUpdateChecker.new(args){ i += 1 }
checker.execute_if_updated
assert_equal 1, i
end
def test_should_not_invoke_the_block_on_first_call_if_it_calculates_last_updated_at_on_load
i = 0
- checker = ActiveSupport::FileUpdateChecker.new(FILES, true){ i += 1 }
+ checker = ActiveSupport::FileUpdateChecker.new(args, true){ i += 1 }
checker.execute_if_updated
assert_equal 0, i
end
def test_should_not_invoke_the_block_if_no_file_has_changed
i = 0
- checker = ActiveSupport::FileUpdateChecker.new(FILES){ i += 1 }
+ checker = ActiveSupport::FileUpdateChecker.new(args){ i += 1 }
5.times { checker.execute_if_updated }
assert_equal 1, i
end
def test_should_invoke_the_block_if_a_file_has_changed
i = 0
- checker = ActiveSupport::FileUpdateChecker.new(FILES){ i += 1 }
+ checker = ActiveSupport::FileUpdateChecker.new(args){ i += 1 }
checker.execute_if_updated
sleep(1)
FileUtils.touch(FILES)
@@ -53,3 +57,13 @@ class FileUpdateCheckerTest < Test::Unit::TestCase
assert_equal 2, i
end
end
+
+class FileUpdateCheckerWithEnumerableTest < Test::Unit::TestCase
+ include FileUpdateCheckerSuite
+ def args; FILES; end
+end
+
+class FileUpdateCheckerWithStringTest < Test::Unit::TestCase
+ include FileUpdateCheckerSuite
+ def args; "{1,2,3}.txt"; end
+end