Don't use "rm" for cleaning tmp/builds

If this directory were to be empty this would result in warnings being printed to STDERR, cluttering spec output. Doing this in Ruby fixes this problem (and also removes the need for shell alltogether).

Don't use "rm" for cleaning tmp/builds
dbc05d4a · Yorick Peterse · c867c225 · dbc05d4a
Commit dbc05d4a authored Oct 02, 2015 by Yorick Peterse
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 2 deletions

setup_builds_storage.rb spec/support/setup_builds_storage.rb +4 -2

No files found.
--- a/spec/support/setup_builds_storage.rb
+++ b/spec/support/setup_builds_storage.rb
@@ -10,8 +10,10 @@ RSpec.configure do |config|
  end
  config.after(:suite) do
-    Dir.chdir(builds_path) do
+    Dir[File.join(builds_path, '*')].each do |path|
-      `ls | grep -v .gitkeep | xargs rm -r`
+      next if File.basename(path) == '.gitkeep'
+      FileUtils.rm_rf(path)
    end
  end
 end