From 6b6b27126d7ecc1dde9a6bc166a7e1daa3af35d2 Mon Sep 17 00:00:00 2001 From: Markus Reiter Date: Fri, 21 Oct 2016 08:57:39 +0200 Subject: Reorder and rename test files. --- Library/Homebrew/dev-cmd/tests.rb | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) (limited to 'Library/Homebrew/dev-cmd') diff --git a/Library/Homebrew/dev-cmd/tests.rb b/Library/Homebrew/dev-cmd/tests.rb index ee7fa9275..df9a00a16 100644 --- a/Library/Homebrew/dev-cmd/tests.rb +++ b/Library/Homebrew/dev-cmd/tests.rb @@ -8,7 +8,7 @@ module Homebrew module_function def tests - (HOMEBREW_LIBRARY/"Homebrew").cd do + HOMEBREW_LIBRARY_PATH.cd do ENV.delete "HOMEBREW_VERBOSE" ENV.delete "VERBOSE" ENV["HOMEBREW_NO_ANALYTICS_THIS_RUN"] = "1" @@ -44,8 +44,8 @@ module Homebrew # Make it easier to reproduce test runs. ENV["SEED"] = ARGV.next if ARGV.include? "--seed" - files = Dir["test/test_*.rb"] - files -= Dir["test/test_os_mac_*.rb"] unless OS.mac? + files = Dir.glob("test/test/**/*_test.rb") + .reject { |p| !OS.mac? && p.start_with?("test/test/os/mac/") } opts = [] opts << "--serialize-stdout" if ENV["CI"] @@ -54,16 +54,14 @@ module Homebrew args << "--trace" if ARGV.include? "--trace" if ARGV.value("only") - ENV["HOMEBREW_TESTS_ONLY"] = "1" - test_name, test_method = ARGV.value("only").split("/", 2) - files = ["test/test_#{test_name}.rb"] + test_name, test_method = ARGV.value("only").split(":", 2) + files = Dir.glob("test/test/{#{test_name},#{test_name}/**/*}_test.rb") args << "--name=test_#{test_method}" if test_method end args += ARGV.named.select { |v| v[/^TEST(OPTS)?=/] } - system "bundle", "exec", "parallel_test", *opts, - "--", *args, "--", *files + system "bundle", "exec", "parallel_test", *opts, "--", *args, "--", *files Homebrew.failed = !$?.success? -- cgit v1.2.3 From e3be9c068bd2168d3bbf077a55abb15a87eaf5bd Mon Sep 17 00:00:00 2001 From: Markus Reiter Date: Mon, 24 Oct 2016 18:57:57 +0200 Subject: Update `.rubocop_todo.yml`. --- Library/Homebrew/dev-cmd/audit.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'Library/Homebrew/dev-cmd') diff --git a/Library/Homebrew/dev-cmd/audit.rb b/Library/Homebrew/dev-cmd/audit.rb index c7461fc5a..df3b5fafe 100644 --- a/Library/Homebrew/dev-cmd/audit.rb +++ b/Library/Homebrew/dev-cmd/audit.rb @@ -761,7 +761,7 @@ class FormulaAuditor bin_names.each do |name| ["system", "shell_output", "pipe_output"].each do |cmd| if text =~ /(def test|test do).*#{cmd}[\(\s]+['"]#{Regexp.escape name}[\s'"]/m - problem %(fully scope test #{cmd} calls e.g. #{cmd} "\#{bin}/#{name}") + problem %Q(fully scope test #{cmd} calls e.g. #{cmd} "\#{bin}/#{name}") end end end -- cgit v1.2.3 From 4efdbbde06bc8785870315f7a334694e59b33528 Mon Sep 17 00:00:00 2001 From: Markus Reiter Date: Mon, 24 Oct 2016 23:11:23 +0200 Subject: Rename `fs_leak_log` to `fs_leak.log` and move to `tmp`. --- Library/Homebrew/dev-cmd/tests.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'Library/Homebrew/dev-cmd') diff --git a/Library/Homebrew/dev-cmd/tests.rb b/Library/Homebrew/dev-cmd/tests.rb index df9a00a16..656225516 100644 --- a/Library/Homebrew/dev-cmd/tests.rb +++ b/Library/Homebrew/dev-cmd/tests.rb @@ -65,7 +65,7 @@ module Homebrew Homebrew.failed = !$?.success? - if (fs_leak_log = HOMEBREW_LIBRARY/"Homebrew/test/fs_leak_log").file? + if (fs_leak_log = HOMEBREW_LIBRARY_PATH/"tmp/fs_leak.log").file? fs_leak_log_content = fs_leak_log.read unless fs_leak_log_content.empty? opoo "File leak is detected" -- cgit v1.2.3 From 7457af2b231528e1287281f6cfe651600d5f3f5d Mon Sep 17 00:00:00 2001 From: Markus Reiter Date: Sun, 13 Nov 2016 22:00:15 +0100 Subject: Move test files back directly to `test/`. --- Library/Homebrew/dev-cmd/tests.rb | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'Library/Homebrew/dev-cmd') diff --git a/Library/Homebrew/dev-cmd/tests.rb b/Library/Homebrew/dev-cmd/tests.rb index 656225516..c032b3437 100644 --- a/Library/Homebrew/dev-cmd/tests.rb +++ b/Library/Homebrew/dev-cmd/tests.rb @@ -44,8 +44,8 @@ module Homebrew # Make it easier to reproduce test runs. ENV["SEED"] = ARGV.next if ARGV.include? "--seed" - files = Dir.glob("test/test/**/*_test.rb") - .reject { |p| !OS.mac? && p.start_with?("test/test/os/mac/") } + files = Dir.glob("test/**/*_test.rb") + .reject { |p| !OS.mac? && p.start_with?("test/os/mac/") } opts = [] opts << "--serialize-stdout" if ENV["CI"] @@ -55,7 +55,7 @@ module Homebrew if ARGV.value("only") test_name, test_method = ARGV.value("only").split(":", 2) - files = Dir.glob("test/test/{#{test_name},#{test_name}/**/*}_test.rb") + files = Dir.glob("test/{#{test_name},#{test_name}/**/*}_test.rb") args << "--name=test_#{test_method}" if test_method end -- cgit v1.2.3