Use test doubles instead of 'true'

This commit is contained in:
sophia 2021-03-08 16:49:52 -06:00
parent 66fc0d84ea
commit 7c8e204b8c
2 changed files with 3 additions and 3 deletions

View File

@ -78,7 +78,7 @@ describe VagrantPlugins::DockerProvider::Provider do
it "returns the host machine object" do
allow(machine.provider_config).to receive(:vagrant_vagrantfile).and_return("/path/to/Vagrantfile")
allow(machine.provider_config).to receive(:vagrant_machine).and_return(:default)
allow(machine).to receive(:env).and_return(true)
allow(machine).to receive(:env).and_return(double("env"))
allow(machine.env).to receive(:root_path).and_return("/.vagrant.d")
allow(machine.env).to receive(:home_path).and_return("/path/to")
allow(machine.env).to receive(:ui_class).and_return(true)

View File

@ -81,7 +81,7 @@ describe VagrantPlugins::Puppet::Provisioner::Puppet do
let(:environment_paths) { ["/etc/puppet/environment"] }
it "builds structured facts if set" do
allow(machine).to receive(:guest).and_return(true)
allow(machine).to receive(:guest).and_return(double("guest"))
allow(machine.guest).to receive(:capability?).and_return(false)
allow(config).to receive(:environment_path).and_return(environment_paths)
allow(config).to receive(:environment).and_return("production")
@ -105,7 +105,7 @@ describe VagrantPlugins::Puppet::Provisioner::Puppet do
end
it "does not build structured facts if not set" do
allow(machine).to receive(:guest).and_return(true)
allow(machine).to receive(:guest).and_return(double("guest"))
allow(machine.guest).to receive(:capability?).and_return(false)
allow(config).to receive(:environment_path).and_return(environment_paths)
allow(config).to receive(:environment).and_return("production")