diff options
author | Godfrey Chan <godfreykfc@gmail.com> | 2015-11-27 01:29:53 -0800 |
---|---|---|
committer | Godfrey Chan <godfreykfc@gmail.com> | 2015-11-27 01:32:38 -0800 |
commit | 9d7d12c0044751a494162760cd018fb66eab200f (patch) | |
tree | 1ea2bec82a759a65cb982d2917ded8d1546407f7 | |
parent | 6be6d8d234015fb753db5b5f7c195d5d08e13e10 (diff) | |
download | rails-9d7d12c0044751a494162760cd018fb66eab200f.tar.gz rails-9d7d12c0044751a494162760cd018fb66eab200f.tar.bz2 rails-9d7d12c0044751a494162760cd018fb66eab200f.zip |
Partially revert 14b20ce
The failure have returned, so disabling the tests again. (The upgrade is
fine to keep.)
-rw-r--r-- | activesupport/test/file_evented_update_checker_test.rb | 5 | ||||
-rwxr-xr-x | ci/travis.rb | 10 |
2 files changed, 13 insertions, 2 deletions
diff --git a/activesupport/test/file_evented_update_checker_test.rb b/activesupport/test/file_evented_update_checker_test.rb index 071449d399..ec3a7e28f3 100644 --- a/activesupport/test/file_evented_update_checker_test.rb +++ b/activesupport/test/file_evented_update_checker_test.rb @@ -5,6 +5,11 @@ require 'file_update_checker_shared_tests' class FileEventedUpdateCheckerTest < ActiveSupport::TestCase include FileUpdateCheckerSharedTests + def setup + skip if ENV['LISTEN'] == '0' + super + end + def new_checker(files = [], dirs = {}, &block) ActiveSupport::FileEventedUpdateChecker.new(files, dirs, &block).tap do wait diff --git a/ci/travis.rb b/ci/travis.rb index b29e06dd5c..658d66c6b4 100755 --- a/ci/travis.rb +++ b/ci/travis.rb @@ -111,8 +111,14 @@ class Build end def env - # Provide extra ENV variables for the build here - {} + if activesupport? && !isolated? + # There is a known issue with the listen tests that casuses files to be + # incorrectly GC'ed even when they are still in-use. The current is to + # only run them in isolation to avoid randomly failing our test suite. + { 'LISTEN' => '0' } + else + {} + end end def run_bug_report_templates |