aboutsummaryrefslogtreecommitdiffstats
path: root/Library/Homebrew/test
diff options
context:
space:
mode:
authorMarkus Reiter2017-06-24 07:01:35 +0200
committerMarkus Reiter2017-06-24 09:15:37 +0200
commita1f9032b9574dcb27b34a6deb4c9730839daf0df (patch)
tree97ccd340be363f7e18dfe879cc2a588ff2135f8d /Library/Homebrew/test
parentf4f1f1a6108970bf646aad82305978262c671ee2 (diff)
downloadbrew-a1f9032b9574dcb27b34a6deb4c9730839daf0df.tar.bz2
🗑️ Use AppleScript instead of Swift for trashing.
Diffstat (limited to 'Library/Homebrew/test')
-rw-r--r--Library/Homebrew/test/spec_helper.rb10
-rw-r--r--Library/Homebrew/test/utils/trash_spec.rb32
2 files changed, 8 insertions, 34 deletions
diff --git a/Library/Homebrew/test/spec_helper.rb b/Library/Homebrew/test/spec_helper.rb
index 03b14720b..75540caad 100644
--- a/Library/Homebrew/test/spec_helper.rb
+++ b/Library/Homebrew/test/spec_helper.rb
@@ -68,12 +68,18 @@ RSpec.configure do |config|
end
config.around(:each) do |example|
+ def find_files
+ Find.find(TEST_TMPDIR)
+ .reject { |f| File.basename(f) == ".DS_Store" }
+ .map { |f| f.sub(TEST_TMPDIR, "") }
+ end
+
begin
TEST_DIRECTORIES.each(&:mkpath)
@__homebrew_failed = Homebrew.failed?
- @__files_before_test = Find.find(TEST_TMPDIR).map { |f| f.sub(TEST_TMPDIR, "") }
+ @__files_before_test = find_files
@__argv = ARGV.dup
@__env = ENV.to_hash # dup doesn't work on ENV
@@ -106,7 +112,7 @@ RSpec.configure do |config|
CoreTap.instance.path/"formula_renames.json",
]
- files_after_test = Find.find(TEST_TMPDIR).map { |f| f.sub(TEST_TMPDIR, "") }
+ files_after_test = find_files
diff = Set.new(@__files_before_test) ^ Set.new(files_after_test)
expect(diff).to be_empty, <<-EOS.undent
diff --git a/Library/Homebrew/test/utils/trash_spec.rb b/Library/Homebrew/test/utils/trash_spec.rb
deleted file mode 100644
index 9f2f7df15..000000000
--- a/Library/Homebrew/test/utils/trash_spec.rb
+++ /dev/null
@@ -1,32 +0,0 @@
-require "open3"
-
-describe "trash", :needs_macos do
- let(:executable) { HOMEBREW_LIBRARY_PATH/"utils/trash.swift" }
- let(:dir) { mktmpdir }
- let(:file) { dir/"new_file" }
-
- it "moves existing files to the trash" do
- FileUtils.touch file
-
- expect(file).to exist
-
- out, err, status = Open3.capture3(executable, file)
-
- expect(out).to match %r{moved #{file} to .*/\.Trash/\.*}
- expect(err).to be_empty
- expect(status).to be_a_success
-
- expect(file).not_to exist
-
- trashed_path = out.sub(/^moved #{Regexp.escape(file.to_s)} to (.*)\n$/, '\1')
- FileUtils.rm_f trashed_path
- end
-
- it "fails when files don't exist" do
- out, err, status = Open3.capture3(executable, file)
-
- expect(out).to be_empty
- expect(err).to eq "could not move #{file} to trash\n"
- expect(status).to be_a_failure
- end
-end