admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

Improve unit test to make sure the sudo version is used.

Browse Source
This commit is contained in:
JBYoshi 2023-05-11 12:44:35 -05:00
parent 2e8232de39
commit a1780c842c
No known key found for this signature in database
GPG Key ID: AE4430116622D05D
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
Library/Homebrew/test/cask/artifact/app_spec.rb
View File

@ -340,17 +340,20 @@ describe Cask::Artifact::App, :cask do
contents_path = target_path.join("Contents")
allow(target_path).to receive(:writable?).and_return false
expect(app.target).to receive(:writable?).at_least(:once).and_return false
allow(command).to receive(:run!).and_call_original
allow(command).to receive(:run!)
.with("/bin/cp", args: ["-pR", source_path.join("Contents"), contents_path],
sudo: true)
.and_wrap_original do |original_method, *args, **kwargs|
original_method.call(*args, sudo_as_root: false, **kwargs)
original_method.call(*args, **kwargs, sudo: false)
end
expect(FileUtils).not_to receive(:move).with(source_path.join("Contents"), contents_path)
app.uninstall_phase(command: command, force: force, successor: cask)
expect(contents_path).not_to exist
expect(target_path).to exist
expect(source_path.join("Contents")).to exist
app.install_phase(command: command, adopt: adopt, force: force, predecessor: cask)
expect(contents_path).to exist