diff options
author | Marc A. Paradise <marc.paradise@gmail.com> | 2019-05-13 16:42:50 -0400 |
---|---|---|
committer | Marc A. Paradise <marc.paradise@gmail.com> | 2019-05-13 17:25:32 -0400 |
commit | 4a28a2ffe6d830634017ccb5cb19749faa304dca (patch) | |
tree | b3023725d4128e9162afd703dbe828816a832e77 | |
parent | 082432c5a741c14b4276a255cc5cb8d90f44ed09 (diff) | |
download | chef-4a28a2ffe6d830634017ccb5cb19749faa304dca.tar.gz |
Ensure tests are aware of CLI default values
We added default value for session-timeout, but the tests
weren't aware of that because config was mocked.
Signed-off-by: Marc A. Paradise <marc.paradise@gmail.com>
-rw-r--r-- | spec/unit/knife/bootstrap_spec.rb | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/spec/unit/knife/bootstrap_spec.rb b/spec/unit/knife/bootstrap_spec.rb index 551ca3f18b..99fa861a07 100644 --- a/spec/unit/knife/bootstrap_spec.rb +++ b/spec/unit/knife/bootstrap_spec.rb @@ -880,7 +880,7 @@ describe Chef::Knife::Bootstrap do logger: Chef::Log, # not configurable ca_trust_file: "no trust", max_wait_until_ready: 9999, - operation_timeout: 9999, + operation_timeout: 60, ssl_peer_fingerprint: "ABCDEF", winrm_transport: "kerberos", winrm_basic_auth_only: true, @@ -956,7 +956,9 @@ describe Chef::Knife::Bootstrap do context "and no values are provided from Chef::Config or CLI" do before do - knife.config = {} + # We will use knife's actual config since these tests + # have assumptions based on CLI default values + knife.merge_configs end let(:expected_result) do { @@ -1128,7 +1130,9 @@ describe Chef::Knife::Bootstrap do end context "and no values are provided from Chef::Config or CLI" do before do - knife.config = {} + # We will use knife's actual config since these tests + # have assumptions based on CLI default values + knife.merge_configs end let(:expected_result) do |