diff options
| author | Markus Reiter | 2017-06-24 07:01:35 +0200 |
|---|---|---|
| committer | Markus Reiter | 2017-06-24 09:15:37 +0200 |
| commit | a1f9032b9574dcb27b34a6deb4c9730839daf0df (patch) | |
| tree | 97ccd340be363f7e18dfe879cc2a588ff2135f8d /Library/Homebrew/test | |
| parent | f4f1f1a6108970bf646aad82305978262c671ee2 (diff) | |
| download | brew-a1f9032b9574dcb27b34a6deb4c9730839daf0df.tar.bz2 | |
🗑️ Use AppleScript instead of Swift for trashing.
Diffstat (limited to 'Library/Homebrew/test')
| -rw-r--r-- | Library/Homebrew/test/spec_helper.rb | 10 | ||||
| -rw-r--r-- | Library/Homebrew/test/utils/trash_spec.rb | 32 |
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 |
